Bug 27974

Summary: Replace obsolete title-string sorting: Circulation templates
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, gmcharlt, jonathan.druart, koha-bugs, kyle.m.hall, martin.renvoize, oleonard, sally.healey, testopia, tomascohen
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
Bug Depends on:    
Bug Blocks: 27934    
Attachments: Bug 27974: Replace obsolete title-string sorting: Circulation templates
Bug 27974: Replace obsolete title-string sorting: Circulation templates
Bug 27974: Replace obsolete title-string sorting: Circulation templates
Bug 27974: (follow-up) Address missing instance
Bug 27974: Replace obsolete title-string sorting: Circulation templates
Bug 27974: (follow-up) Address missing instance

Description Owen Leonard 2021-03-17 16:32:38 UTC
This patch will modify the following templates to replace the old "title-string" DataTables sorting method with the built-in method using "data-order" attributes.

  intranet-tmpl/prog/en/modules/circ/checkout-notes.tt
  intranet-tmpl/prog/en/modules/circ/on-site_checkouts.tt
  intranet-tmpl/prog/en/modules/circ/overdue.tt
  intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
  intranet-tmpl/prog/en/modules/circ/transfers_to_send.tt
  intranet-tmpl/prog/en/modules/circ/transferstoreceive.tt
  intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt
Comment 1 Owen Leonard 2021-03-17 17:35:58 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2021-03-17 18:04:15 UTC Comment hidden (obsolete)
Comment 3 Martin Renvoize 2021-03-18 10:46:11 UTC
Created attachment 118456 [details] [review]
Bug 27974: Replace obsolete title-string sorting: Circulation templates

This patch modifies several circulation templates to replace the use of
the "title-string" DataTables sorting method with the newer
"data-order" attribute.

To test, apply the patch and view the following pages to confirm that
columns containing dates sort correctly when using any setting of the
"dateformat" system preference:

- Circulation -> Checkout notes
  - To populate the table, enable the "AllowCheckoutNotes" system
    preference enabled, log in to the OPAC as a user with checkouts.
    From the "Your summary" page use the "Report a problem" fields to
    submit notes.
- Circulation -> On-site checkouts
  - Requires the "OnSiteCheckouts" system preference to be enabled and
    and for some items to be checked out as on-site checkouts.
- Circulation -> Overdues
- Circulation -> Holds to pull
- Circulation -> Transfers to send
  - Requires the "StockRotation" preference to be enabled, some rotas
    to be set up, and for the stockrotation cron job to be executed.
- Circulation -> Transfers to receive
- Circulation -> Holds queue

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2021-03-18 10:46:55 UTC
Great bit of cleaning up, thanks Owen.

Signing off
Comment 5 Owen Leonard 2021-03-25 13:45:30 UTC
Created attachment 118798 [details] [review]
Bug 27974: (follow-up) Address missing instance

- Circulation -> Article requests
  - Both the "Pending" and "Processing" tabs.
Comment 6 Katrin Fischer 2021-03-27 14:44:52 UTC
Created attachment 118895 [details] [review]
Bug 27974: Replace obsolete title-string sorting: Circulation templates

This patch modifies several circulation templates to replace the use of
the "title-string" DataTables sorting method with the newer
"data-order" attribute.

To test, apply the patch and view the following pages to confirm that
columns containing dates sort correctly when using any setting of the
"dateformat" system preference:

- Circulation -> Checkout notes
  - To populate the table, enable the "AllowCheckoutNotes" system
    preference enabled, log in to the OPAC as a user with checkouts.
    From the "Your summary" page use the "Report a problem" fields to
    submit notes.
- Circulation -> On-site checkouts
  - Requires the "OnSiteCheckouts" system preference to be enabled and
    and for some items to be checked out as on-site checkouts.
- Circulation -> Overdues
- Circulation -> Holds to pull
- Circulation -> Transfers to send
  - Requires the "StockRotation" preference to be enabled, some rotas
    to be set up, and for the stockrotation cron job to be executed.
- Circulation -> Transfers to receive
- Circulation -> Holds queue

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Katrin Fischer 2021-03-27 14:44:55 UTC
Created attachment 118896 [details] [review]
Bug 27974: (follow-up) Address missing instance

- Circulation -> Article requests
  - Both the "Pending" and "Processing" tabs.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Katrin Fischer 2021-03-27 14:45:41 UTC
I wonder if we should change the module to Templates as that would make them all sort together in release notes and as we expect no behaviour change.
Comment 9 Martin Renvoize 2021-03-27 15:55:58 UTC
Good idea to change the module :)
Comment 10 Jonathan Druart 2021-04-06 13:32:01 UTC
Is that correct to remove "html" for on-site_checkouts.tt?
Comment 11 Jonathan Druart 2021-04-06 13:58:18 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 12 Fridolin Somers 2021-04-15 11:23:07 UTC
Enhancement not pushed to 20.11.x
Comment 13 Fridolin Somers 2021-04-15 11:25:45 UTC
Enhancement not pushed to 20.11.x