Remove open-ils.permacrud service

[Galen Charlton]

As Dan Wells noted during a presentation at the 2017 conference, the open-ils.permacrud service was effectively deprecated by the advent of open-ils.pcrud. Consequently, the last few invocations of it should be changed to use open-ils.pcrud and the open-ils.permacrud service removed.

Grep tells us:

Open-ILS/web/js/dojo/openils/widget/TranslatorPopup.js:223: OpenSRF.CachedClientSession('open-ils.permacrud').request({
Open-ILS/web/js/dojo/openils/widget/TranslatorPopup.js:224: method : 'open-ils.permacrud.' + method + '.i18n',
Open-ILS/web/js/ui/default/vandelay/vandelay.js:175: ['open-ils.permacrud', 'open-ils.permacrud.search.cbs.atomic'],
Open-ILS/web/js/ui/default/vandelay/vandelay.js:288: ['open-ils.permacrud', 'open-ils.permacrud.search.vqbrad'],
Open-ILS/web/js/ui/default/vandelay/vandelay.js:314: ['open-ils.permacrud', 'open-ils.permacrud.search.vqarad'],
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1752: var method = 'open-ils.permacrud.update.' + rec.classname;
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1755: ['open-ils.permacrud', method],
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1769: var method = 'open-ils.permacrud.search.vqbr';
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1774: ['open-ils.permacrud', method],
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1902: var method = 'open-ils.permacrud' + (isCreate ? '.create.' : '.update.')
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1909: ['open-ils.permacrud', method],
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1934: var method = 'open-ils.permacrud.delete.' + (isAuth ? 'vqarad' : 'vqbrad');
Open-ILS/web/js/ui/default/vandelay/vandelay.js:1937: ['open-ils.permacrud', method],
Open-ILS/xul/staff_client/chrome/content/cat/opac.js:711: 'open-ils.permacrud',
Open-ILS/xul/staff_client/chrome/content/cat/opac.js:712: 'open-ils.permacrud.delete.sre',
Open-ILS/xul/staff_client/chrome/content/cat/opac.js:750: var method = 'open-ils.permacrud.update.' + rec.classname;
Open-ILS/xul/staff_client/chrome/content/cat/opac.js:753: 'open-ils.permacrud', method,

Evergreen master

[Bill Erickson (berick)]

Here's a branch to remove the offending code, but I have not yet tested the code paths where open-ils.permacrud was replaced with open-ils.pcrud. In theory, it should just work -- confirmation pending.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1680566-deprecate-permacrud

[Jeff Godin (jgodin)]

Branch here for testing and signoff: user/jeff/lp1680566_remove_permacrud

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

[Bill Erickson (berick)]

Thanks, Jeff. You caught a few things I missed (and added release, notes, yay). We still need to merge my .js file changes (migrating to open-ils.pcrud) into the final branch.

[Bill Erickson (berick)]

Scratch my previous comment, I missed one of Jeff's commit. +1 to Jeff's branch.

[Galen Charlton (gmc)]

Unfortunately, this isn't ready for 3.1. For create, update, and delete actions, open-ils.pcrud turns out to not be a drop-in replacement for open-ils.permacrud, as the former wants an explicit client session and a transaction opened before doing CUD, whereas open-ils.permacrud did not.

The branch collab/gmcharlt/lp1680566_more_permacrud_removal_work contains a patch that address the mismatch for the following areas:

    - creating, updating, and deleting bib and authority record display
      attributes in Vandelay
    - saving records when the XUL MARC editor is invoked from Vandelay
    - the translator popup-dialog (e.g., as used by the copy status admin page)

Left to do is handling deleting MFHDs and invoking the MARC editor on them (in the XUL client); note that that breakage would be moot with the release of 3.2 and the removal of the XUL client.