View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0020069 | mantisbt | code cleanup | public | 2015-09-01 19:05 | 2019-03-16 20:20 |
Reporter | cproensa | Assigned To | atrol | ||
Priority | low | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Target Version | 2.20.0 | Fixed in Version | 2.20.0 | ||
Summary | 0020069: default_email_on_status, misleading comments in config_defaults | ||||
Description | These two variables, in "config_defaults.inc" have a comment which reads "Unused" /**
/**
| ||||
Tags | No tags attached. | ||||
@todo comments mean that there is something in code that has not been finished yet. You will also find a note in administrators guide that the option is not yet implemented. I don't see any place in code where the settings are used for something that can be seen by a user. Do you agree that I can close this issue? |
|
Hello atrol class UserPreferences { which is later user in a config_get(). They set the status for options in user preferencies: |
|
You are right, Seems we should change the source code comment and the administrators guide. |
|
Fis is part of PR https://github.com/mantisbt/mantisbt/pull/1454 |
|
MantisBT: master f980b902 2019-01-25 00:09 Details Diff |
Correct documentation of email preferences Fixes 0020069 |
Affected Issues 0020069 |
|
mod - config_defaults_inc.php | Diff File | ||
mod - docbook/Admin_Guide/en-US/config/defaults.xml | Diff File | ||
MantisBT: master 7d5c990b 2019-02-02 10:36 Details Diff |
Minor doc corrections Address PR comments Issue #0020069 |
Affected Issues 0020069 |
|
mod - docbook/Admin_Guide/en-US/User_Management.xml | Diff File | ||
mod - docbook/Admin_Guide/en-US/config/defaults.xml | Diff File |