If UseCashRegisters and EnablePointOfSale preferences are enabled but there are no cash registers defined, a message will appear in several places in Koha saying "You must have at least one cash register associated with the branch." If the user has permission, it would be helpful to include a link directly to new cash register creation. Also, the term "library" should be used instead of "branch."
Created attachment 108092 [details] [review] Bug 26194: Add link to cash register management from message about missing registers In places where the interface warns the user about missing cash registers we should provide users with permission a direct link to the cash register management page. To test, apply the patch and enable the UseCashRegisters and EnablePointOfSale preferences. If necessary, go to Administration -> Cash registers and remove any cash registers for your library. - Log in to the staff interface as a user with "manage_cash_registers" permission. - Locate a patron who has outstanding fines. - Under Accounting -> Make a payment, click the "Pay" button next to a fine. - The page should show the message about missing cash registers with a button which take you directly to the "Add new cash register" form. - Perform the same check by selecting the checkbox next to a fine and clicking the "Pay amount" button and the "Pay selected" button. - Test the message shown at: - Tools -> Cashup registers. - Point of sale -> Register details - Point of sale -> Library details - Perform the same tests when logged in as a user without permission to manage cash registers. The messages shown in the previous steps should omit the button linking to cash register management.
Nice.. will test now :)
Created attachment 108127 [details] [review] Bug 26194: Add link to cash register management from message about missing registers In places where the interface warns the user about missing cash registers we should provide users with permission a direct link to the cash register management page. To test, apply the patch and enable the UseCashRegisters and EnablePointOfSale preferences. If necessary, go to Administration -> Cash registers and remove any cash registers for your library. - Log in to the staff interface as a user with "manage_cash_registers" permission. - Locate a patron who has outstanding fines. - Under Accounting -> Make a payment, click the "Pay" button next to a fine. - The page should show the message about missing cash registers with a button which take you directly to the "Add new cash register" form. - Perform the same check by selecting the checkbox next to a fine and clicking the "Pay amount" button and the "Pay selected" button. - Test the message shown at: - Tools -> Cashup registers. - Point of sale -> Register details - Point of sale -> Library details - Perform the same tests when logged in as a user without permission to manage cash registers. The messages shown in the previous steps should omit the button linking to cash register management. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 108128 [details] [review] Bug 26194: (follow-up) Default to current branch When adding a new register, it would be helpful to default to the current logged in branch.
Works well.. I do wonder if the block could be factored out into an include.. but we could/should probably do that on another bug report. I've added a followup to fix the 'Add register' form so it uses the PROCESS block from HTMLHelpers and as such we get the currently logged in branch selected in the dropdown by default (which with these links in place makes allot more sense than defaulting to the first one listed). Signing off.
Minor niggle to be looked at during QA.. the alert box styling has changed from 'alert' to 'info' (yellow background to blue background).. and the button styling doesn't look completely right to me yet... ?
Created attachment 108313 [details] [review] Bug 26194: Add link to cash register management from message about missing registers In places where the interface warns the user about missing cash registers we should provide users with permission a direct link to the cash register management page. To test, apply the patch and enable the UseCashRegisters and EnablePointOfSale preferences. If necessary, go to Administration -> Cash registers and remove any cash registers for your library. - Log in to the staff interface as a user with "manage_cash_registers" permission. - Locate a patron who has outstanding fines. - Under Accounting -> Make a payment, click the "Pay" button next to a fine. - The page should show the message about missing cash registers with a button which take you directly to the "Add new cash register" form. - Perform the same check by selecting the checkbox next to a fine and clicking the "Pay amount" button and the "Pay selected" button. - Test the message shown at: - Tools -> Cashup registers. - Point of sale -> Register details - Point of sale -> Library details - Perform the same tests when logged in as a user without permission to manage cash registers. The messages shown in the previous steps should omit the button linking to cash register management. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 108314 [details] [review] Bug 26194: (follow-up) Default to current branch When adding a new register, it would be helpful to default to the current logged in branch. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
(In reply to Martin Renvoize from comment #6) > Minor niggle to be looked at during QA.. the alert box styling has changed > from 'alert' to 'info' (yellow background to blue background).. and the > button styling doesn't look completely right to me yet... ? I think that's ok, because blue = info, not indicating a real problem. I beleive the button looks similar in other spots too.
Pushed to master for 20.11, thanks to everybody involved!
enhancement will not be backported to 20.05.x