Most libraries have the word library in their name - and even if they don't showing 'library' next to the library's name is not really necessary on the list of acq funds.
Created attachment 24694 [details] Library 2 times
Created attachment 24695 [details] [review] [Bug 11608] Remove the word 'library' from funds list The list of funds on the main acq page shows the library's name followed by the word 'library'. I don't think this word is necessary as most libraries have the word 'library' in their names. Even if they don't they probably just want their library name to show without extra words. To test: * Visit the acquisitions module before applying * See the library's name + 'library' above the funds list * Apply patch * Look again at funds list and 'library' should be gone
Created attachment 24700 [details] [review] Bug 11608: Remove the word 'library' from funds list The list of funds on the main acq page shows the library's name followed by the word 'library'. I don't think this word is necessary as most libraries have the word 'library' in their names. Even if they don't they probably just want their library name to show without extra words. To test: * Visit the acquisitions module before applying * See the library's name + 'library' above the funds list * Apply patch * Look again at funds list and 'library' should be gone Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 24701 [details] [review] Bug 11608: Remove the word 'library' from funds list The list of funds on the main acq page shows the library's name followed by the word 'library'. I don't think this word is necessary as most libraries have the word 'library' in their names. Even if they don't they probably just want their library name to show without extra words. To test: * Visit the acquisitions module before applying * See the library's name + 'library' above the funds list * Apply patch * Look again at funds list and 'library' should be gone Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Amended patch: remove a space
Created attachment 24702 [details] [review] Bug 11608: Delete unused commented lines in template On the way, these lines are commented and can be deleted safely. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 24727 [details] [review] [PASSED QA] Bug 11608: Remove the word 'library' from funds list The list of funds on the main acq page shows the library's name followed by the word 'library'. I don't think this word is necessary as most libraries have the word 'library' in their names. Even if they don't they probably just want their library name to show without extra words. To test: * Visit the acquisitions module before applying * See the library's name + 'library' above the funds list * Apply patch * Look again at funds list and 'library' should be gone Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Amended patch: remove a space Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Works as described.
Created attachment 24728 [details] [review] [PASSED QA] Bug 11608: Delete unused commented lines in template On the way, these lines are commented and can be deleted safely. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Removes an HTML comment with unused code.
Pushed to master. Thanks, Nicole!
Pushed to 3.14.x, will be in 3.14.07
Pushed to 3.12.x will be in 3.12.13