Add possibility to edit order notes when the basket is closed. BibLibre infos: Repository: git://git.biblibre.com/koha_biblibre.git Branch: stetienne/ft/MT7603
Created attachment 7068 [details] [review] Proposed patch When you are on parcel.pl or basket.pl you can now add or edit a note for each order. To test: Create orders with and without note. Edit/Add the note on basket.pl page. Close the basket. Check you can add/edit the order note on parcel.pl page.
The script devel/misc/testbugzillapatches.pl has detected that this bug does not apply anymore, so i'm marking this bug as "does not apply". The script is supposed to handle conflicts related to updatedatabase, but no other cases of merge conflict. So the conflict may be trivial to fix. If it happens that the patch attached does apply cleanly, please send me an email, there's something wrong in the script. It's possible, as it's a recent script. If the patch applies does not apply as I think, please rebase and resubmit, thanks
Created attachment 7736 [details] [review] Bug 7166 Adds edit order notes when the basket is closed Rebased patch
Hi Jonathan, your patch does not apply, I suspect 7113, but there is also a conflict in neworderempty.pl. Can you please fix and resubmit? Thank you.
Created attachment 7848 [details] [review] Bug 7166 Adds edit order notes when the basket is closed Rebased patch
Hi Jonathan, I have tested your patch and it works nicely. I think perhaps display and usability can be improved a little bit. What I found is not enough for a 'failed qa', but perhaps you can consider the suggestions for a follow up: 1) I think the note gets a bit lost in the colum with the title information. Perhaps a separate column displaying the notes and the links to add or change notes would be a bit nicer. 2) After saving the note you are not redirected to the page basket or parcel page, but first have to confirm again. I think redirecting directly to the page would be a bit nicer and save one step. 3) Not sure, but perhaps a better label for the button on the new order page would be "Save" or "Update" instead of "Change". I will attach some screenshots, so others can perhaps add their opinion without applying the patch.
Created attachment 7887 [details] [review] [SIGNED-OFF] Bug 7166: Adds edit order notes when the basket is closed When you are on parcel.pl or basket.pl you can now add or edit a note for each order. To test: Create orders with and without note. Edit/Add the note on basket.pl page Close the basket. Check you can add/edit the order note on parcel.pl page Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Feature works nicely. I added some notes about and ideas for usability and display to the bug report.
Created attachment 7888 [details] new order note page
Created attachment 7889 [details] Order notes on basket page Screenshot shows how the order note gets a bit lost/hidden within the other information in that column.
Created attachment 7943 [details] [review] Bug 7166 Followup redirect after editing the note This patch redirects to the correct page and changes the button label to "Save". Katrin, I fear that putting the note in a new column deforms the table, if this note is too long What do you thing about a "mouseover popup" to display the note ?
Created attachment 7950 [details] [review] Follow-up fix: Markup corrections and improvements Correcting markup issues with modordernotes.pl: - Adding "rows" and "cols" attributes to <textarea> (required) - Converting cancel button to link to stay consistent with other pages - Changing page layout to fixed & centered (not sidebar needed) - Expanding information in breadcrumbs to match information on basket.pl Changing markup around note add and edit links: Moving output of order note to the end of the table cell, putting it in its own paragraph, and adding a "Note:" label
I noticed a very strange bug: When I'm on parcel.pl I don't see notes which I added or modified via modordernotes.pl. I thought notes were stored in the same place either way, but I see an example where parcel.pl doesn't show any notes for an order, but when I click "add note" the form on modordernotes.pl shows the data I entered previously.
Created attachment 7955 [details] [review] Bug 7166 Followup Adds edit order notes when the basket Bug 7166: Followup: Adds edit order notes when the basket is closed FIX field aqorders.notes Hi Owen, In fact, there is an ambiguous field between biblio and aqorders tables. I fix the problem with this patch.
Something went wrong ! Applying: Bug 7166: Adds edit order notes when the basket is closed Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... Auto-merging acqui/neworderempty.pl Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt Failed to merge in the changes. Patch failed at 0001 Bug 7166: Adds edit order notes when the basket is closed
Created attachment 8289 [details] [review] Bug 7166: Adds edit order notes when the basket is closed Rebased and Squashed patch
Edit/Add the note on basket.pl page and parcel.pl page. Everything is OK.
Mostly a template-only change, with a small reordering of select columns in C4::Acquisition::GetPendingOrders(). Adds valuable functionality in it's own page and script. Marking as Passed QA.
Ok in master