When in the OPAC and I click on "New suggestion", only an empty page comes up. The form is missing. I didn't see anything in the logs. Not sure if this is related to CSRF or not?
Created attachment 162907 [details] [review] Bug 36274: Use right op on opac-suggestions.tt To test: 1. click on "New suggestion", only an empty page comes up. The form is missing. 2. Apply patch and try again. 3. Play with suggestions making sure everything looks as excepted.
(In reply to Katrin Fischer from comment #0) > When in the OPAC and I click on "New suggestion", only an empty page comes > up. The form is missing. > I didn't see anything in the logs. > Not sure if this is related to CSRF or not? Yes, related to CSRF but a trivial fix.
This works to display the form I think there's still a problem with the check for duplicates. I filled out the form (title, author, isbn, and year) using information from a book already in the catalog and when I submitted it I got a blank page.
Created attachment 162935 [details] [review] Bug 36274: Correct links leading to purchase suggestion page To test: 1. APPLY PATCH 2. As a logged in OPAC user go to 'Your account' > Purchase suggestion. The form should display. 3. Make a suggestion and confirm it works. 4. Try to make a suggestion using title, author, isbn information from an item that already exists. 5. See the "A similar document already exists:" warning. Confirm the suggestion anyways. 6. Set MaxTotalSuggestions to 4 and NumberOfSuggestionDays to 2. Make sure you can still navigate to the purchase suggestion form. 7. Do an OPAC search that will yield no results as a logged in user. 8. Click the "Make a purchase suggestion" link, making sure it works. 9. Turn on AnonSuggestions and log out of the OPAC. 10. Do an OPAC search that will yield no results. Again click the "Make a purchase suggestion", confirm it works.
(In reply to Owen Leonard from comment #3) > This works to display the form I think there's still a problem with the > check for duplicates. I filled out the form (title, author, isbn, and year) > using information from a book already in the catalog and when I submitted it > I got a blank page. Good catch. I approached this the wrong way. My new patch should be better.
Created attachment 162994 [details] [review] Bug 36274: Correct links leading to purchase suggestion page To test: 1. APPLY PATCH 2. As a logged in OPAC user go to 'Your account' > Purchase suggestion. The form should display. 3. Make a suggestion and confirm it works. 4. Try to make a suggestion using title, author, isbn information from an item that already exists. 5. See the "A similar document already exists:" warning. Confirm the suggestion anyways. 6. Set MaxTotalSuggestions to 4 and NumberOfSuggestionDays to 2. Make sure you can still navigate to the purchase suggestion form. 7. Do an OPAC search that will yield no results as a logged in user. 8. Click the "Make a purchase suggestion" link, making sure it works. 9. Turn on AnonSuggestions and log out of the OPAC. 10. Do an OPAC search that will yield no results. Again click the "Make a purchase suggestion", confirm it works. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 163166 [details] [review] Bug 36274: Correct links leading to purchase suggestion page To test: 1. APPLY PATCH 2. As a logged in OPAC user go to 'Your account' > Purchase suggestion. The form should display. 3. Make a suggestion and confirm it works. 4. Try to make a suggestion using title, author, isbn information from an item that already exists. 5. See the "A similar document already exists:" warning. Confirm the suggestion anyways. 6. Set MaxTotalSuggestions to 4 and NumberOfSuggestionDays to 2. Make sure you can still navigate to the purchase suggestion form. 7. Do an OPAC search that will yield no results as a logged in user. 8. Click the "Make a purchase suggestion" link, making sure it works. 9. Turn on AnonSuggestions and log out of the OPAC. 10. Do an OPAC search that will yield no results. Again click the "Make a purchase suggestion", confirm it works. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Pushed for 24.05! Well done everyone, thank you!
So depends on Bug 34478 right ?
(In reply to Fridolin Somers from comment #9) > So depends on Bug 34478 right ? Correct
Bug 34478 not in 23.11.x