outdated terminology remaining in the fm_idl.xml file

[Christine Burns]

Evergreen 3.3

I noticed new instances of the word "volume" in Server Admin. Jane Sandberg took a look and identified the issue.

There are several <class> and <field> elements with outdated terminology in their reporter:label attributes in the fm_idl.xml file.

For example, the reporter:label attribute on this line should be "Call Number", but it is "Call Number/Volume": https://github.com/evergreen-library-system/Evergreen/blob/master/Open-ILS/examples/fm_IDL.xml#L3095

New references to outdated terminology "Volume" in Evergreen 3.3

Admin -> Server Admin -> Call Number Prefixes

The page header is "Call Number/Volume Prefix Configuration" & the button = New Call Number/Volume Prefix

Admin -> Server Admin -> Call Number Suffixes

The page header is "Call Number/Volume Suffix Configuration"

Button = New Call Number/Volume Suffix

All label attributes including "Volume" should be updated so Volume is removed.

[Galen Charlton (gmc)]

More: various instances of "copy location" should now be "shelving location"

[Shula Link (slink-g)]

To clarify - would the related permissions to copy location also need to be updated to shelving location for consistence, e.g. UPDATE_COPY_LOCATION?

[Jane Sandberg (sandbergja)]

I think it's safest to leave the permissions as is. Changing their names would also require changes to the code, database migrations, and some careful testing to make sure nobody loses access to things they need.

[Galen Charlton (gmc)]

Agreed, the permission codes should be left as is, but it would be fine to update their stock descriptions.

[Shula Link (slink-g)]

I *think* this got all the labels for both, and the associated translation files:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=1561196389b5accbf489c0ca92f51b06085baf2b

[Jason Boyer (jboyer)]

Hi Shula, I have some feedback about your changes.

First, the initial issue is caused in part by the IDL file in Open-ILS/examples/fm_IDL.xml that doesn't appear to be addressed by your patch, so that will need to be looked at. There are several reporter:label fields still set to Copy Location.

Even though changing comments won't generally be visible to end users changing them is a good idea to keep devs from accidentally re-introducing old terminology.

All of the past upgrade scripts and the translations can (and should) be ignored. Any non-comment changes made to the files in Open-ILS/src/sql/Pg/ would also be performed in a new file you write and put in the upgrade directory. (It looks like that's just the stuff in 950.data.seed-values.sql currently.) The reason for this is that old upgrade scripts are treated as if they're encased in stone; if you want to make new changes you just need to add a new upgrade script to the list. There have been a couple times that this convention has been broken but that's usually to prevent unexpected data loss or crashes discovered after an upgrade.

Translations can be ignored because they'll be rebuilt automatically from your changes to other files.