[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=14172 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Galen Charlton gmcha...@gmail.com changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||gmcha...@gmail.com --- Comment #20 from Galen Charlton gmcha...@gmail.com --- Pushed to master. Thanks, Jonathan! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Attachment #17575|0 |1 is obsolete|| --- Comment #18 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 19016 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19016action=edit Bug 9987: Remove DB field aqorders.biblioitemnunmber The DB field aqorders.biblioitemnumber seems to be unused except to get the itype on the spent.pl page. This information can be retrieved uising another sql join. Test plan: Try a complete workflow in the acquisition module: create an order, receive it, play with the syspref AcqCreateItem. Check that no regression is found and that the data for existing orders don't change. Signed-off-by: Mathieu Saby mathieu.s...@univ-rennes2.fr Signed-off-by: Kyle M Hall k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added CC||k...@bywatersolutions.com --- Comment #19 from Kyle M Hall k...@bywatersolutions.com --- Passes koha-qa.pl and everything look good. Removes unused subroutines and references aqorders.biblioitemnumber as well as the field itself. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #12 from Jonathan Druart jonathan.dru...@biblibre.com --- (In reply to comment #11) I did not test 17436 because I am not sure it is related to the same problem. Could you please confirm it, and provide a test plan ? On ordered.pl and spent.pl, itemtype codes are displayed instead of their descriptions. This patch just change that. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #13 from mathieu saby mathieu.s...@univ-rennes2.fr --- thank you, I thought it was a bad attachment... I will test that this afternoon. Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Attachment #17436|0 |1 is obsolete|| --- Comment #14 from Jonathan Druart jonathan.dru...@biblibre.com --- Comment on attachment 17436 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17436 Bug 9987: Add itemtype description instead of code I opened a new bug 10090 for it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 mathieu saby mathieu.s...@univ-rennes2.fr changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #15 from mathieu saby mathieu.s...@univ-rennes2.fr --- Ok, so I sign off bug 9987. Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 mathieu saby mathieu.s...@univ-rennes2.fr changed: What|Removed |Added Attachment #17435|0 |1 is obsolete|| --- Comment #16 from mathieu saby mathieu.s...@univ-rennes2.fr --- Created attachment 17575 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17575action=edit [PATCH][SIGNED OFF] Bug 9987: Remove DB field aqorders.biblioitemnunmber -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added CC||colin.campbell@ptfs-europe. ||com --- Comment #17 from Jonathan Druart jonathan.dru...@biblibre.com --- *** Bug 10091 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #11 from mathieu saby mathieu.s...@univ-rennes2.fr --- Hello Jonathan attachement 17435 is working well. I did not test 17436 because I am not sure it is related to the same problem. Could you please confirm it, and provide a test plan ? Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #8 from Jonathan Druart jonathan.dru...@biblibre.com --- (In reply to comment #7) I have this error when I try to apply 17184 Auto-merging installer/data/mysql/updatedatabase.pl CONFLICT (content): Merge conflict in installer/data/mysql/updatedatabase.pl Auto-merging C4/Acquisition.pm Failed to merge in the changes. Patch failed at 0001 Bug 9987: Remove DB field aqorders.biblioitemnunmber When you have resolved this problem run git am --resolved. If you would prefer to skip this patch, instead run git am --skip. To restore the original branch and stop patching run git am --abort. M. Saby Mathieu, It is a conflict on updatedatabase.pl, it is really easy to fix for testing (edit the file, merge conflict, git add /path/to/updatedatabase.pl, git am --resolved). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Attachment #17184|0 |1 is obsolete|| Attachment #17185|0 |1 is obsolete|| --- Comment #9 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 17435 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17435action=edit Bug 9987: Remove DB field aqorders.biblioitemnunmber The DB field aqorders.biblioitemnumber seems to be unused except to get the itype on the spent.pl page. This information can be retrieved uising another sql join. Test plan: Try a complete workflow in the acquisition module: create an order, receive it, play with the syspref AcqCreateItem. Check that no regression is found and that the data for existing orders don't change. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #10 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 17436 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17436action=edit Bug 9987: Add itemtype description instead of code In ordered.pl and spent.pl, the codes are displayed, instead of descriptions. Links for the ordernumber should be changed. In ordered.pl, we are redirected to the receive page. In spent.pl, the links are deleted. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 mathieu saby mathieu.s...@univ-rennes2.fr changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #7 from mathieu saby mathieu.s...@univ-rennes2.fr --- I have this error when I try to apply 17184 Auto-merging installer/data/mysql/updatedatabase.pl CONFLICT (content): Merge conflict in installer/data/mysql/updatedatabase.pl Auto-merging C4/Acquisition.pm Failed to merge in the changes. Patch failed at 0001 Bug 9987: Remove DB field aqorders.biblioitemnunmber When you have resolved this problem run git am --resolved. If you would prefer to skip this patch, instead run git am --skip. To restore the original branch and stop patching run git am --abort. M. Saby -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #4 from Jonathan Druart jonathan.dru...@biblibre.com --- (In reply to comment #3) Jonathan, you suppress 2 subs in C4::Acquisition. Is it necessary? Are you sure they will not be usefull ? Yes, grep is my friend. git grep GetOrderNumber git grep ModOrderBibliotemNumber -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #5 from mathieu saby mathieu.s...@univ-rennes2.fr --- OK. But are you sure they cannot be usefull for a developer in the future? Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #6 from Jonathan Druart jonathan.dru...@biblibre.com --- (s)he will rewrite them :) Both use the field I delete and are not currently in use. I don't see any usefulness to keep them. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #3 from mathieu saby mathieu.s...@univ-rennes2.fr --- Jonathan, you suppress 2 subs in C4::Acquisition. Is it necessary? Are you sure they will not be usefull ? Mathieu Saby -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|koha-b...@lists.koha-commun |jonathan.dru...@biblibre.co |ity.org |m -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=7593 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #2 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 17185 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17185action=edit Bug 9987: Add itemtype description instead of code In ordered.pl and spent.pl, the codes are displayed, instead of descriptions. Links for the ordernumber should be changed. In ordered.pl, we are redirected to the receive page. In spent.pl, the links are deleted. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 --- Comment #1 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 17184 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=17184action=edit Bug 9987: Remove DB field aqorders.biblioitemnunmber The DB field aqorders.biblioitemnumber seems to be unused except to get the itype on the spent.pl page. This information can be retrieved uising another sql join. Test plan: Try a complete workflow in the acquisition module: create an order, receive it, play with the syspref AcqCreateItem. Check that no regression is found and that the data for existing orders don't change. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Status|ASSIGNED|Needs Signoff Patch complexity|--- |Medium patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 9987] biblioitemnumber is useless for orders.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9987 mathieu saby mathieu.s...@univ-rennes2.fr changed: What|Removed |Added CC||mathieu.saby@univ-rennes2.f ||r -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
