Re: [development] preg_match bug or regex help needed?

31 Dec 2007

      While this response wasn't the exact solution I was looking for, I want
to say "Thank You". 

When I tried to simplify the problem for the devel list, I actually
transposed a couple things wrong, ... so the double ++ was just a typo,
and I also left of the style= part of the regex,  which made Bevan's
solution possible.  But this convinced me to at put something in code,
even if it was a few extra lines of code.

BTW, this is for the http://drupal.org/project/nicedit.  If anyone else
is not quite satisfied with our wysiwyg editor options, please join me
in working on this new editor option for Drupal.  There are a couple of
open problems documented on the project page, but I think that this is
getting close to usable.

Bevan Rudge wrote:
...
And testing with random whitespace;
...
$text = ' style="  font-style  : italic  ; font-weight  : bold  ;  " ';
$regex = "\s*([a-z][a-z0-9\-]*)\s*:\s*([a-z][a-z0-9\-]*)\s*;";
if (preg_match_all('/'. $regex .'/i', $text, $matches, PREG_SET_ORDER)) {
  print_r($matches);
}
$style = array();
foreach ($matches as $match) {
  $style[$match[1]] = $match[2];
}
print_r($style);
works
Array
...
(
    [0] => Array
        (
            [0] =>   font-style  : italic  ;
            [1] => font-style
            [2] => italic
        )
[1] => Array
        (
            [0] =>  font-weight  : bold  ;
            [1] => font-weight
            [2] => bold
        )
)
Array
(
    [font-style] => italic
    [font-weight] => bold
)
Testing with commonly practiced and supported syntactical errors and
irregularities;
...
$text = 'style=font-style:italic;font-weight:bold';
doesn't all work
...
Array
(
    [0] => Array
        (
            [0] => font-style:italic;
            [1] => font-style
            [2] => italic
        )
)
Array
(
    [font-style] => italic
)
It also doesn't work for non-textual properties, and property names starting
with '-' exclude the '-';
$text = 'style=border-width: 0 2em 10px 0; border-left: 1px solid #000;
-moz-border-radius: foo;';
outputs
...
Array
(
    [0] => Array
        (
            [0] => moz-border-radius: foo;
            [1] => moz-border-radius
            [2] => foo
        )
)
Array
(
    [moz-border-radius] => foo
)
This regex deals with those issues (but not the missing trailing semicolon
';' $regex = "\s*([a-z\-][a-z0-9\-]*)\s*:\s*([^;]*)\s*;";
I'm not sure what the best way to deal with that is given the context.
Perhaps something like
...
$text = 'font-style:italic;font-weight:bold';
$regex = "\s*([a-z\-][a-z0-9\-]*)\s*:\s*([^;]*)\s*";
$styles = explode(';', $text);
$all_matches = array();
foreach($styles as $style) {
  if (preg_match('/'. $regex .'/i', $style, $matches)) {
    print_r($matches);
    $all_matches[] = $matches;
  }
}
$style = array();
foreach ($all_matches as $match) {
  $style[$match[1]] = $match[2];
}
print_r($style);
Bevan/

Re: [development] preg_match bug or regex help needed?

Doug Green