Bug 9384 - With OPACPatronDetails set to 'don't allow' a form shows
Summary: With OPACPatronDetails set to 'don't allow' a form shows
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Kyle M Hall
QA Contact: Jonathan Druart
URL: cgi-bin/koha/opac-memberentry.pl
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-14 17:24 UTC by Nicole C. Engard
Modified: 2016-12-05 21:22 UTC (History)
3 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:


Attachments
Bug 9384 - With OPACPatronDetails set to 'don't allow' a form shows (2.90 KB, patch)
2013-01-15 12:53 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 9384 - With OPACPatronDetails set to 'don't allow' a form shows (3.26 KB, patch)
2013-01-15 12:56 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 9384 - With OPACPatronDetails set to 'don't allow' a form shows (3.34 KB, patch)
2013-01-31 00:18 UTC, Liz Rea
Details | Diff | Splinter Review
Bug 9384 - With OPACPatronDetails set to 'don't allow' a form shows (3.40 KB, patch)
2013-01-31 08:52 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nicole C. Engard 2013-01-14 17:24:17 UTC
In master because of the new feature to allow patrons to register for a card via the opac when you have  OPACPatronDetails  set to 'don't allow' it still shows a from for the patron details, but you can't edit the form. It would be better if this showed as a text display like it used to so that it's not confusing to patrons (even with the note about seeing the librarian).

Nicole
Comment 1 Kyle M Hall 2013-01-14 19:46:12 UTC
The code for the old form was never removed so it should be trivial to enable Koha to pick which one to show based on whether or not OPAC registration is enabled.
Comment 2 Kyle M Hall 2013-01-15 12:53:17 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall 2013-01-15 12:56:18 UTC Comment hidden (obsolete)
Comment 4 Liz Rea 2013-01-31 00:18:07 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2013-01-31 08:52:03 UTC
QA comment:
Works as expected, simple patch.
Marked as Passed QA.
Comment 6 Jonathan Druart 2013-01-31 08:52:18 UTC
Created attachment 14971 [details] [review]
Bug 9384 - With OPACPatronDetails set to 'don't allow' a form shows

Because of the new feature to allow patrons to register for a card
via the opac when you have OPACPatronDetails  set to 'don't allow'
it still shows a from for the patron details, but you can't edit the form.
It would be better if this showed as a text display like it used to so
that it's not confusing to patrons ( even with the note about seeing
the librarian ).

Test Plan:
1) Apply patch
2) Log into OPAC
3) Set OPACPatronDetails to "Allow"
4) Refresh the page and click "my personal details"
5) Verify you get the editable form ( opac-memberentry.pl )
6) Set OPACPatronDetails to "Don't allow"
7) Refresh the page and click "my personal details" again
8) Verify you get the non-editable non-form details page ( opac-userupdate.pl )

Signed-off-by: Liz Rea <liz@catalyst.net.nz>
Passes tests, works as advertised

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 7 Jared Camins-Esakov 2013-01-31 19:30:47 UTC
This patch works as far as it goes, but it does not add any check to opac-memberentry.pl to redirect the user if they end up on that page with OPACPatronDetails set to "Don't allow," and it really needs to before I push it.
Comment 8 Jonathan Druart 2013-02-01 08:32:47 UTC
(In reply to comment #7)
> This patch works as far as it goes, but it does not add any check to
> opac-memberentry.pl to redirect the user if they end up on that page with
> OPACPatronDetails set to "Don't allow," and it really needs to before I push
> it.

So I think the followup will delete some tests in the opac-memberentry.tt template ([% IF OPACPatronDetails %]).
Comment 9 Marc Véron 2016-05-23 15:20:58 UTC
Is this bug still valid?
Comment 10 Owen Leonard 2016-05-23 15:37:10 UTC
As far as I can tell this is no longer valid.