Bug 34889 - Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization
Summary: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTM...
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Owen Leonard
QA Contact: Katrin Fischer
URL:
Keywords:
Depends on: 31383 34869
Blocks: 26050 34894
  Show dependency treegraph
 
Reported: 2023-09-22 15:17 UTC by Owen Leonard
Modified: 2023-12-11 21:03 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact: Caroline Cyr La Rose
Documentation submission: https://gitlab.com/koha-community/koha-manual/-/merge_requests/777
Text to go in the release notes:
Version(s) released in:
23.11.00
Circulation function:


Attachments
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (10.04 KB, patch)
2023-09-22 16:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (10.09 KB, patch)
2023-09-25 00:00 UTC, David Nind
Details | Diff | Splinter Review
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (10.18 KB, patch)
2023-10-07 14:54 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (9.97 KB, patch)
2023-10-20 18:38 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: (follow-up) Revise database update following Bug 31383 (2.45 KB, patch)
2023-10-25 15:04 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: (follow-up) Add entry to HTML customization help include (1.12 KB, patch)
2023-10-25 15:13 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (10.24 KB, patch)
2023-10-25 15:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: (follow-up) Revise database update following Bug 31383 (2.45 KB, patch)
2023-10-25 15:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: (follow-up) Add entry to HTML customization help include (1.12 KB, patch)
2023-10-25 15:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (10.24 KB, patch)
2023-10-26 11:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: (follow-up) Revise database update following Bug 31383 (2.73 KB, patch)
2023-10-26 11:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: (follow-up) Add entry to HTML customization help include (1.12 KB, patch)
2023-10-26 11:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization (10.24 KB, patch)
2023-11-05 11:01 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 34889: (follow-up) Revise database update following Bug 31383 (2.78 KB, patch)
2023-11-05 11:01 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 34889: (follow-up) Add entry to HTML customization help include (1.18 KB, patch)
2023-11-05 11:01 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2023-09-22 15:17:19 UTC
Building on 22318, the PatronSelfRegistrationAdditionalInstructions system preference should be converted to an HTML customization option.
Comment 1 Owen Leonard 2023-09-22 16:03:25 UTC Comment hidden (obsolete)
Comment 2 Lucas Gass (lukeg) 2023-09-24 15:11:02 UTC
Owen,

I am having trouble getting content to appear when language packs are installed.

I have installed both fr-FR and es-ES, turned them on, and added content in the right place for that language. Then I am switching to that language and registering. 

Maybe I am missing something but can you check once more?
Comment 3 David Nind 2023-09-25 00:00:07 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2023-10-07 14:54:15 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2023-10-07 14:55:08 UTC
(In reply to Lucas Gass from comment #2)
> Owen,
> 
> I am having trouble getting content to appear when language packs are
> installed.
> 
> I have installed both fr-FR and es-ES, turned them on, and added content in
> the right place for that language. Then I am switching to that language and
> registering. 
> 
> Maybe I am missing something but can you check once more?

I did some double checking and it seemed to work all ok. Some thing to keep in mind is that we need to set a publication date for things to show up at all. That's something that trapped me at first.
Comment 6 Tomás Cohen Arazi (tcohen) 2023-10-20 18:05:19 UTC
Please rebase ASAP. Thanks!
Comment 7 Owen Leonard 2023-10-20 18:38:52 UTC Comment hidden (obsolete)
Comment 8 Tomás Cohen Arazi (tcohen) 2023-10-20 19:30:19 UTC
Hi, I'm sorry to say that having pushed bug 31383 made this require some more work on the atomicupdate, as the language-specific portion of the `additional_contents` table has been moved into `additional_contents_localizations`.
Comment 9 Tomás Cohen Arazi (tcohen) 2023-10-25 12:56:15 UTC
PIng
Comment 10 Owen Leonard 2023-10-25 15:04:50 UTC Comment hidden (obsolete)
Comment 11 Tomás Cohen Arazi (tcohen) 2023-10-25 15:12:56 UTC
Look at my reason to fail on bug 34869.
Comment 12 Owen Leonard 2023-10-25 15:13:54 UTC Comment hidden (obsolete)
Comment 13 Owen Leonard 2023-10-25 15:54:54 UTC
Created attachment 157826 [details] [review]
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization

This patch moves the PatronSelfRegistrationAdditionalInstructions system
preference into HTML customizations, making it possible to have
language- and library-specific content.

To test you should have some content in the
PatronSelfRegistrationAdditionalInstructions system preference before
applying the patch. Apply the patch and run the database update process.

 - In the staff client, go to Tools -> HTML customizations and verify
   that the content previously in the
   PatronSelfRegistrationAdditionalInstructions preference is
   now stored there.
 - The HTML customization entry form should offer
   PatronSelfRegistrationAdditionalInstructions as a choice under
   "Display location."
 - Go to Administration -> System preferences and enable
   "PatronSelfRegistration." Set "PatronSelfRegistrationVerifyByEmail"
   to "Don't require."
 - Go to the OPAC and click "Register here."
 - Submit a new registration and confirm that the content that was
   previously stored in the system preference is shown correctly on the
   registration confirmation page.
 - Update and reinstall active translations (for instance fr-FR):
   - cd misc/translator/
   - perl translate update fr-FR
   - perl translate install fr-FR
 - Enable the translation if necessary under Administration -> System
   preferences -> language.
 - Enable the "opaclanguagesdisplay" preference if necessary.
 - Edit the PatronSelfRegistrationAdditionalInstructions HTML
   customization and add unique content to the "fr-FR" tab.
 - Go to the OPAC and switch to your updated translation.
   Complete the online registration process again to confirm that the
   content you added for your translation shows up correctly on the
   confirmation page.
 - Go to Administration -> System preferences and confirm that the
   PatronSelfRegistrationAdditionalInstructions  preference has been
   removed.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 14 Owen Leonard 2023-10-25 15:54:56 UTC
Created attachment 157827 [details] [review]
Bug 34889: (follow-up) Revise database update following Bug 31383
Comment 15 Owen Leonard 2023-10-25 15:54:58 UTC
Created attachment 157828 [details] [review]
Bug 34889: (follow-up) Add entry to HTML customization help include
Comment 16 Owen Leonard 2023-10-26 11:54:36 UTC
Created attachment 157910 [details] [review]
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization

This patch moves the PatronSelfRegistrationAdditionalInstructions system
preference into HTML customizations, making it possible to have
language- and library-specific content.

To test you should have some content in the
PatronSelfRegistrationAdditionalInstructions system preference before
applying the patch. Apply the patch and run the database update process.

 - In the staff client, go to Tools -> HTML customizations and verify
   that the content previously in the
   PatronSelfRegistrationAdditionalInstructions preference is
   now stored there.
 - The HTML customization entry form should offer
   PatronSelfRegistrationAdditionalInstructions as a choice under
   "Display location."
 - Go to Administration -> System preferences and enable
   "PatronSelfRegistration." Set "PatronSelfRegistrationVerifyByEmail"
   to "Don't require."
 - Go to the OPAC and click "Register here."
 - Submit a new registration and confirm that the content that was
   previously stored in the system preference is shown correctly on the
   registration confirmation page.
 - Update and reinstall active translations (for instance fr-FR):
   - cd misc/translator/
   - perl translate update fr-FR
   - perl translate install fr-FR
 - Enable the translation if necessary under Administration -> System
   preferences -> language.
 - Enable the "opaclanguagesdisplay" preference if necessary.
 - Edit the PatronSelfRegistrationAdditionalInstructions HTML
   customization and add unique content to the "fr-FR" tab.
 - Go to the OPAC and switch to your updated translation.
   Complete the online registration process again to confirm that the
   content you added for your translation shows up correctly on the
   confirmation page.
 - Go to Administration -> System preferences and confirm that the
   PatronSelfRegistrationAdditionalInstructions  preference has been
   removed.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 17 Owen Leonard 2023-10-26 11:54:39 UTC
Created attachment 157911 [details] [review]
Bug 34889: (follow-up) Revise database update following Bug 31383
Comment 18 Owen Leonard 2023-10-26 11:54:41 UTC
Created attachment 157912 [details] [review]
Bug 34889: (follow-up) Add entry to HTML customization help include
Comment 19 Katrin Fischer 2023-11-05 11:01:22 UTC
Created attachment 158447 [details] [review]
Bug 34889: Convert PatronSelfRegistrationAdditionalInstructions system preference to HTML customization

This patch moves the PatronSelfRegistrationAdditionalInstructions system
preference into HTML customizations, making it possible to have
language- and library-specific content.

To test you should have some content in the
PatronSelfRegistrationAdditionalInstructions system preference before
applying the patch. Apply the patch and run the database update process.

 - In the staff client, go to Tools -> HTML customizations and verify
   that the content previously in the
   PatronSelfRegistrationAdditionalInstructions preference is
   now stored there.
 - The HTML customization entry form should offer
   PatronSelfRegistrationAdditionalInstructions as a choice under
   "Display location."
 - Go to Administration -> System preferences and enable
   "PatronSelfRegistration." Set "PatronSelfRegistrationVerifyByEmail"
   to "Don't require."
 - Go to the OPAC and click "Register here."
 - Submit a new registration and confirm that the content that was
   previously stored in the system preference is shown correctly on the
   registration confirmation page.
 - Update and reinstall active translations (for instance fr-FR):
   - cd misc/translator/
   - perl translate update fr-FR
   - perl translate install fr-FR
 - Enable the translation if necessary under Administration -> System
   preferences -> language.
 - Enable the "opaclanguagesdisplay" preference if necessary.
 - Edit the PatronSelfRegistrationAdditionalInstructions HTML
   customization and add unique content to the "fr-FR" tab.
 - Go to the OPAC and switch to your updated translation.
   Complete the online registration process again to confirm that the
   content you added for your translation shows up correctly on the
   confirmation page.
 - Go to Administration -> System preferences and confirm that the
   PatronSelfRegistrationAdditionalInstructions  preference has been
   removed.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 20 Katrin Fischer 2023-11-05 11:01:25 UTC
Created attachment 158448 [details] [review]
Bug 34889: (follow-up) Revise database update following Bug 31383

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 21 Katrin Fischer 2023-11-05 11:01:28 UTC
Created attachment 158449 [details] [review]
Bug 34889: (follow-up) Add entry to HTML customization help include

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 22 Tomás Cohen Arazi (tcohen) 2023-11-08 20:42:49 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 23 Fridolin Somers 2023-11-11 01:19:43 UTC
Enhancement not pushed to 23.05.x