P tags around images function

[Fortyfive]

Hi Cooper, your preg_replace does not seem to work:

return preg_replace( '/

\s_(<a .>)?\s(<img ._ />)\s_(</a>)?\s_</p>/iU', '\1\2\3', $content );

Yet this one does for me:

return preg_replace('/

\s_?(<a ._?><img.?></a>|<img.?>)?\s*</p>/s', '\1\2\3', $content);

Wondered if this was a bug - are you having problems with the original? Could you also offer any advice on the validity of the syntax in my suggested replacement?

Ref: http://atg2.hostfortyfive.co.uk/about

Thanks

[cdukes]

I'm able to reproduce this. I'll look at some other regex options and update in the next release.

[Fortyfive]

Great, look forward to it.

[cdukes]

I've decided to remove this function for now. I'm not crazy about maintaining regexes, and I think trying to strip these tags out after the fact is the wrong approach.

I've opened a ticket, requesting a filter on wpautop()'s block elements list. If this is added, we'd be able to prevent the <img>s from ever being wrapped with <p>. Please voice your support for the change if you agree.

Thanks.

[Fortyfive]

Will do, thanks Cooper.