Bad </p> tag removal

esarrouy - February 19, 2009 - 15:33
Project:Inline and link Drupal objects (Linodef)
Version:6.x-1.0-rc2
Component:Filter
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I tested Linodef using FCKEditor (I wich I could use a non WYSIWYG one like BUEditor but some people just can't understand them :) and I had a little problem: a <\p>\n\n tag is inserted that Linodef doesn't detect. I modified a few lines in includes\linodef-filter.inc (arround line 209) and everything works fine :

// ESY (E. Sarrouy, 2009/02/19) Added the following line to remove spaces and other characters ('\n','\t',etc.)
$output = trim($output);
// Remove annoying <p> tag enclosing $output created by line break converter.
if (substr($output, 0, 3) == '<p>') {
    $output = substr($output, 3);
}
if (substr($output, -4, 4) == '</p>') { // ESY (E. Sarrouy, 2009/02/19) changed -5 to -4
    $output = substr($output, 0, -4);   // ESY (E. Sarrouy, 2009/02/19) changed -5 to -4
}

#1

Roi Danton - March 4, 2009 - 13:27

Thx for the suggestion! This is already fixed in 6.x-1.0-dev. For more information see post 9 of this issue: #357319: Problems with "pass-through" of filtered layout of embedded fields.

#2

Roi Danton - March 4, 2009 - 13:27
Status:active» fixed

#3

System Message - March 18, 2009 - 13:30
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.