With the addition of opac-memberentry.pl and the patron update moderation system in the staff client the old system should be removed: opac-userdetails.pl and opac-userupdate.pl. Several OPAC templates link to these pages even though they should be obsolete.
Created attachment 17964 [details] [review] Bug 10201 - Old OPAC patron update system should be removed The old pages for viewing and updating patron details in the OPAC have been superceded by the new script opac-memberentry.pl. This patch removes he old scripts and templates and corrects links to them. This patch also removes reference to opac-userupdate.tt from opac-patron-image.pl and replaces the authentication process with one which uses check_cookie_auth, based on the example of opac-tags.pl. To test, edit a patron record and set the "Gone no address" flag. Log in to the OPAC with that account and view the patron details page. The warning about out of date contact information should link to the new update page. Next, attempt to place a hold. You should see the same warning, and it should also link to the new update page. Test the display of patron images: Log in as a user who has an image associated with their account and navigate to /cgi-bin/koha/opac-patron-image.pl. Their patron image should display. A search of Koha source files should return no results for the missing scripts or templates.
Created attachment 17977 [details] [review] [SIGNED OFF] Bug 10201 - Old OPAC patron update system should be removed The old pages for viewing and updating patron details in the OPAC have been superceded by the new script opac-memberentry.pl. This patch removes he old scripts and templates and corrects links to them. This patch also removes reference to opac-userupdate.tt from opac-patron-image.pl and replaces the authentication process with one which uses check_cookie_auth, based on the example of opac-tags.pl. To test, edit a patron record and set the "Gone no address" flag. Log in to the OPAC with that account and view the patron details page. The warning about out of date contact information should link to the new update page. Next, attempt to place a hold. You should see the same warning, and it should also link to the new update page. Test the display of patron images: Log in as a user who has an image associated with their account and navigate to /cgi-bin/koha/opac-patron-image.pl. Their patron image should display. A search of Koha source files should return no results for the missing scripts or templates. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Created attachment 17993 [details] [review] Bug 10201 [Follow-up] Old OPAC patron update system should be removed The previous patch missed removal of one template, kohaerror.tt, which was used only by opac-userupdate.pl. This patch removes it. To test, apply the patch and search Koha source files for references to kohaerror. There should be none.
Created attachment 18000 [details] [review] Bug 10201 [Follow-up] Old OPAC patron update system should be removed The previous patch missed removal of one template, kohaerror.tt, which was used only by opac-userupdate.pl. This patch removes it. To test, apply the patch and search Koha source files for references to kohaerror. There should be none. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Created attachment 18007 [details] [review] [PASSED QA] Bug 10201 [Follow-up] Old OPAC patron update system should be removed The previous patch missed removal of one template, kohaerror.tt, which was used only by opac-userupdate.pl. This patch removes it. To test, apply the patch and search Koha source files for references to kohaerror. There should be none. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All tests and QA script pass. Updating the German po files with this patch applied no new strings appeared.
Created attachment 18008 [details] [review] [PASSED QA] Bug 10201 - Old OPAC patron update system should be removed The old pages for viewing and updating patron details in the OPAC have been superceded by the new script opac-memberentry.pl. This patch removes he old scripts and templates and corrects links to them. This patch also removes reference to opac-userupdate.tt from opac-patron-image.pl and replaces the authentication process with one which uses check_cookie_auth, based on the example of opac-tags.pl. To test, edit a patron record and set the "Gone no address" flag. Log in to the OPAC with that account and view the patron details page. The warning about out of date contact information should link to the new update page. Next, attempt to place a hold. You should see the same warning, and it should also link to the new update page. Test the display of patron images: Log in as a user who has an image associated with their account and navigate to /cgi-bin/koha/opac-patron-image.pl. Their patron image should display. A search of Koha source files should return no results for the missing scripts or templates. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Comments on second patch.
Created attachment 18009 [details] [review] [PASSED QA] Bug 10201 [Follow-up] Old OPAC patron update system should be removed The previous patch missed removal of one template, kohaerror.tt, which was used only by opac-userupdate.pl. This patch removes it. To test, apply the patch and search Koha source files for references to kohaerror. There should be none. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All tests and QA script pass. Updating the German po files with this patch applied no new strings appeared.
This patch has been pushed to master and 3.12.x.
Not needed for 3.10.x and below