WP_Review_Me - Code Metrics - Inspection of "Update image src" - julien731/WP-Review-Me - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 686edc...1e24fa )

by Julien

created 2016-07-03 14:31 UTC

WP_Review_Me A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	396
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	11
Bugs	0	Features	0

Metric	Value
wmc	29
c	11
b	0
f	0
lcom	1
cbo	1
dl	0
loc	396
rs	10

14 Methods

Rating	Name	Size	Complexity
A	get_defaults()	17	1
A	is_wp_compatible()	9	2
A	is_php_compatible()	9	2
A	spit_error()	7	1
A	is_time()	16	3
A	setup_date()	3	1
A	get_review_link()	23	3
A	maybe_prompt()	12	2
A	__construct()	20	1
B	init()	54	6
A	get_review_link_tag()	7	1
B	script()	26	1
B	dismiss_notice()	39	4
A	get_message()	9	1

<?php

/**
 * WP Review Me
 *
 * A lightweight library to help you get more reviews for your WordPress theme/plugin.
 *
 * LICENSE: This program is free software; you can redistribute it and/or modify it under the terms of the GNU
 * General Public License as published by the Free Software Foundation; either version 3 of the License, or (at
 * your option) any later version. This program is distributed in the hope that it will be useful, but WITHOUT ANY
 * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
 * General Public License for more details. You should have received a copy of the GNU General Public License along
 * with this program. If not, see <http://opensource.org/licenses/gpl-license.php>
 *
 * @package   WP Review Me
 * @author    Julien Liabeuf <[email protected]>
 * @version   1.0
 * @license   GPL-2.0+
 * @link      https://julienliabeuf.com
 * @copyright 2016 Julien Liabeuf
 */

// If this file is called directly, abort.
if ( ! defined( 'WPINC' ) ) {
	die;
}

if ( ! class_exists( 'WP_Review_Me' ) ) {

	class WP_Review_Me {
namespace YourVendor;

class YourClass { }

		/**
		 * Library version
		 *
		 * @since 1.0
		 * @var string
		 */
		public $version = '1.0';

		/**
		 * Required version of PHP.
		 *
		 * @since 1.0
		 * @var string
		 */
		public $php_version_required = '5.5';

		/**
		 * Minimum version of WordPress required to use the library
		 *
		 * @since 1.0
		 * @var string
		 */
		public $wordpress_version_required = '4.2';

		/**
		 * Holds the unique identifying key for this particular instance
		 *
		 * @since 1.0
		 * @var string
		 */
		protected $key;

		/**
		 * Link unique ID
		 *
		 * @since 1.0
		 * @var string
		 */
		public $link_id;

		/**
		 * WP_Review_Me constructor.
		 *
		 * @since 1.0
		 *
		 * @param array $args Object settings
		 */
		public function __construct( $args ) {

			$args             = wp_parse_args( $args, $this->get_defaults() );
			$this->days       = $args['days_after'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->type       = $args['type'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->slug       = $args['slug'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->rating     = $args['rating'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->message    = $args['message'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->link_label = $args['link_label'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->cap        = $args['cap'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
			$this->scope      = $args['scope'];
class MyClass { }

$x = new MyClass();
$x->foo = true;

			// Set the unique identifying key for this instance
			$this->key     = 'wrm_' . substr( md5( plugin_basename( __FILE__ ) ), 0, 20 );
			$this->link_id = 'wrm-review-link-' . $this->key;

			// Instantiate
			$this->init();

		}

		/**
		 * Get default object settings
		 *
		 * @since 1.0
		 * @return array
		 */
		protected function get_defaults() {

			$defaults = array(
				'days_after' => 10,
				'type'       => '',
				'slug'       => '',
				'rating'     => 5,
				'message'    => sprintf( esc_html__( 'Hey! It&#039;s been a little while that you&#039;ve been using this product. You might not realize it, but user reviews are such a great help to us. We would be so grateful if you could take a minute to leave a review on WordPress.org. Many thanks in advance :)', 'wp-review-me' ) ),
				'link_label' => esc_html__( 'Click here to leave your review', 'wp-review-me' ),
				// Parameters used in WP Dismissible Notices Handler
				'cap'        => 'administrator',
				'scope'      => 'global',
			);

			return $defaults;

		}

		/**
		 * Initialize the library
		 *
		 * @since 1.0
		 * @return void
		 */
		private function init() {

			// Make sure WordPress is compatible
			if ( ! $this->is_wp_compatible() ) {
				$this->spit_error(
					sprintf(
						esc_html__( 'The library can not be used because your version of WordPress is too old. You need version %s at least.', 'wp-review-me' ),
						$this->wordpress_version_required
					)
				);

				return;
			}

			// Make sure PHP is compatible
			if ( ! $this->is_php_compatible() ) {
				$this->spit_error(
					sprintf(
						esc_html__( 'The library can not be used because your version of PHP is too old. You need version %s at least.', 'wp-review-me' ),
						$this->php_version_required
					)
				);

				return;
			}

			// Make sure the dependencies are loaded
			if ( ! function_exists( 'dnh_register_notice' ) ) {

				$dnh_file = trailingslashit( plugin_dir_path( __FILE__ ) ) . 'vendor/julien731/wp-dismissible-notices-handler/handler.php';

				if ( file_exists( $dnh_file ) ) {
					require( $dnh_file );
				}

				if ( ! function_exists( 'dnh_register_notice' ) ) {
					$this->spit_error(
						sprintf(
							esc_html__( 'Dependencies are missing. Please run a %s.', 'wp-review-me' ),
							'<code>composer install</code>'
						)
					);

					return;
				}
			}

			add_action( 'admin_footer', array( $this, 'script' ) );
			add_action( 'wp_ajax_wrm_clicked_review', array( $this, 'dismiss_notice' ) );

			// And let's roll... maybe.
			$this->maybe_prompt();

		}

		/**
		 * Check if the current WordPress version fits the requirements
		 *
		 * @since  1.0
		 * @return boolean
		 */
		private function is_wp_compatible() {

			if ( version_compare( get_bloginfo( 'version' ), $this->wordpress_version_required, '<' ) ) {
				return false;
			}

			return true;

		}

		/**
		 * Check if the version of PHP is compatible with this library
		 *
		 * @since  1.0
		 * @return boolean
		 */
		private function is_php_compatible() {

			if ( version_compare( phpversion(), $this->php_version_required, '<' ) ) {
				return false;
			}

			return true;

		}

		/**
		 * Spits an error message at the top of the admin screen
		 *
		 * @since 1.0
		 *
		 * @param string $error Error message to spit
		 *
		 * @return void
		 */
		protected function spit_error( $error ) {
			printf(
				'<div style="margin: 20px; text-align: center;"><strong>%1$s</strong> %2$s</pre></div>',
				esc_html__( 'WP Review Me Error:', 'wp-review-me' ),
				wp_kses_post( $error )
			);
		}

		/**
		 * Check if it is time to ask for a review
		 *
		 * @since 1.0
		 * @return bool
		 */
		public function is_time() {

			$installed = get_option( $this->key, false );

			if ( false === $installed ) {
				$this->setup_date();
				$installed = time();
			}

			if ( $installed + ( $this->days + 86400 ) > time() ) {
				return false;
			}

			return true;

		}

		/**
		 * Save the current date as the installation date
		 *
		 * @since 1.0
		 * @return void
		 */
		protected function setup_date() {
			update_option( $this->key, time() );
		}

		/**
		 * Get the review link
		 *
		 * @since 1.0
		 * @return string
		 */
		protected function get_review_link() {

			$link = 'https://wordpress.org/support/view/';

			switch ( $this->type ) {

				case 'theme':
					$link .= 'theme-reviews/';
					break;

				case 'plugin':
					$link .= 'plugin-reviews/';
					break;

			}

			$link .= $this->slug;
			$link = add_query_arg( 'rate', $this->rating, $link );
			$link = esc_url( $link . '#postform' );

			return $link;

		}

		/**
		 * Get the complete link tag
		 *
		 * @since 1.0
		 * @return string
		 */
		protected function get_review_link_tag() {

			$link = $this->get_review_link();

			return "<a href='$link' target='_blank' id='$this->link_id'>$this->link_label</a>";

		}

		/**
		 * Trigger the notice if it is time to ask for a review
		 *
		 * @since 1.0
		 * @return void
		 */
		protected function maybe_prompt() {

			if ( ! $this->is_time() ) {
				return;
			}

			dnh_register_notice( $this->key, 'updated', $this->get_message(), array(
				'scope' => $this->scope,
				'cap'   => $this->cap
			) );

		}

		/**
		 * Echo the JS script in the admin footer
		 *
		 * @since 1.0
		 * @return void
		 */
		public function script() { ?>

			<script>
				jQuery(document).ready(function($) {
					$('#<?php echo $this->link_id; ?>').on('click', wrmDismiss);
					function wrmDismiss() {

						var data = {
							action: 'wrm_clicked_review',
							id: '<?php echo $this->link_id; ?>'
						};

						jQuery.ajax({
							type:'POST',
							url: ajaxurl,
							data: data,
							success:function( data ){
								console.log(data);
							}
						});

					}
				});
			</script>

		<?php }

		/**
		 * Dismiss the notice when the review link is clicked
		 *
		 * @since 1.0
		 * @return void
		 */
		public function dismiss_notice() {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}

			if ( empty( $_POST ) ) {
				echo 'missing POST';
				die();

			}

			if ( ! isset( $_POST['id'] ) ) {
				echo 'missing ID';
				die();

			}

			$id = sanitize_text_field( $_POST['id'] );

			if ( $id !== $this->link_id ) {
				echo "not this instance's job";
				die();

			}

			// Get the DNH notice ID ready
			$notice_id = DNH()->get_id( str_replace( 'wrm-review-link-', '', $id ) );
			$dismissed = DNH()->dismiss_notice( $notice_id );
			
			echo $dismissed;

			/**
			 * Fires right after the notice has been dismissed. This allows for various integrations to perform additional tasks.
			 *
			 * @since 1.0
			 *
			 * @param string $id        The notice ID
			 * @param string $notice_id The notice ID as defined by the DNH class
			 */
			do_action( 'wrm_after_notice_dismissed', $id, $notice_id );

			// Stop execution here
			die();


		}

		/**
		 * Get the review prompt message
		 *
		 * @since 1.0
		 * @return string
		 */
		protected function get_message() {

			$message = $this->message;
			$link    = $this->get_review_link_tag();
			$message = $message . ' ' . $link;

			return wp_kses_post( $message );

		}

	}

}

1			<?php
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 29 and the first side effect is on line 24. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2			/**
3			* WP Review Me
4			*
5			* A lightweight library to help you get more reviews for your WordPress theme/plugin.
6			*
7			* LICENSE: This program is free software; you can redistribute it and/or modify it under the terms of the GNU
8			* General Public License as published by the Free Software Foundation; either version 3 of the License, or (at
9			* your option) any later version. This program is distributed in the hope that it will be useful, but WITHOUT ANY
10			* WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
11			* General Public License for more details. You should have received a copy of the GNU General Public License along
12			* with this program. If not, see <http://opensource.org/licenses/gpl-license.php>
13			*
14			* @package WP Review Me
15			* @author Julien Liabeuf <[email protected]>
16			* @version 1.0
17			* @license GPL-2.0+
18			* @link https://julienliabeuf.com
19			* @copyright 2016 Julien Liabeuf
20			*/
21
22			// If this file is called directly, abort.
23			if ( ! defined( 'WPINC' ) ) {
24			die;
25			}
26
27			if ( ! class_exists( 'WP_Review_Me' ) ) {
28
29			class WP_Review_Me {
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
30
31			/**
32			* Library version
33			*
34			* @since 1.0
35			* @var string
36			*/
37			public $version = '1.0';
38
39			/**
40			* Required version of PHP.
41			*
42			* @since 1.0
43			* @var string
44			*/
45			public $php_version_required = '5.5';
46
47			/**
48			* Minimum version of WordPress required to use the library
49			*
50			* @since 1.0
51			* @var string
52			*/
53			public $wordpress_version_required = '4.2';
54
55			/**
56			* Holds the unique identifying key for this particular instance
57			*
58			* @since 1.0
59			* @var string
60			*/
61			protected $key;
62
63			/**
64			* Link unique ID
65			*
66			* @since 1.0
67			* @var string
68			*/
69			public $link_id;
70
71			/**
72			* WP_Review_Me constructor.
73			*
74			* @since 1.0
75			*
76			* @param array $args Object settings
77			*/
78			public function __construct( $args ) {
79
80			$args = wp_parse_args( $args, $this->get_defaults() );
81			$this->days = $args['days_after'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `days` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
82			$this->type = $args['type'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `type` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
83			$this->slug = $args['slug'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `slug` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
84			$this->rating = $args['rating'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `rating` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
85			$this->message = $args['message'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `message` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
86			$this->link_label = $args['link_label'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `link_label` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
87			$this->cap = $args['cap'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `cap` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
88			$this->scope = $args['scope'];
			0 ignored issues – show Bug introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The property `scope` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
89
90			// Set the unique identifying key for this instance
91			$this->key = 'wrm_' . substr( md5( plugin_basename( __FILE__ ) ), 0, 20 );
92			$this->link_id = 'wrm-review-link-' . $this->key;
93
94			// Instantiate
95			$this->init();
96
97			}
98
99			/**
100			* Get default object settings
101			*
102			* @since 1.0
103			* @return array
104			*/
105			protected function get_defaults() {
106
107			$defaults = array(
108			'days_after' => 10,
109			'type' => '',
110			'slug' => '',
111			'rating' => 5,
112			'message' => sprintf( esc_html__( 'Hey! It's been a little while that you've been using this product. You might not realize it, but user reviews are such a great help to us. We would be so grateful if you could take a minute to leave a review on WordPress.org. Many thanks in advance :)', 'wp-review-me' ) ),
113			'link_label' => esc_html__( 'Click here to leave your review', 'wp-review-me' ),
114			// Parameters used in WP Dismissible Notices Handler
115			'cap' => 'administrator',
116			'scope' => 'global',
117			);
118
119			return $defaults;
120
121			}
122
123			/**
124			* Initialize the library
125			*
126			* @since 1.0
127			* @return void
128			*/
129			private function init() {
130
131			// Make sure WordPress is compatible
132			if ( ! $this->is_wp_compatible() ) {
133			$this->spit_error(
134			sprintf(
135			esc_html__( 'The library can not be used because your version of WordPress is too old. You need version %s at least.', 'wp-review-me' ),
136			$this->wordpress_version_required
137			)
138			);
139
140			return;
141			}
142
143			// Make sure PHP is compatible
144			if ( ! $this->is_php_compatible() ) {
145			$this->spit_error(
146			sprintf(
147			esc_html__( 'The library can not be used because your version of PHP is too old. You need version %s at least.', 'wp-review-me' ),
148			$this->php_version_required
149			)
150			);
151
152			return;
153			}
154
155			// Make sure the dependencies are loaded
156			if ( ! function_exists( 'dnh_register_notice' ) ) {
157
158			$dnh_file = trailingslashit( plugin_dir_path( __FILE__ ) ) . 'vendor/julien731/wp-dismissible-notices-handler/handler.php';
159
160			if ( file_exists( $dnh_file ) ) {
161			require( $dnh_file );
162			}
163
164			if ( ! function_exists( 'dnh_register_notice' ) ) {
165			$this->spit_error(
166			sprintf(
167			esc_html__( 'Dependencies are missing. Please run a %s.', 'wp-review-me' ),
168			'<code>composer install</code>'
169			)
170			);
171
172			return;
173			}
174			}
175
176			add_action( 'admin_footer', array( $this, 'script' ) );
177			add_action( 'wp_ajax_wrm_clicked_review', array( $this, 'dismiss_notice' ) );
178
179			// And let's roll... maybe.
180			$this->maybe_prompt();
181
182			}
183
184			/**
185			* Check if the current WordPress version fits the requirements
186			*
187			* @since 1.0
188			* @return boolean
189			*/
190			private function is_wp_compatible() {
191
192			if ( version_compare( get_bloginfo( 'version' ), $this->wordpress_version_required, '<' ) ) {
193			return false;
194			}
195
196			return true;
197
198			}
199
200			/**
201			* Check if the version of PHP is compatible with this library
202			*
203			* @since 1.0
204			* @return boolean
205			*/
206			private function is_php_compatible() {
207
208			if ( version_compare( phpversion(), $this->php_version_required, '<' ) ) {
209			return false;
210			}
211
212			return true;
213
214			}
215
216			/**
217			* Spits an error message at the top of the admin screen
218			*
219			* @since 1.0
220			*
221			* @param string $error Error message to spit
222			*
223			* @return void
224			*/
225			protected function spit_error( $error ) {
226			printf(
227			'<div style="margin: 20px; text-align: center;"><strong>%1$s</strong> %2$s</pre></div>',
228			esc_html__( 'WP Review Me Error:', 'wp-review-me' ),
229			wp_kses_post( $error )
230			);
231			}
232
233			/**
234			* Check if it is time to ask for a review
235			*
236			* @since 1.0
237			* @return bool
238			*/
239			public function is_time() {
240
241			$installed = get_option( $this->key, false );
242
243			if ( false === $installed ) {
244			$this->setup_date();
245			$installed = time();
246			}
247
248			if ( $installed + ( $this->days + 86400 ) > time() ) {
249			return false;
250			}
251
252			return true;
253
254			}
255
256			/**
257			* Save the current date as the installation date
258			*
259			* @since 1.0
260			* @return void
261			*/
262			protected function setup_date() {
263			update_option( $this->key, time() );
264			}
265
266			/**
267			* Get the review link
268			*
269			* @since 1.0
270			* @return string
271			*/
272			protected function get_review_link() {
273
274			$link = 'https://wordpress.org/support/view/';
275
276			switch ( $this->type ) {
277
278			case 'theme':
279			$link .= 'theme-reviews/';
280			break;
281
282			case 'plugin':
283			$link .= 'plugin-reviews/';
284			break;
285
286			}
287
288			$link .= $this->slug;
289			$link = add_query_arg( 'rate', $this->rating, $link );
290			$link = esc_url( $link . '#postform' );
291
292			return $link;
293
294			}
295
296			/**
297			* Get the complete link tag
298			*
299			* @since 1.0
300			* @return string
301			*/
302			protected function get_review_link_tag() {
303
304			$link = $this->get_review_link();
305
306			return "<a href='$link' target='_blank' id='$this->link_id'>$this->link_label</a>";
307
308			}
309
310			/**
311			* Trigger the notice if it is time to ask for a review
312			*
313			* @since 1.0
314			* @return void
315			*/
316			protected function maybe_prompt() {
317
318			if ( ! $this->is_time() ) {
319			return;
320			}
321
322			dnh_register_notice( $this->key, 'updated', $this->get_message(), array(
323			'scope' => $this->scope,
324			'cap' => $this->cap
325			) );
326
327			}
328
329			/**
330			* Echo the JS script in the admin footer
331			*
332			* @since 1.0
333			* @return void
334			*/
335			public function script() { ?>
336
337			<script>
338			jQuery(document).ready(function($) {
339			$('#<?php echo $this->link_id; ?>').on('click', wrmDismiss);
340			function wrmDismiss() {
341
342			var data = {
343			action: 'wrm_clicked_review',
344			id: '<?php echo $this->link_id; ?>'
345			};
346
347			jQuery.ajax({
348			type:'POST',
349			url: ajaxurl,
350			data: data,
351			success:function( data ){
352			console.log(data);
353			}
354			});
355
356			}
357			});
358			</script>
359
360			<?php }
361
362			/**
363			* Dismiss the notice when the review link is clicked
364			*
365			* @since 1.0
366			* @return void
367			*/
368			public function dismiss_notice() {
			0 ignored issues – show Coding Style introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report `dismiss_notice` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
369
370			if ( empty( $_POST ) ) {
371			echo 'missing POST';
372			die();
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The method `dismiss_notice()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
373			}
374
375			if ( ! isset( $_POST['id'] ) ) {
376			echo 'missing ID';
377			die();
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The method `dismiss_notice()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
378			}
379
380			$id = sanitize_text_field( $_POST['id'] );
381
382			if ( $id !== $this->link_id ) {
383			echo "not this instance's job";
384			die();
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The method `dismiss_notice()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
385			}
386
387			// Get the DNH notice ID ready
388			$notice_id = DNH()->get_id( str_replace( 'wrm-review-link-', '', $id ) );
389			$dismissed = DNH()->dismiss_notice( $notice_id );
390
391			echo $dismissed;
392
393			/**
394			* Fires right after the notice has been dismissed. This allows for various integrations to perform additional tasks.
395			*
396			* @since 1.0
397			*
398			* @param string $id The notice ID
399			* @param string $notice_id The notice ID as defined by the DNH class
400			*/
401			do_action( 'wrm_after_notice_dismissed', $id, $notice_id );
402
403			// Stop execution here
404			die();
			0 ignored issues – show Coding Style Compatibility introduced 2016-04-18 16:22 UTC by Report Bug Copy Issue Report The method `dismiss_notice()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
405
406			}
407
408			/**
409			* Get the review prompt message
410			*
411			* @since 1.0
412			* @return string
413			*/
414			protected function get_message() {
415
416			$message = $this->message;
417			$link = $this->get_review_link_tag();
418			$message = $message . ' ' . $link;
419
420			return wp_kses_post( $message );
421
422			}
423
424			}
425
426			}

julien731 / WP-Review-Me

Push — develop ( 686edc...1e24fa )

WP_Review_Me A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

14 Methods

Duplication Side-by-Side

Filter issues like