Bug 27681

Summary: Style OPAC self-registration CAPTCHA as uppercase
Product: Koha Reporter: Owen Leonard <oleonard>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: barbara.johnson, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This changes the CAPTCHA field in the OPAC self-registration form to automatically change any character entered to be upper case.
Version(s) released in:
21.05.00
Circulation function:
Bug Depends on: 27668    
Bug Blocks:    
Attachments: Bug 27681: Style OPAC self-registration CAPTCHA as uppercase
Bug 27681: Style OPAC self-registration CAPTCHA as uppercase
Bug 27681: Style OPAC self-registration CAPTCHA as uppercase

Description Owen Leonard 2021-02-11 13:06:29 UTC
Although the OPAC self-registration CAPTCHA is case-insensitive (Bug 16283), there's no way for the user to know that. Since the CAPTCHA is always in all uppercase we should make it easier on the user by adding "text-transform:uppercase" to the form field so that they don't need to physically make the text uppercase.
Comment 1 Owen Leonard 2021-02-11 13:25:41 UTC
Created attachment 116715 [details] [review]
Bug 27681: Style OPAC self-registration CAPTCHA as uppercase

This patch adds an inline "text-transform:uppercase" style attribute to
the OPAC self-registration CAPTCHA field so that the user doesn't have
to match the case of the CAPTCHA they're shown.

To test, apply the patch and start the process of self-registration in
the OPAC.

- At the end of the form, under verification, confirm that any text you
  type in the verification field is uppercase by default.
- Confirm that the CAPTCHA is accepted when you submit the form.
Comment 2 Barbara Johnson 2021-02-12 17:28:52 UTC
Text entered in the CAPTCHA field is all uppercase.  But when I submit the form I'm getting an error - Branchcode not allowed at /kohadevbox/koha/opac/opac-memberentry.pl line 160 followed by 52 more lines of stuff.  I had this problem recently with another bug so maybe I'm doing something wrong.
Comment 3 ByWater Sandboxes 2021-02-19 19:40:37 UTC
Created attachment 117095 [details] [review]
Bug 27681: Style OPAC self-registration CAPTCHA as uppercase

This patch adds an inline "text-transform:uppercase" style attribute to
the OPAC self-registration CAPTCHA field so that the user doesn't have
to match the case of the CAPTCHA they're shown.

To test, apply the patch and start the process of self-registration in
the OPAC.

- At the end of the form, under verification, confirm that any text you
  type in the verification field is uppercase by default.
- Confirm that the CAPTCHA is accepted when you submit the form.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Comment 4 Barbara Johnson 2021-02-19 19:41:21 UTC
I re-tested this two times and it worked as described so I signed off.
Comment 5 Katrin Fischer 2021-03-04 12:53:52 UTC
The depending bug is missing sign-off, so is blocking this from QA for the moment.
Comment 6 Katrin Fischer 2021-03-04 13:00:50 UTC
Created attachment 117717 [details] [review]
Bug 27681: Style OPAC self-registration CAPTCHA as uppercase

This patch adds an inline "text-transform:uppercase" style attribute to
the OPAC self-registration CAPTCHA field so that the user doesn't have
to match the case of the CAPTCHA they're shown.

To test, apply the patch and start the process of self-registration in
the OPAC.

- At the end of the form, under verification, confirm that any text you
  type in the verification field is uppercase by default.
- Confirm that the CAPTCHA is accepted when you submit the form.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Katrin Fischer 2021-03-04 13:02:30 UTC
Hm, as I had already spent quite some time here, I decided to SO the dependency and move this to PQA - not the dependency please.
Comment 8 Jonathan Druart 2021-03-09 16:20:04 UTC
Pushed to master for 21.05, thanks to everybody involved!