Hi all, I don't know if I'm just not doing it right, but I cannot seem to be able to delete an HTML customization. To replicate : 1. Go to Tools > HTML customizations 2. Click on New entry 3. Fill out the form (I tried it with OpacSuggestionInstructions and OpacLoginInstructions) 4. Click Save 5. In the list of HTML customizations, click on the Delete button at the end of the row -> Note that it goes back to Additional contents (News) instead of Additional contents (HTML customizations) 6. Go back to Tools > HTML customizations -> Note that the entry you deleted is still there Since it goes back to Additional contents (News) after deleting, I'm wondering if it has something to do with the separation of News and HTML customizations... Caroline
Created attachment 125223 [details] [review] Bug 29019: Unable to delete HTML customization This patch makes some corrections to additional contents to allow content to be deleted. The wrong parameter was being passed to the script. The script was also not handling multiple deletions correctly. The patch also adds a "category" parameter to the delete operation so that the page is redirected correctly. The patch also changes some strings which referred to "news" referencing operations which might be performed on news or on HTML customizations, e.g.: "Are you sure you want to delete the selected content?" instead of "Are you sure you want to delete the selected news?" To test, apply the patch and go to Tools -> News. - Create multiple news items if necessary. - Test the "Delete" button corresponding to a single news item: - Clicking the button should ask you to confirm. - Check that the wording of the message is correct. - After confirming the news item should be deleted. - Ideally, test on news items which are on the second page of the DataTable of news items. - Test the process of deleting multiple news items at once: - Check the checkbox next to multiple items. - Click the "Delete selected" button at the bottom of the page. - Check that the wording of the confirmation message is correct. - After you confirm, the items should be deleted. - Repeat these tests under Tools -> HTML customizations to confirm that redirects work correctly. After deleting an HTML customization you should be redirected back to the list of HTML customizations.
Created attachment 125278 [details] [review] Bug 29019: Unable to delete HTML customization This patch makes some corrections to additional contents to allow content to be deleted. The wrong parameter was being passed to the script. The script was also not handling multiple deletions correctly. The patch also adds a "category" parameter to the delete operation so that the page is redirected correctly. The patch also changes some strings which referred to "news" referencing operations which might be performed on news or on HTML customizations, e.g.: "Are you sure you want to delete the selected content?" instead of "Are you sure you want to delete the selected news?" To test, apply the patch and go to Tools -> News. - Create multiple news items if necessary. - Test the "Delete" button corresponding to a single news item: - Clicking the button should ask you to confirm. - Check that the wording of the message is correct. - After confirming the news item should be deleted. - Ideally, test on news items which are on the second page of the DataTable of news items. - Test the process of deleting multiple news items at once: - Check the checkbox next to multiple items. - Click the "Delete selected" button at the bottom of the page. - Check that the wording of the confirmation message is correct. - After you confirm, the items should be deleted. - Repeat these tests under Tools -> HTML customizations to confirm that redirects work correctly. After deleting an HTML customization you should be redirected back to the list of HTML customizations. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Created attachment 125339 [details] [review] Bug 29019: Unable to delete HTML customization This patch makes some corrections to additional contents to allow content to be deleted. The wrong parameter was being passed to the script. The script was also not handling multiple deletions correctly. The patch also adds a "category" parameter to the delete operation so that the page is redirected correctly. The patch also changes some strings which referred to "news" referencing operations which might be performed on news or on HTML customizations, e.g.: "Are you sure you want to delete the selected content?" instead of "Are you sure you want to delete the selected news?" To test, apply the patch and go to Tools -> News. - Create multiple news items if necessary. - Test the "Delete" button corresponding to a single news item: - Clicking the button should ask you to confirm. - Check that the wording of the message is correct. - After confirming the news item should be deleted. - Ideally, test on news items which are on the second page of the DataTable of news items. - Test the process of deleting multiple news items at once: - Check the checkbox next to multiple items. - Click the "Delete selected" button at the bottom of the page. - Check that the wording of the confirmation message is correct. - After you confirm, the items should be deleted. - Repeat these tests under Tools -> HTML customizations to confirm that redirects work correctly. After deleting an HTML customization you should be redirected back to the list of HTML customizations. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 125394 [details] [review] Bug 29019: Restore eval call But fix it (pass an arrayref). We need the eval to catch possible errors.
Pushed to master for 21.11, thanks to everybody involved!
Affected feature is not part of 21.05.x