diff --git a/src-ui/messages.xlf b/src-ui/messages.xlf index 29f7c7441..f3285d1b2 100644 --- a/src-ui/messages.xlf +++ b/src-ui/messages.xlf @@ -737,7 +737,7 @@ src/app/components/manage/settings/settings.component.ts - 692 + 696 @@ -936,15 +936,15 @@ src/app/components/manage/settings/settings.component.html - 370 + 371 src/app/components/manage/settings/settings.component.html - 421 + 423 src/app/components/manage/settings/settings.component.html - 455 + 457 src/app/components/manage/tasks/tasks.component.html @@ -1294,7 +1294,99 @@ src/app/components/manage/settings/settings.component.html - 491 + 493 + + + + Consume Folder + + src/app/components/common/edit-dialog/consumption-template-edit-dialog/consumption-template-edit-dialog.component.ts + 25 + + + + API Upload + + src/app/components/common/edit-dialog/consumption-template-edit-dialog/consumption-template-edit-dialog.component.ts + 29 + + + + Mail Fetch + + src/app/components/common/edit-dialog/consumption-template-edit-dialog/consumption-template-edit-dialog.component.ts + 33 + + + + Create new consumption template + + src/app/components/common/edit-dialog/consumption-template-edit-dialog/consumption-template-edit-dialog.component.ts + 83 + + + + Edit consumption template + + src/app/components/common/edit-dialog/consumption-template-edit-dialog/consumption-template-edit-dialog.component.ts + 87 + + + + Matching algorithm + + src/app/components/common/edit-dialog/correspondent-edit-dialog/correspondent-edit-dialog.component.html + 10 + + + src/app/components/common/edit-dialog/document-type-edit-dialog/document-type-edit-dialog.component.html + 11 + + + src/app/components/common/edit-dialog/storage-path-edit-dialog/storage-path-edit-dialog.component.html + 11 + + + src/app/components/common/edit-dialog/tag-edit-dialog/tag-edit-dialog.component.html + 13 + + + + Matching pattern + + src/app/components/common/edit-dialog/correspondent-edit-dialog/correspondent-edit-dialog.component.html + 11 + + + src/app/components/common/edit-dialog/document-type-edit-dialog/document-type-edit-dialog.component.html + 12 + + + src/app/components/common/edit-dialog/storage-path-edit-dialog/storage-path-edit-dialog.component.html + 12 + + + src/app/components/common/edit-dialog/tag-edit-dialog/tag-edit-dialog.component.html + 14 + + + + Case insensitive + + src/app/components/common/edit-dialog/correspondent-edit-dialog/correspondent-edit-dialog.component.html + 12 + + + src/app/components/common/edit-dialog/document-type-edit-dialog/document-type-edit-dialog.component.html + 13 + + + src/app/components/common/edit-dialog/storage-path-edit-dialog/storage-path-edit-dialog.component.html + 13 + + + src/app/components/common/edit-dialog/tag-edit-dialog/tag-edit-dialog.component.html + 15 @@ -1477,6 +1569,14 @@ src/app/components/manage/settings/settings.component.html 135 + + src/app/components/manage/settings/settings.component.html + 346 + + + src/app/components/manage/settings/settings.component.html + 384 + Create new user group @@ -1525,7 +1625,7 @@ src/app/components/manage/settings/settings.component.html - 420 + 422 @@ -1612,11 +1712,11 @@ src/app/components/manage/settings/settings.component.html - 396 + 398 src/app/components/manage/settings/settings.component.html - 482 + 484 src/app/components/manage/tasks/tasks.component.html @@ -1691,7 +1791,7 @@ src/app/components/manage/settings/settings.component.html - 371 + 372 @@ -1941,19 +2041,19 @@ src/app/components/manage/settings/settings.component.html - 346 + 347 src/app/components/manage/settings/settings.component.html - 383 + 385 src/app/components/manage/settings/settings.component.html - 435 + 437 src/app/components/manage/settings/settings.component.html - 470 + 472 @@ -2182,11 +2282,11 @@ src/app/components/manage/settings/settings.component.html - 422 + 424 src/app/components/manage/settings/settings.component.html - 443 + 445 @@ -2402,17 +2502,28 @@ src/app/components/manage/settings/settings.component.html - 382 + 383 src/app/components/manage/settings/settings.component.html - 434 + 436 src/app/components/manage/settings/settings.component.html 469 + + Edit permissions also grant viewing permissions + + src/app/components/common/input/permissions/permissions-form/permissions-form.component.html + 64 + + + src/app/components/manage/settings/settings.component.html + 206 + + Add item @@ -2469,18 +2580,25 @@ 35 - - Set Permissions + + Set permissions src/app/components/common/permissions-dialog/permissions-dialog.component.ts 26 + + Edit permissions for + + src/app/components/common/permissions-dialog/permissions-dialog.component.ts + 30 + + Note that permissions set here will override any existing permissions src/app/components/common/permissions-dialog/permissions-dialog.component.ts - 43 + 55 @@ -2512,7 +2630,7 @@ src/app/components/manage/settings/settings.component.html - 408 + 410 @@ -3042,15 +3160,15 @@ src/app/components/manage/settings/settings.component.html - 372 + 373 src/app/components/manage/settings/settings.component.html - 423 + 425 src/app/components/manage/settings/settings.component.html - 458 + 460 src/app/components/manage/tasks/tasks.component.html @@ -3451,19 +3569,19 @@ src/app/components/manage/settings/settings.component.ts - 797 + 801 src/app/components/manage/settings/settings.component.ts - 849 + 853 src/app/components/manage/settings/settings.component.ts - 908 + 912 src/app/components/manage/settings/settings.component.ts - 966 + 970 @@ -3482,19 +3600,19 @@ src/app/components/manage/settings/settings.component.ts - 799 + 803 src/app/components/manage/settings/settings.component.ts - 851 + 855 src/app/components/manage/settings/settings.component.ts - 910 + 914 src/app/components/manage/settings/settings.component.ts - 968 + 972 @@ -4680,6 +4798,10 @@ src/app/components/manage/management-list/management-list.component.ts 300 + + src/app/components/manage/settings/settings.component.ts + 1010 + Start tour @@ -5042,308 +5164,315 @@ No mail accounts defined. src/app/components/manage/settings/settings.component.html - 352 + 353 Mail rules src/app/components/manage/settings/settings.component.html - 358 + 359 Add Rule src/app/components/manage/settings/settings.component.html - 363 + 364 No mail rules defined. src/app/components/manage/settings/settings.component.html - 389 + 391 Users & Groups src/app/components/manage/settings/settings.component.html - 403 + 405 Add User src/app/components/manage/settings/settings.component.html - 413 + 415 Add Group src/app/components/manage/settings/settings.component.html - 448 + 450 Error retrieving groups src/app/components/manage/settings/settings.component.ts - 302 + 304 Error retrieving mail rules src/app/components/manage/settings/settings.component.ts - 326 + 328 Error retrieving mail accounts src/app/components/manage/settings/settings.component.ts - 334 + 336 Error retrieving users src/app/components/manage/settings/settings.component.ts - 351 + 354 Saved view "" deleted. src/app/components/manage/settings/settings.component.ts - 554 + 558 Settings saved src/app/components/manage/settings/settings.component.ts - 676 + 680 Settings were saved successfully. src/app/components/manage/settings/settings.component.ts - 677 + 681 Settings were saved successfully. Reload is required to apply some changes. src/app/components/manage/settings/settings.component.ts - 681 + 685 Reload now src/app/components/manage/settings/settings.component.ts - 682 + 686 Use system language src/app/components/manage/settings/settings.component.ts - 700 + 704 Use date format of display language src/app/components/manage/settings/settings.component.ts - 707 + 711 Error while storing settings on server. src/app/components/manage/settings/settings.component.ts - 727 + 731 Password has been changed, you will be logged out momentarily. src/app/components/manage/settings/settings.component.ts - 769 + 773 Saved user "". src/app/components/manage/settings/settings.component.ts - 776 + 780 Error saving user. src/app/components/manage/settings/settings.component.ts - 787 + 791 Confirm delete user account src/app/components/manage/settings/settings.component.ts - 795 + 799 This operation will permanently delete this user account. src/app/components/manage/settings/settings.component.ts - 796 + 800 Deleted user src/app/components/manage/settings/settings.component.ts - 805 + 809 Error deleting user. src/app/components/manage/settings/settings.component.ts - 812 + 816 Saved group "". src/app/components/manage/settings/settings.component.ts - 830 + 834 Error saving group. src/app/components/manage/settings/settings.component.ts - 839 + 843 Confirm delete user group src/app/components/manage/settings/settings.component.ts - 847 + 851 This operation will permanently delete this user group. src/app/components/manage/settings/settings.component.ts - 848 + 852 Deleted group src/app/components/manage/settings/settings.component.ts - 857 + 861 Error deleting group. src/app/components/manage/settings/settings.component.ts - 864 + 868 Saved account "". src/app/components/manage/settings/settings.component.ts - 887 + 891 Error saving account. src/app/components/manage/settings/settings.component.ts - 898 + 902 Confirm delete mail account src/app/components/manage/settings/settings.component.ts - 906 + 910 This operation will permanently delete this mail account. src/app/components/manage/settings/settings.component.ts - 907 + 911 Deleted mail account src/app/components/manage/settings/settings.component.ts - 916 + 920 Error deleting mail account. src/app/components/manage/settings/settings.component.ts - 925 + 929 Saved rule "". src/app/components/manage/settings/settings.component.ts - 945 + 949 Error saving rule. src/app/components/manage/settings/settings.component.ts - 956 + 960 Confirm delete mail rule src/app/components/manage/settings/settings.component.ts - 964 + 968 This operation will permanently delete this mail rule. src/app/components/manage/settings/settings.component.ts - 965 + 969 Deleted mail rule src/app/components/manage/settings/settings.component.ts - 974 + 978 Error deleting mail rule. src/app/components/manage/settings/settings.component.ts - 982 + 986 + + + + Permissions updated + + src/app/components/manage/settings/settings.component.ts + 1006 diff --git a/src-ui/src/app/components/common/edit-dialog/mail-rule-edit-dialog/mail-rule-edit-dialog.component.html b/src-ui/src/app/components/common/edit-dialog/mail-rule-edit-dialog/mail-rule-edit-dialog.component.html index 869ba12ec..acf855f27 100644 --- a/src-ui/src/app/components/common/edit-dialog/mail-rule-edit-dialog/mail-rule-edit-dialog.component.html +++ b/src-ui/src/app/components/common/edit-dialog/mail-rule-edit-dialog/mail-rule-edit-dialog.component.html @@ -6,7 +6,7 @@
-
+
@@ -15,7 +15,7 @@
-
+

Paperless will only process mails that match all of the filters specified below.

@@ -23,7 +23,7 @@
-
+

Assignments specified here will supersede any consumption templates.

diff --git a/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.html b/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.html index 2ed42fd62..863c9ecee 100644 --- a/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.html +++ b/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.html @@ -5,7 +5,7 @@
-

+

diff --git a/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.spec.ts b/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.spec.ts index bf9b55c95..47f245d0a 100644 --- a/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.spec.ts +++ b/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.spec.ts @@ -19,7 +19,7 @@ const set_permissions = { users: [1], groups: [], }, - edit: { + change: { users: [1], groups: [], }, @@ -78,6 +78,10 @@ describe('PermissionsDialogComponent', () => { }) it('should return permissions', () => { + expect(component.permissions).toEqual({ + owner: null, + set_permissions: null, + }) component.form.get('permissions_form').setValue(set_permissions) expect(component.permissions).toEqual(set_permissions) }) @@ -87,4 +91,16 @@ describe('PermissionsDialogComponent', () => { component.cancelClicked() expect(closeSpy).toHaveBeenCalled() }) + + it('should support edit permissions on object', () => { + let obj = { + id: 1, + name: 'account1', + owner: set_permissions.owner, + permissions: set_permissions.set_permissions, + } + component.object = obj + expect(component.title).toEqual(`Edit permissions for ${obj.name}`) + expect(component.permissions).toEqual(set_permissions) + }) }) diff --git a/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.ts b/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.ts index f1b922175..b3aa6d2df 100644 --- a/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.ts +++ b/src-ui/src/app/components/common/permissions-dialog/permissions-dialog.component.ts @@ -1,6 +1,7 @@ import { Component, EventEmitter, Input, Output } from '@angular/core' import { FormControl, FormGroup } from '@angular/forms' import { NgbActiveModal } from '@ng-bootstrap/ng-bootstrap' +import { ObjectWithPermissions } from 'src/app/data/object-with-permissions' import { PaperlessUser } from 'src/app/data/paperless-user' import { UserService } from 'src/app/services/rest/user.service' @@ -11,6 +12,7 @@ import { UserService } from 'src/app/services/rest/user.service' }) export class PermissionsDialogComponent { users: PaperlessUser[] + private o: ObjectWithPermissions = undefined constructor( public activeModal: NgbActiveModal, @@ -23,7 +25,23 @@ export class PermissionsDialogComponent { public confirmClicked = new EventEmitter() @Input() - title = $localize`Set Permissions` + title = $localize`Set permissions` + + @Input() + set object(o: ObjectWithPermissions) { + this.o = o + this.title = $localize`Edit permissions for ` + o['name'] + this.form.patchValue({ + permissions_form: { + owner: o.owner, + set_permissions: o.permissions, + }, + }) + } + + get object(): ObjectWithPermissions { + return this.o + } form = new FormGroup({ permissions_form: new FormControl(), diff --git a/src-ui/src/app/components/manage/settings/settings.component.html b/src-ui/src/app/components/manage/settings/settings.component.html index cc5e7a15e..e2e4ed5c5 100644 --- a/src-ui/src/app/components/manage/settings/settings.component.html +++ b/src-ui/src/app/components/manage/settings/settings.component.html @@ -343,6 +343,7 @@
+
@@ -380,6 +381,7 @@
+
diff --git a/src-ui/src/app/components/manage/settings/settings.component.spec.ts b/src-ui/src/app/components/manage/settings/settings.component.spec.ts index 1bcbe37f8..929182da8 100644 --- a/src-ui/src/app/components/manage/settings/settings.component.spec.ts +++ b/src-ui/src/app/components/manage/settings/settings.component.spec.ts @@ -52,6 +52,9 @@ import { TagsComponent } from '../../common/input/tags/tags.component' import { TextComponent } from '../../common/input/text/text.component' import { PageHeaderComponent } from '../../common/page-header/page-header.component' import { SettingsComponent } from './settings.component' +import { PermissionsDialogComponent } from '../../common/permissions-dialog/permissions-dialog.component' +import { PermissionsFormComponent } from '../../common/input/permissions/permissions-form/permissions-form.component' +import { IfOwnerDirective } from 'src/app/directives/if-owner.directive' const savedViews = [ { id: 1, name: 'view1' }, @@ -70,8 +73,8 @@ const mailAccounts = [ { id: 2, name: 'account2' }, ] const mailRules = [ - { id: 1, name: 'rule1', owner: 1 }, - { id: 2, name: 'rule2', owner: 2 }, + { id: 1, name: 'rule1', owner: 1, account: 1 }, + { id: 2, name: 'rule2', owner: 2, account: 2 }, ] describe('SettingsComponent', () => { @@ -110,6 +113,9 @@ describe('SettingsComponent', () => { MailRuleEditDialogComponent, PermissionsUserComponent, PermissionsGroupComponent, + IfOwnerDirective, + PermissionsDialogComponent, + PermissionsFormComponent, ], providers: [CustomDatePipe, DatePipe, PermissionsGuard], imports: [ @@ -591,4 +597,69 @@ describe('SettingsComponent', () => { expect(listAllSpy).toHaveBeenCalled() expect(toastInfoSpy).toHaveBeenCalledWith('Deleted mail rule') }) + + it('should support edit permissions on mail rule objects', () => { + completeSetup() + const perms = { + owner: 99, + set_permissions: { + view: { + users: [1], + groups: [2], + }, + change: { + users: [3], + groups: [4], + }, + }, + } + let modal: NgbModalRef + modalService.activeInstances.subscribe((refs) => (modal = refs[0])) + const toastErrorSpy = jest.spyOn(toastService, 'showError') + const toastInfoSpy = jest.spyOn(toastService, 'showInfo') + const rulePatchSpy = jest.spyOn(mailRuleService, 'patch') + component.editPermissions(mailRules[0] as PaperlessMailRule) + expect(modal).not.toBeUndefined() + let dialog = modal.componentInstance as PermissionsDialogComponent + expect(dialog.object).toEqual(mailRules[0]) + + rulePatchSpy.mockReturnValueOnce( + throwError(() => new Error('error saving perms')) + ) + dialog.confirmClicked.emit(perms) + expect(rulePatchSpy).toHaveBeenCalled() + expect(toastErrorSpy).toHaveBeenCalled() + rulePatchSpy.mockReturnValueOnce(of(mailRules[0] as PaperlessMailRule)) + dialog.confirmClicked.emit(perms) + expect(toastInfoSpy).toHaveBeenCalledWith('Permissions updated') + + modalService.dismissAll() + }) + + it('should support edit permissions on mail account objects', () => { + completeSetup() + const perms = { + owner: 99, + set_permissions: { + view: { + users: [1], + groups: [2], + }, + change: { + users: [3], + groups: [4], + }, + }, + } + let modal: NgbModalRef + modalService.activeInstances.subscribe((refs) => (modal = refs[0])) + const accountPatchSpy = jest.spyOn(mailAccountService, 'patch') + component.editPermissions(mailAccounts[0] as PaperlessMailAccount) + expect(modal).not.toBeUndefined() + let dialog = modal.componentInstance as PermissionsDialogComponent + expect(dialog.object).toEqual(mailAccounts[0]) + dialog = modal.componentInstance as PermissionsDialogComponent + dialog.confirmClicked.emit(perms) + expect(accountPatchSpy).toHaveBeenCalled() + }) }) diff --git a/src-ui/src/app/components/manage/settings/settings.component.ts b/src-ui/src/app/components/manage/settings/settings.component.ts index 520cd9520..9c7586007 100644 --- a/src-ui/src/app/components/manage/settings/settings.component.ts +++ b/src-ui/src/app/components/manage/settings/settings.component.ts @@ -51,6 +51,8 @@ import { PermissionType, PermissionsService, } from 'src/app/services/permissions.service' +import { PermissionsDialogComponent } from '../../common/permissions-dialog/permissions-dialog.component' +import { AbstractPaperlessService } from 'src/app/services/rest/abstract-paperless-service' enum SettingsNavIDs { General = 1, @@ -307,14 +309,14 @@ export class SettingsComponent (!this.mailAccounts || !this.mailRules) ) { this.mailAccountService - .listAll() + .listAll(null, null, { full_perms: true }) .pipe(first()) .subscribe({ next: (r) => { this.mailAccounts = r.results this.mailRuleService - .listAll() + .listAll(null, null, { full_perms: true }) .pipe(first()) .subscribe({ next: (r) => { @@ -346,6 +348,7 @@ export class SettingsComponent .subscribe({ next: (r) => { this.users = r.results + this.initialize(false) }, error: (e) => { this.toastService.showError($localize`Error retrieving users`, e) @@ -382,9 +385,8 @@ export class SettingsComponent } } - if (this.users && this.groups) { + if (this.users) { this.emptyGroup(this.usersGroup) - this.emptyGroup(this.groupsGroup) for (let user of this.users) { storeData.usersGroup[user.id.toString()] = { @@ -411,7 +413,9 @@ export class SettingsComponent }) ) } - + } + if (this.groups) { + this.emptyGroup(this.groupsGroup) for (let group of this.groups) { storeData.groupsGroup[group.id.toString()] = { id: group.id, @@ -887,10 +891,12 @@ export class SettingsComponent $localize`Saved account "${newMailAccount.name}".` ) this.mailAccountService.clearCache() - this.mailAccountService.listAll().subscribe((r) => { - this.mailAccounts = r.results - this.initialize() - }) + this.mailAccountService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailAccounts = r.results + this.initialize() + }) }) modal.componentInstance.failed .pipe(takeUntil(this.unsubscribeNotifier)) @@ -915,10 +921,12 @@ export class SettingsComponent modal.close() this.toastService.showInfo($localize`Deleted mail account`) this.mailAccountService.clearCache() - this.mailAccountService.listAll().subscribe((r) => { - this.mailAccounts = r.results - this.initialize(true) - }) + this.mailAccountService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailAccounts = r.results + this.initialize(true) + }) }, error: (e) => { this.toastService.showError( @@ -944,11 +952,13 @@ export class SettingsComponent .subscribe((newMailRule) => { this.toastService.showInfo($localize`Saved rule "${newMailRule.name}".`) this.mailRuleService.clearCache() - this.mailRuleService.listAll().subscribe((r) => { - this.mailRules = r.results + this.mailRuleService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailRules = r.results - this.initialize(true) - }) + this.initialize(true) + }) }) modal.componentInstance.failed .pipe(takeUntil(this.unsubscribeNotifier)) @@ -973,10 +983,12 @@ export class SettingsComponent modal.close() this.toastService.showInfo($localize`Deleted mail rule`) this.mailRuleService.clearCache() - this.mailRuleService.listAll().subscribe((r) => { - this.mailRules = r.results - this.initialize(true) - }) + this.mailRuleService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailRules = r.results + this.initialize(true) + }) }, error: (e) => { this.toastService.showError($localize`Error deleting mail rule.`, e) @@ -984,4 +996,30 @@ export class SettingsComponent }) }) } + + editPermissions(object: PaperlessMailRule | PaperlessMailAccount) { + const modal = this.modalService.open(PermissionsDialogComponent, { + backdrop: 'static', + }) + const dialog: PermissionsDialogComponent = + modal.componentInstance as PermissionsDialogComponent + dialog.object = object + modal.componentInstance.confirmClicked.subscribe((permissions) => { + modal.componentInstance.buttonsEnabled = false + const service: AbstractPaperlessService< + PaperlessMailRule | PaperlessMailAccount + > = 'account' in object ? this.mailRuleService : this.mailAccountService + object.owner = permissions['owner'] + object['set_permissions'] = permissions['set_permissions'] + service.patch(object).subscribe({ + next: () => { + this.toastService.showInfo($localize`Permissions updated`) + modal.close() + }, + error: (e) => { + this.toastService.showError($localize`Error updating permissions`, e) + }, + }) + }) + } }