WordPress.org

1. amereservant
   Member
   Posted 8 months 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
   Posted 8 months 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.

Reply

You must log in to post.