[Koha-bugs] [Bug 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Bug 17562 depends on bug 13321, which changed state. Bug 13321 Summary: Fix tax and prices calculation https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321 What|Removed |Added Status|Pushed to Master|RESOLVED Resolution|--- |FIXED -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Jonathan Druart changed: What|Removed |Added Depends on||13321 --- Comment #1 from Jonathan Druart --- Certainly caused by bug 13321 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321 [Bug 13321] Fix tax and prices calculation -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Jonathan Druart changed: What|Removed |Added Status|ASSIGNED|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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 --- Comment #2 from Jonathan Druart --- Created attachment 57249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57249&action=edit Bug 17562: Fix Acquisition.t I find these tests too much complicated and decided to remove them. They depends on the number of fields in the DB and any time we change this table we need to update the tests. As I don't understand the tests, I prefer to simply remove them, they don't seem very useful. Feel free to provide a counter patch if you think we should keep them. Test plan: prove t/db_dependent/Acquisition.t should return green -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Josef Moravec changed: What|Removed |Added Attachment #57249|0 |1 is obsolete|| --- Comment #3 from Josef Moravec --- Created attachment 57251 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57251&action=edit [SIGNED-OFF] Bug 17562: Fix Acquisition.t I find these tests too much complicated and decided to remove them. They depends on the number of fields in the DB and any time we change this table we need to update the tests. As I don't understand the tests, I prefer to simply remove them, they don't seem very useful. Feel free to provide a counter patch if you think we should keep them. Test plan: prove t/db_dependent/Acquisition.t should return green Signed-off-by: Josef Moravec -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Josef Moravec changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Jonathan Druart changed: What|Removed |Added Blocks||13726 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13726 [Bug 13726] Koha::Acquisition::Bookseller should use Koha::Object -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Tomás Cohen Arazi 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Tomás Cohen Arazi changed: What|Removed |Added Attachment #57251|0 |1 is obsolete|| --- Comment #4 from Tomás Cohen Arazi --- Created attachment 57396 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=57396&action=edit Bug 17562: Fix Acquisition.t I find these tests too much complicated and decided to remove them. They depends on the number of fields in the DB and any time we change this table we need to update the tests. As I don't understand the tests, I prefer to simply remove them, they don't seem very useful. Feel free to provide a counter patch if you think we should keep them. Test plan: prove t/db_dependent/Acquisition.t should return green Signed-off-by: Josef Moravec Signed-off-by: Tomas Cohen Arazi This tests should be removed as this patch does. -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com Version|unspecified |master QA Contact|testo...@bugs.koha-communit |tomasco...@gmail.com |y.org | -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #5 from Kyle M Hall --- Pushed to master for 16.11, 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Mason James changed: What|Removed |Added CC||m...@kohaaloha.com Status|Pushed to Master|Pushed to Stable --- Comment #6 from Mason James --- (In reply to Kyle M Hall from comment #5) > Pushed to master for 16.11, thanks Jonathan! blocked by DB changes, passing for 16.05 -- 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 17562] Acquisition.t is broken
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17562 Mason James changed: What|Removed |Added Status|Pushed to Stable|Pushed to Master -- 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/