gravityview / GravityView

includes/extensions/entry-notes/partials/note-add-note.php

Spacing +2 added lines, -2 removed lines

@@ -21,10 +21,10 @@
 		{email_fields}
 
 		<div class="gv-note-content-container">
-			<label for="gv-note-content-{entry_slug}" class="screen-reader-text"><?php echo GravityView_Field_Notes::strings('content-label'); ?></label>
+			<label for="gv-note-content-{entry_slug}" class="screen-reader-text"><?php echo GravityView_Field_Notes::strings( 'content-label' ); ?></label>
 			<textarea name="gv-note-content" id="gv-note-content-{entry_slug}"></textarea>
 		</div>
 
-		<button type="submit" class="button gv-add-note-submit"><?php echo GravityView_Field_Notes::strings('add-note'); ?></button>
+		<button type="submit" class="button gv-add-note-submit"><?php echo GravityView_Field_Notes::strings( 'add-note' ); ?></button>
 	</div>
 </form>
\ No newline at end of file

includes/admin/field-types/type_html.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 * @return string
 	 */
 	function get_field_desc() {
-		return !empty( $this->field['desc'] ) ? $this->field['desc'] : '';
+		return ! empty( $this->field[ 'desc' ] ) ? $this->field[ 'desc' ] : '';
 	}
 
 }

includes/admin/field-types/type_checkboxes.php

Spacing +6 added lines, -6 removed lines

@@ -22,7 +22,7 @@ 
 	}
 
 	function render_input( $override_input = null ) {
-		if( isset( $override_input ) ) {
+		if ( isset( $override_input ) ) {
 			echo $override_input;
 			return;
 		}
@@ -30,18 +30,18 @@ 
 		?>
 		<ul class="gv-setting-list">
 		<?php
-		foreach( $this->field['options'] as $value => $label ) { ?>
-			<li <?php if( isset( $label['requires'] ) ) { printf( 'class="gv-sub-setting" data-requires="%s"', $label['requires'] ); } ?>>
+		foreach ( $this->field[ 'options' ] as $value => $label ) { ?>
+			<li <?php if ( isset( $label[ 'requires' ] ) ) { printf( 'class="gv-sub-setting" data-requires="%s"', $label[ 'requires' ] ); } ?>>
 				<label>
 				<input name="<?php printf( '%s[%s]', esc_attr( $this->name ), esc_attr( $value ) ); ?>" type="hidden"
 				       value="0"/>
 				<input name="<?php printf( '%s[%s]', esc_attr( $this->name ), esc_attr( $value ) ); ?>"
 				       id="<?php echo $this->get_field_id(); ?>" type="checkbox"
 				       value="1" <?php checked( ! empty( $this->value[ $value ] ) ); ?> />
-				<?php echo esc_html( $label['label'] ); ?>
+				<?php echo esc_html( $label[ 'label' ] ); ?>
 				</label>
-				<?php if( ! empty( $label['desc'] ) ) {
-					printf( '<span class="howto">%s</span>', $label['desc'] );
+				<?php if ( ! empty( $label[ 'desc' ] ) ) {
+					printf( '<span class="howto">%s</span>', $label[ 'desc' ] );
 				}
 				?>
 			</li>

includes/extensions/entry-notes/fields/notes.php

Spacing +13 added lines, -13 removed lines

@@ -9,9 +9,9 @@ 
 $gravityview_view = GravityView_View::getInstance();
 
 $visibility_settings = $gravityview_view->getCurrentFieldSetting( 'notes' );
-$show_notes_logged_out = ( ! empty( $visibility_settings['view'] ) && ! empty( $visibility_settings['view_loggedout'] ) );
+$show_notes_logged_out = ( ! empty( $visibility_settings[ 'view' ] ) && ! empty( $visibility_settings[ 'view_loggedout' ] ) );
 
-if(	! GVCommon::has_cap( array( 'gravityview_view_entry_notes', 'gravityview_add_entry_notes', 'gravityview_delete_entry_notes' ) ) && ! $show_notes_logged_out ) {
+if ( ! GVCommon::has_cap( array( 'gravityview_view_entry_notes', 'gravityview_add_entry_notes', 'gravityview_delete_entry_notes' ) ) && ! $show_notes_logged_out ) {
 	return;
 }
 
@@ -25,20 +25,20 @@ 
 do_action( 'gravityview/field/notes/scripts' );
 
 $entry = $gravityview_view->getCurrentEntry();
-$notes = GravityView_Entry_Notes::get_notes( $entry['id'] );
+$notes = GravityView_Entry_Notes::get_notes( $entry[ 'id' ] );
 $strings = GravityView_Field_Notes::strings();
-$entry_slug = GravityView_API::get_entry_slug( $entry['id'], $entry );
+$entry_slug = GravityView_API::get_entry_slug( $entry[ 'id' ], $entry );
 
-$show_add = ! empty( $visibility_settings['add'] );
-$show_delete = ( ! empty( $visibility_settings['delete'] ) && GVCommon::has_cap( 'gravityview_delete_entry_notes' ) );
-$show_notes = $show_notes_logged_out || ( ! empty( $visibility_settings['view'] ) && GVCommon::has_cap( 'gravityview_view_entry_notes' ) );
+$show_add = ! empty( $visibility_settings[ 'add' ] );
+$show_delete = ( ! empty( $visibility_settings[ 'delete' ] ) && GVCommon::has_cap( 'gravityview_delete_entry_notes' ) );
+$show_notes = $show_notes_logged_out || ( ! empty( $visibility_settings[ 'view' ] ) && GVCommon::has_cap( 'gravityview_view_entry_notes' ) );
 
 $container_class = ( sizeof( $notes ) > 0 ? 'gv-has-notes' : 'gv-no-notes' );
 $container_class .= $show_notes ? ' gv-show-notes' : ' gv-hide-notes';
 ?>
 <div class="gv-notes <?php echo $container_class; ?>">
 <?php
-	if( $show_notes ) {
+	if ( $show_notes ) {
 ?>
 	<form method="post" class="gv-notes-list">
 		<?php if ( $show_delete ) { wp_nonce_field( 'gv_delete_notes_' . $entry_slug, 'gv_delete_notes' ); } ?>
@@ -46,21 +46,21 @@ 
 			<input type="hidden" name="action" value="gv_delete_notes" />
 			<input type="hidden" name="entry-slug" value="<?php echo esc_attr( $entry_slug ); ?>" />
 			<table>
-				<caption><?php echo $strings['caption']; ?></caption>
+				<caption><?php echo $strings[ 'caption' ]; ?></caption>
 				<?php
 				if ( $show_delete ) {
 				?>
 				<thead>
 					<tr>
 						<th colspan="2">
-							<label><input type="checkbox" value="" class="gv-notes-toggle" title="<?php echo $strings['toggle-notes']; ?>"><span class="screen-reader-text"><?php echo $strings['toggle-notes']; ?></span></label>
-							<button type="submit" class="button button-small gv-notes-delete"><?php echo $strings['delete']; ?></button>
+							<label><input type="checkbox" value="" class="gv-notes-toggle" title="<?php echo $strings[ 'toggle-notes' ]; ?>"><span class="screen-reader-text"><?php echo $strings[ 'toggle-notes' ]; ?></span></label>
+							<button type="submit" class="button button-small gv-notes-delete"><?php echo $strings[ 'delete' ]; ?></button>
 						</th>
 					</tr>
 				</thead>
 				<?php } ?>
 				<tbody>
-					<tr class="gv-notes-no-notes"><td colspan="2"><?php echo $strings['no-notes']; ?></td></tr>
+					<tr class="gv-notes-no-notes"><td colspan="2"><?php echo $strings[ 'no-notes' ]; ?></td></tr>
 					<?php
 						foreach ( $notes as $note ) {
 							echo GravityView_Field_Notes::display_note( $note, $show_delete );
@@ -73,7 +73,7 @@ 
 <?php
 	} // End if can view notes
 
-	if( $show_add ) {
+	if ( $show_add ) {
 		echo do_shortcode( '[gv_note_add]' );
 	}
 ?>

includes/plugin-and-theme-hooks/class-gravityview-theme-hooks-church-themes.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 */
 	public function if_gravityview_return_true( $has_content = false ) {
 
-		if( ! class_exists( 'GravityView_frontend' ) ) {
+		if ( ! class_exists( 'GravityView_frontend' ) ) {
 			return $has_content;
 		}
 

includes/admin/class-gravityview-admin-no-conflict.php

Braces +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
 
 		// Dequeue other jQuery styles even if no-conflict is off.
 		// Terrible-looking tabs help no one.
-		if( !empty( $wp_styles->registered ) )  {
+		if( !empty( $wp_styles->registered ) ) {
 			foreach ($wp_styles->registered as $key => $style) {
 				if( preg_match( '/^(?:wp\-)?jquery/ism', $key ) ) {
 					wp_dequeue_style( $key );
@@ -189,8 +189,8 @@ 
 
 		//unregistering scripts
 		$registered = array();
-		foreach( $wp_objects->registered as $handle => $script_registration ){
-			if( in_array( $handle, $required_objects ) ){
+		foreach( $wp_objects->registered as $handle => $script_registration ) {
+			if( in_array( $handle, $required_objects ) ) {
 				$registered[ $handle ] = $script_registration;
 			}
 		}

Spacing +22 added lines, -22 removed lines

@@ -18,7 +18,7 @@ 
 	 */
 	public function __construct() {
 
-		if( ! is_admin() ) { return; }
+		if ( ! is_admin() ) { return; }
 		
 		$this->add_hooks();
 	}
@@ -32,13 +32,13 @@ 
 	 */
 	private function add_hooks() {
 		//Hooks for no-conflict functionality
-		add_action( 'wp_print_scripts', array( $this, 'no_conflict_scripts' ), 1000);
-		add_action( 'admin_print_footer_scripts', array( $this, 'no_conflict_scripts' ), 9);
+		add_action( 'wp_print_scripts', array( $this, 'no_conflict_scripts' ), 1000 );
+		add_action( 'admin_print_footer_scripts', array( $this, 'no_conflict_scripts' ), 9 );
 
-		add_action( 'wp_print_styles', array( $this, 'no_conflict_styles' ), 1000);
-		add_action( 'admin_print_styles', array( $this, 'no_conflict_styles' ), 11);
-		add_action( 'admin_print_footer_scripts', array( $this, 'no_conflict_styles' ), 1);
-		add_action( 'admin_footer', array( $this, 'no_conflict_styles' ), 1);
+		add_action( 'wp_print_styles', array( $this, 'no_conflict_styles' ), 1000 );
+		add_action( 'admin_print_styles', array( $this, 'no_conflict_styles' ), 11 );
+		add_action( 'admin_print_footer_scripts', array( $this, 'no_conflict_styles' ), 1 );
+		add_action( 'admin_footer', array( $this, 'no_conflict_styles' ), 1 );
 	}
 
 	/**
@@ -51,13 +51,13 @@ 
 	function no_conflict_scripts() {
 		global $wp_scripts;
 
-		if( ! gravityview()->request->is_admin( '', null ) ) {
+		if ( ! gravityview()->request->is_admin( '', null ) ) {
 			return;
 		}
 
 		$no_conflict_mode = gravityview()->plugin->settings->get( 'no-conflict-mode' );
 
-		if( empty( $no_conflict_mode ) ) {
+		if ( empty( $no_conflict_mode ) ) {
 			return;
 		}
 
@@ -110,15 +110,15 @@ 
 	function no_conflict_styles() {
 		global $wp_styles;
 
-		if( ! gravityview()->request->is_admin( '', null ) ) {
+		if ( ! gravityview()->request->is_admin( '', null ) ) {
 			return;
 		}
 
 		// Dequeue other jQuery styles even if no-conflict is off.
 		// Terrible-looking tabs help no one.
-		if( !empty( $wp_styles->registered ) )  {
-			foreach ($wp_styles->registered as $key => $style) {
-				if( preg_match( '/^(?:wp\-)?jquery/ism', $key ) ) {
+		if ( ! empty( $wp_styles->registered ) ) {
+			foreach ( $wp_styles->registered as $key => $style ) {
+				if ( preg_match( '/^(?:wp\-)?jquery/ism', $key ) ) {
 					wp_dequeue_style( $key );
 				}
 			}
@@ -127,7 +127,7 @@ 
 		$no_conflict_mode = gravityview()->plugin->settings->get( 'no-conflict-mode' );
 
 		// If no conflict is off, jQuery will suffice.
-		if( empty( $no_conflict_mode ) ) {
+		if ( empty( $no_conflict_mode ) ) {
 			return;
 		}
 
@@ -154,7 +154,7 @@ 
 		/**
 		 * @action `gravityview_remove_conflicts_after` Runs after no-conflict styles are removed. You can re-add styles here.
 		 */
-		do_action('gravityview_remove_conflicts_after');
+		do_action( 'gravityview_remove_conflicts_after' );
 	}
 
 	/**
@@ -178,9 +178,9 @@ 
 
 		//reset queue
 		$queue = array();
-		foreach( $wp_objects->queue as $object ) {
-			if( in_array( $object, $required_objects ) || preg_match('/gravityview|gf_|gravityforms/ism', $object ) ) {
-				$queue[] = $object;
+		foreach ( $wp_objects->queue as $object ) {
+			if ( in_array( $object, $required_objects ) || preg_match( '/gravityview|gf_|gravityforms/ism', $object ) ) {
+				$queue[ ] = $object;
 			}
 		}
 		$wp_objects->queue = $queue;
@@ -189,8 +189,8 @@ 
 
 		//unregistering scripts
 		$registered = array();
-		foreach( $wp_objects->registered as $handle => $script_registration ){
-			if( in_array( $handle, $required_objects ) ){
+		foreach ( $wp_objects->registered as $handle => $script_registration ) {
+			if ( in_array( $handle, $required_objects ) ) {
 				$registered[ $handle ] = $script_registration;
 			}
 		}
@@ -205,7 +205,7 @@ 
 	 * @param array $registered [description]
 	 * @param array $scripts    [description]
 	 */
-	private function add_script_dependencies($registered, $scripts) {
+	private function add_script_dependencies( $registered, $scripts ) {
 
 		//gets all dependent scripts linked to the $scripts array passed
 		do {
@@ -214,7 +214,7 @@ 
 				$deps = isset( $registered[ $script ] ) && is_array( $registered[ $script ]->deps ) ? $registered[ $script ]->deps : array();
 				foreach ( $deps as $dep ) {
 					if ( ! in_array( $dep, $scripts ) && ! in_array( $dep, $dependents ) ) {
-						$dependents[] = $dep;
+						$dependents[ ] = $dep;
 					}
 				}
 			}

includes/admin/field-types/type_radio.php

Spacing +2 added lines, -2 removed lines

@@ -19,12 +19,12 @@
 	}
 
 	function render_input( $override_input = null ) {
-		if( isset( $override_input ) ) {
+		if ( isset( $override_input ) ) {
 			echo $override_input;
 			return;
 		}
 
-		foreach( $this->field['options'] as $value => $label ) : ?>
+		foreach ( $this->field[ 'options' ] as $value => $label ) : ?>
 		<label class="<?php echo $this->get_label_class(); ?>">
 			<input name="<?php echo esc_attr( $this->name ); ?>" id="<?php echo $this->get_field_id(); ?>-<?php echo esc_attr( $value ); ?>" type="radio" value="<?php echo esc_attr( $value ); ?>" <?php checked( $value, $this->value, true ); ?> />&nbsp;<?php echo esc_html( $label ); ?>
 		</label>

includes/plugin-and-theme-hooks/class-gravityview-plugin-hooks-gravityformsquiz.php

Spacing +5 added lines, -5 removed lines

@@ -48,25 +48,25 @@
 		
 		$quiz_fields = GFAPI::get_fields_by_type( $form, 'quiz' );
 		
-		if( ! empty( $quiz_fields ) ) {
+		if ( ! empty( $quiz_fields ) ) {
 			
-			$fields['gquiz_score']   = array(
+			$fields[ 'gquiz_score' ] = array(
 				'label' => __( 'Quiz Score Total', 'gravityview' ),
 				'type'  => 'quiz_score',
 				'desc'  => __( 'Displays the number of correct Quiz answers the user submitted.', 'gravityview' ),
 			);
-			$fields['gquiz_percent'] = array(
+			$fields[ 'gquiz_percent' ] = array(
 				'label' => __( 'Quiz Percentage Grade', 'gravityview' ),
 				'type'  => 'quiz_percent',
 				'desc'  => __( 'Displays the percentage of correct Quiz answers the user submitted.', 'gravityview' ),
 			);
-			$fields['gquiz_grade']   = array(
+			$fields[ 'gquiz_grade' ] = array(
 				/* translators: This is a field type used by the Gravity Forms Quiz Addon. "A" is 100-90, "B" is 89-80, "C" is 79-70, etc.  */
 				'label' => __( 'Quiz Letter Grade', 'gravityview' ),
 				'type'  => 'quiz_grade',
 				'desc'  => __( 'Displays the Grade the user achieved based on Letter Grading configured in the Quiz Settings.', 'gravityview' ),
 			);
-			$fields['gquiz_is_pass'] = array(
+			$fields[ 'gquiz_is_pass' ] = array(
 				'label' => __( 'Quiz Pass/Fail', 'gravityview' ),
 				'type'  => 'quiz_is_pass',
 				'desc'  => __( 'Displays either Passed or Failed based on the Pass/Fail settings configured in the Quiz Settings.', 'gravityview' ),

includes/fields/class-gravityview-field-password.php

Braces +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 *
 	 * @return string If a custom field label isn't set, return the field label for the password field
 	 */
-	function field_label( $label = '', $field = array(), $form = array(), $entry = array() ){
+	function field_label( $label = '', $field = array(), $form = array(), $entry = array() ) {
 
 		// If using a custom label, no need to fetch the parent label
 		if( ! is_numeric( $field['id'] ) || ! empty( $field['custom_label'] ) ) {

Spacing +12 added lines, -12 removed lines

@@ -49,17 +49,17 @@ 
 	 *
 	 * @return string If a custom field label isn't set, return the field label for the password field
 	 */
-	function field_label( $label = '', $field = array(), $form = array(), $entry = array() ){
+	function field_label( $label = '', $field = array(), $form = array(), $entry = array() ) {
 
 		// If using a custom label, no need to fetch the parent label
-		if( ! is_numeric( $field['id'] ) || ! empty( $field['custom_label'] ) ) {
+		if ( ! is_numeric( $field[ 'id' ] ) || ! empty( $field[ 'custom_label' ] ) ) {
 			return $label;
 		}
 
-		$field_object = GFFormsModel::get_field( $form, $field['id'] );
+		$field_object = GFFormsModel::get_field( $form, $field[ 'id' ] );
 
-		if( $field_object && 'password' === $field_object->type ) {
-			$label = $field['label'];
+		if ( $field_object && 'password' === $field_object->type ) {
+			$label = $field[ 'label' ];
 		}
 
 		return $label;
@@ -79,17 +79,17 @@ 
 	function add_form_fields( $fields = array(), $form = array(), $include_parent_field = true ) {
 
 		foreach ( $fields as $key => $field ) {
-			if( 'password' === $field['type'] ) {
+			if ( 'password' === $field[ 'type' ] ) {
 
 				// The Enter Password input
-				if( floor( $key ) === floatval( $key ) ) {
+				if ( floor( $key ) === floatval( $key ) ) {
 
-					if( ! empty( $field['parent'] ) ) {
-						$field['label']      = $field['parent']->label;
-						$field['adminOnly']  = $field['parent']->adminOnly;
-						$field['adminLabel'] = $field['parent']->adminLabel;
+					if ( ! empty( $field[ 'parent' ] ) ) {
+						$field[ 'label' ]      = $field[ 'parent' ]->label;
+						$field[ 'adminOnly' ]  = $field[ 'parent' ]->adminOnly;
+						$field[ 'adminLabel' ] = $field[ 'parent' ]->adminLabel;
 						// Don't show as a child input
-						unset( $field['parent'] );
+						unset( $field[ 'parent' ] );
 					}
 
 					$fields[ $key ] = $field;

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 	 * @param \GV\Entry		$entry	The entry.
 	 * @param \GV\Request	$request The request context.
 	 *
-	 * @return mixed $value The filtered value.
+	 * @return string $value The filtered value.
 	 */
 	public function get_value( $value, $field, $view, $source, $entry, $request ) {
 		/** Passwords should never be exposed. */