When receiving several issues for a subscription (Serials > Serial collection > Multi receiving), the reception date is identical to the publication date instead of being the today date.
The BULAC (http://www.bulac.fr/) would like to sponsor this development ! As long as “Change sponsored?” is set on “Seeking developer”, anyone wanting to quote us for the development of such a feature or just ask for more information is welcome to write to us: <sigb@bulac.fr>.
Created attachment 74375 [details] [review] Bug 18327: Add the ability to set the received date to today on multi receiving serials When receiving several issues for a subscription (Serials > Serial collection > Multi receiving), the reception date is always identical to the publication date. In some use cases we would like to set this "date received" value to today. Note: "date received" refers to the DB column serial.planneddate To make this possible this patch replaces the JS prompt with a modal dialog asking for 1. the number of issues to receive 2. if the received date must be set to today Ergonomic note: bootstrap styled buttons are used, but they do not display correctly We may need to improve that (later) Test plan: - Receive 1 or more serials using the "Multi receiving" buttons Note that this button appears twice, on the "serial collection information" and the "serial edition" pages - Try with and without the new checkbox ticked and confirm the behaviour is correct (i.e. date received is set to today or set to the publish date) - Make sure "Edit serials" and "Generate next" buttons still work as before
Created attachment 74376 [details] [review] Bug 18327: Same change for serials-edit And use an include file to avoid copy/paste
Created attachment 74390 [details] [review] Bug 18327: Add the ability to set the received date to today on multi receiving serials When receiving several issues for a subscription (Serials > Serial collection > Multi receiving), the reception date is always identical to the publication date. In some use cases we would like to set this "date received" value to today. Note: "date received" refers to the DB column serial.planneddate To make this possible this patch replaces the JS prompt with a modal dialog asking for 1. the number of issues to receive 2. if the received date must be set to today Ergonomic note: bootstrap styled buttons are used, but they do not display correctly We may need to improve that (later) Test plan: - Receive 1 or more serials using the "Multi receiving" buttons Note that this button appears twice, on the "serial collection information" and the "serial edition" pages - Try with and without the new checkbox ticked and confirm the behaviour is correct (i.e. date received is set to today or set to the publish date) - Make sure "Edit serials" and "Generate next" buttons still work as before Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 74391 [details] [review] Bug 18327: Same change for serials-edit And use an include file to avoid copy/paste Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
The use of checkbox works as expected. Behaviours of others buttons are the same as before. I messed bug 18237 when I first this one, I confused myself between numbers... I hope it won't cause touble.
Only some cosmetics: 1) With the patch the buttons on the serials collection and serial edit pages are no longer aligned and look very different. Can we fix this? 2) In the pop-up the submit button is labelled "Multi receiving". I think OK would be better and more consistent with other views (also translations). 3) Can we shorten the input field for entering the number of items and give the text labels more space? Text is already crowded on multiple lines and it will get worse with translations to verbose language like German. :) Works well. Setting to failed to get some attention :)
(In reply to Katrin Fischer from comment #7) > Only some cosmetics: > > 1) With the patch the buttons on the serials collection and serial edit > pages are no longer aligned and look very different. Can we fix this? It's in the commit message, I do not know how to do. I will need Owen's help. > 2) In the pop-up the submit button is labelled "Multi receiving". I think OK > would be better and more consistent with other views (also translations). > > 3) Can we shorten the input field for entering the number of items and give > the text labels more space? Text is already crowded on multiple lines and it > will get worse with translations to verbose language like German. :) follow-up is coming later today.
Created attachment 74456 [details] [review] Bug 18327: Rename submit button with 'OK'
Created attachment 74457 [details] [review] Bug 18327: Set label width to 50% in modal
(In reply to Katrin Fischer from comment #7) > 3) Can we shorten the input field for entering the number of items and give > the text labels more space? Text is already crowded on multiple lines and it > will get worse with translations to verbose language like German. :) I have extended the label, it looks better to me as it. What do you think?
Hm, I will leave it up to you with the follow ups. I am not sure about the pop-up either, maybe Owen will be able to take a look. Functionally it works well.
Created attachment 74513 [details] [review] Bug 18327: Add the ability to set the received date to today on multi receiving serials When receiving several issues for a subscription (Serials > Serial collection > Multi receiving), the reception date is always identical to the publication date. In some use cases we would like to set this "date received" value to today. Note: "date received" refers to the DB column serial.planneddate To make this possible this patch replaces the JS prompt with a modal dialog asking for 1. the number of issues to receive 2. if the received date must be set to today Ergonomic note: bootstrap styled buttons are used, but they do not display correctly We may need to improve that (later) Test plan: - Receive 1 or more serials using the "Multi receiving" buttons Note that this button appears twice, on the "serial collection information" and the "serial edition" pages - Try with and without the new checkbox ticked and confirm the behaviour is correct (i.e. date received is set to today or set to the publish date) - Make sure "Edit serials" and "Generate next" buttons still work as before Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 74514 [details] [review] Bug 18327: Same change for serials-edit And use an include file to avoid copy/paste Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 74515 [details] [review] Bug 18327: Rename submit button with 'OK' Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 74516 [details] [review] Bug 18327: Set label width to 50% in modal Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Owen, could you take a look maybe at the GUI bits here for some of your magic?
Created attachment 74535 [details] [review] Bug 18327: Style 'Multi receiving' button like others
I have removed the '+' icon, but it did not exist prior to these patches anyway.
Pushed to master for 18.05, thanks to everybody involved!