Project

General

Profile

Bug #1392

Better field values separation for install.xml

Added by Vasily Bezruchkin over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Urgent
Category:
Core
Target version:
Start date:
January 21, 2014
Due date:
% Done:

100%

Estimated time:

Description

Comma does not seem to be the best separator for the multiple values in install.xml

foreach (explode(',', $this->_attributes['values']) as $k => $v)
{
    $array = explode('||', $v);
    if (!isset($array[1]))
    {
        $values[$k + 1] = $v;
    }
    else
    {
        if ($array[0] == '-')
        {
            $array[0] = iaSanitize::alias($array[1]);
        }
        $values[$array[0]] = $array[1];
    }
}

Discuss this.

History

#1

Updated by Vasily Bezruchkin over 5 years ago

  • Priority changed from Normal to Urgent
  • Target version changed from 3.2.7 to 3.1.3
#2

Updated by Batyr Mackenov over 5 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Added the :: as a higher-priority separator. Commas could still be used.

commit:804db83

Also available in: Atom PDF