[Bug 14343] New: Links to results pages don't work in Receive Shipment list
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Bug ID: 14343 Summary: Links to results pages don't work in Receive Shipment list Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: koha@univ-lyon3.fr QA Contact: testopia@bugs.koha-community.org Created attachment 39905 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39905&action=edit Numbered links have incorrect url Links to result pages don't work in Receive Shipment List (but fortunately, Next and Previous buttons work) It's because the booksellerid is not furnished in the url. Test Plan : 1) Go to Acquisitions module, enter a bookseller name that you know you can get many page of invoices for and search for it. 2) click on Receive shipment button. 3) On bottom of the first results page, click on page number 2 link. (cf joined screencast) You'll see that the results include invoices from other booksellerid. Indeed, I suppose that you get results from all booksellerid. Intall patch and redo 3 steps. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@univ-lyon3.fr --- Comment #1 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Created attachment 39907 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39907&action=edit Bug 14343 Incorrect links to results pages in Receive shipment list The problem is already existing in 3.14 version and maybe previous ones. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #39907|0 |1 is obsolete| | --- Comment #2 from M. Tompsett <mtompset@hotmail.com> --- Created attachment 39910 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39910&action=edit Bug 14343 Incorrect links to results pages in Receive Shipment List Numbered links have incorrect url. Links to result pages don't work in Receive Shipment List (but fortunately, Next and Previous buttons work) It's because the booksellerid is not furnished in the url. Test Plan : 1) Go to Acquisitions module, enter a bookseller name that you know you can get many page of invoices for and search for it. 2) click on Receive shipment button. 3) On bottom of the first results page, click on page number 2 link. (cf joined screencast) You'll see that the results include invoices from other booksellerid. Indeed, I suppose that you get results from all booksellerid. Intall patch and redo 3 steps. NOTE: I did not follow this test plan. I read the acqui/parcels.pl code. The template parameter numbers is assigned in a function which has no reference to booksellerid at all! Additionally, the booksellerid is set directly elsewhere. It is also strange that the booksellerid references before and after this loop do not use the numbers.booksellerid, but just booksellerid. The change from numbers.booksellerid to booksellerid is correct! Signed-off-by: Mark Tompsett <mtompset@hotmail.com> -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset@hotmail.com --- Comment #3 from M. Tompsett <mtompset@hotmail.com> --- This applies cleanly as far back at 3.8.x -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think we removed this kind of double paging at some point after introducing the datatable, does this appear in master? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Mark write he didn't follow the test plan - please, can someone check in the GUI? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |koha@univ-lyon3.fr |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 --- Comment #6 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- I confirm that the double paging (one from datatable and another in html) still appears in Master but the datatable one does'nt work either (next and previous buttons are disabled,show entries does not get far more than 20 results) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Zeno Tajoli <z.tajoli@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Koha Team Rennes 2 <koha@listes.univ-rennes2.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@listes.univ-rennes2.fr Status|Needs Signoff |Signed Off --- Comment #7 from Koha Team Rennes 2 <koha@listes.univ-rennes2.fr> --- Hi, This patch applies on 3.16 but there is a two page gap between the result numbers and the pages actually displayed (for example, clicking on number 10 will lead to page 8). Don't know if this problem has already been reported somewhere else. Koha Team Rennes 2 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #39910|0 |1 is obsolete| | --- Comment #8 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42402 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42402&action=edit Bug 14343 Incorrect links to results pages in Receive Shipment List Numbered links have incorrect url. Links to result pages don't work in Receive Shipment List (but fortunately, Next and Previous buttons work) It's because the booksellerid is not furnished in the url. Test Plan : 1) Go to Acquisitions module, enter a bookseller name that you know you can get many page of invoices for and search for it. 2) click on Receive shipment button. 3) On bottom of the first results page, click on page number 2 link. (cf joined screencast) You'll see that the results include invoices from other booksellerid. Indeed, I suppose that you get results from all booksellerid. Intall patch and redo 3 steps. NOTE: I did not follow this test plan. I read the acqui/parcels.pl code. The template parameter numbers is assigned in a function which has no reference to booksellerid at all! Additionally, the booksellerid is set directly elsewhere. It is also strange that the booksellerid references before and after this loop do not use the numbers.booksellerid, but just booksellerid. The change from numbers.booksellerid to booksellerid is correct! Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42403 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42403&action=edit Bug 14343: Remove the DT pagination It does not make sense to have 2 paginations here. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org Severity|minor |normal -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com Status|Passed QA |Pushed to Master --- Comment #10 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patches pushed to master. Thanks Lyon3 Team! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz Status|Pushed to Master |Pushed to Stable --- Comment #11 from Chris Cormack <chris@bigballofwax.co.nz> --- Pushed to 3.20.x will be in 3.20.4 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14343 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Links to results pages |Links to results pages |don't work in Receive |don't work in receive |Shipment list |shipment list -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org