During checkout the withdrawn information is displayed in a message : "Item has been withdrawn". Like in other pages and like notforloan, we should display the withdrawn authorized value description.
Created attachment 82616 [details] [review] Bug 21877: Show authorized value description for withdrawn in checkout During checkout the withdrawn information is displayed in a message : "Item has been withdrawn". Like in other pages and like notforloan, we should display the withdrawn authorized value description. This patch adds this display. Add <span> so that translation does not change. Also adds class 'co-withdrawn' (like in checkin 'ci-*' classes) to ease hidding this new information via CSS. Test plan : 1) On a catalog with items.withdrawn defined with authorized values category WITHDRAWN 2) Define in WITHDRAWN an authorized values 1 with description 'dropped in trash' 3) Define in WITHDRAWN an authorized values 2 with description empty 4) Edit an item with withdrawn=1 5) Checkout this item => You see 'Item has been withdrawn (dropped in trash)' 6) Edit an item with withdrawn=2 7) Checkout this item => You see 'Item has been withdrawn'
Created attachment 82624 [details] [review] Bug 21877: Show authorized value description for withdrawn in checkout During checkout the withdrawn information is displayed in a message : "Item has been withdrawn". Like in other pages and like notforloan, we should display the withdrawn authorized value description. This patch adds this display. Add <span> so that translation does not change. Also adds class 'co-withdrawn' (like in checkin 'ci-*' classes) to ease hidding this new information via CSS. Test plan : 1) On a catalog with items.withdrawn defined with authorized values category WITHDRAWN 2) Define in WITHDRAWN an authorized values 1 with description 'dropped in trash' 3) Define in WITHDRAWN an authorized values 2 with description empty 4) Edit an item with withdrawn=1 5) Checkout this item => You see 'Item has been withdrawn (dropped in trash)' 6) Edit an item with withdrawn=2 7) Checkout this item => You see 'Item has been withdrawn' Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>
Why are not you doing it the same way as notforloan? Or provide tests, but I'd prefer consistency...
(In reply to Jonathan Druart from comment #3) > Why are not you doing it the same way as notforloan? > > Or provide tests, but I'd prefer consistency... In Circulation.pm its the same : $needsconfirmation{item_notforloan} = $item->{'notforloan'}; $issuingimpossible{item_withdrawn} = $item->{'withdrawn'}; In circulation.tt : notforloan uses AuthorisedValues.GetByCode( authvalcode_notforloan ... authvalcode_notforloan is computed in circulation.pl this useful because it is used several times in circulation.tt. But for withdrawn the authorised value is used only once. Using AuthorisedValues.GetDescriptionByKohaField is better for MVC separation no ? I could create another bug to change notforloan maybe ?
Created attachment 83400 [details] [review] Bug 21877: [ALTERNATIVE-PATCH] Display authorized value description for withdrawn in checkout
(In reply to Jonathan Druart from comment #5) > Created attachment 83400 [details] [review] [review] > Bug 21877: [ALTERNATIVE-PATCH] Display authorized value description for > withdrawn in checkout Hi Frido, This is what I had in mind. It will avoid adding more stuff to C4::Circulation as well as unify the way we display/fetch the values I would prefer this approach and provide selenium tests. Note that another occurrence of notforloan must be replaced in circulation.tt What do you think?
(In reply to Jonathan Druart from comment #6) > (In reply to Jonathan Druart from comment #5) > > Created attachment 83400 [details] [review] [review] [review] > > Bug 21877: [ALTERNATIVE-PATCH] Display authorized value description for > > withdrawn in checkout > > Hi Frido, > This is what I had in mind. > It will avoid adding more stuff to C4::Circulation as well as unify the way > we display/fetch the values > > I would prefer this approach and provide selenium tests. > > Note that another occurrence of notforloan must be replaced in circulation.tt > > What do you think? OK better indeed. I will signoff.
Ah in the end I review all the code and split into 2 patches : - adding withdrawn description - changing the way notforloan description is fetched
Created attachment 84273 [details] [review] Bug 21877: Show authorized value description for withdrawn in checkout During checkout the withdrawn information is displayed in a message : "Item has been withdrawn". Like in other pages and like notforloan, we should display the withdrawn authorized value description. This patch adds this display. Add <span> so that translation does not change. Also adds class 'co-withdrawn' (like in checkin 'ci-*' classes) to ease hidding this new information via CSS. Test plan : 1) On a catalog with items.withdrawn defined with authorized values category WITHDRAWN 2) Define in WITHDRAWN an authorized values 1 with description 'dropped in trash' 3) Define in WITHDRAWN an authorized values 2 with description empty 4) Edit an item with withdrawn=1 5) Checkout this item => You see 'Item has been withdrawn (dropped in trash)' 6) Edit an item with withdrawn=2 7) Checkout this item => You see 'Item has been withdrawn'
Created attachment 84274 [details] [review] Bug 21877: Use AuthorisedValues.GetDescriptionByKohaField for notforloan in checkout Using only TT plugin for authorised value description is a better code. Test plan : Part 1: 1.1) On a catalog with items.notforloan defined with authorized values category NOT_LOAN 1.2) Define in NOT_LOAN an authorized values 1 with description 'at the beach' 1.3) Define in NOT_LOAN an authorized values 2 with description empty Part 2: 2.1) Set preference 'AllowNotForLoanOverride' to 'Don't allow' 2.2) Edit an item with notforloan=1 2.3) Checkout this item => You see 'Item not for loan (at the beach)' 2.4) Edit an item with notforloan=2 2.5) Checkout this item => You see 'Item not for loan' Part 3: 3.1) Set preference 'AllowNotForLoanOverride' to 'Allow' 3.2) Edit an item with notforloan=1 3.3) Checkout this item => You see 'Item is normally not for loan (at the beach)' 3.4) Edit an item with notforloan=2 3.5) Checkout this item => You see 'Item is normally not for loan'
Both patches seems to be fine and I will probably sign them off. When I do a test with NOT_LOAN and AV = 2, I see this : "Item not for loan .". There is a space character created by the dot no matter what the system preference setting is. I wonder if it can be fixed as well too.
Created attachment 84436 [details] [review] Bug 21877: Show authorized value description for withdrawn in checkout During checkout the withdrawn information is displayed in a message : "Item has been withdrawn". Like in other pages and like notforloan, we should display the withdrawn authorized value description. This patch adds this display. Add <span> so that translation does not change. Also adds class 'co-withdrawn' (like in checkin 'ci-*' classes) to ease hidding this new information via CSS. Test plan : 1) On a catalog with items.withdrawn defined with authorized values category WITHDRAWN 2) Define in WITHDRAWN an authorized values 1 with description 'dropped in trash' 3) Define in WITHDRAWN an authorized values 2 with description empty 4) Edit an item with withdrawn=1 5) Checkout this item => You see 'Item has been withdrawn (dropped in trash)' 6) Edit an item with withdrawn=2 7) Checkout this item => You see 'Item has been withdrawn' Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>
Created attachment 84437 [details] [review] Bug 21877: Use AuthorisedValues.GetDescriptionByKohaField for notforloan in checkout Using only TT plugin for authorised value description is a better code. Test plan : Part 1: 1.1) On a catalog with items.notforloan defined with authorized values category NOT_LOAN 1.2) Define in NOT_LOAN an authorized values 1 with description 'at the beach' 1.3) Define in NOT_LOAN an authorized values 2 with description empty Part 2: 2.1) Set preference 'AllowNotForLoanOverride' to 'Don't allow' 2.2) Edit an item with notforloan=1 2.3) Checkout this item => You see 'Item not for loan (at the beach)' 2.4) Edit an item with notforloan=2 2.5) Checkout this item => You see 'Item not for loan' Part 3: 3.1) Set preference 'AllowNotForLoanOverride' to 'Allow' 3.2) Edit an item with notforloan=1 3.3) Checkout this item => You see 'Item is normally not for loan (at the beach)' 3.4) Edit an item with notforloan=2 3.5) Checkout this item => You see 'Item is normally not for loan' Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>
Created attachment 84448 [details] [review] Bug 21877: Show authorized value description for withdrawn in checkout During checkout the withdrawn information is displayed in a message : "Item has been withdrawn". Like in other pages and like notforloan, we should display the withdrawn authorized value description. This patch adds this display. Add <span> so that translation does not change. Also adds class 'co-withdrawn' (like in checkin 'ci-*' classes) to ease hidding this new information via CSS. Test plan : 1) On a catalog with items.withdrawn defined with authorized values category WITHDRAWN 2) Define in WITHDRAWN an authorized values 1 with description 'dropped in trash' 3) Define in WITHDRAWN an authorized values 2 with description empty 4) Edit an item with withdrawn=1 5) Checkout this item => You see 'Item has been withdrawn (dropped in trash)' 6) Edit an item with withdrawn=2 7) Checkout this item => You see 'Item has been withdrawn' Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 84449 [details] [review] Bug 21877: Use AuthorisedValues.GetDescriptionByKohaField for notforloan in checkout Using only TT plugin for authorised value description is a better code. Test plan : Part 1: 1.1) On a catalog with items.notforloan defined with authorized values category NOT_LOAN 1.2) Define in NOT_LOAN an authorized values 1 with description 'at the beach' 1.3) Define in NOT_LOAN an authorized values 2 with description empty Part 2: 2.1) Set preference 'AllowNotForLoanOverride' to 'Don't allow' 2.2) Edit an item with notforloan=1 2.3) Checkout this item => You see 'Item not for loan (at the beach)' 2.4) Edit an item with notforloan=2 2.5) Checkout this item => You see 'Item not for loan' Part 3: 3.1) Set preference 'AllowNotForLoanOverride' to 'Allow' 3.2) Edit an item with notforloan=1 3.3) Checkout this item => You see 'Item is normally not for loan (at the beach)' 3.4) Edit an item with notforloan=2 3.5) Checkout this item => You see 'Item is normally not for loan' Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Awesome work all! Pushed to master for 19.05
Pushed to 18.11.x for 18.11.03
pushed to 18.05.x for 18.05.09
Pushed to 17.11.x for 17.11.15
Created attachment 121747 [details] [review] Bug 28177: Add date column and column configuration to uploads This patch adds a "Date added" column to the table showing uploaded files. Column configuration is added to the table, and the date column is hidden by default. To test, apply the patch and restart all. - Go to Tools -> Upload. - If necessary, upload multiple files with the same category. - Use the "Search uploads by category" form to find those uploads. - In the table of uploads, confirm that table settings and other DataTable controls work correctly, including the option to show the "Date added" column. - Go to Administration -> Table settings -> Tools -> upload. - Change the settings for the uploads table. - Return to Uploads and confirm that your changes are reflected in the display of the uploads table. Signed-off-by: Salman Ali <salman@outaouais>