md-systems / pathauto

DEPRECATED: Pathauto moved back to drupal.org, use the official repository.
https://www.drupal.org/project/pathauto
29 stars 34 forks source link

Replace checkPlain() function and added html entities decode #89

Closed giancarlosotelo closed 9 years ago

giancarlosotelo commented 9 years ago

Test was failing because Value span-class-text-text-span-has-br-a is equal to value text-has-html-tags. So for the text here (string) SafeMarkup::checkPlain('This <span class="text">text</span>...

in PlainTextOutput::renderFromHtml($output); instead of removing the html tags was just removing html entities generated from checkPlain(); 'This &lt;span class=&quot;text&quot;&gt;text&lt;/

So then we still have the words(span, class, etc) in the output. I am not sure if is the way to fix that, decoding first and then removing the html tags.