Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

when config file is not writable, display a warning message #1153

Closed
mattab opened this issue Feb 11, 2010 · 2 comments
Closed

when config file is not writable, display a warning message #1153

mattab opened this issue Feb 11, 2010 · 2 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Feb 11, 2010

When the config file is not writable, and the config file is written to disk, it currently silently fails. All writes to the config file are done on _destruct which means we can't display an error message to the screen when it fails.

Instead, on the screens that require a writable config file, we should display a warning message that the config file is not writable.

The screens affected are:

  • Plugins management: plugin enable/disable
  • Super user password reset
  • Super user email change
@mattab
Copy link
Member Author

mattab commented May 14, 2010

(In [2183]) Fixes #1153 Adding warning message in UI if config file not writable for the super user "Settings" screen, and in the "Plugins" screen

@mattab
Copy link
Member Author

mattab commented Jul 24, 2010

(In [2664]) Refs #1153 adding warnign for general settings screen as SMTP config requires config file writable

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Development

No branches or pull requests

1 participant