preg_replace on table code - regular expression help

SeenGee

I have modded and setup an implementation of the mozilla midas RTE project, its basically an online wysiwyg for clients in a site backoffice, it allows the client to create tables etc but i need to be able to stop them from using tables over 600 pixels wide, the width is done wih inline css like this

"<table style="width: 700px;" border="1" cellpadding="2" etc....."

I basically need a regular expression that checks the width and reduces it to 600px if its over 600px, if not leave it alone. I am absolutely useless at regular expressions though, would really appreciate some help on this one.

bubblenut

if(preg_match_all('/<table.*?style="width: ([0-9]+)px/', $string, $matches)) {
  $to_replace=array();
  foreach($matches as $match) {
    if($match[1]>600) {
      $to_replace[]=$match[1];
    }
  }
  $to_replace=array_unique($to_replace);
  foreach($to_replace as $width) {
    $string=preg_replace('/(<table.*?style="width: )'.$width.'px/', '\${1}600px', $string);
  }
}

This is completely untested so I'm sure it will need some tweaking. In particular, I remember having some issues with backreferences using the \${1} syntax, I have a funny feeling that when using single quotes you shouldn't escape the $ but I'm not sure. Let me know if you find out 🙂

xblue

hi,

I can't test it now either, but I think it should be possible with just one call to preg_replace

// if the style attribute always comes right after table
// and the width definition is always defined first within it
$string=preg_replace('/(<table style="width: (\d{4,}|[6-9]\d{2})px;/', '<table style="width: 600px;', $string); 

// else it gets slightly more complicated
$string=preg_replace('/(<table (.*? )style="(.*? )width: (\d{4,}|[6-9]\d{2})px;/', '<table $1style="$2width: 600px;', $string);

if the table tags may span over multiple lines, use the s modifier in the 2nd pattern, and if uppercase tags may occur, the i modifier in both.

both patterns actually match 600 which would not need to be replaced, but it shouldn't hurt to replace 600 with 600 😉