Building on 22318, the OPACMySummaryNote system preference should be converted to a Koha news block.
Created attachment 131204 [details] [review] Bug 24221: Move OPACMySummaryNote to HTML customizations in additional-contents To test: 1- Before applying the patch add some content to the system preference OPACMySummaryNote 2- Apply patch 3- restart_all and updatedatabase 4. Go to additional-content.pl ( HTML customizations ) 5. You should now see your content from the system preference under OpacMySummaryNote in HTML customizations 6. Try searching for the system preference, OPACMySummaryNote. It has been removed and you should not be able to find it. 7. Try changing/removing/adding content from OpacMySummaryNote in HTML customizations 8. Make sure your edits are reflected when you look at the user summary page Note: The system preference was named OPACMySummaryNote but in to be more consitent with how we have named other additional_contents I have renamed this OpacMySummaryNote.
Patch doesn't apply. At step 7, content doesn't appear when saving the new entry in Additional contents.
(In reply to shi-yao.wang from comment #2) > Patch doesn't apply. At step 7, content doesn't appear when saving the new > entry in Additional contents. This patch does cleanly apply to current master. As for step 7, I suspect your not adding a title. When I do not add a title to an HTML customization the content does not appear, that is probably a bug but I would request that you file separate bug report. Thanks for testing! Resetting this to NSO.
Created attachment 131581 [details] [review] Bug 24221: Move OPACMySummaryNote to HTML customizations in additional-contents To test: 1- Before applying the patch add some content to the system preference OPACMySummaryNote 2- Apply patch 3- restart_all and updatedatabase 4. Go to additional-content.pl ( HTML customizations ) 5. You should now see your content from the system preference under OpacMySummaryNote in HTML customizations 6. Try searching for the system preference, OPACMySummaryNote. It has been removed and you should not be able to find it. 7. Try changing/removing/adding content from OpacMySummaryNote in HTML customizations 8. Make sure your edits are reflected when you look at the user summary page Note: The system preference was named OPACMySummaryNote but in to be more consitent with how we have named other additional_contents I have renamed this OpacMySummaryNote. Signed-off-by: shiyao <shiyao@inlibro.com>
Shiyao, I have created a new bug report for the problem you were experiencing: See Bug 30272
Created attachment 132013 [details] [review] Bug 24221: Move OPACMySummaryNote to HTML customizations in additional-contents To test: 1- Before applying the patch add some content to the system preference OPACMySummaryNote 2- Apply patch 3- restart_all and updatedatabase 4. Go to additional-content.pl ( HTML customizations ) 5. You should now see your content from the system preference under OpacMySummaryNote in HTML customizations 6. Try searching for the system preference, OPACMySummaryNote. It has been removed and you should not be able to find it. 7. Try changing/removing/adding content from OpacMySummaryNote in HTML customizations 8. Make sure your edits are reflected when you look at the user summary page Note: The system preference was named OPACMySummaryNote but in to be more consitent with how we have named other additional_contents I have renamed this OpacMySummaryNote. Signed-off-by: shiyao <shiyao@inlibro.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 132014 [details] [review] Bug 24221: Improve atomic update Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Like other conversions, I've added to DBRev" : No OpacMySummaryNote preference found. Value was empty or update has already been run. I add a follow-up to remove the HTML id, like done in Bug 28927 "Fix id opacmainuserblock used twice in OPAC"
Created attachment 132264 [details] [review] Bug 24221: (follow-up) Remove id opac-my-summary-note Like in other conversions to news block, we do not need an HTML id in the template, one is created by PROCESS koha_news_block
Pushed to master for 22.05, thanks to everybody involved [U+1F984]