Bug 33463 - 'Actions' column on plugins table should not be sortable
Summary: 'Actions' column on plugins table should not be sortable
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-04-09 12:08 UTC by Katrin Fischer
Modified: 2023-12-28 20:47 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.07


Attachments
Bug 33463: Remove sort for 'Actions' on the plugins table (1.38 KB, patch)
2023-05-11 10:28 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 33463: Remove sort for 'Actions' on the plugins table (1.44 KB, patch)
2023-05-14 11:51 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2023-04-09 12:08:31 UTC
The last column in the plugins table is 'Actions' and should not be sortable.
Comment 1 Jonathan Druart 2023-05-11 10:28:49 UTC
Created attachment 151066 [details] [review]
Bug 33463: Remove sort for 'Actions' on the plugins table

Test plan:
Go go the plugin admin page
upload a plugin
Notice that the "Actions" column is not longuer sortable
Comment 2 Jonathan Druart 2023-05-11 10:30:18 UTC
Looks like there are much more occurrences.

git grep 'Actions</th>'|grep -v NoSort 

We should fix them all.
Comment 3 Katrin Fischer 2023-05-14 11:50:03 UTC
(In reply to Jonathan Druart from comment #2)
> Looks like there are much more occurrences.
> 
> git grep 'Actions</th>'|grep -v NoSort 
> 
> We should fix them all.

I have checked a few and I think there is other mark-up there to prevent them from being sorted sometimes:

koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt:                                    <th class="noExport">Actions</th>
Action column on vendor search is NOT sortable.
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/edifactmsgs.tt:<th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt:                          <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt:                            <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt:        <th>Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt:                            <th class="noExport">Actions</th>
Actions IS sortable.
koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt:                                <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt:                                <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tt:            <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqcontract.tt:                                <th scope="col">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt:                    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/background_jobs.tt:                    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt:                    <th data-class-name="actions noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/cash_registers.tt:                           <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/categories.tt:                    <th scope="col">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt:                        <th data-class-name="actions noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt:    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt:    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/classsources.tt:    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/credit_types.tt:                                    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/debit_types.tt:                                    <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/edi_accounts.tt:       <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/edi_ean_accounts.tt:            <th class="noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/identity_provider_domains.tt:                    <th data-class-name="actions noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/identity_providers.tt:                    <th data-class-name="actions noExport">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/items_search_fields.tt:                                            <th class="noExport">Actions</th>
Table is not sortable at all.
koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc-overlay-rules.tt:                            <th>Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/matching-rules.tt:                                    <th class="noExport">Actions</th>
Table is not sortable at all.
koha-tmpl/intranet-tmpl/prog/en/modules/admin/restrictions.tt:                        <th scope="col">Actions</th>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt:                            <th class="noExport">Actions</th>
Actions is NOT sortable.

I feel this might be a nice excercise for maybe a newbiew/intro session as they require taking a closer look maybe? I feel we can still push this one.
Comment 4 Katrin Fischer 2023-05-14 11:51:27 UTC
Created attachment 151174 [details] [review]
Bug 33463: Remove sort for 'Actions' on the plugins table

Test plan:
Go go the plugin admin page
upload a plugin
Notice that the "Actions" column is not longuer sortable

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Katrin Fischer 2023-05-14 11:54:35 UTC
Filed:
Bug 33730 - Action column should not be sortable

Easy fix, excercising QAM super powers here: PQA
Comment 6 Tomás Cohen Arazi 2023-05-15 11:52:03 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Pedro Amorim 2023-06-05 16:25:27 UTC
Nice work everyone!

Pushed to 22.11.x for next release