Description
Aleisha Amohia
2021-01-21 00:31:45 UTC
Created attachment 115487 [details] [review] Bug 27503 Work In Progress: Rename system preference virtualshelves to AllowLists Created attachment 115496 [details] [review] Bug 27503: Rename system preference virtualshelves to AllowLists This patch renames virtualshelves to AllowLists Test plan: 1. In Koha Administration, search System preferences for virtualshelves 2. Note the virtualshelves preference 3. Apply the patch and run updatedatabase.pl 4. Search system preferances for virtualshelves, there should be no results 5. Search system preferances for AllowLists 6. ensure that the preferance runs correctly, as if it were virtualshelves. Created attachment 115499 [details] [review] Bug 27503: Rename system preference virtualshelves to AllowLists This patch renames virtualshelves to AllowLists Test plan: 1. In Koha Administration, search System preferences for virtualshelves 2. Note the virtualshelves preference 3. Apply the patch and run updatedatabase.pl 4. Search system preferances for virtualshelves, there should be no results 5. Search system preferances for AllowLists 6. ensure that the preferance runs correctly, as if it were virtualshelves. We've had a bit of a discussion about that on on IRC earlier our day. Allowlists is a little confusing as the term is also used as a replacement for "whitelist". As this only appears to affect the OPAC list feature, i'd suggest OpacLists (similar to the cart one. What do you think? I think in that case, OPACLists is a good alternative. We would also want to move the syspref to opac.pref and change the description to be clearer. Created attachment 115647 [details] [review] Bug 27503: (follow up) Rename AllowLists to OpacLists This patch renames virtualshelves to OpacLists Test plan: 1. In Koha Administration, search System preferences for virtualshelves 2. Note the virtualshelves preference 3. Apply the patch and run updatedatabase.pl 4. Search system preferances for virtualshelves, there should be no results 5. Search system preferances for OpacLists 6. ensure that the preferance runs correctly, as if it were virtualshelves. Hi James/Aleisha/Katrin. I like OpacLists much better than virtualshelves (or AllowLists), so a nice improvement! A couple of things I noticed in testing: 1. Should it be OpacLists or OPACLists - the naming of system preferences with either OPAC or Opac in the name is a bit of a mixed bag! I think I would prefer OPAC, so it would then be OPACLists. 2. Database update - is there supposed to be a database update for this change? There is nothing in the patch, so running updatedatabase doesn't do anything. 3. Text for system preference description - Opac should be spelt as OPAC to be consistent with other descriptions for system preferences. Also "saved lists of books" - as the lists could include catalog records for many types of things, maybe a more generic wording could be used, such as records or items. My suggestion would by: "Allow staff and patrons to create and view saved lists of items in the OPAC: " David Hi David, > 1. Should it be OpacLists or OPACLists - the naming of system preferences > with either OPAC or Opac in the name is a bit of a mixed bag! I think I > would prefer OPAC, so it would then be OPACLists. I don't have a preference there, but OPACLists makes sense. > 2. Database update - is there supposed to be a database update for this > change? There is nothing in the patch, so running updatedatabase doesn't do > anything. We definitely need a database update. It looks like something went wrong here, as the initial patch made many more changes to files and included a database update. But doesn't have the new name now :( I believe git bz is to blame. The follow-up had the exact same subject, so one got overwritten with the other. It also needs to be fixed to reflect the new name we picked. > 3. Text for system preference description - Opac should be spelt as OPAC to > be consistent with other descriptions for system preferences. Also "saved > lists of books" - as the lists could include catalog records for many types > of things, maybe a more generic wording could be used, such as records or > items. My suggestion would by: > > "Allow staff and patrons to create and view saved lists of items in the > OPAC: " Actually, I would much prefer records, as lists can only be used on record level and this would lead to a lot of wrong translations (I know in English lists have items, but not the same word in other languages :) ) Comment on attachment 115647 [details] [review] Bug 27503: (follow up) Rename AllowLists to OpacLists Not sure whether this is the best way - renaming title for the follow up patch so it is different from the initial patch name. Comment on attachment 115647 [details] [review] Bug 27503: (follow up) Rename AllowLists to OpacLists Rename file name for follow up patch. Patch no longer applies, I will have a go at rescuing it (not sure how to do exactly, but will try and work it out). |