?
avatar eaxs
eaxs
10 Jul 2012

I think there's a simple error in: "/libraries/joomla/registry/format/ini.php" which caused the method to always return an empty object:

// Validate the key.
if (preg_match('/[^A-Z0-9_]/i', $key))
{
    // Maybe throw exception?
    continue;
}

Shouldn't it be "not" (!):

if (!preg_match('/[^A-Z0-9_]/i', $key))

To reproduce:

$params = JRegistry::getInstance('test');
$params->loadFile('/path/to/ini_file', 'INI');
avatar eaxs eaxs - open - 10 Jul 2012
avatar phproberto
phproberto - comment - 27 Jul 2012

I think you are true. Sent a pull request to the platform repo:

joomla/joomla-platform#1415

If approved I'll send it also to the joomla-cms

avatar eaxs
eaxs - comment - 28 Jul 2012

Cool, thanks!

avatar nicksavov
nicksavov - comment - 5 May 2013

Hi guys,

What's the status on this one? With the new reabsorption of the platform by the CMS, we can apply this directly to the CMS, if needed.

avatar brianteeman brianteeman - close - 13 Oct 2013
avatar zero-24 zero-24 - close - 13 Oct 2013
avatar brianteeman
brianteeman - comment - 13 Oct 2013

Thanks for reporting this. At this time we are only using github as the place to submit code fixes so I am closing the report here. The actual reporting of issues and testing fixes is still taking place on Joomlacode.

As it has been some time since you opened this issue can you please confirm that it is still valid with the current Master or Joomla 3.2 beta.

If it is still valid please can you open an item on the Joomlacode tracker in the appropriate area.

CMS Bug Reports: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8103

CMS Feature Requests: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8549

avatar zero-24 zero-24 - change - 7 Jul 2015
Labels Added: ?
Removed: ?
Build staging

Add a Comment

Login with GitHub to post a comment