---- Reported by oleonard@myacpl.org 2009-09-25 19:09:49 ---- Creating a template for add_message.pl as a standalone interface would accomplish a couple of goals: 1. The process of adding a message could be made less dependent on JavaScript. Any link to add a new message could load the page itself if JavaScript wasn't enabled. 2. We could add an "Add message" button to the standard patron toolbar, enabling staff to add a message from anywhere. The "Add message" button could trigger the display of the form when clicked from circulation.pl or could take the user to add_message.pl --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:13 UTC --- This bug was previously known as _bug_ 3669 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3669 Actual time not defined. Setting to 0.0 CC member jwagner@ptfs.com does not have an account here
Created attachment 49497 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. Sponsored-by: Catalyst IT
I like the feature and it works fine on the Checkout screen. The new toolbar button appears on other screens as well (e.g. Details or Fines), but nothing happens.
I think this should be "Failed QA" until a solution is available which makes the button functional on all pages with that toolbar. Prompted by Bug 16125 I was working on a patch very similar to this one. I can finish it up and post it as an alternative, or drop it if I'm stepping on toes again.
Created attachment 49610 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar (ie Details, Fines etc) Sponsored-by: Catalyst IT
Hi Aleisha The modal opens now fine from all pages with the members toolbar. However, if I add a message and click "Save", I'm not taken back to the page where I came from (what I would expect), but to the patron's checkout page. This is due to a redirect in in circ/add_message.pl, line 55 that is hard coded to circulation.pl It could take in account the referrer - what do you think about?
(In reply to Marc Véron from comment #5) > Hi Aleisha > > The modal opens now fine from all pages with the members toolbar. > However, if I add a message and click "Save", I'm not taken back to the page > where I came from (what I would expect), but to the patron's checkout page. > > This is due to a redirect in in circ/add_message.pl, line 55 that is hard > coded to circulation.pl > > It could take in account the referrer - what do you think about? That is a great idea but I'm not really sure how to do it! Any suggestions? Or does anyone else know how?
(In reply to Aleisha Amohia from comment #6) > (In reply to Marc Véron from comment #5) > > Hi Aleisha > > > > The modal opens now fine from all pages with the members toolbar. > > However, if I add a message and click "Save", I'm not taken back to the page > > where I came from (what I would expect), but to the patron's checkout page. > > > > This is due to a redirect in in circ/add_message.pl, line 55 that is hard > > coded to circulation.pl > > > > It could take in account the referrer - what do you think about? > > That is a great idea but I'm not really sure how to do it! Any suggestions? > Or does anyone else know how? I think you can make use of $input->referer() in circ/add_message.pl
Created attachment 49714 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT: Once the form has been submitted the page will redirect to where you submitted the form from. To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar (ie Details, Fines etc) 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT
Created attachment 49716 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT: Once the form has been submitted the page will redirect to where you submitted the form from. To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar (ie Details, Fines etc) 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Aleisha, new files should be correctly indented (even if it is created with c/p code). It does not seem to work for me, the messages are not saved if created from another page than circulation.pl. Moreover the branchcode is not filled and predefined notes do not appear.
Comment on attachment 49716 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar Review of attachment 49716 [details] [review]: ----------------------------------------------------------------- ::: circ/add_message.pl @@ +51,4 @@ > } > )->store; > > +my $url = $input->referer(); http://perldoc.perl.org/CGI.html "referer() Return the URL of the page the browser was viewing prior to fetching your script. Not available for all browsers." -- might be a good idea to // "/cgi...$borrowernumber";
Created attachment 49833 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT
Overall this looks great. I am curious why you created a new include file for the add message form but put the modal markup in members-toolbar.inc. Why not just put all the markup in members-toolbar.inc?
(In reply to Owen Leonard from comment #13) > Overall this looks great. I am curious why you created a new include file > for the add message form but put the modal markup in members-toolbar.inc. > Why not just put all the markup in members-toolbar.inc? It was just how I'd seen it done elsewhere so I followed that. I can move it all into members-toolbar tomorrow if you think that would be better.
Created attachment 50044 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT
Created attachment 50047 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Comment on attachment 50047 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar Review of attachment 50047 [details] [review]: ----------------------------------------------------------------- Your patch adds a swp file koha-tmpl/intranet-tmpl/prog/en/includes/.member-add-message.inc.swp ::: members/boraccount.pl @@ +106,4 @@ > ); > } > > +my $branch = C4::Context->userenv->{'branch'}; This value can be retrieved from the templates using LoginBranchcode, it will avoid to pass it from all the scripts. Keep in mind: if you are duplicating code, you are doing something wrong ;) @@ +108,5 @@ > > +my $branch = C4::Context->userenv->{'branch'}; > + > +# get authorised values with type of BOR_NOTES > +my $canned_notes = GetAuthorisedValues("BOR_NOTES"); Same here, the AuthorisedValues template plugin will help you.
Created attachment 50100 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar EDIT5: Removing swp file and unnecessary code duplication by utilising plugins To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
The redirect is failing for me on circulation.pl if I arrive at the page via a form submission. It works if I come to the page via circulation.pl?borrowernumber=X or circulation.pl?findborrower=X. The same is true for the batch checkout page. If I click the batch checkout link, then try to add a message the redirect works. If I perform a batch checkout and then try to add a message the redirect fails. In by cases what I mean by "fails" is that I'm returned to a "blank" circulation.pl with no patron loaded. This is true of paycollect.pl too, so I assume it's true for any page which loads the patron record via a POST operation.
(In reply to Owen Leonard from comment #19) > The redirect is failing for me on circulation.pl if I arrive at the page via > a form submission. It works if I come to the page via > circulation.pl?borrowernumber=X or circulation.pl?findborrower=X. > > The same is true for the batch checkout page. If I click the batch checkout > link, then try to add a message the redirect works. If I perform a batch > checkout and then try to add a message the redirect fails. > > In by cases what I mean by "fails" is that I'm returned to a "blank" > circulation.pl with no patron loaded. > > This is true of paycollect.pl too, so I assume it's true for any page which > loads the patron record via a POST operation. I'm a bit confused - the button works fine for me from the paycollect page etc. Not sure what's going wrong for you!
(In reply to Aleisha Amohia from comment #20) > (In reply to Owen Leonard from comment #19) > > The redirect is failing for me on circulation.pl if I arrive at the page via > > a form submission. It works if I come to the page via > > circulation.pl?borrowernumber=X or circulation.pl?findborrower=X. > > > > The same is true for the batch checkout page. If I click the batch checkout > > link, then try to add a message the redirect works. If I perform a batch > > checkout and then try to add a message the redirect fails. > > > > In by cases what I mean by "fails" is that I'm returned to a "blank" > > circulation.pl with no patron loaded. > > > > This is true of paycollect.pl too, so I assume it's true for any page which > > loads the patron record via a POST operation. > > I'm a bit confused - the button works fine for me from the paycollect page > etc. Not sure what's going wrong for you! To reproduce: - Go to Batch check out page - Enter some barcodes, click "Check out" - On the next page, instead of "Confirm Check out" click "Add message" - Enter a message and click "Save" Result: Redirection to /cgi-bin/koha/circ/circulation.pl with text "This patron does not exist. Find another patron?"
Created attachment 50976 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar EDIT5: Removing swp file and unnecessary code duplication by utilising plugins EDIT6: Batch checkout fixes To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl turn on BatchCheckouts syspref and put in appropriate patron category circ/circulation.pl?borrowernumber=X&batch=1 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
(In reply to Marc Véron from comment #21) > To reproduce: > - Go to Batch check out page > - Enter some barcodes, click "Check out" > - On the next page, instead of "Confirm Check out" click "Add message" > - Enter a message and click "Save" > > Result: Redirection to /cgi-bin/koha/circ/circulation.pl with text "This > patron does not exist. Find another patron?" hi Marc, finally figured out what the problem was - the batch checkout form submits to circulation.pl so the Add message form will always redirect to an empty circulation page. I've given a more specific URL for the form to submit to now. This is the best possible fix I could think of! Unless you can think of a better way?
Hi Aleisha, I went through the test plan, everything seems to work OK, including returning to Batch checkout (Confirmatoion page, Comment #21). However, with patch I get an error in the JavaScript console: ReferenceError: MSG_ADD_MESSAGE is not defined (...)/intranet-tmpl/prog/en/js/pages/circulation.js Line: 17 Without patch, the error does not occur.
Created attachment 51353 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar EDIT5: Removing swp file and unnecessary code duplication by utilising plugins EDIT6: Batch checkout fixes EDIT7: Ensure you do not get JS error from Comment 24 (Batch checkout syspref must be turned on) To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl turn on BatchCheckouts syspref and put in appropriate patron category circ/circulation.pl?borrowernumber=X&batch=1 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 51374 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar EDIT5: Removing swp file and unnecessary code duplication by utilising plugins EDIT6: Batch checkout fixes EDIT7: Ensure you do not get JS error from Comment 24 (Batch checkout syspref must be turned on) To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl turn on BatchCheckouts syspref and put in appropriate patron category circ/circulation.pl?borrowernumber=X&batch=1 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. (Re-tested for comment #24) Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 52277 [details] [review] Bug 3669: Remove parameters passed to action of form Sounds weird to pass parameters to the action of a POST form. This patch adds a workaround.
It would be nicer to get a message to inform the librarian that the message has been added successfully. Actually it would have been better to ajaxify this script, to avoid the reload and the problematic of POST form. Owen, do you think this is an acceptable solution and then we should ajaxify this script?
I think this patch is an improvement and seems to work well now as far as I can tell. Small glitch with and without patch: if you don't enter a message it takes a while for the page to load, but there is no message saved.
I agree that this is an improvement and I don't think we should wait for an AJAX version (but it's a good idea).
Created attachment 52641 [details] [review] Bug 3669: Moving 'Add a new message' into a pop up box and adding to patron toolbar This patch moves all the code into an include which can be accessed from the 'Add a new message' link or the patron toolbar. EDIT: Ensuring modal opens on all pages EDIT2: Once the form has been submitted the page will redirect to where you submitted the form from. EDIT3: Fixing indentation of member-add-message.inc Ensuring messages save on every page Fill branch Ensure predefined notes appear Form redirects to where it was submitted from, or just the circulation page. EDIT4: Removing the new include file and moving it all into the modal in the toolbar EDIT5: Removing swp file and unnecessary code duplication by utilising plugins EDIT6: Batch checkout fixes EDIT7: Ensure you do not get JS error from Comment 24 (Batch checkout syspref must be turned on) To test: 1) Go to a patron circ page (circ/circulation.pl?borrowernumber=X) 2) Click 'Add a new message' under Messages 3) Confirm this brings up the modal to add a message. Confirm clicking Save saves your message. If it is an OPAC message (for the user), confirm it shows as expected on the OPAC. 4) Click Cancel. Confirm this closes the modal. 5) Click 'Add message' button in toolbar. Complete steps 3 and 4. 6) Confirm the modal opens from all other pages with the members toolbar. Confirm predefined notes shows. circ/circulation.pl members/moremember.pl members/routing-lists.pl members/statistics.pl members/boraccount.pl members/pay.pl members/maninvoice.pl members/mancredit.pl members/readingrec.pl members/notices.pl members/member-flags.pl members/member-password.pl members/paycollect.pl members/files.pl turn on BatchCheckouts syspref and put in appropriate patron category circ/circulation.pl?borrowernumber=X&batch=1 7) Confirm that the page redirects to where you submitted the form from once you have submitted it. 8) Go to Check Out tab and confirm your message saved (or OPAC, wherever you saved it) Sponsored-by: Catalyst IT Followed test plan, works as expected. (Re-tested for comment #24) Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 52642 [details] [review] Bug 3669: Remove parameters passed to action of form Sounds weird to pass parameters to the action of a POST form. This patch adds a workaround. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(In reply to Katrin Fischer from comment #29) > Small glitch with and without patch: if you don't enter a message it takes a > while for the page to load, but there is no message saved. I don't notice that (under plack).
Was thinking maybe the field should be 'required' somehow - but it's not a blocker :)
Created attachment 52796 [details] [review] Bug 3669 [QA Followup] - Give the text area some breathing room Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 16.11, thanks Aleisha, Jonathan!