View GitHub Repository GravityView

Schedule Inspection
Subscribe
Completed
Push — develop ( 31c9ca...168da6 )
by Zack
38:08 queued 17:59
created
Status
Category
Toggle Patches
includes/class-gravityview-roles-capabilities.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch 
@@ -33,7 +33,7 @@  discard block
 
 block discarded – undo
33 33 
 	 */
34 34 
 	public static function get_instance() {
35 35
36 
-		if( ! self::$instance ) {
36 
+		if ( ! self::$instance ) {
37 37 
 			self::$instance = new self;
38 38 
 		}
39 39 
 
@@ -56,7 +56,7 @@  discard block
 
 block discarded – undo
56 56 
 		add_filter( 'members_get_capabilities', array( 'GravityView_Roles_Capabilities', 'merge_with_all_caps' ) );
57 57 
 		add_action( 'members_register_cap_groups', array( $this, 'members_register_cap_group' ), 20 );
58 58 
 		add_filter( 'user_has_cap', array( $this, 'filter_user_has_cap' ), 10, 4 );
59 
-        add_action( 'admin_init', array( $this, 'add_caps') );
59 
+        add_action( 'admin_init', array( $this, 'add_caps' ) );
60 60 
 	}
61 61
62 62 
 
@@ -77,18 +77,18 @@  discard block
 
 block discarded – undo
77 77 
 	public function filter_user_has_cap( $usercaps = array(), $caps = array(), $args = array(), $user = NULL ) {
78 78
79 79 
 		// Empty caps_to_check array
80 
-		if( ! $usercaps || ! $caps ) {
80 
+		if ( ! $usercaps || ! $caps ) {
81 81 
 			return $usercaps;
82 82 
 		}
83 83
84 84 
 		/**
85 85 
 		 * Enable all GravityView caps_to_check if `gravityview_full_access` is enabled
86 86 
 		 */
87 
-		if( ! empty( $usercaps['gravityview_full_access'] ) ) {
87 
+		if ( ! empty( $usercaps[ 'gravityview_full_access' ] ) ) {
88 88
89 89 
 			$all_gravityview_caps = self::all_caps();
90 90
91 
-			foreach( $all_gravityview_caps as $gv_cap ) {
91 
+			foreach ( $all_gravityview_caps as $gv_cap ) {
92 92 
 				$usercaps[ $gv_cap ] = true;
93 93 
 			}
94 94 
 
@@ -306,7 +306,7 @@  discard block
 
 block discarded – undo
306 306 
 		$all = $administrator;
307 307
308 308 
 		// If role is set, return caps_to_check for just that role.
309 
-		if( $single_role ) {
309 
+		if ( $single_role ) {
310 310 
 			$caps = isset( ${$single_role} ) ? ${$single_role} : false;
311 311 
 			return $flat_array ? $caps : array( $single_role => $caps );
312 312 
 		}
 
@@ -355,9 +355,9 @@  discard block
 
 block discarded – undo
355 355
356 356 
 		if ( true === $allow_logged_out ) {
357 357
358 
-			$all_caps = self::all_caps('editor');
358 
+			$all_caps = self::all_caps( 'editor' );
359 359
360 
-			if( array_intersect( $all_caps, (array) $caps_to_check ) ) {
360 
+			if ( array_intersect( $all_caps, (array)$caps_to_check ) ) {
361 361 
 				return true;
362 362 
 			}
363 363 
 		}
 
@@ -378,13 +378,13 @@  discard block
 
 block discarded – undo
378 378 
 		$caps_to_check = self::maybe_add_full_access_caps( $caps_to_check );
379 379
380 380 
 		foreach ( $caps_to_check as $cap ) {
381 
-			if( ! is_null( $object_id ) ) {
381 
+			if ( ! is_null( $object_id ) ) {
382 382 
 				$has_cap = $user_id ? user_can( $user_id, $cap, $object_id ) : current_user_can( $cap, $object_id );
383 383 
 			} else {
384 384 
 				$has_cap = $user_id ? user_can( $user_id, $cap ) : current_user_can( $cap );
385 385 
 			}
386 386 
 			// At the first successful response, stop checking
387 
-			if( $has_cap ) {
387 
+			if ( $has_cap ) {
388 388 
 				break;
389 389 
 			}
390 390 
 		}
 
@@ -408,15 +408,15 @@  discard block
 
 block discarded – undo
408 408 
 		$all_gravityview_caps = self::all_caps();
409 409
410 410 
 		// Are there any $caps_to_check that are from GravityView?
411 
-		if( $has_gravityview_caps = array_intersect( $caps_to_check, $all_gravityview_caps ) ) {
412 
-			$caps_to_check[] = 'gravityview_full_access';
411 
+		if ( $has_gravityview_caps = array_intersect( $caps_to_check, $all_gravityview_caps ) ) {
412 
+			$caps_to_check[ ] = 'gravityview_full_access';
413 413 
 		}
414 414
415 415 
 		$all_gravity_forms_caps = class_exists( 'GFCommon' ) ? GFCommon::all_caps() : array();
416 416
417 417 
 		// Are there any $caps_to_check that are from Gravity Forms?
418 
-		if( $all_gravity_forms_caps = array_intersect( $caps_to_check, $all_gravity_forms_caps ) ) {
419 
-			$caps_to_check[] = 'gform_full_access';
418 
+		if ( $all_gravity_forms_caps = array_intersect( $caps_to_check, $all_gravity_forms_caps ) ) {
419 
+			$caps_to_check[ ] = 'gform_full_access';
420 420 
 		}
421 421
422 422 
 		return array_unique( $caps_to_check );
 
@@ -438,7 +438,7 @@  discard block
 
 block discarded – undo
438 438 
 			$capabilities = self::all_caps();
439 439
440 440 
 			// Loop through each role and remove GV caps_to_check
441 
-			foreach( $wp_roles->get_names() as $role_slug => $role_name ) {
441 
+			foreach ( $wp_roles->get_names() as $role_slug => $role_name ) {
442 442 
 				foreach ( $capabilities as $cap ) {
443 443 
 					$wp_roles->remove_cap( $role_slug, $cap );
444 444 
 				}
Please login to merge, or discard this patch.
future/loader.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -25,7 +25,7 @@  discard block
 
 block discarded – undo
25 25 
 			printf( '<body style="padding: 0; margin: 0; font-family: -apple-system, BlinkMacSystemFont, \'Segoe UI\', Roboto, Oxygen-Sans, Ubuntu, Cantarell, \'Helvetica Neue\', sans-serif;">' );
26 26 
 			printf( '<img src="' . plugins_url( 'assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ) . '" alt="The GravityView Astronaut Says:" style="float: left; height: 60px; margin-right : 10px;" />' );
27 27 
 			printf( __( "%sGravityView requires PHP Version %s or newer.%s \n\nYou're using Version %s. Please ask your host to upgrade your server's PHP.", 'gravityview' ),
28 
-				'<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION , "</h3>\n\n", $version );
28 
+				'<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION, "</h3>\n\n", $version );
29 29 
 			printf( '</body>' );
30 30 
 		}
31 31 
 
@@ -34,7 +34,7 @@  discard block
 
 block discarded – undo
34 34 
 }
35 35
36 36 
 /** The future branch of GravityView requires PHP 5.3+ namespaces and SPL. */
37 
-if ( version_compare( phpversion(), '5.3.0' , '<' ) ) {
37 
+if ( version_compare( phpversion(), '5.3.0', '<' ) ) {
38 38 
 	require GRAVITYVIEW_DIR . 'future/_stubs.php';
39 39
40 40 
 /** All looks fine. */
Please login to merge, or discard this patch.
future/includes/class-gv-shortcode.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -76,7 +76,7 @@  discard block
 
 block discarded – undo
76 76 
 	 */
77 77 
 	public static function remove() {
78 78 
 		$shortcode = new static();
79 
-		unset( self::$shortcodes[$shortcode->name] );
79 
+		unset( self::$shortcodes[ $shortcode->name ] );
80 80 
 		remove_shortcode( $shortcode->name );
81 81 
 	}
82 82 
 
@@ -108,14 +108,14 @@  discard block
 
 block discarded – undo
108 108 
 		preg_match_all( '/' . get_shortcode_regex() . '/', $content, $matches, PREG_SET_ORDER );
109 109
110 110 
 		foreach ( $matches as $shortcode ) {
111 
-			$shortcode_name = $shortcode[2];
111 
+			$shortcode_name = $shortcode[ 2 ];
112 112
113 
-			$shortcode_atts = shortcode_parse_atts( $shortcode[3] );
114 
-			$shortcode_content = $shortcode[5];
113 
+			$shortcode_atts = shortcode_parse_atts( $shortcode[ 3 ] );
114 
+			$shortcode_content = $shortcode[ 5 ];
115 115
116 116 
 			/** This is a registered GravityView shortcode. */
117 
-			if ( !empty( self::$shortcodes[$shortcode_name] ) ) {
118 
-				$shortcode = clone self::$shortcodes[$shortcode_name];
117 
+			if ( ! empty( self::$shortcodes[ $shortcode_name ] ) ) {
118 
+				$shortcode = clone self::$shortcodes[ $shortcode_name ];
119 119 
 			} else {
120 120 
 				/** This is some generic shortcode. */
121 121 
 				$shortcode = new self;
Please login to merge, or discard this patch.
future/includes/class-gv-field-gravityforms.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -27,17 +27,17 @@  discard block
 
 block discarded – undo
27 27 
 	 * @return \GV\GF_Field|null The field implementation or null on error.
28 28 
 	 */
29 29 
 	public static function from_configuration( $configuration ) {
30 
-		if ( empty( $configuration['id'] ) || ! is_numeric( $configuration['id'] ) ) {
30 
+		if ( empty( $configuration[ 'id' ] ) || ! is_numeric( $configuration[ 'id' ] ) ) {
31 31 
 			gravityview()->log->error( 'Invalid configuration[id] supplied.' );
32 32 
 			return null;
33 33 
 		}
34 34
35 
-		if ( empty( $configuration['form_id'] ) || ! $form = \GV\GF_Form::by_id( $configuration['form_id'] )  ) {
35 
+		if ( empty( $configuration[ 'form_id' ] ) || ! $form = \GV\GF_Form::by_id( $configuration[ 'form_id' ] ) ) {
36 36 
 			gravityview()->log->error( 'Invalid configuration[form_id] supplied.' );
37 37 
 			return null;
38 38 
 		}
39 39
40 
-		$field = self::by_id( $form, $configuration['id'] );
40 
+		$field = self::by_id( $form, $configuration[ 'id' ] );
41 41
42 42 
 		if ( ! $field ) {
43 43 
 			gravityview()->log->error( 'Invalid configuration[id] supplied.' );
 
@@ -111,7 +111,7 @@  discard block
 
 block discarded – undo
111 111
112 112 
 		/** This is a complex Gravity Forms input. */
113 113 
 		if ( $input = \GFFormsModel::get_input( $this->field, $this->ID ) ) {
114 
-			$label = ! empty( $input['customLabel'] ) ? $input['customLabel'] : $input['label'];
114 
+			$label = ! empty( $input[ 'customLabel' ] ) ? $input[ 'customLabel' ] : $input[ 'label' ];
115 115 
 		} else {
116 116 
 			/** This is a field with one label. */
117 117 
 			$label = $this->field->get_field_label( true, $this->label );
Please login to merge, or discard this patch.
future/includes/class-gv-template-entry-legacy.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -34,7 +34,7 @@
 
 block discarded – undo
34 34 
 		global $post;
35 35
36 36 
 		if ( $post ) {
37 
-			$context['post'] = $post;
37 
+			$context[ 'post' ] = $post;
38 38 
 		}
39 39
40 40 
 		\GV\Mocks\Legacy_Context::push( $context );
Please login to merge, or discard this patch.
future/includes/class-gv-core.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -113,14 +113,14 @@
 
 block discarded – undo
113 113
114 114 
 		/** Require critical legacy core files. @todo Deprecate */
115 115 
 		require_once $this->plugin->dir( 'includes/helper-functions.php' );
116 
-		require_once $this->plugin->dir( 'includes/class-common.php');
117 
-		require_once $this->plugin->dir( 'includes/connector-functions.php');
116 
+		require_once $this->plugin->dir( 'includes/class-common.php' );
117 
+		require_once $this->plugin->dir( 'includes/connector-functions.php' );
118 118 
 		require_once $this->plugin->dir( 'includes/class-gravityview-compatibility.php' );
119 119 
 		require_once $this->plugin->dir( 'includes/class-gravityview-roles-capabilities.php' );
120 120 
 		require_once $this->plugin->dir( 'includes/class-gravityview-admin-notices.php' );
121 121 
 		require_once $this->plugin->dir( 'includes/class-admin.php' );
122 
-		require_once $this->plugin->dir( 'includes/class-post-types.php');
123 
-		require_once $this->plugin->dir( 'includes/class-cache.php');
122 
+		require_once $this->plugin->dir( 'includes/class-post-types.php' );
123 
+		require_once $this->plugin->dir( 'includes/class-cache.php' );
124 124
125 125 
 		/**
126 126 
 		 * GravityView extensions and widgets.
Please login to merge, or discard this patch.
future/includes/class-gv-template-entry.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch 
@@ -113,18 +113,18 @@
 
 block discarded – undo
113 113 
 		global $post;
114 114
115 115 
 		if ( ! $this->request->is_view() && $post ) {
116 
-			$specifics []= sprintf( '%spost-%d-view-%d-entry-%d-%s.php', $slug_dir, $post->ID, $this->view->ID, $this->entry->ID, $slug_name );
117 
-			$specifics []= sprintf( '%spost-%d-entry-%d-%s.php', $slug_dir, $post->ID, $this->entry->ID, $slug_name );
118 
-			$specifics []= sprintf( '%spost-%d-view-%d-%s.php', $slug_dir, $post->ID, $this->view->ID, $slug_name );
119 
-			$specifics []= sprintf( '%spost-%d-%s.php', $slug_dir, $post->ID, $slug_name );
116 
+			$specifics [ ] = sprintf( '%spost-%d-view-%d-entry-%d-%s.php', $slug_dir, $post->ID, $this->view->ID, $this->entry->ID, $slug_name );
117 
+			$specifics [ ] = sprintf( '%spost-%d-entry-%d-%s.php', $slug_dir, $post->ID, $this->entry->ID, $slug_name );
118 
+			$specifics [ ] = sprintf( '%spost-%d-view-%d-%s.php', $slug_dir, $post->ID, $this->view->ID, $slug_name );
119 
+			$specifics [ ] = sprintf( '%spost-%d-%s.php', $slug_dir, $post->ID, $slug_name );
120 120 
 		}
121 121
122 
-		$specifics []= sprintf( '%sview-%d-entry-%d-%s.php', $slug_dir, $this->view->ID, $this->entry->ID, $slug_name );
123 
-		$specifics []= sprintf( '%sform-%d-entry-%d-%s.php', $slug_dir, $this->view->form->ID, $this->entry->ID, $slug_name );
124 
-		$specifics []= sprintf( '%sview-%d-%s.php', $slug_dir, $this->view->ID, $slug_name );
125 
-		$specifics []= sprintf( '%sform-%d-%s.php', $slug_dir, $this->view->form->ID, $slug_name );
122 
+		$specifics [ ] = sprintf( '%sview-%d-entry-%d-%s.php', $slug_dir, $this->view->ID, $this->entry->ID, $slug_name );
123 
+		$specifics [ ] = sprintf( '%sform-%d-entry-%d-%s.php', $slug_dir, $this->view->form->ID, $this->entry->ID, $slug_name );
124 
+		$specifics [ ] = sprintf( '%sview-%d-%s.php', $slug_dir, $this->view->ID, $slug_name );
125 
+		$specifics [ ] = sprintf( '%sform-%d-%s.php', $slug_dir, $this->view->form->ID, $slug_name );
126 126
127 
-		$specifics []= sprintf( '%sentry-%d-%s.php', $slug_dir, $this->entry->ID, $slug_name );
127 
+		$specifics [ ] = sprintf( '%sentry-%d-%s.php', $slug_dir, $this->entry->ID, $slug_name );
128 128
129 129 
 		return array_merge( $specifics, $templates );
130 130 
 	}
Please login to merge, or discard this patch.
future/includes/class-gv-collection-entry-sort.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 
 		if ( $this->field ) {
79 79 
 			return array(
80 80 
 				'key' => $this->field->ID,
81 
-				'direction' => $this->direction ? : self::ASC,
81 
+				'direction' => $this->direction ?: self::ASC,
82 82 
 				'is_numeric' => self::ALPHA ? true : false,
83 83 
 			);
84 84 
 		}
Please login to merge, or discard this patch.
future/includes/class-gv-wrappers.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -50,8 +50,8 @@
 
 block discarded – undo
50 50 
 		/**
51 51 
 		 * By array.
52 52 
 		 */
53 
-		if ( is_array( $view ) && ! empty( $view['id'] ) ) {
54 
-			return $this->get( $view['id'] );
53 
+		if ( is_array( $view ) && ! empty( $view[ 'id' ] ) ) {
54 
+			return $this->get( $view[ 'id' ] );
55 55 
 		}
56 56
57 57 
 		/**
Please login to merge, or discard this patch.