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 | ||