Bug 27538

Summary: Cells in the bottom filtering row of the "Holds to pull" table shifted
Product: Koha Reporter: Peter Vashchuk <stalkernoid>
Component: CirculationAssignee: Peter Vashchuk <stalkernoid>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: andrewfh, caroline.cyr-la-rose, fridolin.somers, gmcharlt, jonathan.druart, kyle.m.hall, nugged, victor
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00,20.11.03,20.05.09
Bug Depends on: 23485    
Bug Blocks:    
Attachments: Bug 27538: fix correlation of fields in Holds to Pull table
Bug 27538: fix correlation of fields in Holds to Pull table
Bug 27538: fix correlation of fields in Holds to Pull table

Description Peter Vashchuk 2021-01-24 19:44:29 UTC

    
Comment 1 Peter Vashchuk 2021-01-24 19:46:14 UTC
Created attachment 115730 [details] [review]
Bug 27538: fix correlation of fields in Holds to Pull table

Bottom filtering row of "Holds to pull" table do not correlate to other ones, columns diverged from "Call numbers" cell: it should be under "Available call numbers" while it's under "Available barcodes", and so on to the last cell (while number of cells still adjusted).

To reproduce:
    1) Head to the Holds to Pull page, check that bottom filtering row shifted to the left by one, starting from the "Call number" column and don't correlate to the upper table (for example, "Call numbers" cell should be under "Available call numbers" while it's under "Available barcodes"), and so on for all cells till the end of the filtering row.
    2) Apply patch.
    3) Refresh Holds to Pull page and ensure that all filtering row columns correlate to other rows columns correctly.
Comment 2 Owen Leonard 2021-01-25 14:45:07 UTC
Created attachment 115777 [details] [review]
Bug 27538: fix correlation of fields in Holds to Pull table

Bottom filtering row of "Holds to pull" table do not correlate to other
ones, columns diverged from "Call numbers" cell: it should be under
"Available call numbers" while it's under "Available barcodes", and so
on to the last cell (while number of cells still adjusted).

To reproduce:
   1) Head to the Holds to Pull page, check that bottom filtering row
      shifted to the left by one, starting from the "Call number"
      column and don't correlate to the upper table (for example, "Call
      numbers" cell should be under "Available call numbers" while it's
      under "Available barcodes"), and so on for all cells till the end
      of the filtering row.
   2) Apply patch.
   3) Refresh Holds to Pull page and ensure that all filtering row
      columns correlate to other rows columns correctly.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Katrin Fischer 2021-01-31 13:16:18 UTC
Created attachment 116100 [details] [review]
Bug 27538: fix correlation of fields in Holds to Pull table

Bottom filtering row of "Holds to pull" table do not correlate to other
ones, columns diverged from "Call numbers" cell: it should be under
"Available call numbers" while it's under "Available barcodes", and so
on to the last cell (while number of cells still adjusted).

To reproduce:
   1) Head to the Holds to Pull page, check that bottom filtering row
      shifted to the left by one, starting from the "Call number"
      column and don't correlate to the upper table (for example, "Call
      numbers" cell should be under "Available call numbers" while it's
      under "Available barcodes"), and so on for all cells till the end
      of the filtering row.
   2) Apply patch.
   3) Refresh Holds to Pull page and ensure that all filtering row
      columns correlate to other rows columns correctly.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Jonathan Druart 2021-02-01 16:17:55 UTC
Caused by:
  commit fe84c3acba33af7a2fb27643f6b5f9129c329b36
  Bug 23485: Show barcode in holds to pull
Comment 5 Jonathan Druart 2021-02-01 16:27:01 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Fridolin Somers 2021-02-05 16:11:45 UTC
Pushed to 20.11.x for 20.11.03
Comment 7 Andrew Fuerste-Henry 2021-02-09 21:19:42 UTC
Pushed to 20.05.x for 20.05.09
Comment 8 Victor Grousset/tuxayo 2021-02-11 03:09:01 UTC
Missing dependencies for 19.11.x, it shouldn't be affected, no backport.