Attachment 157458 Details for Bug 35108 – Bug 35108: Hide optionally empty fields label entries, these are:

[patch] Bug 35108: Hide optionally empty fields label entries, these are:

Bug-35108-Hide-optionally-empty-fields-label-entri.patch (text/plain), 11.10 KB, created by Pedro Amorim on 2023-10-19 14:23:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pedro Amorim

Created: 2023-10-19 14:23:22 UTC

Size: 11.10 KB

patch

obsolete

>From bfd9b6f7f9426284203f0f9484b1da0bafea11f5 Mon Sep 17 00:00:00 2001
>From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
>Date: Thu, 19 Oct 2023 14:13:48 +0000
>Subject: [PATCH] Bug 35108: Hide optionally empty fields label entries, these
> are:
>
>orderid, biblio_id, batch already was the case, type, type_disclaimer_value, type_disclaimer_date, cost, price_paid, notesstaff, notesopac
>
>Test plan, k-t-d:
>1) Install FreeForm and BLDSS backends, run:
>bash <(curl -s https://raw.githubusercontent.com/ammopt/koha-ill-dev/master/start-ill-dev-plus.sh)
>2) Create a new FreeForm request, visit:
>/cgi-bin/koha/ill/ill-requests.pl?method=create&backend=FreeForm
>5) Set type 'Book', put '42' in the cardnumber, and pick a library. Click 'create'.
>6) Notice how there is a lot of unnecessary empty field entries
>7) Apply patch, restart plack (koha-plack --restart kohadev)
>8) Refresh the page, notice all the empty fields entries are no longer displayed
>---
> .../prog/en/modules/ill/ill-requests.tt | 114 ++++++++++--------
> 1 file changed, 65 insertions(+), 49 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>index 0d6dd28b7a..f77af50d0f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
>@@ -631,10 +631,12 @@
> <h4>Details from library</h4>
> <div class="rows">
> <ol>
>- <li class="orderid">
>- Order ID:
>- [% IF request.orderid %][% request.orderid | html %][% ELSE %]N/A[% END %]
>- </li>
>+ [% IF request.orderid %]
>+ <li class="orderid">
>+ Order ID:
>+ [% request.orderid | html %]
>+ </li>
>+ [% END %]
> <li class="borrowernumber">
> Patron:
> [% borrowerlink = "/cgi-bin/koha/members/moremember.pl" _ "?borrowernumber=" _ request.patron.borrowernumber %]
>@@ -642,15 +644,12 @@
> [% request.patron.firstname _ " " _ request.patron.surname _ " [" _ request.patron.cardnumber _ "]" | html %]
> </a>
> </li>
>-
>- <li class="biblio_id">
>- Bibliographic record ID:
>- [% IF request.biblio_id %]
>+ [% IF request.biblio_id %]
>+ <li class="biblio_id">
>+ Bibliographic record ID:
> <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% request.biblio_id | uri %]">[% request.biblio_id | html %]</a>
>- [% ELSE %]
>- N/A
>- [% END %]
>- </li>
>+ </li>
>+ [% END %]
> <li class="branchcode">
> Library:
> [% Branches.GetName(request.branchcode) | html %]
>@@ -667,58 +666,75 @@
> [% END %]
> </li>
> [% IF request.ill_batch > 0 %]
>- <li class="batch">
>- Batch:
>- <a href="/cgi-bin/koha/ill/ill-requests.pl?batch_id=[% request.batch.ill_batch_id | uri %]">
>- [% request.batch.name | html %]
>- </a>
>- </li>
>+ <li class="batch">
>+ Batch:
>+ <a href="/cgi-bin/koha/ill/ill-requests.pl?batch_id=[% request.batch.ill_batch_id | uri %]">
>+ [% request.batch.name | html %]
>+ </a>
>+ </li>
> [% END %]
> <li class="updated">
> Last updated:
> [% request.updated | $KohaDates with_hours => 1 %]
> </li>
>- <li class="medium">
>- Request type:
>- [% type = request.get_type %]
>- [% IF type %][% type | html %][% ELSE %]N/A[% END %]
>- </li>
>- <li class="type_disclaimer_value">
>- Type disclaimer value:
>- [% type_disclaimer_value = request.get_type_disclaimer_value %]
>- [% IF type_disclaimer_value %][% type_disclaimer_value | html %][% ELSE %]N/A[% END %]
>- </li>
>- <li class="type_disclaimer_date">
>- Type disclaimer date:
>- [% type_disclaimer_date = request.get_type_disclaimer_date %]
>- [% IF type_disclaimer_date %][% type_disclaimer_date | html %][% ELSE %]N/A[% END %]
>- </li>
>- <li class="cost">
>- Cost:
>- [% IF request.cost %][% request.cost | $Price %][% ELSE %]N/A[% END %]
>- </li>
>- <li class="price_paid">
>- Price paid:
>- [% IF request.price_paid %][% request.price_paid | $Price %][% ELSE %]N/A[% END %]
>- </li>
>- <li class="notesstaff">
>- Staff notes:
>- [% request.notesstaff | html %]
>- </li>
>- <li class="notesopac">
>- Notes:
>- [% request.notesopac | html %]
>- </li>
>+ [% type = request.get_type %]
>+ [% IF type %]
>+ <li class="medium">
>+ Request type:
>+ [% type | html %]
>+ </li>
>+ [% END %]
>+ [% type_disclaimer_value = request.get_type_disclaimer_value %]
>+ [% IF type_disclaimer_value %]
>+ <li class="type_disclaimer_value">
>+ Type disclaimer value:
>+ [% type_disclaimer_value = request.get_type_disclaimer_value %]
>+ [% type_disclaimer_value | html %]
>+ </li>
>+ [% END %]
>+ [% type_disclaimer_date = request.get_type_disclaimer_date %]
>+ [% IF type_disclaimer_date %]
>+ <li class="type_disclaimer_date">
>+ Type disclaimer date:
>+ [% type_disclaimer_date | html %]
>+ </li>
>+ [% END %]
>+ [% IF request.cost %]
>+ <li class="cost">
>+ Cost:
>+ [% request.cost | $Price %]
>+ </li>
>+ [% END %]
>+ [% IF request.price_paid %]
>+ <li class="price_paid">
>+ Price paid:
>+ [% request.price_paid | $Price %]
>+ </li>
>+ [% END %]
>+ [% IF request.notesstaff %]
>+ <li class="notesstaff">
>+ Staff notes:
>+ [% request.notesstaff | html %]
>+ </li>
>+ [% END %]
>+ [% IF request.notesopac %]
>+ <li class="notesopac">
>+ Notes:
>+ [% request.notesopac | html %]
>+ </li>
>+ [% END %]
> </ol>
> </div>
> <div class="rows">
> <h4>Details from supplier ([% request.backend | html %])</h4>
> <ol>
> [% FOREACH meta IN request.metadata %]
>+ [% IF meta.value %]
> <li class="requestmeta-[% meta.key.replace('\s','_') | html %]">
> [% meta.key.replace('_',' ') | html %]:
> [% meta.value | html %]
> </li>
>+ [% END %]
> [% END %]
> </ol>
> </div>
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35108: 157458 | 157463 | 157464 | 159657 | 159658 | 159688 | 159689 | 159752 | 159753 | 165136 | 165137