Attachment 115250 Details for Bug 27465 – Bug 27465: Add column visibility to admin/cities.pl

[patch] Bug 27465: Add column visibility to admin/cities.pl

Bug-27465-Add-column-visibility-to-admincitiespl.patch (text/plain), 2.70 KB, created by Jonathan Druart on 2021-01-18 15:21:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2021-01-18 15:21:48 UTC

Size: 2.70 KB

patch

obsolete

>From 6aab2909158782a115cbf40d5ea1e98c63cafd8d Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 18 Jan 2021 16:18:09 +0100
>Subject: [PATCH] Bug 27465: Add column visibility to admin/cities.pl
>
>We want admin/cities.pl to be using as much DT features as possible to be used as an example.
>
>Test plan:
>Add some cities, go to admin/cities.pl confirm that you can show/hide
>column.
>
>Go to the tables settings admin page and confirm that you can set some
>default settings from there.
>---
> admin/columns_settings.yml                        | 15 +++++++++++++++
> .../intranet-tmpl/prog/en/modules/admin/cities.tt |  5 ++++-
> 2 files changed, 19 insertions(+), 1 deletion(-)
>
>diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml
>index d825168439..1b9db9ea05 100644
>--- a/admin/columns_settings.yml
>+++ b/admin/columns_settings.yml
>@@ -155,6 +155,21 @@ modules:
>               cannot_be_toggled: 1
>               cannot_be_modified: 1
>   admin:
>+    cities:
>+      table_cities:
>+        columns:
>+            -
>+              columnname: city_id
>+            -
>+              columnname: city_name
>+            -
>+              columnname: city_state
>+            -
>+              columnname: city_zipcode
>+            -
>+              columnname: city_country
>+            -
>+              columnname: actions
>     currency:
>       currencies-table:
>         columns:
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt
>index ebf9ffad4e..24a2d26d30 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt
>@@ -1,6 +1,7 @@
> [% USE raw %]
> [% USE Asset %]
> [% SET footerjs = 1 %]
>+[% USE TablesSettings %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Koha &rsaquo; Administration &rsaquo; [% IF op =='add_form' %]Cities &rsaquo; [% IF city.cityid %] Modify city[% ELSE %] New city[% END %][% ELSE %][% IF op == 'delete_confirm' %]Cities &rsaquo; Confirm deletion of city[% ELSE %] Cities[% END %][% END %]</title>
> [% INCLUDE 'doc-head-close.inc' %]
>@@ -167,8 +168,10 @@
> [% MACRO jsinclude BLOCK %]
>     [% Asset.js("js/admin-menu.js") | $raw %]
>     [% INCLUDE 'datatables.inc' %]
>+    [% INCLUDE 'columns_settings.inc' %]
>     <script>
> 
>+        var columns_settings = [% TablesSettings.GetColumns( 'admin', 'cities', 'table_cities', 'json' ) | $raw %];
>         $(document).ready(function() {
>             var cities_table_url = '/api/v1/cities?';
> 
>@@ -233,7 +236,7 @@
>                         "orderable": false
>                     }
>                 ]
>-            }, [], 1);
>+            }, columns_settings, 1);
> 
>         });
>     </script>
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 27465: 115250 | 119233 | 119238 | 119799