Bugzilla – Attachment 12109 Details for
Bug 7067
allow patron self registration via the opac
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 7067 - OPAC Borrower Self Registration - Followup
Bug-7067---OPAC-Borrower-Self-Registration---Follo.patch (text/plain), 13.88 KB, created by
Kyle M Hall (khall)
on 2012-09-10 15:46:27 UTC
(
hide
)
Description:
Bug 7067 - OPAC Borrower Self Registration - Followup
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2012-09-10 15:46:27 UTC
Size:
13.88 KB
patch
obsolete
>From 1e647bfac171294c4ac474c9a597370984533b57 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Mon, 10 Sep 2012 08:06:09 -0400 >Subject: [PATCH] Bug 7067 - OPAC Borrower Self Registration - Followup >Content-Type: text/plain; charset="utf-8" > >--- > .../prog/en/modules/members/members-update.tt | 113 ++++++++++---------- > .../opac-tmpl/prog/en/modules/opac-memberentry.tt | 26 ++++- > members/members-update-do.pl | 2 +- > opac/opac-memberentry.pl | 22 ++-- > 4 files changed, 92 insertions(+), 71 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt >index 8878b7f..4495054 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt >@@ -64,72 +64,75 @@ > <div id="yui-main"> > <div class="yui-b"> > <div class="yui-g"> >+ [% IF PendingModifications %] >+ <form method="post" action="members-update-do.pl"> > >- <form method="post" action="members-update-do.pl"> >- >- <table> >- <thead> >- <tr> >- <th colspan="3">Action</th> >- <th rowspan="2">Patron</th> >- <th rowspan="2">Changes</th> >- </tr> >- >- <tr> >- <th>Approve</th> >- <th>Deny</th> >- <th>Ignore</th> >- </tr> >- </thead> >+ <table> >+ <thead> >+ <tr> >+ <th colspan="3">Action</th> >+ <th rowspan="2">Patron</th> >+ <th rowspan="2">Changes</th> >+ </tr> > >- <tbody> >- [% FOREACH pm IN PendingModifications %] >- [% SET borrowernumber = pm.borrowernumber %] > <tr> >- <td> >- <input type="radio" name="modify_[% pm.borrowernumber %]" value="approve" /> >- </td> >- <td> >- <input type="radio" name="modify_[% pm.borrowernumber %]" value="deny" /> >- </td> >- <td> >- <input type="radio" name="modify_[% pm.borrowernumber %]" value="ignore" checked="checked"/> >- </td> >+ <th>Approve</th> >+ <th>Deny</th> >+ <th>Ignore</th> >+ </tr> >+ </thead> >+ >+ <tbody> >+ [% FOREACH pm IN PendingModifications %] >+ [% SET borrowernumber = pm.borrowernumber %] >+ <tr> >+ <td> >+ <input type="radio" name="modify_[% pm.borrowernumber %]" value="approve" /> >+ </td> >+ <td> >+ <input type="radio" name="modify_[% pm.borrowernumber %]" value="deny" /> >+ </td> >+ <td> >+ <input type="radio" name="modify_[% pm.borrowernumber %]" value="ignore" checked="checked"/> >+ </td> > >- <td> >- [% borrowers.$borrowernumber.firstname %] [% borrowers.$borrowernumber.surname %] >- </td> >+ <td> >+ [% borrowers.$borrowernumber.firstname %] [% borrowers.$borrowernumber.surname %] >+ </td> > >- <td> >- <table> >- <tr> >- <th>Field</th> >- <th>From</th> >- <th>To</th> >- </tr> >+ <td> >+ <table> >+ <tr> >+ <th>Field</th> >+ <th>From</th> >+ <th>To</th> >+ </tr> > > >- [% FOREACH key IN pm.keys %] >- [% IF field_display_names.$key %] >- [% IF ( ( pm.$key OR borrowers.$borrowernumber.$key ) && ( pm.$key != borrowers.$borrowernumber.$key ) ) %] >- <tr> >- <td>[% field_display_names.$key %]</td> >- <td>[% borrowers.$borrowernumber.$key %]</td> >- <td>[% pm.$key %]</td> >- </td> >+ [% FOREACH key IN pm.keys %] >+ [% IF field_display_names.$key %] >+ [% IF ( ( pm.$key OR borrowers.$borrowernumber.$key ) && ( pm.$key != borrowers.$borrowernumber.$key ) ) %] >+ <tr> >+ <td>[% field_display_names.$key %]</td> >+ <td>[% borrowers.$borrowernumber.$key %]</td> >+ <td>[% pm.$key %]</td> >+ </td> >+ [% END %] > [% END %] > [% END %] >- [% END %] >- </table> >- </td> >- </tr> >- [% END %] >- </tbody> >- </table> >+ </table> >+ </td> >+ </tr> >+ [% END %] >+ </tbody> >+ </table> > >- <p><input type="submit" /></p> >+ <p><input type="submit" /></p> > >- </form> >+ </form> >+ [% ELSE %] >+ <p>There are no pending patron modifications.</p> >+ [% END %] > </div> > </div> > </div> >diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt >index 38c0bc7..fab3ff5 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt >@@ -7,7 +7,16 @@ > > <script type="text/javascript"> > $(document).ready(function() { >- $( "#borrower_dateofbirth" ).datepicker({ yearRange: "c-120:c" }); >+ [% IF OPACPatronDetails %] >+ $( "#borrower_dateofbirth" ).datepicker({ yearRange: "c-120:c" }); >+ [% ELSE %] >+ $("#memberentry-form :input").attr('readonly', true); >+ $("#borrower_branchcode").attr('disabled',true); >+ $("#borrower_title").attr('disabled',true); >+ $('#memberentry-form :radio').attr('disabled',true); >+ $('span.required').remove(); >+ $('label.required').removeClass('required'); >+ [% END %] > }); > </script> > </head> >@@ -21,6 +30,9 @@ > <div class="yui-b"> > <div class="yui-g"> > <div id="useraccount" class="container"> >+ [% UNLESS OPACPatronDetails %] >+ <div>To make changes to your record please contact the library.</div> >+ [% END %] > > [% IF empty_mandatory_fields %] > <div class="dialog alert">You have not filled out all required fields. Please fill in all missing fields and resubmit.</div> >@@ -30,7 +42,7 @@ > <div class="dialog alert">You typed in the wrong characters in the box before submitting. Please try again.</div> > [% END %] > >- <form method="post"> >+ <form method="post" id="memberentry-form"> > > [% UNLESS > hidden.defined('branchcode') >@@ -134,7 +146,9 @@ > > <input type="text" id="borrower_dateofbirth" name="borrower_dateofbirth" value="[% borrower.dateofbirth | $KohaDates %]" readonly="readonly" size="10" /> > >- <a href="#" style="font-size:85%;text-decoration:none;" onclick="document.getElementById('borrower_dateofbirth').value='';return false;">Clear date</a><p></p> >+ [% IF OPACPatronDetails %] >+ <a href="#" style="font-size:85%;text-decoration:none;" onclick="document.getElementById('borrower_dateofbirth').value='';return false;">Clear date</a><p></p> >+ [% END %] > > [% IF mandatory.defined('dateofbirth') %]<span class="required">Required</span>[% END %] > </li> >@@ -690,8 +704,10 @@ > [% END %] > > [% IF action == 'edit' %] >- <input type="hidden" name="action" value="update" /> >- <input type="submit" value="Submit Update Request" /> >+ [% IF OPACPatronDetails %] >+ <input type="hidden" name="action" value="update" /> >+ <input type="submit" value="Submit Update Request" /> >+ [% END %] > [% ELSE %] > <input type="hidden" name="action" value="create" /> > <input type="submit" value="Submit" /> >diff --git a/members/members-update-do.pl b/members/members-update-do.pl >index 147975a..e7b7b28 100755 >--- a/members/members-update-do.pl >+++ b/members/members-update-do.pl >@@ -61,4 +61,4 @@ foreach my $param (@params) { > } > } > >-print $query->redirect("/cgi-bin/koha/mainpage.pl"); >+print $query->redirect("/cgi-bin/koha/members/members-update.pl"); >diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl >index cf24176..36ddd24 100755 >--- a/opac/opac-memberentry.pl >+++ b/opac/opac-memberentry.pl >@@ -30,11 +30,6 @@ use C4::Branch qw(GetBranchesLoop); > my $cgi = new CGI; > my $dbh = C4::Context->dbh; > >-unless ( C4::Context->preference('PatronSelfRegistration') ) { >- print $cgi->redirect("/cgi-bin/koha/opac-main.pl"); >- exit; >-} >- > my ( $template, $borrowernumber, $cookie ) = get_template_and_user( > { > template_name => "opac-memberentry.tmpl", >@@ -44,11 +39,17 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user( > } > ); > >+unless ( C4::Context->preference('PatronSelfRegistration') || $borrowernumber ) { >+ print $cgi->redirect("/cgi-bin/koha/opac-main.pl"); >+ exit; >+} >+ > $template->param( >- hidden => GetHiddenFields(), >- mandatory => GetMandatoryFields(), >- member_titles => GetTitles(), >- branches => GetBranchesLoop() >+ hidden => GetHiddenFields(), >+ mandatory => GetMandatoryFields(), >+ member_titles => GetTitles(), >+ branches => GetBranchesLoop(), >+ OPACPatronDetails => C4::Context->preference('OPACPatronDetails'), > ); > > my $action = $cgi->param('action') || q{}; >@@ -167,7 +168,8 @@ elsif ($borrowernumber) { #Display logged in borrower's data > $action = 'edit'; > > $template->param( >- borrower => GetMember( borrowernumber => $borrowernumber ) ); >+ borrower => GetMember( borrowernumber => $borrowernumber ), >+ ); > } > > my $captcha = random_string("CCCCC"); >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 7067
:
11960
|
11961
|
11962
|
11965
|
11966
|
11967
|
11968
|
11970
|
11973
|
11978
|
12006
|
12007
|
12013
|
12104
|
12109
|
12114
|
12116
|
12129
|
12135
|
12136
|
12138
|
12153
|
12154
|
12171
|
12190
|
12191
|
12196
|
12197
|
12242
|
12243
|
12244
|
12245
|
12277
|
12278
|
13038
|
13039
|
13040
|
13048
|
13053
|
13259
|
13260
|
13744
|
13745
|
13746
|
13747
|
13748
|
13765
|
13766
|
13767
|
13768
|
13769
|
13984
|
14001
|
14085
|
14086
|
14087
|
14088
|
14089
|
14090
|
14092
|
14093
|
14119
|
14190
|
14191
|
14193
|
14194
|
14282
|
14283