The table of orders on the basket summary page includes a lot of columns with different prices. Two are not labelled well: - ecost tax inc. - ecost tax exc. Apart from the capitalization issue, I think it's not clear that this should mean 'estimated cost'. I suggest changing this to the term used on the order form: - Budgeted cost tax inc. - Budgeted cost tax exc.
Created attachment 95448 [details] [review] Bug 24033 : Fix column labelling on basket summary page (ecost) When viewing a basket, some of the column names are unclear and are different than the terms used for the same field on the order form. This patch edits the column titles to match
Created attachment 95449 [details] [review] Bug 24033 : Fix column labelling on basket summary page (ecost) When viewing a basket, some of the column names are unclear and are different than the terms used for the same field on the order form. This patch edits the column titles to match To test: 1)In acquisitions, create a budget. 2)In acquisitions, create a fund. 3)In acquisitions, create a vendor. 4)In acquisitions, create a basket and add a test item to the basket. 5)Observe the column titles ecost tax inc. and ecost tax exc. Check the column visibility settings if you don't see the columns. 6)Apply the patch. 7)Observe the column titles have been updated.
Created attachment 95710 [details] [review] Bug 24033 : Fix column labelling on basket summary page (ecost) When viewing a basket, some of the column names are unclear and are different than the terms used for the same field on the order form. This patch edits the column titles to match the order f Signed-off-by: George Williams <george@nekls.org>
Created attachment 95741 [details] [review] k
Created attachment 95742 [details] [review] Bug 24033: Fix column labelling on basket summary page (ecost) When viewing a basket, some of the column names are unclear and are different than the terms used for the same field on the order form. This patch edits the column titles to match the order form. To test: 1) In acquisitions, create a budget. 2) In acquisitions, create a fund. 3) In acquisitions, create a vendor. 4) In acquisitions, create a basket and add a test item to the basket. 5) Observe the column titles ecost tax inc. and ecost tax exc. Check the column visibility settings if you don't see the columns. 6) Apply the patch. 7) Observe the column titles have been updated. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Hi Lisette, nice patch! Just some small things for your next one: - The first line always needs to start with 'Bug xxxx:'. - Something seems to happen to the breaks in your patches - no problem, I've fixed it. How do you write the commit message? Maybe try to add an extra empty line after the first.
(In reply to Katrin Fischer from comment #6) > Hi Lisette, nice patch! > > Just some small things for your next one: > > - The first line always needs to start with 'Bug xxxx:'. > - Something seems to happen to the breaks in your patches - no problem, I've > fixed it. How do you write the commit message? Maybe try to add an extra > empty line after the first. Katrin, Thanks for fixing it. I'm using the devbox. I'll do: 1) git commit basket.tt 2) Write the commit notes, following the instructions on the wiki that I found here: https://wiki.koha-community.org/wiki/Commit_messages 3) :wq! to write and quit. 4) git bz attach -e 24403 HEAD 5) Put all the line breaks back in and change the status. I did think it was quirky that I had to put the line breaks in again. That was also why I had 2 patches on this one. The first time I forgot to -e so I re-attached it after editing.
(In reply to Lisette Scheer from comment #7) > (In reply to Katrin Fischer from comment #6) > > Hi Lisette, nice patch! > > > > Just some small things for your next one: > > > > - The first line always needs to start with 'Bug xxxx:'. > > - Something seems to happen to the breaks in your patches - no problem, I've > > fixed it. How do you write the commit message? Maybe try to add an extra > > empty line after the first. > > > Katrin, > Thanks for fixing it. > I'm using the devbox. > I'll do: > 1) git commit basket.tt > 2) Write the commit notes, following the instructions on the wiki that I > found here: https://wiki.koha-community.org/wiki/Commit_messages > 3) :wq! to write and quit. > 4) git bz attach -e 24403 HEAD > 5) Put all the line breaks back in and change the status. > > I did think it was quirky that I had to put the line breaks in again. > That was also why I had 2 patches on this one. The first time I forgot to -e > so I re-attached it after editing. Hm something is odd there, you shouldn't have to do anything on bugzilla and it doesn't really work well this way. It sounds like you are using vi in the devbox. One can change the default editor for git via the config files for the devbox, maybe this could improve things... but it SHOULD work with vi too :(
Nice work everyone! Pushed to master for 20.05
Pushed to 19.11.01