3433 posts Ruben Bristian
  • Most Wanted Bounty Winner
  • Elite Author
  • Sold between 250 000 and 1 000 000 dollars
  • Has been a member for 6-7 years
  • Repeatedly Helped protect Envato Marketplaces against copyright violations
  • Won a Competition
  • Bought between 100 and 499 items
  • Exclusive Author
  • Referred between 500 and 999 users
+5 more
KrownThemes says

I am aware of all the answers from the other threads regarding this issue(especially bitfade’s class and the tutorial with the shortcodes filter order). I’m just trying an alternative and wanted to be sure that i can pass review:

 function rbFilterShortcodes($content){
     $filtered = str_replace(
         array(
             'autop"></p>',
             '<p></p>',
             '<p></div>'
         ),
         array(
             'autop">',
             '',
             '</div>'
         ),
         $content);
     return $filtered;
 }

add_filter("the_content", "rbFilterShortcodes", 99);

All of my shortcodes have the “autop” class at the end, so this is how i’m searching for them.. Also, there are more weird stuff happening, such as the last two filters..

Is it safe to work like this?

3433 posts Ruben Bristian
  • Most Wanted Bounty Winner
  • Elite Author
  • Sold between 250 000 and 1 000 000 dollars
  • Has been a member for 6-7 years
  • Repeatedly Helped protect Envato Marketplaces against copyright violations
  • Won a Competition
  • Bought between 100 and 499 items
  • Exclusive Author
  • Referred between 500 and 999 users
+5 more
KrownThemes says

3433 posts Ruben Bristian
  • Most Wanted Bounty Winner
  • Elite Author
  • Sold between 250 000 and 1 000 000 dollars
  • Has been a member for 6-7 years
  • Repeatedly Helped protect Envato Marketplaces against copyright violations
  • Won a Competition
  • Bought between 100 and 499 items
  • Exclusive Author
  • Referred between 500 and 999 users
+5 more
KrownThemes says

bump! :D

194 posts Dreams Money Can Buy
  • Power Elite Author
  • Power Elite Author: Sold between 1 000 000 - 1 999 999 dollars
  • Bought between 10 and 49 items
  • Referred between 200 and 499 users
  • United Kingdom
  • Has been a member for 3-4 years
  • Exclusive Author
SwiftIdeas says

I’m not sure if the reviewers would have anything against it, but does it work well for you?

- Ed

3433 posts Ruben Bristian
  • Most Wanted Bounty Winner
  • Elite Author
  • Sold between 250 000 and 1 000 000 dollars
  • Has been a member for 6-7 years
  • Repeatedly Helped protect Envato Marketplaces against copyright violations
  • Won a Competition
  • Bought between 100 and 499 items
  • Exclusive Author
  • Referred between 500 and 999 users
+5 more
KrownThemes says

I’m not sure if the reviewers would have anything against it, but does it work well for you? - Ed

Thanks :)

It works really well for me.. Actually, it works the best, since there are always some weird paragraphs appearing before or after elements.. So i can also fix stuff like: <p></section> for example..

194 posts Dreams Money Can Buy
  • Power Elite Author
  • Power Elite Author: Sold between 1 000 000 - 1 999 999 dollars
  • Bought between 10 and 49 items
  • Referred between 200 and 499 users
  • United Kingdom
  • Has been a member for 3-4 years
  • Exclusive Author
SwiftIdeas says

I have had issues before where shortcodes on the same line as a paragraph would cause erroneous tags. Seems that it would also be better as it wouldn’t interfere with any 3rd party shortcodes!

So for every shortcode you create, you have a class on the outer wrap of “autop”?

Might need to take a little look at re-working the way we do it ;)

- Ed

3433 posts Ruben Bristian
  • Most Wanted Bounty Winner
  • Elite Author
  • Sold between 250 000 and 1 000 000 dollars
  • Has been a member for 6-7 years
  • Repeatedly Helped protect Envato Marketplaces against copyright violations
  • Won a Competition
  • Bought between 100 and 499 items
  • Exclusive Author
  • Referred between 500 and 999 users
+5 more
KrownThemes says

So for every shortcode you create, you have a class on the outer wrap of “autop”?

yep .. :D

38 posts
  • Elite Author
  • Referred between 10 and 49 users
  • Sold between 50 000 and 100 000 dollars
  • Exclusive Author
  • Envato Studio (Microlancer) Beta Tester
  • Bought between 10 and 49 items
  • Has been a member for 3-4 years
wpstrong says

My way is using jQuery :

    
//Clean empty p elements

$('#content p:empty').remove();

It helps a bit

by
by
by
by
by
by