Skip to content

Commit

Permalink
Do not request a confirmation after the shared folders ACL page has b…
Browse files Browse the repository at this point in the history
…een saved and is exited.

Signed-off-by: Volker Theile <votdev@gmx.de>
  • Loading branch information
votdev committed Dec 4, 2022
1 parent 5786bc2 commit f6b4e85
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
3 changes: 2 additions & 1 deletion deb/openmediavault/debian/changelog
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
openmediavault (6.1.1-1) stable; urgency=low

*
* Do not request a confirmation after the shared folders ACL page
has been saved and is exited.

-- Volker Theile <volker.theile@openmediavault.org> Sun, 04 Dec 2022 13:19:18 +0100

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,10 @@ export class FormPageComponent
return this.form.formGroup.dirty;
}

markAsPristine(): void {
this.form.formGroup.markAsPristine();
}

loadData(): void {
const request = this.config.request;
if (_.isString(request?.service) && _.isPlainObject(request?.get)) {
Expand Down Expand Up @@ -372,7 +376,7 @@ export class FormPageComponent
// At this point we can assume the form values have been
// submitted and stored, so we can safely mark the form as
// pristine again.
this.form.formGroup.markAsPristine();
this.markAsPristine();
// Display a success notification?
const notificationTitle = _.get(this.routeConfig, 'data.notificationTitle');
if (!_.isEmpty(notificationTitle)) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -358,6 +358,7 @@ export class SharedFolderAclFormPageComponent extends IsDirtyFormPageComponent i
})
)
.subscribe(() => {
this.page.markAsPristine();
this.notificationService.show(
NotificationType.success,
_.get(this.page.routeConfig, 'data.notificationTitle')
Expand Down

0 comments on commit f6b4e85

Please sign in to comment.