Possible variable $this found

Resolved Sultan
(@sultan_semmaiyahoocom)

11 years, 10 months ago

Hello,

I got the following message. What should i replace instead of $this i think it tell me that dont use variable.

1. RECOMMENDED: Possible variable $this found in translation function in class-tgm-plugin-activation.php. Translation function calls must NOT contain PHP variables.

Line 503: echo '<p><a href=''>menu, admin_url( $this->parent_url_slug ) ) . '' title='' . esc_attr( $this->strings['return'] ) . '' target='_parent'>' . __( $this->strings['return'],'clear' ) . '</a></p>';

2. i got this
Line 10: require_once(‘OAuth.php’);

3.
iframe was found in the file single.php iframes are sometimes used to load unwanted adverts and code on your site.

Line 88: <iframe src=’http://player.vimeo.com/video/<?php echo $post_type[‘vimeo’];?>

4.the most important

WARNING: curl_init was found in the file twitteroauth.php File operations should use the WP_Filesystem methods instead of direct PHP filesystem calls.

Line 195: $ci = curl_init();

WARNING: curl_exec was found in the file twitteroauth.php File operations should use the WP_Filesystem methods instead of direct PHP filesystem calls.

Line 221: $response = curl_exec($ci);

https://ww.wp.xz.cn/plugins/theme-check/

Viewing 3 replies - 1 through 3 (of 3 total)

Plugin Author Samuel Wood (Otto)
(@otto42)

ww.wp.xz.cn Admin

11 years, 10 months ago

Translation functions like __() don’t work on variables, only on strings.

Please fix your i18n code to follow the rules.

Thread Starter Sultan
(@sultan_semmaiyahoocom)

11 years, 10 months ago

thank you for such a great plugin ok i will use plugin fro twitter and also dont like__().

please can you help me in the followings also.

1.INFO: iframe was found in the file single.php iframes are sometimes used to load unwanted adverts and code on your site.
Line 88: <iframe src=’http://player.vimeo.com/video/<?php echo $post_type[‘vimeo’];?>

2.INFO: clear.php The theme appears to use include or require. If these are being used to include separate sections of a template from independent files, then get_template_part() should be used instead.

Line 87: include ( $class );
Line 146: require_once ( ‘clear-core-functions.php’ ); // Contains core functions for

3.INFO: Possible hard-coded links were found in the file panel_header.php.

4.RECOMMENDED: No reference to add_theme_support( “custom-header”, $args ) was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.

5.RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.

6.RECOMMENDED: get_theme_data found in the file class-api.php. Deprecated since version 3.4. Use wp_get_theme() instead.

Line 169: $theme_data = get_theme_data( get_stylesheet_directory() . ‘/style.css’ );
Line 5:

Thread Starter Sultan
(@sultan_semmaiyahoocom)

11 years, 10 months ago

only three info reaming now.

1.
INFO: Possible hard-coded links were found in the file panel_header.php.

Line 5:

2.
INFO: iframe was found in the file single.php iframes are sometimes used to load unwanted adverts and code on your site.

Line 88: <iframe src=’http://player.vimeo.com/video/<?php echo $post_type[‘vimeo’];?>

3.
INFO: clear-widget-init.php The theme appears to use include or require. If these are being used to include separate sections of a template from independent files, then get_template_part() should be used instead.

Line 113: include ( ‘clear-module-extras.php’ );
Line 13: require_once ( ‘utilities/wp.php’ );

please help me to remove these also.

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Possible variable $this found’ is closed to new replies.