TPAC "my list" still referred to as "bookbag" in some places

[Remington Steed]

The TPAC renamed "bookbags" to "lists". Most "bookbag" occurrences are in URL parameters and logs, but a few are more prominent. We should at least correct the following:

src/templates/opac/myopac/lists.tt2
369:[% |l %]Sharing a Bookbag means that the contents
370:of the Bookbag will be visible to others.
371:To see the public view of a shared Bookbag,
372:click on the Bookbag's name in the Bookbag list.[% END %]

src/templates/opac/parts/anon_list.tt2
35:<optgroup label="[% l('Move selected items to bookbag:') %]">
-- Visible in TPAC via: login, add item to temporary list, go to My Account > My Lists, under Temporary List click dropdown Actions for these items, in bold: "Move selected items to bookbag".

src/templates/opac/parts/record/body.tt2
11:[% l("This record has been deleted from the database. We recommend that you remove this title from any bookbags it may have been added to.") %]

src/sql/Pg/950.data.seed-values.sql
4967:INSERT INTO container.biblio_record_entry_bucket_type (code,label) VALUES ('bookbag', oils_i18n_gettext('bookbag', 'Book Bag', 'cbrebt', 'label'));
-- No longer used?

src/sql/Pg/950.data.seed-values.sql
10272:'Produce a CSV file representing a bookbag',

src/sql/Pg/950.data.seed-values.sql
10284:'Facilitates produce a CSV file representing a bookbag by introducing an "items" variable into the TT environment, sorted as dictated according to user params',
-- can we change this wording while we are here? "Facilitates producing" maybe?

src/sql/Pg/950.data.seed-values.sql
10296:'Bookbag CSV', 'container.biblio_record_entry_bucket.csv', 'ContainerCSV',

src/sql/Pg/950.data.seed-values.sql
12057:'Present a warning dialog to the patron when a patron adds a book to a temporary book bag.',
-- Adds a row to config.org_unit_setting_type with this description, but the corresponding label says "book list" instead of "book bag".

src/sql/Pg/950.data.seed-values.sql
12093:'Default list to use when adding to a bookbag',
-- also at line 12099

[Remington Steed (rjs7)]

I have pushed a branch to address all of the above occurrences of "bookbag". Comments are welcome.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/finish_renaming_bookbag_to_list

user/rsteed/finish_renaming_bookbag_to_list

[Melissa Ceraso (mceraso)]

I'm going to take a look at this.

[Ben Shum (bshum)]

We started reviewing this and noticed that with some recent changes in master, it no longer applied cleanly when cherry-picking (the message in lists.tt2 moved positioning).

Also, missing an upgrade script to apply the seed data changes?

[Remington Steed (rjs7)]

Rebased to master, resolved the conflict, added an upgrade script and force pushed to the same branch. Thanks for reviewing, Ben!

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/finish_renaming_bookbag_to_list

user/rsteed/finish_renaming_bookbag_to_list

[Ben Shum (bshum)]

Thanks Remington! Looks good at surface glance. My only concern is the part of the upgrade script where we replace the template for Bookbag CSV. I'd be concerned about what if someone had modified their own template for different information. Us replacing it wholesale would potential break that.

I think in the past, some folks have done some special replace commands to change a few words in a template, rather than copying the whole thing. Maybe we can use something like that here. I'll try to find some examples to work off from.

[Ben Shum (bshum)]

Unrelated, taking off the 2.5 milestone target since it affects string changes again and backporting those might not be good.

[Remington Steed (rjs7)]

I force pushed to my above branch to follow Ben's recommendation. Now it simply replaces "bookbag" with "book list" in the template, rather than replacing the whole template.

[Ben Shum (bshum)]

Thanks Remington! This has been picked to master for inclusion in 2.7.