Bug 6104 - Modifying an item causes it to lose its branch
Summary: Modifying an item causes it to lose its branch
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: 3.4
Hardware: All All
: PATCH-Sent (DO NOT USE) critical
Assignee: Paul Poulain
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-07 08:33 UTC by Paul Poulain
Modified: 2019-06-27 09:24 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
proposed patch (1.05 KB, patch)
2011-04-07 08:34 UTC, Paul Poulain 		Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Poulain 2011-04-07 08:33:26 UTC 
Under certain circumstances, the item branch, when modified is lost.

Step to reproduce:
* IndependantBranches=OFF
* have 2 branches
* connect as a librarian in branch A
* try to modify an item from branch B
=> the form now select branch A for the item
Comment 1 Paul Poulain 2011-04-07 08:34:56 UTC 
Created attachment 3768 [details] [review]
proposed patch
Comment 2 Sophie MEYNIEUX 2011-04-07 09:24:08 UTC 
I have tried to modify a item before applying the patch. Homebranch and holdingbranch are set to my branch whatever value they had before.

I have applied the patch and tried to modify an item. Homebranch and holding branch keep their previous value.
Comment 3 Chris Cormack 2011-04-07 22:30:48 UTC 
Patch has not hit the mailing list yet, can it be attached to the bug.
Comment 4 Chris Cormack 2011-04-11 21:53:37 UTC 
This still needs the signed off patch, I really would like to push this, but no signed off patch attached here, or on the mailing list
Comment 5 Chris Cormack 2011-04-14 06:31:17 UTC 
Patch finally came through on the mailing list (i suspect it got held up for moderation) pushed now
Comment 6 Katrin Fischer 2011-04-24 12:00:11 UTC 
Successfully tested on 3.4.