From da3287763591a55651cc4279d37187e49d656b1b Mon Sep 17 00:00:00 2001 From: shamoon <4887959+shamoon@users.noreply.github.com> Date: Fri, 22 Sep 2023 11:28:23 -0700 Subject: [PATCH] Move mail settings to its own component --- src-ui/e2e/admin/settings.spec.ts | 23 - src-ui/src/app/app-routing.module.ts | 21 +- src-ui/src/app/app.component.ts | 10 +- src-ui/src/app/app.module.ts | 2 + .../admin/settings/settings.component.html | 96 +--- .../admin/settings/settings.component.spec.ts | 255 +--------- .../admin/settings/settings.component.ts | 471 ++++-------------- .../app-frame/app-frame.component.html | 21 +- .../manage/mail/mail.component.html | 83 +++ .../manage/mail/mail.component.scss | 0 .../manage/mail/mail.component.spec.ts | 304 +++++++++++ .../components/manage/mail/mail.component.ts | 232 +++++++++ .../app/services/rest/mail-account.service.ts | 4 +- 13 files changed, 771 insertions(+), 751 deletions(-) create mode 100644 src-ui/src/app/components/manage/mail/mail.component.html create mode 100644 src-ui/src/app/components/manage/mail/mail.component.scss create mode 100644 src-ui/src/app/components/manage/mail/mail.component.spec.ts create mode 100644 src-ui/src/app/components/manage/mail/mail.component.ts diff --git a/src-ui/e2e/admin/settings.spec.ts b/src-ui/e2e/admin/settings.spec.ts index 0dc2532e7..00bdde029 100644 --- a/src-ui/e2e/admin/settings.spec.ts +++ b/src-ui/e2e/admin/settings.spec.ts @@ -2,24 +2,6 @@ import { test, expect } from '@playwright/test' const REQUESTS_HAR = 'e2e/admin/requests/api-settings.har' -test('should post settings on save', async ({ page }) => { - await page.routeFromHAR(REQUESTS_HAR, { notFound: 'fallback' }) - await page.goto('/settings') - await page.getByLabel('Use system setting').click() - await page.getByRole('button', { name: 'Save' }).scrollIntoViewIfNeeded() - const updatePromise = page.waitForRequest((request) => { - const data = request.postDataJSON() - const isValid = data['settings'] != null - return ( - isValid && - request.method() === 'POST' && - request.url().includes('/api/ui_settings/') - ) - }) - await page.getByRole('button', { name: 'Save' }).click() - await updatePromise -}) - test('should activate / deactivate save button when settings change', async ({ page, }) => { @@ -89,11 +71,6 @@ test('should support tab direct navigation', async ({ page }) => { 'aria-selected', 'true' ) - await page.goto('/settings/mail') - await expect(page.getByRole('tab', { name: 'Mail' })).toHaveAttribute( - 'aria-selected', - 'true' - ) await page.goto('/settings/usersgroups') await expect( page.getByRole('tab', { name: 'Users & Groups' }) diff --git a/src-ui/src/app/app-routing.module.ts b/src-ui/src/app/app-routing.module.ts index f3e6b518a..95a4c9f24 100644 --- a/src-ui/src/app/app-routing.module.ts +++ b/src-ui/src/app/app-routing.module.ts @@ -22,6 +22,7 @@ import { PermissionType, } from './services/permissions.service' import { ConsumptionTemplatesListComponent } from './components/manage/consumption-templates-list/consumption-templates-list.component' +import { MailComponent } from './components/manage/mail/mail.component' export const routes: Routes = [ { path: '', redirectTo: 'dashboard', pathMatch: 'full' }, @@ -143,6 +144,10 @@ export const routes: Routes = [ }, }, }, + { + path: 'settings/mail', + redirectTo: '/mail', + }, { path: 'settings', component: SettingsComponent, @@ -167,11 +172,6 @@ export const routes: Routes = [ }, }, }, - { - path: 'settings/:section', - component: SettingsComponent, - canDeactivate: [DirtyFormGuard], - }, { path: 'tasks', component: TasksComponent, @@ -194,6 +194,17 @@ export const routes: Routes = [ }, }, }, + { + path: 'mail', + component: MailComponent, + canActivate: [PermissionsGuard], + data: { + requiredPermission: { + action: PermissionAction.View, + type: PermissionType.MailAccount, + }, + }, + }, ], }, diff --git a/src-ui/src/app/app.component.ts b/src-ui/src/app/app.component.ts index ca8e5ab23..1bed38752 100644 --- a/src-ui/src/app/app.component.ts +++ b/src-ui/src/app/app.component.ts @@ -179,6 +179,14 @@ export class AppComponent implements OnInit, OnDestroy { offset: 0, }, }, + { + anchorId: 'tour.mail', + content: $localize`Manage e-mail accounts and rules for automatically importing documents.`, + route: '/mail', + backdropConfig: { + offset: 0, + }, + }, { anchorId: 'tour.file-tasks', content: $localize`File Tasks shows you documents that have been consumed, are waiting to be, or may have failed during the process.`, @@ -189,7 +197,7 @@ export class AppComponent implements OnInit, OnDestroy { }, { anchorId: 'tour.settings', - content: $localize`Check out the settings for various tweaks to the web app, toggle settings for saved views or setup e-mail checking.`, + content: $localize`Check out the settings for various tweaks to the web app, toggle settings for saved views or manage users.`, route: '/settings', backdropConfig: { offset: 0, diff --git a/src-ui/src/app/app.module.ts b/src-ui/src/app/app.module.ts index f53f24adc..c2313dadd 100644 --- a/src-ui/src/app/app.module.ts +++ b/src-ui/src/app/app.module.ts @@ -125,6 +125,7 @@ import localeSv from '@angular/common/locales/sv' import localeTr from '@angular/common/locales/tr' import localeUk from '@angular/common/locales/uk' import localeZh from '@angular/common/locales/zh' +import { MailComponent } from './components/manage/mail/mail.component' registerLocaleData(localeAf) registerLocaleData(localeAr) @@ -237,6 +238,7 @@ function initializeApp(settings: SettingsService) { ShareLinksDropdownComponent, ConsumptionTemplatesListComponent, ConsumptionTemplateEditDialogComponent, + MailComponent, ], imports: [ BrowserModule, diff --git a/src-ui/src/app/components/admin/settings/settings.component.html b/src-ui/src/app/components/admin/settings/settings.component.html index e2e4ed5c5..ad411aab7 100644 --- a/src-ui/src/app/components/admin/settings/settings.component.html +++ b/src-ui/src/app/components/admin/settings/settings.component.html @@ -11,7 +11,7 @@
Administration
diff --git a/src-ui/src/app/components/manage/mail/mail.component.html b/src-ui/src/app/components/manage/mail/mail.component.html new file mode 100644 index 000000000..3b8518cad --- /dev/null +++ b/src-ui/src/app/components/manage/mail/mail.component.html @@ -0,0 +1,83 @@ + + + + +

+ Mail accounts + +

+
    + +
  • +
    +
    Name
    +
    Server
    +
    Actions
    +
    +
    • + +
  • +
    +
    +
    {{account.imap_server}}
    +
    +
    + + + +
    +
    +
    +
    • + +
    No mail accounts defined.
    +
+ + + +

+ Mail rules + +

+
    + +
  • +
    +
    Name
    +
    Account
    +
    Actions
    +
    +
    • + +
  • +
    +
    +
    {{(mailAccountService.getCached(rule.account) | async)?.name}}
    +
    +
    + + + +
    +
    +
    +
    • + +
    No mail rules defined.
    +
+ + +
+ + +
diff --git a/src-ui/src/app/components/manage/mail/mail.component.scss b/src-ui/src/app/components/manage/mail/mail.component.scss new file mode 100644 index 000000000..e69de29bb diff --git a/src-ui/src/app/components/manage/mail/mail.component.spec.ts b/src-ui/src/app/components/manage/mail/mail.component.spec.ts new file mode 100644 index 000000000..1793232f0 --- /dev/null +++ b/src-ui/src/app/components/manage/mail/mail.component.spec.ts @@ -0,0 +1,304 @@ +import { ComponentFixture, TestBed } from '@angular/core/testing' + +import { MailComponent } from './mail.component' +import { DatePipe } from '@angular/common' +import { HttpClientTestingModule } from '@angular/common/http/testing' +import { RouterTestingModule } from '@angular/router/testing' +import { + NgbModule, + NgbAlertModule, + NgbModalRef, + NgbModal, +} from '@ng-bootstrap/ng-bootstrap' +import { NgSelectModule } from '@ng-select/ng-select' +import { of, throwError } from 'rxjs' +import { routes } from 'src/app/app-routing.module' +import { PaperlessMailAccount } from 'src/app/data/paperless-mail-account' +import { PaperlessMailRule } from 'src/app/data/paperless-mail-rule' +import { IfOwnerDirective } from 'src/app/directives/if-owner.directive' +import { IfPermissionsDirective } from 'src/app/directives/if-permissions.directive' +import { PermissionsGuard } from 'src/app/guards/permissions.guard' +import { CustomDatePipe } from 'src/app/pipes/custom-date.pipe' +import { SafeHtmlPipe } from 'src/app/pipes/safehtml.pipe' +import { MailAccountService } from 'src/app/services/rest/mail-account.service' +import { MailRuleService } from 'src/app/services/rest/mail-rule.service' +import { ConfirmDialogComponent } from '../../common/confirm-dialog/confirm-dialog.component' +import { MailAccountEditDialogComponent } from '../../common/edit-dialog/mail-account-edit-dialog/mail-account-edit-dialog.component' +import { MailRuleEditDialogComponent } from '../../common/edit-dialog/mail-rule-edit-dialog/mail-rule-edit-dialog.component' +import { CheckComponent } from '../../common/input/check/check.component' +import { NumberComponent } from '../../common/input/number/number.component' +import { PasswordComponent } from '../../common/input/password/password.component' +import { PermissionsFormComponent } from '../../common/input/permissions/permissions-form/permissions-form.component' +import { PermissionsGroupComponent } from '../../common/input/permissions/permissions-group/permissions-group.component' +import { PermissionsUserComponent } from '../../common/input/permissions/permissions-user/permissions-user.component' +import { SelectComponent } from '../../common/input/select/select.component' +import { TextComponent } from '../../common/input/text/text.component' +import { PageHeaderComponent } from '../../common/page-header/page-header.component' +import { PermissionsDialogComponent } from '../../common/permissions-dialog/permissions-dialog.component' +import { PermissionsService } from 'src/app/services/permissions.service' +import { ToastService } from 'src/app/services/toast.service' +import { TagsComponent } from '../../common/input/tags/tags.component' +import { FormsModule, ReactiveFormsModule } from '@angular/forms' +import { EditDialogMode } from '../../common/edit-dialog/edit-dialog.component' + +const mailAccounts = [ + { id: 1, name: 'account1' }, + { id: 2, name: 'account2' }, +] +const mailRules = [ + { id: 1, name: 'rule1', owner: 1, account: 1 }, + { id: 2, name: 'rule2', owner: 2, account: 2 }, +] + +describe('MailComponent', () => { + let component: MailComponent + let fixture: ComponentFixture + let mailAccountService: MailAccountService + let mailRuleService: MailRuleService + let modalService: NgbModal + let toastService: ToastService + let permissionsService: PermissionsService + + beforeEach(() => { + TestBed.configureTestingModule({ + declarations: [ + MailComponent, + PageHeaderComponent, + IfPermissionsDirective, + CustomDatePipe, + ConfirmDialogComponent, + CheckComponent, + SafeHtmlPipe, + SelectComponent, + TextComponent, + PasswordComponent, + NumberComponent, + MailAccountEditDialogComponent, + MailRuleEditDialogComponent, + IfOwnerDirective, + TagsComponent, + PermissionsUserComponent, + PermissionsGroupComponent, + PermissionsDialogComponent, + PermissionsFormComponent, + ], + providers: [CustomDatePipe, DatePipe, PermissionsGuard], + imports: [ + NgbModule, + HttpClientTestingModule, + RouterTestingModule.withRoutes(routes), + FormsModule, + ReactiveFormsModule, + NgbAlertModule, + NgSelectModule, + ], + }).compileComponents() + + mailAccountService = TestBed.inject(MailAccountService) + mailRuleService = TestBed.inject(MailRuleService) + modalService = TestBed.inject(NgbModal) + toastService = TestBed.inject(ToastService) + permissionsService = TestBed.inject(PermissionsService) + jest.spyOn(permissionsService, 'currentUserCan').mockReturnValue(true) + jest + .spyOn(permissionsService, 'currentUserHasObjectPermissions') + .mockReturnValue(true) + jest + .spyOn(permissionsService, 'currentUserOwnsObject') + .mockReturnValue(true) + + fixture = TestBed.createComponent(MailComponent) + component = fixture.componentInstance + fixture.detectChanges() + }) + + function completeSetup(excludeService = null) { + if (excludeService !== mailAccountService) { + jest.spyOn(mailAccountService, 'listAll').mockReturnValue( + of({ + all: mailAccounts.map((a) => a.id), + count: mailAccounts.length, + results: (mailAccounts as PaperlessMailAccount[]).concat([]), + }) + ) + } + if (excludeService !== mailRuleService) { + jest.spyOn(mailRuleService, 'listAll').mockReturnValue( + of({ + all: mailRules.map((r) => r.id), + count: mailRules.length, + results: (mailRules as PaperlessMailRule[]).concat([]), + }) + ) + } + + fixture = TestBed.createComponent(MailComponent) + component = fixture.componentInstance + fixture.detectChanges() + } + + it('should show errors on load if load mailAccounts failure', () => { + const toastErrorSpy = jest.spyOn(toastService, 'showError') + jest + .spyOn(mailAccountService, 'listAll') + .mockImplementation(() => + throwError(() => new Error('failed to load mail accounts')) + ) + completeSetup(mailAccountService) + expect(toastErrorSpy).toBeCalled() + }) + + it('should show errors on load if load mailRules failure', () => { + const toastErrorSpy = jest.spyOn(toastService, 'showError') + jest + .spyOn(mailRuleService, 'listAll') + .mockImplementation(() => + throwError(() => new Error('failed to load mail rules')) + ) + completeSetup(mailRuleService) + expect(toastErrorSpy).toBeCalled() + }) + + it('should support edit / create mail account, show error if needed', () => { + completeSetup() + let modal: NgbModalRef + modalService.activeInstances.subscribe((refs) => (modal = refs[0])) + component.editMailAccount(mailAccounts[0] as PaperlessMailAccount) + let editDialog = modal.componentInstance as MailAccountEditDialogComponent + const toastErrorSpy = jest.spyOn(toastService, 'showError') + const toastInfoSpy = jest.spyOn(toastService, 'showInfo') + editDialog.failed.emit() + expect(toastErrorSpy).toBeCalled() + editDialog.succeeded.emit(mailAccounts[0]) + expect(toastInfoSpy).toHaveBeenCalledWith( + `Saved account "${mailAccounts[0].name}".` + ) + editDialog.cancel() + component.editMailAccount() + }) + + it('should support delete mail account, show error if needed', () => { + completeSetup() + let modal: NgbModalRef + modalService.activeInstances.subscribe((refs) => (modal = refs[0])) + component.deleteMailAccount(mailAccounts[0] as PaperlessMailAccount) + const deleteDialog = modal.componentInstance as ConfirmDialogComponent + const deleteSpy = jest.spyOn(mailAccountService, 'delete') + const toastErrorSpy = jest.spyOn(toastService, 'showError') + const toastInfoSpy = jest.spyOn(toastService, 'showInfo') + const listAllSpy = jest.spyOn(mailAccountService, 'listAll') + deleteSpy.mockReturnValueOnce( + throwError(() => new Error('error deleting mail account')) + ) + deleteDialog.confirm() + expect(toastErrorSpy).toBeCalled() + deleteSpy.mockReturnValueOnce(of(true)) + deleteDialog.confirm() + expect(listAllSpy).toHaveBeenCalled() + expect(toastInfoSpy).toHaveBeenCalledWith('Deleted mail account') + }) + + it('should support edit / create mail rule, show error if needed', () => { + completeSetup() + let modal: NgbModalRef + modalService.activeInstances.subscribe((refs) => (modal = refs[0])) + component.editMailRule(mailRules[0] as PaperlessMailRule) + const editDialog = modal.componentInstance as MailRuleEditDialogComponent + const toastErrorSpy = jest.spyOn(toastService, 'showError') + const toastInfoSpy = jest.spyOn(toastService, 'showInfo') + editDialog.failed.emit() + expect(toastErrorSpy).toBeCalled() + editDialog.succeeded.emit(mailRules[0]) + expect(toastInfoSpy).toHaveBeenCalledWith( + `Saved rule "${mailRules[0].name}".` + ) + editDialog.cancel() + component.editMailRule() + }) + + it('should support delete mail rule, show error if needed', () => { + completeSetup() + let modal: NgbModalRef + modalService.activeInstances.subscribe((refs) => (modal = refs[0])) + component.deleteMailRule(mailRules[0] as PaperlessMailRule) + const deleteDialog = modal.componentInstance as ConfirmDialogComponent + const deleteSpy = jest.spyOn(mailRuleService, 'delete') + const toastErrorSpy = jest.spyOn(toastService, 'showError') + const toastInfoSpy = jest.spyOn(toastService, 'showInfo') + const listAllSpy = jest.spyOn(mailRuleService, 'listAll') + deleteSpy.mockReturnValueOnce( + throwError(() => new Error('error deleting mail rule')) + ) + deleteDialog.confirm() + expect(toastErrorSpy).toBeCalled() + deleteSpy.mockReturnValueOnce(of(true)) + deleteDialog.confirm() + 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/mail/mail.component.ts b/src-ui/src/app/components/manage/mail/mail.component.ts new file mode 100644 index 000000000..6143ad3c8 --- /dev/null +++ b/src-ui/src/app/components/manage/mail/mail.component.ts @@ -0,0 +1,232 @@ +import { Component, OnInit, OnDestroy } from '@angular/core' +import { NgbModal } from '@ng-bootstrap/ng-bootstrap' +import { Subject, first, takeUntil } from 'rxjs' +import { ObjectWithPermissions } from 'src/app/data/object-with-permissions' +import { PaperlessMailAccount } from 'src/app/data/paperless-mail-account' +import { PaperlessMailRule } from 'src/app/data/paperless-mail-rule' +import { + PermissionsService, + PermissionAction, +} from 'src/app/services/permissions.service' +import { AbstractPaperlessService } from 'src/app/services/rest/abstract-paperless-service' +import { MailAccountService } from 'src/app/services/rest/mail-account.service' +import { MailRuleService } from 'src/app/services/rest/mail-rule.service' +import { ToastService } from 'src/app/services/toast.service' +import { ConfirmDialogComponent } from '../../common/confirm-dialog/confirm-dialog.component' +import { EditDialogMode } from '../../common/edit-dialog/edit-dialog.component' +import { MailAccountEditDialogComponent } from '../../common/edit-dialog/mail-account-edit-dialog/mail-account-edit-dialog.component' +import { MailRuleEditDialogComponent } from '../../common/edit-dialog/mail-rule-edit-dialog/mail-rule-edit-dialog.component' +import { PermissionsDialogComponent } from '../../common/permissions-dialog/permissions-dialog.component' +import { ComponentWithPermissions } from '../../with-permissions/with-permissions.component' + +@Component({ + selector: 'pngx-mail', + templateUrl: './mail.component.html', + styleUrls: ['./mail.component.scss'], +}) +export class MailComponent + extends ComponentWithPermissions + implements OnInit, OnDestroy +{ + mailAccounts: PaperlessMailAccount[] = [] + mailRules: PaperlessMailRule[] = [] + + unsubscribeNotifier: Subject = new Subject() + + constructor( + public mailAccountService: MailAccountService, + public mailRuleService: MailRuleService, + private toastService: ToastService, + private modalService: NgbModal, + public permissionsService: PermissionsService + ) { + super() + } + + ngOnInit(): void { + this.mailAccountService + .listAll(null, null, { full_perms: true }) + .pipe(first(), takeUntil(this.unsubscribeNotifier)) + .subscribe({ + next: (r) => { + this.mailAccounts = r.results + }, + error: (e) => { + this.toastService.showError( + $localize`Error retrieving mail accounts`, + e + ) + }, + }) + + this.mailRuleService + .listAll(null, null, { full_perms: true }) + .pipe(first(), takeUntil(this.unsubscribeNotifier)) + .subscribe({ + next: (r) => { + this.mailRules = r.results + }, + error: (e) => { + this.toastService.showError($localize`Error retrieving mail rules`, e) + }, + }) + } + + ngOnDestroy() { + this.unsubscribeNotifier.next(true) + } + + editMailAccount(account: PaperlessMailAccount = null) { + const modal = this.modalService.open(MailAccountEditDialogComponent, { + backdrop: 'static', + size: 'xl', + }) + modal.componentInstance.dialogMode = account + ? EditDialogMode.EDIT + : EditDialogMode.CREATE + modal.componentInstance.object = account + modal.componentInstance.succeeded + .pipe(takeUntil(this.unsubscribeNotifier)) + .subscribe((newMailAccount) => { + this.toastService.showInfo( + $localize`Saved account "${newMailAccount.name}".` + ) + this.mailAccountService.clearCache() + this.mailAccountService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailAccounts = r.results + }) + }) + modal.componentInstance.failed + .pipe(takeUntil(this.unsubscribeNotifier)) + .subscribe((e) => { + this.toastService.showError($localize`Error saving account.`, e) + }) + } + + deleteMailAccount(account: PaperlessMailAccount) { + const modal = this.modalService.open(ConfirmDialogComponent, { + backdrop: 'static', + }) + modal.componentInstance.title = $localize`Confirm delete mail account` + modal.componentInstance.messageBold = $localize`This operation will permanently delete this mail account.` + modal.componentInstance.message = $localize`This operation cannot be undone.` + modal.componentInstance.btnClass = 'btn-danger' + modal.componentInstance.btnCaption = $localize`Proceed` + modal.componentInstance.confirmClicked.subscribe(() => { + modal.componentInstance.buttonsEnabled = false + this.mailAccountService.delete(account).subscribe({ + next: () => { + modal.close() + this.toastService.showInfo($localize`Deleted mail account`) + this.mailAccountService.clearCache() + this.mailAccountService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailAccounts = r.results + }) + }, + error: (e) => { + this.toastService.showError( + $localize`Error deleting mail account.`, + e + ) + }, + }) + }) + } + + editMailRule(rule: PaperlessMailRule = null) { + const modal = this.modalService.open(MailRuleEditDialogComponent, { + backdrop: 'static', + size: 'xl', + }) + modal.componentInstance.dialogMode = rule + ? EditDialogMode.EDIT + : EditDialogMode.CREATE + modal.componentInstance.object = rule + modal.componentInstance.succeeded + .pipe(takeUntil(this.unsubscribeNotifier)) + .subscribe((newMailRule) => { + this.toastService.showInfo($localize`Saved rule "${newMailRule.name}".`) + this.mailRuleService.clearCache() + this.mailRuleService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailRules = r.results + }) + }) + modal.componentInstance.failed + .pipe(takeUntil(this.unsubscribeNotifier)) + .subscribe((e) => { + this.toastService.showError($localize`Error saving rule.`, e) + }) + } + + deleteMailRule(rule: PaperlessMailRule) { + const modal = this.modalService.open(ConfirmDialogComponent, { + backdrop: 'static', + }) + modal.componentInstance.title = $localize`Confirm delete mail rule` + modal.componentInstance.messageBold = $localize`This operation will permanently delete this mail rule.` + modal.componentInstance.message = $localize`This operation cannot be undone.` + modal.componentInstance.btnClass = 'btn-danger' + modal.componentInstance.btnCaption = $localize`Proceed` + modal.componentInstance.confirmClicked.subscribe(() => { + modal.componentInstance.buttonsEnabled = false + this.mailRuleService.delete(rule).subscribe({ + next: () => { + modal.close() + this.toastService.showInfo($localize`Deleted mail rule`) + this.mailRuleService.clearCache() + this.mailRuleService + .listAll(null, null, { full_perms: true }) + .subscribe((r) => { + this.mailRules = r.results + }) + }, + error: (e) => { + this.toastService.showError($localize`Error deleting mail rule.`, e) + }, + }) + }) + } + + 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) + }, + }) + }) + } + + userCanEdit(obj: ObjectWithPermissions): boolean { + return this.permissionsService.currentUserHasObjectPermissions( + PermissionAction.Change, + obj + ) + } + + userIsOwner(obj: ObjectWithPermissions): boolean { + return this.permissionsService.currentUserOwnsObject(obj) + } +} diff --git a/src-ui/src/app/services/rest/mail-account.service.ts b/src-ui/src/app/services/rest/mail-account.service.ts index 33ec182d5..0d9736c5e 100644 --- a/src-ui/src/app/services/rest/mail-account.service.ts +++ b/src-ui/src/app/services/rest/mail-account.service.ts @@ -50,6 +50,8 @@ export class MailAccountService extends AbstractPaperlessService