gravityview / GravityView

includes/class-gravityview-bulk-actions.php

Spacing +20 added lines, -20 removed lines

@@ -54,7 +54,7 @@ 
 
 		$gv_bulk_action = false;
 
-		if( version_compare( GFForms::$version, '2.0', '>=' ) ) {
+		if ( version_compare( GFForms::$version, '2.0', '>=' ) ) {
 			$bulk_action = ( '-1' !== \GV\Utils::_POST( 'action' ) ) ? \GV\Utils::_POST( 'action' ) : \GV\Utils::_POST( 'action2' );
 		} else {
 			// GF 1.9.x - Bulk action 2 is the bottom bulk action select form.
@@ -62,7 +62,7 @@ 
 		}
 
 		// Check the $bulk_action value against GV actions, see if they're the same. I hate strpos().
-		if( ! empty( $bulk_action ) && preg_match( '/^('. implode( '|', self::$bulk_action_prefixes ) .')/ism', $bulk_action ) ) {
+		if ( ! empty( $bulk_action ) && preg_match( '/^(' . implode( '|', self::$bulk_action_prefixes ) . ')/ism', $bulk_action ) ) {
 			$gv_bulk_action = $bulk_action;
 		}
 
@@ -86,7 +86,7 @@ 
 
 		// gforms_entry_list is the nonce that confirms we're on the right page
 		// gforms_update_note is sent when bulk editing entry notes. We don't want to process then.
-		if ( ! ( $bulk_action && \GV\Utils::_POST( 'gforms_entry_list' ) && empty( $_POST['gforms_update_note'] ) ) ) {
+		if ( ! ( $bulk_action && \GV\Utils::_POST( 'gforms_entry_list' ) && empty( $_POST[ 'gforms_update_note' ] ) ) ) {
 			return;
 		}
 
@@ -106,13 +106,13 @@ 
 		}
 
 		// All entries are set to be updated, not just the visible ones
-		if ( ! empty( $_POST['all_entries'] ) ) {
+		if ( ! empty( $_POST[ 'all_entries' ] ) ) {
 
 			// Convert the current entry search into GF-formatted search criteria
 			$search = array(
-				'search_field' => isset( $_POST['f'] ) ? $_POST['f'][0] : 0,
-				'search_value' => isset( $_POST['v'][0] ) ? $_POST['v'][0] : '',
-				'search_operator' => isset( $_POST['o'][0] ) ? $_POST['o'][0] : 'contains',
+				'search_field' => isset( $_POST[ 'f' ] ) ? $_POST[ 'f' ][ 0 ] : 0,
+				'search_value' => isset( $_POST[ 'v' ][ 0 ] ) ? $_POST[ 'v' ][ 0 ] : '',
+				'search_operator' => isset( $_POST[ 'o' ][ 0 ] ) ? $_POST[ 'o' ][ 0 ] : 'contains',
 			);
 
 			$search_criteria = GravityView_frontend::get_search_criteria( $search, $form_id );
@@ -123,7 +123,7 @@ 
 		} else {
 
 			// Changed from 'lead' to 'entry' in 2.0
-			$entries = isset( $_POST['lead'] ) ? $_POST['lead'] : $_POST['entry'];
+			$entries = isset( $_POST[ 'lead' ] ) ? $_POST[ 'lead' ] : $_POST[ 'entry' ];
 
 		}
 
@@ -135,15 +135,15 @@ 
 		$entry_count = count( $entries ) > 1 ? sprintf( __( '%d entries', 'gravityview' ), count( $entries ) ) : __( '1 entry', 'gravityview' );
 
 		switch ( $approved_status ) {
-			case self::$bulk_action_prefixes['approve']:
+			case self::$bulk_action_prefixes[ 'approve' ]:
 				GravityView_Entry_Approval::update_bulk( $entries, GravityView_Entry_Approval_Status::APPROVED, $form_id );
 				self::$bulk_update_message = sprintf( __( '%s approved.', 'gravityview' ), $entry_count );
 				break;
-			case self::$bulk_action_prefixes['unapprove']:
+			case self::$bulk_action_prefixes[ 'unapprove' ]:
 				GravityView_Entry_Approval::update_bulk( $entries, GravityView_Entry_Approval_Status::UNAPPROVED, $form_id );
 				self::$bulk_update_message = sprintf( __( '%s unapproved.', 'gravityview' ), $entry_count );
 				break;
-			case self::$bulk_action_prefixes['disapprove']:
+			case self::$bulk_action_prefixes[ 'disapprove' ]:
 				GravityView_Entry_Approval::update_bulk( $entries, GravityView_Entry_Approval_Status::DISAPPROVED, $form_id );
 				self::$bulk_update_message = sprintf( __( '%s disapproved.', 'gravityview' ), $entry_count );
 				break;
@@ -165,16 +165,16 @@ 
 		$bulk_actions = array(
 			'GravityView' => array(
 				array(
-					'label' => GravityView_Entry_Approval_Status::get_string('approved', 'action'),
-					'value' => sprintf( '%s-%d', self::$bulk_action_prefixes['approve'], $form_id ),
+					'label' => GravityView_Entry_Approval_Status::get_string( 'approved', 'action' ),
+					'value' => sprintf( '%s-%d', self::$bulk_action_prefixes[ 'approve' ], $form_id ),
 				),
 				array(
-					'label' => GravityView_Entry_Approval_Status::get_string('disapproved', 'action'),
-					'value' => sprintf( '%s-%d', self::$bulk_action_prefixes['disapprove'], $form_id ),
+					'label' => GravityView_Entry_Approval_Status::get_string( 'disapproved', 'action' ),
+					'value' => sprintf( '%s-%d', self::$bulk_action_prefixes[ 'disapprove' ], $form_id ),
 				),
 				array(
-					'label' => GravityView_Entry_Approval_Status::get_string('unapproved', 'action'),
-					'value' => sprintf( '%s-%d', self::$bulk_action_prefixes['unapprove'], $form_id ),
+					'label' => GravityView_Entry_Approval_Status::get_string( 'unapproved', 'action' ),
+					'value' => sprintf( '%s-%d', self::$bulk_action_prefixes[ 'unapprove' ], $form_id ),
 				),
 			),
 		);
@@ -191,13 +191,13 @@ 
 		// Sanitize the values, just to be sure.
 		foreach ( $bulk_actions as $key => $group ) {
 
-			if( empty( $group ) ) {
+			if ( empty( $group ) ) {
 				continue;
 			}
 
 			foreach ( $group as $i => $action ) {
-				$bulk_actions[ $key ][ $i ]['label'] = esc_html( $bulk_actions[ $key ][ $i ]['label'] );
-				$bulk_actions[ $key ][ $i ]['value'] = esc_attr( $bulk_actions[ $key ][ $i ]['value'] );
+				$bulk_actions[ $key ][ $i ][ 'label' ] = esc_html( $bulk_actions[ $key ][ $i ][ 'label' ] );
+				$bulk_actions[ $key ][ $i ][ 'value' ] = esc_attr( $bulk_actions[ $key ][ $i ][ 'value' ] );
 			}
 		}