View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009455 | mantisbt | administration | public | 2008-07-30 10:41 | 2009-06-26 12:05 |
| Reporter | jblanchon | Assigned To | jreese | ||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 1.1.2 | ||||
| Summary | 0009455: Database configuration does not correctly deal with array | ||||
| Description | In the database configuration page, since 1.1.2 version, it is not possible to set a configuration option which is of type array. Mantis always consider it as a string. Running the adm_config_report.php page and setting a configuration option which is an array | ||||
| Steps To Reproduce |
For instance : Type : Complex The value is the considered as a string : Type : String | ||||
| Tags | No tags attached. | ||||
|
This is already fixed in SVN r5420 (1.1.x) and r5421 (trunk). |
|
|
This still doesn't work (in 1.1.5) |
|
|
Reminder sent to: giallu IIRC, the way complex types needed to be specified changed at this point. Giallu, can you help him? The documentation in the code specifies:
|
|
|
seweso, if you have problems in setting an option in the interface (anyway, I'd still suggest storing it in the config_inc.php file), please detail what are you doing and what do you get. For this purpose, it is preferred to use the mailing list mantisbt-help@lists.sourceforge.net or our forums http://www.mantisbt.org/forums. |
|
|
It works just fine when you use it as jreese wrote. I had a lot of trouble with this one because the easy way to introduce new entries is to copy the existing ones from the view and alter them (which doesn't work since quotation is not allowed, neither are line breaks) Maybe adding some help right on the configuration admin screen would be a good idea |
|
|
yeas, unfortunately, the strictness is a side effect of the fix for the security issue detailed in 0008976. It would be nice to have an improved system for editing the configuration options, ideas are welcome on the mantisbt-devel mailing list. |
|
|
I changed mine to the following which worked for me, sorry I don't use git so I can't make a patch for it, but can anyone see any reason why this isn't ok? adm_config_set.php line 84 |
|
|
It allows me to cut & paste previous values and edit them, I haven't tested with all array types admittedly, but I haven't needed them at all so don't have time to do that sorry. |
|
|
The last ereg_replace should have '"' included also perhaps but I think as long as one format works people can be happy enough with it. |
|
|
MantisBT: master c6393d1d 2008-07-25 12:37 Details Diff |
Loosen regexp matching as suggested in bug 0009426 git-svn-id: http://mantisbt.svn.sourceforge.net/svnroot/mantisbt/trunk@5421 <a class="text" href="/?p=mantisbt.git;a=object;h=f5dc347c">f5dc347c</a>-c33d-0410-90a0-b07cc1902cb9 |
Affected Issues 0009426, 0009455 |
|
| mod - adm_config_set.php | Diff File | ||