Completed
Push — master ( 29b848...1fd6be )
by Zack
59:32 queued 42:45
created
includes/class-gv-license-handler.php 4 patches
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -295,7 +295,8 @@  discard block
 block discarded – undo
295 295
 			if ( empty( $license_data ) ) {
296 296
 				if ( $is_ajax ) {
297 297
 					exit( json_encode( array() ) );
298
-				} else { // Non-ajax call
298
+				} else {
299
+// Non-ajax call
299 300
 					return json_encode( array() );
300 301
 				}
301 302
 			}
@@ -325,7 +326,8 @@  discard block
 block discarded – undo
325 326
 
326 327
 		if ( $is_ajax ) {
327 328
 			exit( $json );
328
-		} else { // Non-ajax call
329
+		} else {
330
+// Non-ajax call
329 331
 			return ( rgget('format', $data ) === 'object' ) ? $license_data : $json;
330 332
 		}
331 333
 	}
Please login to merge, or discard this patch.
Doc Comments   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -86,7 +86,6 @@  discard block
 block discarded – undo
86 86
 	 *
87 87
 	 * @since 1.19.3
88 88
 	 *
89
-	 * @param bool $force Whether to force checking license, even if AJAX
90 89
 	 *
91 90
 	 * @return void
92 91
 	 */
@@ -272,6 +271,9 @@  discard block
 block discarded – undo
272 271
 		return $implode ? implode( $implode, $extensions ) : $extensions;
273 272
 	}
274 273
 
274
+	/**
275
+	 * @param boolean $echo
276
+	 */
275 277
 	function settings_edd_license_activation( $field, $echo ) {
276 278
 
277 279
 		$script_debug = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
@@ -464,7 +466,7 @@  discard block
 block discarded – undo
464 466
 	 * Generate the status message box HTML based on the current status
465 467
 	 *
466 468
 	 * @since 1.7.4
467
-	 * @param $message
469
+	 * @param string $message
468 470
 	 * @param string $class
469 471
 	 *
470 472
 	 * @return string
Please login to merge, or discard this patch.
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -116,14 +116,14 @@  discard block
 block discarded – undo
116 116
 			// Call the custom API.
117 117
 			$response = wp_remote_post( self::url, array(
118 118
 				'timeout'   => 15,
119
-			    'sslverify' => false,
120
-			    'body'      =>  array(
121
-				    'edd_action' => 'check_license',
122
-				    'license'    => trim( $this->Addon->get_app_setting( 'license_key' ) ),
123
-				    'item_name'  => self::name,
124
-				    'url'        => home_url(),
125
-				    'site_data'  => $this->get_site_data(),
126
-			    ),
119
+				'sslverify' => false,
120
+				'body'      =>  array(
121
+					'edd_action' => 'check_license',
122
+					'license'    => trim( $this->Addon->get_app_setting( 'license_key' ) ),
123
+					'item_name'  => self::name,
124
+					'url'        => home_url(),
125
+					'site_data'  => $this->get_site_data(),
126
+				),
127 127
 			));
128 128
 
129 129
 			// make sure the response came back okay
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
 			'author'    => self::author,
386 386
 			'language'  => get_locale(),
387 387
 			'url'       => home_url(),
388
-		    'beta'      => $this->Addon->get_app_setting( 'beta' ),
388
+			'beta'      => $this->Addon->get_app_setting( 'beta' ),
389 389
 		);
390 390
 
391 391
 		if( !empty( $action ) ) {
@@ -678,7 +678,7 @@  discard block
 block discarded – undo
678 678
 		// Update option with passed data license
679 679
 		$settings = $this->Addon->get_app_settings();
680 680
 
681
-        $settings['license_key'] = $license_data->license_key = trim( $data['license'] );
681
+		$settings['license_key'] = $license_data->license_key = trim( $data['license'] );
682 682
 		$settings['license_key_status'] = $license_data->license;
683 683
 		$settings['license_key_response'] = (array)$license_data;
684 684
 
Please login to merge, or discard this patch.
Spacing   +106 added lines, -106 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 	 * @return GV_License_Handler
48 48
 	 */
49 49
 	public static function get_instance( GravityView_Settings $GFAddOn ) {
50
-		if( empty( self::$instance ) ) {
50
+		if ( empty( self::$instance ) ) {
51 51
 			self::$instance = new self( $GFAddOn );
52 52
 		}
53 53
 		return self::$instance;
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 				    'url'        => home_url(),
124 124
 				    'site_data'  => $this->get_site_data(),
125 125
 			    ),
126
-			));
126
+			) );
127 127
 
128 128
 			// make sure the response came back okay
129 129
 			if ( is_wp_error( $response ) ) {
@@ -149,12 +149,12 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	public function refresh_license_status() {
151 151
 
152
-		if ( defined('DOING_AJAX') && DOING_AJAX ) {
152
+		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
153 153
 			return;
154 154
 		}
155 155
 
156 156
 		// The transient is fresh; don't fetch.
157
-		if( $status = get_transient( self::status_transient_key ) ) {
157
+		if ( $status = get_transient( self::status_transient_key ) ) {
158 158
 			return;
159 159
 		}
160 160
 
@@ -187,49 +187,49 @@  discard block
 block discarded – undo
187 187
 		$theme_data = wp_get_theme();
188 188
 		$theme      = $theme_data->Name . ' ' . $theme_data->Version;
189 189
 
190
-		$data['gv_version']  = GravityView_Plugin::version;
191
-		$data['php_version']  = phpversion();
192
-		$data['wp_version']   = get_bloginfo( 'version' );
193
-		$data['gf_version']  = GFForms::$version;
194
-		$data['server']       = isset( $_SERVER['SERVER_SOFTWARE'] ) ? $_SERVER['SERVER_SOFTWARE'] : '';
195
-		$data['multisite']    = is_multisite();
196
-		$data['theme']        = $theme;
197
-		$data['url']          = home_url();
198
-		$data['license_key']  = GravityView_Settings::get_instance()->get_app_setting( 'license_key' );
199
-		$data['beta']         = GravityView_Settings::get_instance()->get_app_setting( 'beta' );
190
+		$data[ 'gv_version' ]  = GravityView_Plugin::version;
191
+		$data[ 'php_version' ]  = phpversion();
192
+		$data[ 'wp_version' ]   = get_bloginfo( 'version' );
193
+		$data[ 'gf_version' ]  = GFForms::$version;
194
+		$data[ 'server' ]       = isset( $_SERVER[ 'SERVER_SOFTWARE' ] ) ? $_SERVER[ 'SERVER_SOFTWARE' ] : '';
195
+		$data[ 'multisite' ]    = is_multisite();
196
+		$data[ 'theme' ]        = $theme;
197
+		$data[ 'url' ]          = home_url();
198
+		$data[ 'license_key' ]  = GravityView_Settings::get_instance()->get_app_setting( 'license_key' );
199
+		$data[ 'beta' ]         = GravityView_Settings::get_instance()->get_app_setting( 'beta' );
200 200
 
201 201
 		// View Data
202
-		$gravityview_posts = get_posts('numberposts=-1&post_type=gravityview&post_status=publish&order=ASC');
202
+		$gravityview_posts = get_posts( 'numberposts=-1&post_type=gravityview&post_status=publish&order=ASC' );
203 203
 
204 204
 		if ( ! empty( $gravityview_posts ) ) {
205 205
 			$first = array_shift( $gravityview_posts );
206 206
 			$latest = array_pop( $gravityview_posts );
207
-			$data['view_count'] = count( $gravityview_posts );
208
-			$data['view_first'] = $first->post_date;
209
-			$data['view_latest'] = $latest->post_date;
207
+			$data[ 'view_count' ] = count( $gravityview_posts );
208
+			$data[ 'view_first' ] = $first->post_date;
209
+			$data[ 'view_latest' ] = $latest->post_date;
210 210
 		}
211 211
 
212 212
 		// Form counts
213 213
 		if ( class_exists( 'GFFormsModel' ) ) {
214 214
 			$form_data = GFFormsModel::get_form_count();
215
-			$data['forms_total'] = rgar( $form_data, 'total', 0 );
216
-			$data['forms_active'] = rgar( $form_data, 'active', 0 );
217
-			$data['forms_inactive'] = rgar( $form_data, 'inactive', 0 );
218
-			$data['forms_trash'] = rgar( $form_data, 'inactive', 0 );
215
+			$data[ 'forms_total' ] = rgar( $form_data, 'total', 0 );
216
+			$data[ 'forms_active' ] = rgar( $form_data, 'active', 0 );
217
+			$data[ 'forms_inactive' ] = rgar( $form_data, 'inactive', 0 );
218
+			$data[ 'forms_trash' ] = rgar( $form_data, 'inactive', 0 );
219 219
 		}
220 220
 
221 221
 		// Retrieve current plugin information
222
-		if( ! function_exists( 'get_plugins' ) ) {
222
+		if ( ! function_exists( 'get_plugins' ) ) {
223 223
 			include ABSPATH . '/wp-admin/includes/plugin.php';
224 224
 		}
225 225
 
226
-		$data['integrations']     = self::get_related_plugins_and_extensions();
227
-		$data['active_plugins']   = get_option( 'active_plugins', array() );
228
-		$data['inactive_plugins'] = array();
229
-		$data['locale']           = get_locale();
226
+		$data[ 'integrations' ]     = self::get_related_plugins_and_extensions();
227
+		$data[ 'active_plugins' ]   = get_option( 'active_plugins', array() );
228
+		$data[ 'inactive_plugins' ] = array();
229
+		$data[ 'locale' ]           = get_locale();
230 230
 
231 231
 		// Validate request on the GV server
232
-		$data['hash']             = 'gv_version.url.locale:' . sha1( $data['gv_version'] . $data['url'] . $data['locale'] );
232
+		$data[ 'hash' ]             = 'gv_version.url.locale:' . sha1( $data[ 'gv_version' ] . $data[ 'url' ] . $data[ 'locale' ] );
233 233
 
234 234
 		return $data;
235 235
 	}
@@ -261,10 +261,10 @@  discard block
 block discarded – undo
261 261
 
262 262
 				$plugin_data = get_plugin_data( $active_plugin );
263 263
 
264
-				$extensions[] = sprintf( '%s %s', $plugin_data['Name'], $plugin_data['Version'] );
264
+				$extensions[ ] = sprintf( '%s %s', $plugin_data[ 'Name' ], $plugin_data[ 'Version' ] );
265 265
 			}
266 266
 
267
-			if( ! empty( $extensions ) ) {
267
+			if ( ! empty( $extensions ) ) {
268 268
 				set_site_transient( self::related_plugins_key, $extensions, HOUR_IN_SECONDS );
269 269
 			} else {
270 270
 				return 'There was an error fetching related plugins.';
@@ -283,37 +283,37 @@  discard block
 block discarded – undo
283 283
 		$status = trim( $this->Addon->get_app_setting( 'license_key_status' ) );
284 284
 		$key = trim( $this->Addon->get_app_setting( 'license_key' ) );
285 285
 
286
-		if( !empty( $key ) ) {
286
+		if ( ! empty( $key ) ) {
287 287
 			$response = $this->Addon->get_app_setting( 'license_key_response' );
288
-			$response = is_array( $response ) ? (object) $response : json_decode( $response );
288
+			$response = is_array( $response ) ? (object)$response : json_decode( $response );
289 289
 		} else {
290 290
 			$response = array();
291 291
 		}
292 292
 
293 293
 		wp_localize_script( 'gv-admin-edd-license', 'GVGlobals', array(
294 294
 			'license_box' => $this->get_license_message( $response )
295
-		));
295
+		) );
296 296
 
297 297
 
298 298
 		$fields = array(
299 299
 			array(
300 300
 				'name'  => 'edd-activate',
301
-				'value' => __('Activate License', 'gravityview'),
302
-				'data-pending_text' => __('Verifying license…', 'gravityview'),
301
+				'value' => __( 'Activate License', 'gravityview' ),
302
+				'data-pending_text' => __( 'Verifying license…', 'gravityview' ),
303 303
 				'data-edd_action' => 'activate_license',
304 304
 				'class' => 'button-primary',
305 305
 			),
306 306
 			array(
307 307
 				'name'  => 'edd-deactivate',
308
-				'value' => __('Deactivate License', 'gravityview'),
309
-				'data-pending_text' => __('Deactivating license…', 'gravityview'),
308
+				'value' => __( 'Deactivate License', 'gravityview' ),
309
+				'data-pending_text' => __( 'Deactivating license…', 'gravityview' ),
310 310
 				'data-edd_action' => 'deactivate_license',
311 311
 				'class' => ( empty( $status ) ? 'button-primary hide' : 'button-primary' ),
312 312
 			),
313 313
 			array(
314 314
 				'name'  => 'edd-check',
315
-				'value' => __('Check License', 'gravityview'),
316
-				'data-pending_text' => __('Verifying license…', 'gravityview'),
315
+				'value' => __( 'Check License', 'gravityview' ),
316
+				'data-pending_text' => __( 'Verifying license…', 'gravityview' ),
317 317
 				'title' => 'Check the license before saving it',
318 318
 				'data-edd_action' => 'check_license',
319 319
 				'class' => 'button-secondary',
@@ -323,17 +323,17 @@  discard block
 block discarded – undo
323 323
 
324 324
 		$class = 'button gv-edd-action';
325 325
 
326
-		$class .= ( !empty( $key ) && $status !== 'valid' ) ? '' : ' hide';
326
+		$class .= ( ! empty( $key ) && $status !== 'valid' ) ? '' : ' hide';
327 327
 
328 328
 		$disabled_attribute = GVCommon::has_cap( 'gravityview_edit_settings' ) ? false : 'disabled';
329 329
 
330 330
 		$submit = '<div class="gv-edd-button-wrapper">';
331 331
 		foreach ( $fields as $field ) {
332
-			$field['type'] = 'button';
333
-			$field['class'] = isset( $field['class'] ) ? $field['class'] . ' '. $class : $class;
334
-			$field['style'] = 'margin-left: 10px;';
335
-			if( $disabled_attribute ) {
336
-				$field['disabled'] = $disabled_attribute;
332
+			$field[ 'type' ] = 'button';
333
+			$field[ 'class' ] = isset( $field[ 'class' ] ) ? $field[ 'class' ] . ' ' . $class : $class;
334
+			$field[ 'style' ] = 'margin-left: 10px;';
335
+			if ( $disabled_attribute ) {
336
+				$field[ 'disabled' ] = $disabled_attribute;
337 337
 			}
338 338
 			$submit .= $this->Addon->settings_submit( $field, $echo );
339 339
 		}
@@ -352,8 +352,8 @@  discard block
 block discarded – undo
352 352
 	 */
353 353
 	public function setup_edd() {
354 354
 
355
-		if( !class_exists('EDD_SL_Plugin_Updater') ) {
356
-			require_once( GRAVITYVIEW_DIR . 'includes/lib/EDD_SL_Plugin_Updater.php');
355
+		if ( ! class_exists( 'EDD_SL_Plugin_Updater' ) ) {
356
+			require_once( GRAVITYVIEW_DIR . 'includes/lib/EDD_SL_Plugin_Updater.php' );
357 357
 		}
358 358
 
359 359
 		// setup the updater
@@ -391,8 +391,8 @@  discard block
 block discarded – undo
391 391
 		    'beta'      => $this->Addon->get_app_setting( 'beta' ),
392 392
 		);
393 393
 
394
-		if( !empty( $action ) ) {
395
-			$settings['edd_action'] = esc_attr( $action );
394
+		if ( ! empty( $action ) ) {
395
+			$settings[ 'edd_action' ] = esc_attr( $action );
396 396
 		}
397 397
 
398 398
 		$settings = array_map( 'urlencode', $settings );
@@ -406,14 +406,14 @@  discard block
 block discarded – undo
406 406
 	 */
407 407
 	private function _license_get_remote_response( $data, $license = '' ) {
408 408
 
409
-		$api_params = $this->_get_edd_settings( $data['edd_action'], $license );
409
+		$api_params = $this->_get_edd_settings( $data[ 'edd_action' ], $license );
410 410
 
411 411
 		$url = add_query_arg( $api_params, self::url );
412 412
 
413 413
 		$response = wp_remote_get( $url, array(
414 414
 			'timeout'   => 15,
415 415
 			'sslverify' => false,
416
-		));
416
+		) );
417 417
 
418 418
 		if ( is_wp_error( $response ) ) {
419 419
 
@@ -451,11 +451,11 @@  discard block
 block discarded – undo
451 451
 	 */
452 452
 	function get_license_message( $license_data ) {
453 453
 
454
-		if( empty( $license_data ) ) {
454
+		if ( empty( $license_data ) ) {
455 455
 			$message = '';
456 456
 		} else {
457 457
 
458
-			if( ! empty( $license_data->error ) ) {
458
+			if ( ! empty( $license_data->error ) ) {
459 459
 				$class = 'error';
460 460
 				$string_key = $license_data->error;
461 461
 			} else {
@@ -463,7 +463,7 @@  discard block
 block discarded – undo
463 463
 				$string_key = $license_data->license;
464 464
 			}
465 465
 
466
-			$message = sprintf( '<p><strong>%s: %s</strong></p>', $this->strings('status'), $this->strings( $string_key, $license_data ) );
466
+			$message = sprintf( '<p><strong>%s: %s</strong></p>', $this->strings( 'status' ), $this->strings( $string_key, $license_data ) );
467 467
 
468 468
 			$message = $this->generate_license_box( $message, $class );
469 469
 		}
@@ -500,18 +500,18 @@  discard block
 block discarded – undo
500 500
 	 */
501 501
 	public function license_details( $response = array() ) {
502 502
 
503
-		$response = (array) $response;
503
+		$response = (array)$response;
504 504
 
505 505
 		$return = '';
506 506
 		$wrapper = '<span class="gv-license-details" aria-live="polite" aria-busy="false">%s</span>';
507 507
 
508
-		if( ! empty( $response['license_key'] ) ) {
508
+		if ( ! empty( $response[ 'license_key' ] ) ) {
509 509
 
510 510
 			$return .= '<h3>' . esc_html__( 'License Details:', 'gravityview' ) . '</h3>';
511 511
 
512 512
 			if ( in_array( rgar( $response, 'license' ), array( 'invalid', 'deactivated' ) ) ) {
513
-				$return .= $this->strings( $response['license'], $response );
514
-			} elseif ( ! empty( $response['license_name'] ) ) {
513
+				$return .= $this->strings( $response[ 'license' ], $response );
514
+			} elseif ( ! empty( $response[ 'license_name' ] ) ) {
515 515
 
516 516
 				$response_keys = array(
517 517
 					'license_name'   => '',
@@ -526,19 +526,19 @@  discard block
 block discarded – undo
526 526
 				// Make sure all the keys are set
527 527
 				$response = wp_parse_args( $response, $response_keys );
528 528
 
529
-				$login_link = sprintf( '<a href="%s" class="howto" rel="external">%s</a>', esc_url( sprintf( 'https://gravityview.co/wp-login.php?username=%s', $response['customer_email'] ) ), esc_html__( 'Access your GravityView account', 'gravityview' ) );
530
-				$local_text = ( ! empty( $response['is_local'] ) ? '<span class="howto">' . __( 'This development site does not count toward license activation limits', 'gravityview' ) . '</span>' : '' );
529
+				$login_link = sprintf( '<a href="%s" class="howto" rel="external">%s</a>', esc_url( sprintf( 'https://gravityview.co/wp-login.php?username=%s', $response[ 'customer_email' ] ) ), esc_html__( 'Access your GravityView account', 'gravityview' ) );
530
+				$local_text = ( ! empty( $response[ 'is_local' ] ) ? '<span class="howto">' . __( 'This development site does not count toward license activation limits', 'gravityview' ) . '</span>' : '' );
531 531
 				$details    = array(
532
-					'license'     => sprintf( esc_html__( 'License level: %s', 'gravityview' ), esc_html( $response['license_name'] ), esc_html( $response['license_limit'] ) ),
533
-					'licensed_to' => sprintf( esc_html_x( 'Licensed to: %1$s (%2$s)', '1: Customer name; 2: Customer email', 'gravityview' ), esc_html__( $response['customer_name'], 'gravityview' ), esc_html__( $response['customer_email'], 'gravityview' ) ) . $login_link,
534
-					'activations' => sprintf( esc_html__( 'Activations: %d of %s sites', 'gravityview' ), intval( $response['site_count'] ), esc_html( $response['license_limit'] ) ) . $local_text,
535
-					'expires'     => 'lifetime' === $response['expires'] ? '' : sprintf( esc_html__( 'Renew on: %s', 'gravityview' ), date_i18n( get_option( 'date_format' ), strtotime( $response['expires'] ) - DAY_IN_SECONDS ) ),
536
-					'upgrade'     => $this->get_upgrade_html( $response['upgrades'] ),
532
+					'license'     => sprintf( esc_html__( 'License level: %s', 'gravityview' ), esc_html( $response[ 'license_name' ] ), esc_html( $response[ 'license_limit' ] ) ),
533
+					'licensed_to' => sprintf( esc_html_x( 'Licensed to: %1$s (%2$s)', '1: Customer name; 2: Customer email', 'gravityview' ), esc_html__( $response[ 'customer_name' ], 'gravityview' ), esc_html__( $response[ 'customer_email' ], 'gravityview' ) ) . $login_link,
534
+					'activations' => sprintf( esc_html__( 'Activations: %d of %s sites', 'gravityview' ), intval( $response[ 'site_count' ] ), esc_html( $response[ 'license_limit' ] ) ) . $local_text,
535
+					'expires'     => 'lifetime' === $response[ 'expires' ] ? '' : sprintf( esc_html__( 'Renew on: %s', 'gravityview' ), date_i18n( get_option( 'date_format' ), strtotime( $response[ 'expires' ] ) - DAY_IN_SECONDS ) ),
536
+					'upgrade'     => $this->get_upgrade_html( $response[ 'upgrades' ] ),
537 537
 				);
538 538
 
539
-				if ( ! empty( $response['error'] ) && 'expired' === $response['error'] ) {
540
-					unset( $details['upgrade'] );
541
-					$details['expires'] = '<div class="error inline"><p>' . $this->strings( 'expired', $response ) . '</p></div>';
539
+				if ( ! empty( $response[ 'error' ] ) && 'expired' === $response[ 'error' ] ) {
540
+					unset( $details[ 'upgrade' ] );
541
+					$details[ 'expires' ] = '<div class="error inline"><p>' . $this->strings( 'expired', $response ) . '</p></div>';
542 542
 				}
543 543
 
544 544
 				$return .= '<ul><li>' . implode( '</li><li>', array_filter( $details ) ) . '</li></ul>';
@@ -561,11 +561,11 @@  discard block
 block discarded – undo
561 561
 
562 562
 		$output = '';
563 563
 
564
-		if( ! empty( $upgrades ) ) {
564
+		if ( ! empty( $upgrades ) ) {
565 565
 
566 566
 			$locale_parts = explode( '_', get_locale() );
567 567
 
568
-			$is_english = ( 'en' === $locale_parts[0] );
568
+			$is_english = ( 'en' === $locale_parts[ 0 ] );
569 569
 
570 570
 			$output .= '<h4>' . esc_html__( 'Upgrades available:', 'gravityview' ) . '</h4>';
571 571
 
@@ -573,14 +573,14 @@  discard block
 block discarded – undo
573 573
 
574 574
 			foreach ( $upgrades as $upgrade_id => $upgrade ) {
575 575
 
576
-				$upgrade = (object) $upgrade;
576
+				$upgrade = (object)$upgrade;
577 577
 
578 578
 				$anchor_text = sprintf( esc_html_x( 'Upgrade to %1$s for %2$s', '1: GravityView upgrade name, 2: Cost of upgrade', 'gravityview' ), esc_attr( $upgrade->name ), esc_attr( $upgrade->price ) );
579 579
 
580
-				if( $is_english && isset( $upgrade->description ) ) {
580
+				if ( $is_english && isset( $upgrade->description ) ) {
581 581
 					$message = esc_html( $upgrade->description );
582 582
 				} else {
583
-					switch( $upgrade->price_id ) {
583
+					switch ( $upgrade->price_id ) {
584 584
 						// Interstellar
585 585
 						case 1:
586 586
 						default:
@@ -618,16 +618,16 @@  discard block
 block discarded – undo
618 618
 	 */
619 619
 	public function license_call( $array = array() ) {
620 620
 
621
-		$is_ajax = ( defined('DOING_AJAX') && DOING_AJAX );
622
-		$data = empty( $array ) ? $_POST['data'] : $array;
621
+		$is_ajax = ( defined( 'DOING_AJAX' ) && DOING_AJAX );
622
+		$data = empty( $array ) ? $_POST[ 'data' ] : $array;
623 623
 		$has_cap = GVCommon::has_cap( 'gravityview_edit_settings' );
624 624
 
625
-		if ( $is_ajax && empty( $data['license'] ) ) {
626
-			die( - 1 );
625
+		if ( $is_ajax && empty( $data[ 'license' ] ) ) {
626
+			die( -1 );
627 627
 		}
628 628
 
629 629
 		// If the user isn't allowed to edit settings, show an error message
630
-		if( ! $has_cap && empty( $data['all_caps'] ) ) {
630
+		if ( ! $has_cap && empty( $data[ 'all_caps' ] ) ) {
631 631
 			$license_data = new stdClass();
632 632
 			$license_data->error = 'capability';
633 633
 			$license_data->message = $this->get_license_message( $license_data );
@@ -651,11 +651,11 @@  discard block
 block discarded – undo
651 651
 
652 652
 			$json = json_encode( $license_data );
653 653
 
654
-			$update_license = ( ! isset( $data['update'] ) || ! empty( $data['update'] ) );
654
+			$update_license = ( ! isset( $data[ 'update' ] ) || ! empty( $data[ 'update' ] ) );
655 655
 
656
-			$is_check_action_button = ( 'check_license' === $data['edd_action'] && defined( 'DOING_AJAX' ) && DOING_AJAX );
656
+			$is_check_action_button = ( 'check_license' === $data[ 'edd_action' ] && defined( 'DOING_AJAX' ) && DOING_AJAX );
657 657
 
658
-			if( $is_check_action_button ) {
658
+			if ( $is_check_action_button ) {
659 659
 				delete_transient( self::status_transient_key );
660 660
 			}
661 661
 			// Failed is the response from trying to de-activate a license and it didn't work.
@@ -664,7 +664,7 @@  discard block
 block discarded – undo
664 664
 			// most likely a mistake.
665 665
 			else if ( $license_data->license !== 'failed' && $update_license ) {
666 666
 
667
-				if ( ! empty( $data['field_id'] ) ) {
667
+				if ( ! empty( $data[ 'field_id' ] ) ) {
668 668
 					set_transient( self::status_transient_key, $license_data, DAY_IN_SECONDS );
669 669
 				}
670 670
 
@@ -675,7 +675,7 @@  discard block
 block discarded – undo
675 675
 		if ( $is_ajax ) {
676 676
 			exit( $json );
677 677
 		} else { // Non-ajax call
678
-			return ( rgget('format', $data ) === 'object' ) ? $license_data : $json;
678
+			return ( rgget( 'format', $data ) === 'object' ) ? $license_data : $json;
679 679
 		}
680 680
 	}
681 681
 
@@ -689,9 +689,9 @@  discard block
 block discarded – undo
689 689
 		// Update option with passed data license
690 690
 		$settings = $this->Addon->get_app_settings();
691 691
 
692
-        $settings['license_key'] = $license_data->license_key = trim( $data['license'] );
693
-		$settings['license_key_status'] = $license_data->license;
694
-		$settings['license_key_response'] = (array)$license_data;
692
+        $settings[ 'license_key' ] = $license_data->license_key = trim( $data[ 'license' ] );
693
+		$settings[ 'license_key_status' ] = $license_data->license;
694
+		$settings[ 'license_key_response' ] = (array)$license_data;
695 695
 
696 696
 		$this->Addon->update_app_settings( $settings );
697 697
 	}
@@ -706,9 +706,9 @@  discard block
 block discarded – undo
706 706
 
707 707
 		$license_data = is_array( $license_data ) ? (object)$license_data : $license_data;
708 708
 
709
-		if( ! empty( $license_data->renewal_url ) ) {
709
+		if ( ! empty( $license_data->renewal_url ) ) {
710 710
 			$renew_license_url = $license_data->renewal_url;
711
-		} elseif( ! empty( $license_data->license_key ) ) {
711
+		} elseif ( ! empty( $license_data->license_key ) ) {
712 712
 			$renew_license_url = sprintf( 'https://gravityview.co/checkout/?download_id=17&edd_license_key=%s', $license_data->license_key );
713 713
 		} else {
714 714
 			$renew_license_url = 'https://gravityview.co/account/';
@@ -730,31 +730,31 @@  discard block
 block discarded – undo
730 730
 
731 731
 
732 732
 		$strings = array(
733
-			'status' => esc_html__('Status', 'gravityview'),
734
-			'error' => esc_html__('There was an error processing the request.', 'gravityview'),
735
-			'failed'  => esc_html__('Could not deactivate the license. The license key you attempted to deactivate may not be active or valid.', 'gravityview'),
736
-			'site_inactive' => esc_html__('The license key is valid, but it has not been activated for this site.', 'gravityview'),
737
-			'inactive' => esc_html__('The license key is valid, but it has not been activated for this site.', 'gravityview'),
738
-			'no_activations_left' => esc_html__('Invalid: this license has reached its activation limit.', 'gravityview') . ' ' . sprintf( esc_html__('You can manage license activations %son your GravityView account page%s.', 'gravityview'), '<a href="https://gravityview.co/account/#licenses">', '</a>' ),
739
-			'deactivated' => esc_html__('The license has been deactivated.', 'gravityview'),
740
-			'valid' => esc_html__('The license key is valid and active.', 'gravityview'),
741
-			'invalid' => esc_html__('The license key entered is invalid.', 'gravityview'),
742
-			'missing' => esc_html__('Invalid license key.', 'gravityview'),
743
-			'revoked' => esc_html__('This license key has been revoked.', 'gravityview'),
744
-			'expired' => sprintf( esc_html__('This license key has expired. %sRenew your license on the GravityView website%s to receive updates and support.', 'gravityview'), '<a href="'. esc_url( $this->get_license_renewal_url( $license_data ) ) .'">', '</a>' ),
733
+			'status' => esc_html__( 'Status', 'gravityview' ),
734
+			'error' => esc_html__( 'There was an error processing the request.', 'gravityview' ),
735
+			'failed'  => esc_html__( 'Could not deactivate the license. The license key you attempted to deactivate may not be active or valid.', 'gravityview' ),
736
+			'site_inactive' => esc_html__( 'The license key is valid, but it has not been activated for this site.', 'gravityview' ),
737
+			'inactive' => esc_html__( 'The license key is valid, but it has not been activated for this site.', 'gravityview' ),
738
+			'no_activations_left' => esc_html__( 'Invalid: this license has reached its activation limit.', 'gravityview' ) . ' ' . sprintf( esc_html__( 'You can manage license activations %son your GravityView account page%s.', 'gravityview' ), '<a href="https://gravityview.co/account/#licenses">', '</a>' ),
739
+			'deactivated' => esc_html__( 'The license has been deactivated.', 'gravityview' ),
740
+			'valid' => esc_html__( 'The license key is valid and active.', 'gravityview' ),
741
+			'invalid' => esc_html__( 'The license key entered is invalid.', 'gravityview' ),
742
+			'missing' => esc_html__( 'Invalid license key.', 'gravityview' ),
743
+			'revoked' => esc_html__( 'This license key has been revoked.', 'gravityview' ),
744
+			'expired' => sprintf( esc_html__( 'This license key has expired. %sRenew your license on the GravityView website%s to receive updates and support.', 'gravityview' ), '<a href="' . esc_url( $this->get_license_renewal_url( $license_data ) ) . '">', '</a>' ),
745 745
 			'capability' => esc_html__( 'You don\'t have the ability to edit plugin settings.', 'gravityview' ),
746 746
 
747
-			'verifying_license' => esc_html__('Verifying license&hellip;', 'gravityview'),
748
-			'activate_license' => esc_html__('Activate License', 'gravityview'),
749
-			'deactivate_license' => esc_html__('Deactivate License', 'gravityview'),
750
-			'check_license' => esc_html__('Verify License', 'gravityview'),
747
+			'verifying_license' => esc_html__( 'Verifying license&hellip;', 'gravityview' ),
748
+			'activate_license' => esc_html__( 'Activate License', 'gravityview' ),
749
+			'deactivate_license' => esc_html__( 'Deactivate License', 'gravityview' ),
750
+			'check_license' => esc_html__( 'Verify License', 'gravityview' ),
751 751
 		);
752 752
 
753
-		if( empty( $status ) ) {
753
+		if ( empty( $status ) ) {
754 754
 			return $strings;
755 755
 		}
756 756
 
757
-		if( isset( $strings[ $status ] ) ) {
757
+		if ( isset( $strings[ $status ] ) ) {
758 758
 			return $strings[ $status ];
759 759
 		}
760 760
 
Please login to merge, or discard this patch.
plugin-and-theme-hooks/class-gravityview-plugin-hooks-yoast-seo.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 
57 57
 		parent::add_hooks();
58 58
 
59
-		if( gravityview_is_admin_page() ) {
59
+		if ( gravityview_is_admin_page() ) {
60 60
 
61 61
 				// Make Yoast metabox go down to the bottom please.
62 62
 			add_filter( 'wpseo_metabox_prio', array( $this, '__return_low' ) );
@@ -83,8 +83,8 @@  discard block
 block discarded – undo
83 83
 		global $pagenow;
84 84
 
85 85
 		// New View page
86
-		if( $pagenow === 'post-new.php' ) {
87
-			$options['hideeditbox-gravityview'] = true;
86
+		if ( $pagenow === 'post-new.php' ) {
87
+			$options[ 'hideeditbox-gravityview' ] = true;
88 88
 		}
89 89
 
90 90
 		return $options;
Please login to merge, or discard this patch.
includes/plugin-and-theme-hooks/class-gravityview-theme-hooks-woothemes.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
 		$gv_page = gravityview_is_admin_page( '', 'single' );
79 79
 
80 80
 		// New View or Edit View page
81
-		if( $gv_page && $pagenow === 'post-new.php' ) {
81
+		if ( $gv_page && $pagenow === 'post-new.php' ) {
82 82
 			remove_meta_box( 'woothemes-settings', 'gravityview', 'normal' );
83 83
 		}
84 84
 	}
Please login to merge, or discard this patch.
includes/class-debug-bar.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -21,12 +21,12 @@  discard block
 block discarded – undo
21 21
 	function init() {
22 22
 
23 23
 		$icon = is_admin() ? '<i class="icon gv-icon-astronaut-head"></i>&nbsp;' : NULL;
24
-		$this->title( $icon . __('GravityView', 'gravityview') );
24
+		$this->title( $icon . __( 'GravityView', 'gravityview' ) );
25 25
 	}
26 26
 
27 27
 	function get_warnings() {
28 28
 
29
-		if( is_null( $this->warnings ) ) {
29
+		if ( is_null( $this->warnings ) ) {
30 30
 			$this->warnings = GravityView_Logging::get_errors();
31 31
 		}
32 32
 
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 
36 36
 	function get_notices() {
37 37
 
38
-		if( is_null( $this->notices ) ) {
38
+		if ( is_null( $this->notices ) ) {
39 39
 			$this->notices = GravityView_Logging::get_notices();
40 40
 		}
41 41
 
@@ -84,27 +84,27 @@  discard block
 block discarded – undo
84 84
 		</style>
85 85
 		<div id='debug-bar-gravityview'>";
86 86
 
87
-		$output .= '<img src="'.plugins_url('assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ).'" class="alignright" alt="" width="100" height="132" />';
87
+		$output .= '<img src="' . plugins_url( 'assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ) . '" class="alignright" alt="" width="100" height="132" />';
88 88
 
89 89
 
90 90
 		$warnings = $this->get_warnings();
91 91
 		$notices = $this->get_notices();
92 92
 
93
-		if(count($warnings)) {
94
-			$output .= '<h3><span>'.__('Warnings', 'gravityview').'</span></h3>';
93
+		if ( count( $warnings ) ) {
94
+			$output .= '<h3><span>' . __( 'Warnings', 'gravityview' ) . '</span></h3>';
95 95
 			$output .= '<ol>';
96
-			foreach ( $warnings as $key => $notice) {
97
-				if(empty($notice['message'])) { continue; }
98
-				$output .= '<li><a href="#'.sanitize_html_class( 'gv-warning-' . $key ).'">'.strip_tags($notice['message']).'</a></li>';
96
+			foreach ( $warnings as $key => $notice ) {
97
+				if ( empty( $notice[ 'message' ] ) ) { continue; }
98
+				$output .= '<li><a href="#' . sanitize_html_class( 'gv-warning-' . $key ) . '">' . strip_tags( $notice[ 'message' ] ) . '</a></li>';
99 99
 			}
100 100
 			$output .= '</ol><hr />';
101 101
 		}
102
-		if(count($notices)) {
103
-			$output .= '<h3><span>'.__('Logs', 'gravityview').'</span></h3>';
102
+		if ( count( $notices ) ) {
103
+			$output .= '<h3><span>' . __( 'Logs', 'gravityview' ) . '</span></h3>';
104 104
 			$output .= '<ol>';
105
-			foreach ( $notices as $key => $notice) {
106
-				if(empty($notice['message'])) { continue; }
107
-				$output .= '<li><a href="#'.sanitize_html_class( 'gv-notice-' . $key ).'">'.strip_tags($notice['message']).'</a></li>';
105
+			foreach ( $notices as $key => $notice ) {
106
+				if ( empty( $notice[ 'message' ] ) ) { continue; }
107
+				$output .= '<li><a href="#' . sanitize_html_class( 'gv-notice-' . $key ) . '">' . strip_tags( $notice[ 'message' ] ) . '</a></li>';
108 108
 			}
109 109
 			$output .= '</ol><hr />';
110 110
 		}
@@ -112,14 +112,14 @@  discard block
 block discarded – undo
112 112
 		if ( count( $warnings ) ) {
113 113
 			$output .= '<h3>Warnings</h3>';
114 114
 			$output .= '<ol class="debug-bar-php-list">';
115
-			foreach ( $warnings as $key => $notice) { $output .= $this->render_item( $notice, 'gv-warning-'  . $key ); }
115
+			foreach ( $warnings as $key => $notice ) { $output .= $this->render_item( $notice, 'gv-warning-' . $key ); }
116 116
 			$output .= '</ol>';
117 117
 		}
118 118
 
119 119
 		if ( count( $notices ) ) {
120 120
 			$output .= '<h3>Notices</h3>';
121 121
 			$output .= '<ol class="debug-bar-php-list">';
122
-			foreach ( $notices as $key => $notice) { $output .= $this->render_item( $notice, 'gv-notice-' . $key ); }
122
+			foreach ( $notices as $key => $notice ) { $output .= $this->render_item( $notice, 'gv-notice-' . $key ); }
123 123
 			$output .= '</ol>';
124 124
 		}
125 125
 
@@ -133,17 +133,17 @@  discard block
 block discarded – undo
133 133
 	 * @param  string|array $item Unescaped
134 134
 	 * @return string       Escaped HTML
135 135
 	 */
136
-	function esc_html_recursive($item) {
137
-		if(is_object($item)) {
138
-			foreach($item as $key => $value) {
139
-				$item->{$key} = $this->esc_html_recursive($value);
136
+	function esc_html_recursive( $item ) {
137
+		if ( is_object( $item ) ) {
138
+			foreach ( $item as $key => $value ) {
139
+				$item->{$key} = $this->esc_html_recursive( $value );
140 140
 			}
141
-		} else if(is_array($item)) {
142
-			foreach($item as $key => $value) {
143
-				$item[$key] = $this->esc_html_recursive($value);
141
+		} else if ( is_array( $item ) ) {
142
+			foreach ( $item as $key => $value ) {
143
+				$item[ $key ] = $this->esc_html_recursive( $value );
144 144
 			}
145 145
 		} else {
146
-			$item = esc_html($item);
146
+			$item = esc_html( $item );
147 147
 		}
148 148
 		return $item;
149 149
 	}
@@ -159,26 +159,26 @@  discard block
 block discarded – undo
159 159
 
160 160
 		$output = '';
161 161
 
162
-		if(!empty($notice['message'])) {
163
-			$output .= '<a id="'.sanitize_html_class( $anchor ).'"></a>';
162
+		if ( ! empty( $notice[ 'message' ] ) ) {
163
+			$output .= '<a id="' . sanitize_html_class( $anchor ) . '"></a>';
164 164
 			$output .= "<li class='debug-bar-php-notice'>";
165 165
 		}
166 166
 
167 167
 		$output .= '<div class="clear"></div>';
168 168
 
169 169
 		// Title
170
-		$output .= '<div class="gravityview-debug-bar-title">'.esc_attr( $notice['message'] ).'</div>';
170
+		$output .= '<div class="gravityview-debug-bar-title">' . esc_attr( $notice[ 'message' ] ) . '</div>';
171 171
 
172 172
 		// Debugging Output
173
-		if( empty( $notice['data'] ) ) {
174
-			if( !is_null( $notice['data'] ) ) {
175
-				$output .= '<em>'._x('Empty', 'Debugging output data is empty.', 'gravityview' ).'</em>';
173
+		if ( empty( $notice[ 'data' ] ) ) {
174
+			if ( ! is_null( $notice[ 'data' ] ) ) {
175
+				$output .= '<em>' . _x( 'Empty', 'Debugging output data is empty.', 'gravityview' ) . '</em>';
176 176
 			}
177 177
 		} else {
178
-			$output .= sprintf( '<pre>%s</pre>', print_r($this->esc_html_recursive( $notice['data'] ), true) );
178
+			$output .= sprintf( '<pre>%s</pre>', print_r( $this->esc_html_recursive( $notice[ 'data' ] ), true ) );
179 179
 		}
180 180
 
181
-		if(!empty($notice['message'])) {
181
+		if ( ! empty( $notice[ 'message' ] ) ) {
182 182
 			$output .= '</li>';
183 183
 		}
184 184
 
Please login to merge, or discard this patch.
includes/default-widgets.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -5,4 +5,4 @@
 block discarded – undo
5 5
  * @deprecated 1.7.5
6 6
  */
7 7
 
8
-include_once( GRAVITYVIEW_DIR .'includes/widgets/class-gravityview-widget.php' );
8
+include_once( GRAVITYVIEW_DIR . 'includes/widgets/class-gravityview-widget.php' );
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field-html.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@
 block discarded – undo
27 27
 
28 28
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
29 29
 
30
-		unset ( $field_options['search_filter'], $field_options['show_as_link'] );
30
+		unset ( $field_options[ 'search_filter' ], $field_options[ 'show_as_link' ] );
31 31
 
32 32
 		return $field_options;
33 33
 	}
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field-number.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,17 +28,17 @@
 block discarded – undo
28 28
 
29 29
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
30 30
 
31
-		$field_options['number_format'] = array(
31
+		$field_options[ 'number_format' ] = array(
32 32
 			'type' => 'checkbox',
33 33
 			'label' => __( 'Format number?', 'gravityview' ),
34
-			'desc' => __('Display numbers with thousands separators.', 'gravityview'),
34
+			'desc' => __( 'Display numbers with thousands separators.', 'gravityview' ),
35 35
 			'value' => false,
36 36
 		);
37 37
 
38
-		$field_options['decimals'] = array(
38
+		$field_options[ 'decimals' ] = array(
39 39
 			'type' => 'number',
40 40
 			'label' => __( 'Decimals', 'gravityview' ),
41
-			'desc' => __('Precision of the number of decimal places. Leave blank to use existing precision.', 'gravityview'),
41
+			'desc' => __( 'Precision of the number of decimal places. Leave blank to use existing precision.', 'gravityview' ),
42 42
 			'value' => '',
43 43
 			'merge_tags' => false,
44 44
 		);
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field-post-id.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
 
30 30
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
31 31
 
32
-		$this->add_field_support('link_to_post', $field_options );
32
+		$this->add_field_support( 'link_to_post', $field_options );
33 33
 
34 34
 		return $field_options;
35 35
 	}
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field-post-title.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -25,13 +25,13 @@
 block discarded – undo
25 25
 
26 26
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
27 27
 
28
-		if( 'edit' === $context ) {
28
+		if ( 'edit' === $context ) {
29 29
 			return $field_options;
30 30
 		}
31 31
 
32
-		$this->add_field_support('link_to_post', $field_options );
32
+		$this->add_field_support( 'link_to_post', $field_options );
33 33
 
34
-		$this->add_field_support('dynamic_data', $field_options );
34
+		$this->add_field_support( 'dynamic_data', $field_options );
35 35
 
36 36
 		return $field_options;
37 37
 	}
Please login to merge, or discard this patch.