WordPress.org

Ready to get started?Download WordPress

Forums

Meta SEO Pack
Title filters need correcting... (2 posts)

  1. amereservant
    Member
    Posted 2 years ago #

    Hey,
    I was trying to hook your filters with a theme I'm using and I noticed you're applying the filters in the wrong place. I haven't looked through the rest of the code, but here's a small fix I suggest:

    Original

    } else {
        $format = apply_filters( 'msp_get_title_format', get_option( 'msp_title_other' ) );
        $data = apply_filters( 'msp_get_title_data', $data );
    }

    Revision

    } else {
        $format = get_option( 'msp_title_other' );
    }
        $format = apply_filters( 'msp_get_title_format', $format );
        $data   = apply_filters( 'msp_get_title_data', $data );

    This applies to lines 409-412 in the meta-seo-pack.php file.

    Thank you for the great plugin that has been priceless for me!

    http://wordpress.org/extend/plugins/meta-seo-pack/

  2. Daniel Fru?y?ski (sirzooro)
    Member
    Plugin Author

    Posted 2 years ago #

    Hi,
    I did this on purpose, to provide support for non-standard page types only. Your post suggest that these filters may be also useful for other purposes. I will take a look on this, and either move these filters out of else block, or add a new ones.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic