[Bug 15375] New: Translatability: Fix issues on OPAC page 'Placing a hold'
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Bug ID: 15375 Summary: Translatability: Fix issues on OPAC page 'Placing a hold' Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: I18N/L10N Assignee: koha-bugs@lists.koha-community.org Reporter: veron@veron.ch QA Contact: testopia@bugs.koha-community.org CC: frederic@tamil.fr Fix several issues with odd translations on opac-reserve.tt due to string splitting by tags. Example in German: Entschuldigung, Sie können nichts vormerken, weil der Bibliothek Ihre aktuellen Kontaktinformation im Archiv. Example in French: Désolé, vous ne pouvez pas faire de réservation, parce que la bibliothèque n'a de données à jour Information de contact sur fichier. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=15374 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #1 from Marc Véron <veron@veron.ch> --- Created attachment 45697 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45697&action=edit Bug 15375 - Translatability: Fix issues on OPAC page 'Placing a hold' Fix several issues with odd translations on opac-reserve.tt due to string splitting by tags. In addition, streamline messages to make them similar to the same messages on the OPAC summary page (Bug 15374) To test: - Apply patch - Log in to OPAC - with blocked /debarred user (with and without comment and date) - with user who has to much fines - with user who has his card marked as lost - with user with an account that is about to expire - with user with expired account - with user with uncertain address information - Try to put holds - Verify that messages about why holds are not possible is correct and displays nicely and that links in messages work - Carefully examine code - Search for regressions -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Assignee|koha-bugs@lists.koha-commun |veron@veron.ch |ity.org | Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aleishaamohia@hotmail.com Status|Needs Signoff |Failed QA --- Comment #2 from Aleisha Amohia <aleishaamohia@hotmail.com> --- This mostly works, but a couple of comments: - For the user with too many fines, there is no comment about the account being blocked as there are with other restrictions on the summary/main page. Maybe there should be? - For the user whose account is about to expire, the error message reads "i Sorry, none of these items can be placed on hold." and "No available items" which is untrue. I'll be happy to sign off on this when these things are fixed. :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #3 from Marc Véron <veron@veron.ch> --- (In reply to Aleisha Amohia from comment #2) Thanks for testing, Aleisha (...)
- For the user with too many fines, there is no comment about the account being blocked as there are with other restrictions on the summary/main page. Maybe there should be?
Link: [server]/cgi-bin/koha/opac-reserve.pl?biblionumber=1 Without patch, I get for somebody with a very high fine: ---- Sorry, you cannot place holds because you owe 10005.00. ---- With patch, I get: ---- Sorry, you cannot place holds. - You have unpaid fines. Amount: 10,005.00. ----
- For the user whose account is about to expire, the error message reads "i Sorry, none of these items can be placed on hold." and "No available items" which is untrue.
I found the stray 'i' and will remove it wit the next patch. Ah, the test plan regarding "with user with an account that is about to expire" is obsolete, it simply is not tested. That makes sense, since while the card is not yet expired the patron should be able to place a hold. Ok, i found out what happend regarding "No available items": - Put a hold on a title -> OK - Try to put a hold on the same item Result (same with and without patch): ----------- Sorry, none of these items can be placed on hold. Breakfast blast , by Kalman, Bobbie. You have already requested this title. ----------- Do a search with results including the item above, in result list check several items, try to put a hold on the checked items, result: List with the items, they are marked as available / not available ("You already have requested this title"). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #45697|0 |1 is obsolete| | --- Comment #4 from Marc Véron <veron@veron.ch> --- Created attachment 45715 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45715&action=edit Bug 15375 - Translatability: Fix issues on OPAC page 'Placing a hold' Fix several issues with odd translations on opac-reserve.tt due to string splitting by tags. In addition, streamline messages to make them similar to the same messages on the OPAC summary page (Bug 15374) To test: - Apply patch - Log in to OPAC - with blocked /debarred user (with and without comment and date) - with user who has to much fines - with user who has his card marked as lost - with user with expired account - with user with uncertain address information - Try to put holds - Verify that message about why holds are not possible is correct and displays nicely and that links in messages work - Carefully examine code - Search for regressions (Amended: stray i removed, test plan updated, removed account that is about to expire) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@univ-lyon3.fr --- Comment #5 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- I followed the test plan. Users with expired account can place a hold on OPAC, so I didn't see any message. Blocked users with date and with comment : comment and date are displayed in the message. OK Users Blocked users with date and without comment : date are not displayed in the message (Your account has been frozen. Usually the reason for freezing an account is old overdues or damage fees. If shows your account to be clear, please contact the library. ) users who has his card marked as lost : ok, the reason appeared in the message. OK users with uncertain address information : ok, the reason appeared in the message. OK users who try to place more than the max number of reserve : the max of reserve is displayed iin the message. OK Sorry, I can't test for users with fines, because there a strange behaviour on the sandbox. It will be nice to add the restriction expiration date in the message. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #6 from Marc Véron <veron@veron.ch> --- (In reply to Koha Team Lyon 3 from comment #5) (...)
Users with expired account can place a hold on OPAC, so I didn't see any message.
MV: - I did string changes only, the overall behaviour should not have changed. - Do you see the same behaviour as without patch? - Did you log in / log out the expired patron after setting the expiration date to make it expire? - Maybe it is related to the setting of syspref 'BlockExpiredPatronOpacAction' and /or in the patron category (Setting 'Block expired patrons' (...)
Users Blocked users with date and without comment : date are not displayed in the message (Your account has been frozen. Usually the reason for freezing an account is old overdues or damage fees. If shows your account to be clear, please contact the library. )
(...)
It will be nice to add the restriction expiration date in the message.
MV: - Is it the same behaviour as without patch? - Note: The details page displays other text than the 'Placing a hold' page. I did not change it because in this bug I wanted to do string changes for translatability only. We can then later streamline the texts / behaviour of the detail page an the hold page. (...)
Sorry, I can't test for users with fines, because there a strange behaviour on the sandbox.
Thanks for testing! Looking forward to have correct translations! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #7 from Marc Véron <veron@veron.ch> --- Still applies on current master. Note: This patch is about string changes only, it does not change the functionallity. If you find an issue or want to suggest an enhancement regarding functionallity, please file a separate bug. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #8 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Can't sign off on this yet, a few things: Blocked/debarred user - the link to the account page did not work, even though I can see in the code that it should so I am not sure what happened there but it did not redirect me anywhere. Uncertain address information - The link took me to opac-userupdate.pl which doesn't exist - should be to opac-memeberentry.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #9 from Marc Véron <veron@veron.ch> --- Hi Aleisha, Blocked user: I re-tested with patch applied. Logged in to OPAC with a user who has a manual debarrement. Searched a biblio and tried to put a hold (from the search result list). Got yellow info with message as expected with working link to the useres addount page. Uncertain address / wrong link: If I do git grep 'opac-userupdate.pl', I get a result in: koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt I think this is a separate Bug. This patch is about koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt only. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #45715|0 |1 is obsolete| | --- Comment #10 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 46631 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46631&action=edit Bug 15375 - Translatability: Fix issues on OPAC page 'Placing a hold' Fix several issues with odd translations on opac-reserve.tt due to string splitting by tags. In addition, streamline messages to make them similar to the same messages on the OPAC summary page (Bug 15374) To test: - Apply patch - Log in to OPAC - with blocked /debarred user (with and without comment and date) - with user who has to much fines - with user who has his card marked as lost - with user with expired account - with user with uncertain address information - Try to put holds - Verify that message about why holds are not possible is correct and displays nicely and that links in messages work - Carefully examine code - Search for regressions (Amended: stray i removed, test plan updated, removed account that is about to expire) Signed-off-by: Aleisha <aleishaamohia@hotmail.com> I am going to sign off on this because it works as it should and I think the problems I had in comment 8 are a separate bug. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #11 from Aleisha Amohia <aleishaamohia@hotmail.com> --- (In reply to Aleisha Amohia from comment #8)
Can't sign off on this yet, a few things:
Blocked/debarred user - the link to the account page did not work, even though I can see in the code that it should so I am not sure what happened there but it did not redirect me anywhere. Patch for this on Bug 15577
Uncertain address information - The link took me to opac-userupdate.pl which doesn't exist - should be to opac-memberentry.pl Patch for this on Bug 15576
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #12 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Marc, why do you replace div with li? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #13 from Marc Véron <veron@veron.ch> --- (In reply to Jonathan Druart from comment #12)
Marc, why do you replace div with li?
Because it looks nicer if there are multiple messages. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com --- Comment #14 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Marc Véron from comment #13)
(In reply to Jonathan Druart from comment #12)
Marc, why do you replace div with li?
Because it looks nicer if there are multiple messages.
I think it's a great improvement! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #15 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 46724 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46724&action=edit Bug 15375 [QA Followup] - Remove refereces to biblio and biblionumber for patron messages Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #46631|0 |1 is obsolete| | Attachment #46724|0 |1 is obsolete| | --- Comment #16 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 46725 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46725&action=edit Bug 15375 - Translatability: Fix issues on OPAC page 'Placing a hold' Fix several issues with odd translations on opac-reserve.tt due to string splitting by tags. In addition, streamline messages to make them similar to the same messages on the OPAC summary page (Bug 15374) To test: - Apply patch - Log in to OPAC - with blocked /debarred user (with and without comment and date) - with user who has to much fines - with user who has his card marked as lost - with user with expired account - with user with uncertain address information - Try to put holds - Verify that message about why holds are not possible is correct and displays nicely and that links in messages work - Carefully examine code - Search for regressions (Amended: stray i removed, test plan updated, removed account that is about to expire) Signed-off-by: Aleisha <aleishaamohia@hotmail.com> I am going to sign off on this because it works as it should and I think the problems I had in comment 8 are a separate bug. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #17 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 46726 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46726&action=edit Bug 15375 [QA Followup] - Remove refereces to biblio and biblionumber for patron messages Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375 --- Comment #18 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 46727 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46727&action=edit Bug 15375 [QA Followup] - Fix non-functional restriction message and date Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org