import {Component, Input, ViewChild, Renderer2} from '@angular/core';
import {Observable} from 'rxjs';
import {switchMap, map, tap} from 'rxjs/operators';
+import {AuthService} from '@eg/core/auth.service';
import {IdlObject} from '@eg/core/idl.service';
+import {EventService} from '@eg/core/event.service';
+import {NetService} from '@eg/core/net.service';
import {PcrudService} from '@eg/core/pcrud.service';
import {ToastService} from '@eg/share/toast/toast.service';
import {NgbModal, NgbModalOptions} from '@ng-bootstrap/ng-bootstrap';
constructor(
private modal: NgbModal, // required for passing to parent
private toast: ToastService,
+ private auth: AuthService,
+ private net: NetService,
+ private evt: EventService,
private pcrud: PcrudService,
private renderer: Renderer2) {
super(modal); // required for subclassing
getNextCopy(): Observable<any> {
+ if (this.auth.opChangeIsActive()) {
+ // FIXME: kludge for now, opChange has been reverting mid-dialog with batch use when handling permission elevation
+ this.auth.undoOpChange();
+ }
+
if (this.ids.length === 0) {
this.close(this.numSucceeded > 0);
}
.pipe(map(c => this.copy = c));
}
- async replaceOneBarcode(): Promise<any> {
+ replaceOneBarcode() {
this.barcodeExists = false;
// First see if the barcode is in use
return;
}
- this.copy.barcode(this.newBarcode);
- this.pcrud.update(this.copy).toPromise().then(
- async (ok) => {
- this.numSucceeded++;
- this.toast.success(await this.successMsg.current());
- return this.getNextCopy().toPromise();
+ this.net.request(
+ 'open-ils.cat',
+ 'open-ils.cat.update_copy_barcode',
+ this.auth.token(), this.copy.id(), this.newBarcode
+ ).subscribe(
+ (res) => {
+ if (this.evt.parse(res)) {
+ console.error('parsed error response', res);
+ } else {
+ console.log('success', res);
+ this.numSucceeded++;
+ this.successMsg.current().then(m => this.toast.success(m));
+ this.getNextCopy().toPromise();
+ }
},
- async (err) => {
+ (err) => {
+ console.error('error', err);
this.numFailed++;
console.error('Replace barcode failed: ', err);
- this.toast.warning(await this.errorMsg.current());
+ this.errorMsg.current().then(m => this.toast.warning(m));
+ },
+ () => {
+ console.log('finis');
}
);
});
return undef;
}
+__PACKAGE__->register_method(
+ method => "update_copy_barcode",
+ api_name => "open-ils.cat.update_copy_barcode",
+ argc => 3,
+ signature => {
+ desc => q|Updates the barcode for a item, checking for either the UPDATE_COPY permission or the UPDATE_COPY_BARCODE permission.|,
+ params => [
+ {desc => 'Authtoken', type => 'string'},
+ {desc => 'Copy ID', type => 'number'},
+ {desc => 'New Barcode', type => 'string'}
+ ]
+ },
+ return => {desc => 'Returns the copy ID if successful, an ILS event otherwise.', type => 'string'}
+);
+
+sub update_copy_barcode {
+ my ($self, $client, $auth, $copy_id, $barcode) = @_;
+ my $e = new_editor(authtoken => $auth, xact => 1);
+
+ $e->checkauth or return $e->event;
+
+ my $copy = $e->retrieve_asset_copy($copy_id)
+ or return $e->event;
+
+ # make sure there is no undeleted copy (including the same one?) with the same barcode
+ my $existing = $e->search_asset_copy({barcode => $barcode, deleted => 'f'}, {idlist=>1});
+ return OpenILS::Event->new('ITEM_BARCODE_EXISTS') if @$existing;
+
+ # if both of these perm checks fail, we'll report it for UPDATE_COPY_BARCODE as it is more specific
+ return $e->event unless $e->allowed('UPDATE_COPY', $copy->circ_lib) || $e->allowed('UPDATE_COPY_BARCODE', $copy->circ_lib);
+
+ $copy->barcode( $barcode );
+
+ $e->update_asset_copy( $copy ) or return $e->event;
+ $e->commit or return $e->event;
+
+ return $copy->id;
+}
1;
( 640, 'ACCESS_ANGULAR_CIRC', oils_i18n_gettext(640,
'Allow a user to access the experimental Angular circulation interfaces', 'ppl', 'description')),
( 641, 'ADMIN_FUND_ROLLOVER', oils_i18n_gettext(641,
- 'Allow the user to perform fund propagation and rollover', 'ppl', 'description'))
+ 'Allow the user to perform fund propagation and rollover', 'ppl', 'description')),
+ ( 642, 'UPDATE_COPY_BARCODE', oils_i18n_gettext(642,
+ 'Update the barcode for an item.', 'ppl', 'description'))
;
SELECT SETVAL('permission.perm_list_id_seq'::TEXT, 1000);
'CREATE_PAYMENT',
'RENEW_HOLD_OVERRIDE',
'UPDATE_COPY',
+ 'UPDATE_COPY_BARCODE',
'UPDATE_VOLUME',
'ADMIN_TOOLBAR',
'VOLUME_HOLDS');
'MARK_ITEM_ON_ORDER',
'MARK_ITEM_RESHELVING',
'UPDATE_COPY',
+ 'UPDATE_COPY_BARCODE',
'UPDATE_COPY_NOTE',
'UPDATE_IMPORT_ITEM',
'UPDATE_MFHD_RECORD',
'UPDATE_BATCH_COPY',
'UPDATE_BIB_IMPORT_QUEUE',
'UPDATE_COPY',
+ 'UPDATE_COPY_BARCODE',
'UPDATE_FUND',
'UPDATE_FUND_ALLOCATION',
'UPDATE_FUNDING_SOURCE',
--- /dev/null
+BEGIN;
+
+-- check whether patch can be applied
+SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
+
+-- 950.data.seed-values.sql
+
+INSERT INTO permission.perm_list ( id, code, description ) VALUES
+ ( 642, 'UPDATE_COPY_BARCODE', oils_i18n_gettext(642,
+ 'Update the barcode for an item.', 'ppl', 'description'))
+;
+
+-- give this perm to perm groups that already have UPDATE_COPY
+WITH perms_to_add AS
+ (SELECT id FROM
+ permission.perm_list
+ WHERE code IN ('UPDATE_COPY_BARCODE'))
+INSERT INTO permission.grp_perm_map (grp, perm, depth, grantable)
+ SELECT grp, perms_to_add.id as perm, depth, grantable
+ FROM perms_to_add,
+ permission.grp_perm_map
+
+ --- Don't add the permissions if they have already been assigned
+ WHERE grp NOT IN
+ (SELECT DISTINCT grp FROM permission.grp_perm_map
+ INNER JOIN perms_to_add ON perm=perms_to_add.id)
+
+ --- we're going to match the depth of their existing perm
+ AND perm = (
+ SELECT id
+ FROM permission.perm_list
+ WHERE code = 'UPDATE_COPY'
+ );
+
+COMMIT;
}
$scope.copyId = copy.id();
- copy.barcode($scope.barcode2);
- egCore.pcrud.update(copy).then(function(stat) {
- $scope.updateOK = stat;
- $scope.focusBarcode = true;
- holdingsSvc.fetchAgain().then(function (){
- holdingsGridDataProviderRef.refresh();
- });
+ egCore.net.request(
+ 'open-ils.cat',
+ 'open-ils.cat.update_copy_barcode',
+ egCore.auth.token(), $scope.copyId, $scope.barcode2
+ ).then(function(resp) {
+ var evt = egCore.evt.parse(resp);
+ if (evt) {
+ console.log('toast 0 here', evt);
+ } else {
+ $scope.updateOK = stat;
+ $scope.focusBarcode = true;
+ holdingsSvc.fetchAgain().then(function (){
+ holdingsGridDataProviderRef.refresh();
+ });
+ }
});
+ },function(E) {
+ console.log('toast 1 here',E);
+ },function(E) {
+ console.log('toast 2 here',E);
});
$uibModalInstance.close();
}
}
$scope.copyId = copy.id();
- copy.barcode($scope.barcode2);
- egCore.pcrud.update(copy).then(function(stat) {
- $scope.updateOK = stat;
- $scope.focusBarcode = true;
+ egCore.net.request(
+ 'open-ils.cat',
+ 'open-ils.cat.update_copy_barcode',
+ egCore.auth.token(), $scope.copyId, $scope.barcode2
+ ).then(function(resp) {
+ var evt = egCore.evt.parse(resp);
+ if (evt) {
+ console.log('toast 0 here 2', evt);
+ } else {
+ $scope.updateOK = true;
+ $scope.focusBarcode = true;
+ }
});
+
});
});
}
}
$scope.copyId = copy.id();
- copy.barcode($scope.barcode2);
- egCore.pcrud.update(copy).then(function(stat) {
- $scope.updateOK = stat;
- $scope.focusBarcode = true;
- if (stat) service.add_barcode_to_list(copy.barcode());
- $uibModalInstance.close();
+ egCore.net.request(
+ 'open-ils.cat',
+ 'open-ils.cat.update_copy_barcode',
+ egCore.auth.token(), $scope.copyId, $scope.barcode2
+ ).then(function(resp) {
+ var evt = egCore.evt.parse(resp);
+ if (evt) {
+ console.log('toast 0 here 2', evt);
+ } else {
+ $scope.updateOK = true;
+ $scope.focusBarcode = true;
+ if (stat) service.add_barcode_to_list(copy.barcode());
+ $uibModalInstance.close();
+ }
});
});
+ },function(E) {
+ console.log('toast 1 here 2',E);
+ },function(E) {
+ console.log('toast 2 here 2',E);
});
}
* Add patron home library code as a column to the View Holds grid in the staff catalog record details page (LP#1991726)
-
* LP1965446 Option to Disable Title-Level Holds on Bib Records with Parts
This feature adds one global flag and one library setting, respectively:
These customizations are global to the Evergreen installation.
+* LP1929593 UPDATE_COPY_BARCODE permission
+
+ This adds the permission UPDATE_COPY_BARCODE and a new API call,
+
+ open-ils.cat.update_copy_barcode
+
+ which explicitly tests for both UPDATE_COPY_BARCODE and UPDATE_COPY,
+ with either being sufficient for allowing a barcode change. Existing
+ Replace Barcode UI's in both Angular and AngularJS have been modified
+ to use this API call instead of the pcrud service. One side-effect of
+ this has been better surfacing of errors, as errors in pcrud were
+ uncaught and bypassing the normal error handling. This addresses
+ LP1951469.
+
+ The upgrade script gives any permission groups that already have the
+ UPDATE_COPY permission the new UPDATE_COPY_BARCODE permission at the
+ same depth, though it's technically not needed.
+
projects / working / Evergreen.git / commitdiff