Empty P Tags When Using Shortcodes

LovelessDesign said
Do I have to change “the_content” to something else to make it work?

remove_filter( 'the_content', 'wpautop' );
add_filter( 'the_content', 'wpautop' , 99);
add_filter( 'the_content', 'shortcode_unautop',100 );

Using [raw] short codes is fine. Removing these filters is absolutely not fine. Doing so causes huge problems for a lot of plugins.

do_shortcode(ot_get_option( 'textbox1' ));

(With ot_get_option I get the content of a textbox from the theme options)

Hm I really don’t find somehting about it on the www.

mordauk said

LovelessDesign said
Do I have to change “the_content” to something else to make it work?

remove_filter( 'the_content', 'wpautop' );
add_filter( 'the_content', 'wpautop' , 99);
add_filter( 'the_content', 'shortcode_unautop',100 );

Using [raw] short codes is fine. Removing these filters is absolutely not fine. Doing so causes huge problems for a lot of plugins.

No worries now Pippin, we’re not approving themes if they’re stripping out native functionalities.

@LovelessDesign http://wordpress.org/extend/plugins/shortcode-empty-paragraph-fix/

Ivor said

mordauk said

LovelessDesign said
Do I have to change “the_content” to something else to make it work?

remove_filter( 'the_content', 'wpautop' );
add_filter( 'the_content', 'wpautop' , 99);
add_filter( 'the_content', 'shortcode_unautop',100 );

Using [raw] short codes is fine. Removing these filters is absolutely not fine. Doing so causes huge problems for a lot of plugins.

No worries now Pippin, we’re not approving themes if they’re stripping out native functionalities.

@LovelessDesign http://wordpress.org/extend/plugins/shortcode-empty-paragraph-fix/

Thank You for this.

Ivor said
@LovelessDesign http://wordpress.org/extend/plugins/shortcode-empty-paragraph-fix/

Wow, does this really resolve all cases? That’d be cool

    add_filter('the_content', 'shortcode_empty_paragraph_fix');
    function shortcode_empty_paragraph_fix($content)
    {   
        $array = array (
            '<p>[' => '[', 
            ']</p>' => ']', 
            ']<br />' => ']'
        );

        $content = strtr($content, $array);

        return $content;
    }

This plugin does also NOT remove P-Tags for shortcodes I’ve implemented like

do_shortcode(ot_get_option( 'textbox1' ));

EDIT : Ok this would work, but isn’t there a nicer solution?

do_shortcode(shortcode_empty_paragraph_fix(ot_get_option( 'fp_textbox' )));

sevenspark said
Wow, does this really resolve all cases? That’d be cool

    add_filter('the_content', 'shortcode_empty_paragraph_fix');
    function shortcode_empty_paragraph_fix($content)
    {   
        $array = array (
            '<p>[' => '[', 
            ']</p>' => ']', 
            ']<br />' => ']'
        );

        $content = strtr($content, $array);

        return $content;
    }

yes but still there’s a major issue: it gets applied to all shortcodes, including those defined in 3rd party plugins (which is bad).

just spent some time on this, and replaced the above with some regexps:

It basically works the same way but only on your blocklevel shortcodes defined in the array (“col” in the example)

BF

pixelentity said

yes but still there’s a major issue: it gets applied to all shortcodes, including those defined in 3rd party plugins (which is bad).

just spent some time on this, and replaced the above with some regexps:

https://gist.github.com/3776987

It basically works the same way but only on your blocklevel shortcodes defined in the array (“col” in the example)

BF

care to share the full code. did you add it in a filter function?

For what it’s worth, I had been following misguided advice from tutorials that should be taken offline before I ran across this old but useful post by Viper007Bond: http://www.viper007bond.com/2009/11/22/wordpress-code-earlier-shortcodes/

Preprocessing shortcodes that output user-supplied content end up showing correctly without all the annoying paragraph and break formatting issues while third-party plugin shortcodes continue to function normally. I only implemented this recently and so far so good. Anybody have a reason that this is not a good solution?