View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0017533 | mantisbt | customization | public | 2014-07-21 18:01 | 2014-12-08 00:33 |
Reporter | vboctor | Assigned To | vboctor | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | 1.2.17 | ||||
Target Version | 1.3.0-beta.1 | Fixed in Version | 1.3.0-beta.1 | ||
Summary | 0017533: Adding assoc arrays via config page converts numbers to strings | ||||
Description | If you enter the value below without $g_ and replacing ON with 1 and NOBODY with 55 and 100. You will notice that the array will show up in the configuration with quotes around the numbers like '1', '55', and '100' where it should be numbers.
| ||||
Tags | mantishub | ||||
Here is the test data to simplify testing: Name: default_notify_flags |
|
Here is the pull request: The pull request was tested with the case where constants or numbers are supplied. |
|
MantisBT: master 49a89ed5 2014-10-24 22:16 Details Diff |
Fix assoc array handling in config page This issue likely affected integer handling in general. The fix tackles the following: - Detect and handle properly numeric strings. - Don't do constant replacements when the constants are enclosed in quotation marks. - Variable name fix. Fixes 0017533 |
Affected Issues 0017533 |
|
mod - adm_config_set.php | Diff File |