Attachment 88596 Details for Bug 21757 – Bug 21757: (folow-up) Restore patron brief detail page

[patch] Bug 21757: (folow-up) Restore patron brief detail page

Bug-21757-folow-up-Restore-patron-brief-detail-pag.patch (text/plain), 6.41 KB, created by Josef Moravec on 2019-04-24 08:03:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2019-04-24 08:03:12 UTC

Size: 6.41 KB

patch

obsolete

>From ec5486185db0ae8db79cd1b9af43207082cd9eca Mon Sep 17 00:00:00 2001
>From: Josef Moravec <josef.moravec@gmail.com>
>Date: Thu, 18 Apr 2019 08:37:38 +0000
>Subject: [PATCH] Bug 21757: (folow-up) Restore patron brief detail page
>
>Test plan
>1) Try to create patron with same name/datefobirth to trigger the
>duplicate patron check
>2) Click on "View existing record" link on "Duplicate patron record?"
>alert message
>3) You should see the patron data, but withou toolbar, menu, and so on.
>---
> .../prog/en/modules/members/moremember-brief.tt    | 84 ++++++++++++++++++++++
> members/moremember.pl                              | 13 +++-
> 2 files changed, 96 insertions(+), 1 deletion(-)
> create mode 100644 koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-brief.tt
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-brief.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-brief.tt
>new file mode 100644
>index 0000000000..c68fd9d63c
>--- /dev/null
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-brief.tt
>@@ -0,0 +1,84 @@
>+[% USE Koha %]
>+[% USE KohaDates %]
>+[% SET footerjs = 1 %]
>+[% INCLUDE 'doc-head-open.inc' %]
>+<title>Koha &rsaquo; Check duplicate patron</title>
>+[% INCLUDE 'doc-head-close.inc' %]
>+</head>
>+
>+<body id="pat_moremember-brief" class="pat">
>+    <div class="container-fluid">
>+        <div class="row">
>+            <div class="col-xs-12">
>+                <h3>[% INCLUDE 'patron-title.inc' %]</h3>
>+            </div>
>+        </div>
>+        <div class="row">
>+            <div class="col-xs-6">
>+                <div id="patron-information">
>+
>+        [% IF ( patron.othernames ) %]&ldquo;[% patron.othernames | html %]&rdquo;[% END %]
>+        <div class = "address">
>+            <ul>
>+                [% IF Koha.Preference( 'AddressFormat' ) %]
>+                    [% INCLUDE "member-display-address-style-${ Koha.Preference( 'AddressFormat' ) }.inc" %]
>+                [% ELSE %]
>+                    [% INCLUDE 'member-display-address-style-us.inc' %]
>+                [% END %]
>+            </ul>
>+        </div>
>+    <div class="rows">
>+        <ol>
>+            [% IF ( patron.phone ) %]<li><span class="label">Primary phone: </span>[% patron.phone | html %]</li>[% END %]
>+            [% IF ( patron.phonepro ) %]<li><span class="label">Secondary phone: </span>[% patron.phonepro | html %]</li>[% END %]
>+            [% IF ( patron.mobile ) %]<li><span class="label">Other phone: </span>[% patron.mobile | html %]</li>[% END %]
>+            [% IF ( patron.fax ) %]<li><span class="label">Fax: </span>[% patron.fax | html %]</li>[% END %]
>+            [% IF ( patron.email ) %]<li class="email"><span class="label">Primary email:</span><a href="mailto:[% email | uri %]">[% patron.email | html %]</a></li>[% END %]
>+            [% IF ( patron.emailpro ) %]<li class="email"><span class="label">Secondary email: </span><a href="mailto:[% emailpro | uri %]">[% patron.emailpro | html %]</a></li>[% END %]
>+            [% UNLESS ( I ) %]
>+                [% IF ( patron.initials ) %]<li><span class="label">Initials: </span>[% patron.initials | html %]</li>[% END %]
>+                [% IF ( patron.dateofbirth ) %]<li><span class="label">Date of birth:</span>[% patron.dateofbirth | $KohaDates %]</li>[% END %]
>+                [% IF ( patron.sex ) %]<li><span class="label">Gender:</span>[% IF ( patron.sex == 'F' ) %]Female[% ELSIF ( patron.sex == 'M' ) %]Male[% ELSE %][% patron.sex | html %][% END %]</li>[% END %]
>+            [% END %]
>+            [% IF ( guarantees ) %]
>+                <li><span class="label">Guarantees:</span><ul>[% FOREACH guarantee IN guarantees %]<li><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantee.borrowernumber | uri %]">[% INCLUDE 'patron-title.inc' patron=guarantee %]</a></li>[% END %]</ul></li>
>+            [% END %]
>+            [% IF ( guarantor ) %]
>+                <li><span class="label">Guarantor:</span><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantor.borrowernumber | uri %]">[% guarantor.surname | html %], [% guarantor.firstname | html %]</a></li>
>+            [% END %]
>+        </ol>
>+    </div>
>+</div>
>+</div>
>+
>+    <div class="col-xs-6">
>+    <div id="patron-library-details">
>+    <h3>Library use</h3>
>+    <div class="rows">
>+        <ol>
>+            <li><span class="label">Card number: </span>[% patron.cardnumber | html %]</li>
>+            <li><span class="label">Borrowernumber: </span> [% patron.borrowernumber | html %]</li>
>+            <li><span class="label">Category: </span>[% patron.category.description | html %] ([% patron.category.categorycode | html %])</li>
>+            <li><span class="label">Registration date: </span>[% patron.dateenrolled | $KohaDates %]</li>
>+            <li><span class="label">Expiration date: </span>
>+            [% IF ( was_renewed ) %]
>+                <strong class="reregistrinfo">[% patron.dateexpiry | $KohaDates %]</strong>
>+            [% ELSE %]
>+                [% patron.dateexpiry | $KohaDates %]
>+            [% END %]
>+            </li>
>+            <li><span class="label">Library: </span>[% patron.library.branchname | html %]</li>
>+            [% IF ( patron.sort1 ) %]<li><span class="label">Sort field 1:</span>[% AuthorisedValues.GetByCode('Bsort1', patron.sort1) | html %]</li>[% END %]
>+            [% IF ( patron.sort2 ) %]<li><span class="label">Sort field 2:</span>[% AuthorisedValues.GetByCode('Bsort2', patron.sort2) | html %]</li>[% END %]
>+        </ol>
>+    </div>
>+    </div>
>+    </div>
>+</div>
>+<div class="row">
>+    <div class="col-xs-12">
>+        <fieldset class="action"><input type="button" class="close" value="Close window" /></fieldset>
>+    </div>
>+</div>
>+
>+[% INCLUDE 'intranet-bottom.inc' popup_window=1 %]
>diff --git a/members/moremember.pl b/members/moremember.pl
>index d68e6ef146..a874c2d1c9 100755
>--- a/members/moremember.pl
>+++ b/members/moremember.pl
>@@ -53,9 +53,20 @@ BEGIN {
> my $input = CGI->new;
> $debug or $debug = $input->param('debug') || 0;
> 
>+
>+my $print = $input->param('print');
>+
>+my $template_name;
>+
>+if (defined $print and $print eq "brief") {
>+        $template_name = "members/moremember-brief.tt";
>+} else {
>+        $template_name = "members/moremember.tt";
>+}
>+
> my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
>     {
>-        template_name   => "members/moremember.tt",
>+        template_name   => $template_name,
>         query           => $input,
>         type            => "intranet",
>         authnotrequired => 0,
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 21757: 81916 | 85782 | 88007 | 88058 | 88060 | 88146 | 88268 | 88595 | 88596 | 88597 | 88643 | 88644 | 88645