May 2018 - Koha-bugs - lists.koha-community.org

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=20687 Depends on|20687 | --- Comment #10 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Moving 20687 to see also instead of deps. They can be applied independent of each other. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20687 [Bug 20687] Multiple invitations to share lists prevents some users from accepting -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart(a)bugs.koha-c | |ommunity.org, | |nick(a)bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This report includes a db release notes| |revision that removes | |duplicate rows on | |shelfnumber and | |borrowernumber from your | |virtualshelfshares table. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|minor |normal -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 --- Comment #9 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 75051 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75051&action=ed… Bug 11943: Db revision to remove double accepted shares Although it is no problem to have them, we could do a cleanup. This patch just removes duplicate rows from the table. Note: I considered adding a unique index like: ALTER TABLE virtualshelfshares ADD UNIQUE INDEX (shelfnumber, borrowernumber, invitekey); But the possible NULL values in borrowernumber and/or invitekey require additional code changes. So I left it alone. Test plan: [1] Create two records with same borrowernumber and shelfnumber in the shares table, if not present already. [2] Run updatedatabase.pl Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 --- Comment #8 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 75050 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75050&action=ed… Bug 11943: Add unit test for double accepts Test plan: Run t/db_dependent/Virtualshelves.t Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 --- Comment #7 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 75049 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75049&action=ed… Bug 11943: Prevent double accepts in Koha::Virtualshelfshare This 'bug' existed long enough now to finally remove it ;) We do so by deleting the invitation if the borrower already has a share on this list. Actually not that hard. We still need: a unit test and a db revision. Test plan: [1] Share a list. Let user B accept. [2] Without this patch: Share again and let B accept again. [3] Verify that you have two shares for this list in virtualshelfshares. [4] With this patch: Share another list, let B accept. [5] Share this other list again, let B accept again. [6] Verify that virtualshelfshares does not contain double entries now. (Note: This pertains to the second list only.) Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] Prevent double accepts in Koha::Virtualshelfshare
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|AcceptShare should check |Prevent double accepts in |for double accepts |Koha::Virtualshelfshare -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11943] AcceptShare should check for double accepts
by bugzilla-daemon＠bugs.koha-community.org 04 May '18

04 May '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11943 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |20687 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20687 [Bug 20687] Multiple invitations to share lists prevents some users from accepting -- You are receiving this mail because: You are watching all bug changes.

1 0