pronamic / wp-pronamic-ideal

classes/Subscriptions/SubscriptionsDataStoreCPT.php

Spacing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
 		$post_status = $this->get_post_status( $subscription->get_status() );
 
 		if ( ! empty( $post_status ) ) {
-			$data['post_status'] = $post_status;
+			$data[ 'post_status' ] = $post_status;
 		}
 
 		wp_update_post( $data );

src/check-versions.php

Spacing +6 added lines, -6 removed lines

@@ -46,9 +46,9 @@ 
 
 foreach ( $tokens as $token ) {
 	if ( is_array( $token ) ) {
-		$type  = $token[0];
-		$value = $token[1];
-		$line  = $token[2];
+		$type  = $token[ 0 ];
+		$value = $token[ 1 ];
+		$line  = $token[ 2 ];
 
 		if ( in_array( $type, array( T_COMMENT, T_DOC_COMMENT ), true ) && false !== strpos( $value, 'Plugin Name' ) ) {
 			$file_header = $value;
@@ -90,14 +90,14 @@ 
 	exit( 1 );
 }
 
-if ( $default_properties['version'] !== $pkg->version ) {
+if ( $default_properties[ 'version' ] !== $pkg->version ) {
 	printf( '❌  ' );
-	printf( 'The "version" property value "%s" in class "%s" does not match.', $default_properties['version'], $plugin_class_reflection->getName() );
+	printf( 'The "version" property value "%s" in class "%s" does not match.', $default_properties[ 'version' ], $plugin_class_reflection->getName() );
 
 	exit( 1 );
 }
 
 printf( '✅  ' );
-printf( 'The "version" property value "%s" in class "%s" matches.', $default_properties['version'], $plugin_class_reflection->getName() );
+printf( 'The "version" property value "%s" in class "%s" matches.', $default_properties[ 'version' ], $plugin_class_reflection->getName() );
 
 echo PHP_EOL;

classes/Admin/AdminGatewayPostType.php

Spacing +21 added lines, -21 removed lines

@@ -153,7 +153,7 @@ 
 							$name = __( 'Dashboard', 'pronamic_ideal' );
 						}
 
-						$content[] = sprintf(
+						$content[ ] = sprintf(
 							'<a href="%s" target="_blank">%s</a>',
 							esc_attr( $url ),
 							esc_html( ucfirst( $name ) )
@@ -179,7 +179,7 @@ 
 		}
 
 		if ( intval( get_option( 'pronamic_pay_config_id' ) ) === $post->ID ) {
-			$post_states['pronamic_pay_config_default'] = __( 'Default', 'pronamic_ideal' );
+			$post_states[ 'pronamic_pay_config_default' ] = __( 'Default', 'pronamic_ideal' );
 		}
 
 		return $post_states;
@@ -273,9 +273,9 @@ 
 		);
 
 		foreach ( $fields as $field ) {
-			if ( isset( $field['meta_key'], $field['filter'] ) ) {
-				$name   = $field['meta_key'];
-				$filter = $field['filter'];
+			if ( isset( $field[ 'meta_key' ], $field[ 'filter' ] ) ) {
+				$name   = $field[ 'meta_key' ];
+				$filter = $field[ 'filter' ];
 
 				$definition[ $name ] = $filter;
 			}
@@ -283,28 +283,28 @@ 
 
 		$data = filter_input_array( INPUT_POST, $definition );
 
-		if ( ! empty( $data['_pronamic_gateway_id'] ) ) {
+		if ( ! empty( $data[ '_pronamic_gateway_id' ] ) ) {
 			$integrations = $this->plugin->gateway_integrations;
 
-			if ( isset( $integrations[ $data['_pronamic_gateway_id'] ] ) ) {
-				$integration = $integrations[ $data['_pronamic_gateway_id'] ];
+			if ( isset( $integrations[ $data[ '_pronamic_gateway_id' ] ] ) ) {
+				$integration = $integrations[ $data[ '_pronamic_gateway_id' ] ];
 			}
 
 			if ( $integration ) {
 				$settings = $integration->get_settings();
 
 				foreach ( $fields as $field ) {
-					if ( isset( $field['default'], $field['meta_key'], $data[ $field['meta_key'] ] ) ) {
+					if ( isset( $field[ 'default' ], $field[ 'meta_key' ], $data[ $field[ 'meta_key' ] ] ) ) {
 						// Remove default value if not applicable to the selected gateway.
-						if ( isset( $field['methods'] ) ) {
-							$clean_default = array_intersect( $settings, $field['methods'] );
+						if ( isset( $field[ 'methods' ] ) ) {
+							$clean_default = array_intersect( $settings, $field[ 'methods' ] );
 
 							if ( empty( $clean_default ) ) {
-								$meta_value = get_post_meta( $post_id, $field['meta_key'], true );
+								$meta_value = get_post_meta( $post_id, $field[ 'meta_key' ], true );
 
 								// Only remove value if not saved before.
 								if ( empty( $meta_value ) ) {
-									$data[ $field['meta_key'] ] = null;
+									$data[ $field[ 'meta_key' ] ] = null;
 
 									continue;
 								}
@@ -312,13 +312,13 @@ 
 						}
 
 						// Set the default value if empty.
-						if ( empty( $data[ $field['meta_key'] ] ) ) {
-							$default = $field['default'];
+						if ( empty( $data[ $field[ 'meta_key' ] ] ) ) {
+							$default = $field[ 'default' ];
 
-							if ( is_array( $default ) && 2 === count( $default ) && Util::class_method_exists( $default[0], $default[1] ) ) {
-								$data[ $field['meta_key'] ] = call_user_func( $default, $field );
+							if ( is_array( $default ) && 2 === count( $default ) && Util::class_method_exists( $default[ 0 ], $default[ 1 ] ) ) {
+								$data[ $field[ 'meta_key' ] ] = call_user_func( $default, $field );
 							} else {
-								$data[ $field['meta_key'] ] = $default;
+								$data[ $field[ 'meta_key' ] ] = $default;
 							}
 						}
 					}
@@ -368,14 +368,14 @@ 
 			0  => '', // Unused. Messages start at index 1.
 			1  => __( 'Configuration updated.', 'pronamic_ideal' ),
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352799&filters[translation_id]=37947229
-			2  => $messages['post'][2],
+			2  => $messages[ 'post' ][ 2 ],
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352800&filters[translation_id]=37947870
-			3  => $messages['post'][3],
+			3  => $messages[ 'post' ][ 3 ],
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352798&filters[translation_id]=37947230
 			4  => __( 'Configuration updated.', 'pronamic_ideal' ),
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352801&filters[translation_id]=37947231
 			// translators: %s: date and time of the revision.
-			5  => isset( $_GET['revision'] ) ? sprintf( __( 'Configuration restored to revision from %s.', 'pronamic_ideal' ), wp_post_revision_title( (int) $_GET['revision'], false ) ) : false, // WPCS: CSRF ok.
+			5  => isset( $_GET[ 'revision' ] ) ? sprintf( __( 'Configuration restored to revision from %s.', 'pronamic_ideal' ), wp_post_revision_title( (int) $_GET[ 'revision' ], false ) ) : false, // WPCS: CSRF ok.
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352802&filters[translation_id]=37949178
 			6  => __( 'Configuration published.', 'pronamic_ideal' ),
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352803&filters[translation_id]=37947232

classes/Plugin.php

Spacing +4 added lines, -4 removed lines

@@ -476,7 +476,7 @@ 
 		);
 
 		if ( $payment_method ) {
-			$args['post__in'] = PaymentMethods::get_config_ids( $payment_method );
+			$args[ 'post__in' ] = PaymentMethods::get_config_ids( $payment_method );
 		}
 
 		$query = new WP_Query( $args );
@@ -687,10 +687,10 @@ 
 		$payment->set_credit_card( $data->get_credit_card() );
 
 		// User Agent (@see https://github.com/WordPress/WordPress/blob/4.9.4/wp-includes/comment.php#L1962-L1965).
-		$payment->user_agent = isset( $_SERVER['HTTP_USER_AGENT'] ) ? $_SERVER['HTTP_USER_AGENT'] : null;
+		$payment->user_agent = isset( $_SERVER[ 'HTTP_USER_AGENT' ] ) ? $_SERVER[ 'HTTP_USER_AGENT' ] : null;
 
 		// IP (@see https://github.com/WordPress/WordPress/blob/4.9.4/wp-includes/comment.php#L1957-L1960).
-		$payment->user_ip = isset( $_SERVER['REMOTE_ADDR'] ) ? $_SERVER['REMOTE_ADDR'] : null;
+		$payment->user_ip = isset( $_SERVER[ 'REMOTE_ADDR' ] ) ? $_SERVER[ 'REMOTE_ADDR' ] : null;
 
 		return self::start_payment( $payment, $gateway );
 	}
@@ -713,7 +713,7 @@ 
 
 		if ( empty( $amount ) ) {
 			// Keep track of free payments to update during shutdown.
-			pronamic_pay_plugin()->payments_module->free[] = $payment->get_id();
+			pronamic_pay_plugin()->payments_module->free[ ] = $payment->get_id();
 
 			return $payment;
 		}

classes/Admin/AdminSubscriptionPostType.php

Spacing +5 added lines, -5 removed lines

@@ -72,10 +72,10 @@ 
 		$screen = get_current_screen();
 
 		if ( self::POST_TYPE === $screen->post_type ) {
-			if ( ! isset( $vars['post_status'] ) ) {
-				$vars['post_status'] = array_keys( SubscriptionPostType::get_states() );
+			if ( ! isset( $vars[ 'post_status' ] ) ) {
+				$vars[ 'post_status' ] = array_keys( SubscriptionPostType::get_states() );
 
-				$vars['post_status'][] = 'publish';
+				$vars[ 'post_status' ][ ] = 'publish';
 			}
 		}
 
@@ -113,8 +113,8 @@ 
 	 * @return array
 	 */
 	public function sortable_columns( $sortable_columns ) {
-		$sortable_columns['pronamic_subscription_title'] = 'ID';
-		$sortable_columns['pronamic_subscription_date']  = 'date';
+		$sortable_columns[ 'pronamic_subscription_title' ] = 'ID';
+		$sortable_columns[ 'pronamic_subscription_date' ]  = 'date';
 
 		return $sortable_columns;
 	}

classes/Admin/AdminPaymentPostType.php

Spacing +14 added lines, -14 removed lines

@@ -93,10 +93,10 @@ 
 		$screen = get_current_screen();
 
 		if ( self::POST_TYPE === $screen->post_type ) {
-			if ( ! isset( $vars['post_status'] ) ) {
-				$vars['post_status'] = array_keys( PaymentPostType::get_payment_states() );
+			if ( ! isset( $vars[ 'post_status' ] ) ) {
+				$vars[ 'post_status' ] = array_keys( PaymentPostType::get_payment_states() );
 
-				$vars['post_status'][] = 'publish';
+				$vars[ 'post_status' ][ ] = 'publish';
 			}
 		}
 
@@ -126,7 +126,7 @@ 
 
 			Plugin::update_payment( $payment, false );
 
-			$this->admin_notices[] = array(
+			$this->admin_notices[ ] = array(
 				'type'    => 'info',
 				'message' => __( 'Payment status updated.', 'pronamic_ideal' ),
 			);
@@ -140,8 +140,8 @@ 
 		foreach ( $this->admin_notices as $notice ) {
 			printf(
 				'<div class="notice notice-%1$s"><p>%2$s</p></div>',
-				esc_attr( $notice['type'] ),
-				esc_html( $notice['message'] )
+				esc_attr( $notice[ 'type' ] ),
+				esc_html( $notice[ 'message' ] )
 			);
 		}
 	}
@@ -196,8 +196,8 @@ 
 	 * @return array
 	 */
 	public function default_hidden_columns( $hidden ) {
-		$hidden[] = 'pronamic_payment_gateway';
-		$hidden[] = 'pronamic_payment_description';
+		$hidden[ ] = 'pronamic_payment_gateway';
+		$hidden[ ] = 'pronamic_payment_description';
 
 		return $hidden;
 	}
@@ -209,9 +209,9 @@ 
 	 * @return array
 	 */
 	public function sortable_columns( $sortable_columns ) {
-		$sortable_columns['pronamic_payment_title']  = 'ID';
-		$sortable_columns['pronamic_payment_amount'] = 'pronamic_payment_amount';
-		$sortable_columns['pronamic_payment_date']   = 'date';
+		$sortable_columns[ 'pronamic_payment_title' ]  = 'ID';
+		$sortable_columns[ 'pronamic_payment_amount' ] = 'pronamic_payment_amount';
+		$sortable_columns[ 'pronamic_payment_date' ]   = 'date';
 
 		return $sortable_columns;
 	}
@@ -551,14 +551,14 @@ 
 			0  => '', // Unused. Messages start at index 1.
 			1  => __( 'Payment updated.', 'pronamic_ideal' ),
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352799&filters[translation_id]=37947229.
-			2  => $messages['post'][2],
+			2  => $messages[ 'post' ][ 2 ],
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352800&filters[translation_id]=37947870.
-			3  => $messages['post'][3],
+			3  => $messages[ 'post' ][ 3 ],
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352798&filters[translation_id]=37947230.
 			4  => __( 'Payment updated.', 'pronamic_ideal' ),
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352801&filters[translation_id]=37947231.
 			// translators: %s: date and time of the revision
-			5  => isset( $_GET['revision'] ) ? sprintf( __( 'Payment restored to revision from %s.', 'pronamic_ideal' ), wp_post_revision_title( (int) $_GET['revision'], false ) ) : false, // WPCS: CSRF ok.
+			5  => isset( $_GET[ 'revision' ] ) ? sprintf( __( 'Payment restored to revision from %s.', 'pronamic_ideal' ), wp_post_revision_title( (int) $_GET[ 'revision' ], false ) ) : false, // WPCS: CSRF ok.
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352802&filters[translation_id]=37949178.
 			6  => __( 'Payment published.', 'pronamic_ideal' ),
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352803&filters[translation_id]=37947232.

src/repositories.php

Spacing +4 added lines, -4 removed lines

@@ -75,11 +75,11 @@ 
 		// Git flow
 		chdir( $git_dir );
 
-		if ( isset( $argv[1] ) && 'develop' === $argv[1] ) {
+		if ( isset( $argv[ 1 ] ) && 'develop' === $argv[ 1 ] ) {
 			`git checkout develop`;
 		}
 
-		if ( isset( $argv[1] ) && 'pull' === $argv[1] ) {
+		if ( isset( $argv[ 1 ] ) && 'pull' === $argv[ 1 ] ) {
 			$command = 'git pull';
 
 			echo $command, PHP_EOL;
@@ -87,8 +87,8 @@ 
 			echo shell_exec( $command ), PHP_EOL;
 		}
 
-		if ( isset( $argv[1], $argv[2] ) && in_array( $argv[1], array( 'git', 'composer', 'yarn' ) ) ) {
-			$command = sprintf( '%s %s', $argv[1], $argv[2] );
+		if ( isset( $argv[ 1 ], $argv[ 2 ] ) && in_array( $argv[ 1 ], array( 'git', 'composer', 'yarn' ) ) ) {
+			$command = sprintf( '%s %s', $argv[ 1 ], $argv[ 2 ] );
 		}
 
 		if ( null !== $command ) {

tests/Payments/PaymentTest.php

Spacing +24 added lines, -24 removed lines

@@ -46,10 +46,10 @@ 
 
 	public function get_and_set_provider() {
 		return array(
-			array( 'set_amount',         'get_amount',         new Money( 89.95, 'EUR' ) ),
-			array( 'set_id',             'get_id',             uniqid()    ),
-			array( 'set_transaction_id', 'get_transaction_id', uniqid()    ),
-			array( 'set_status',         'get_status',         'completed' ),
+			array( 'set_amount', 'get_amount', new Money( 89.95, 'EUR' ) ),
+			array( 'set_id', 'get_id', uniqid() ),
+			array( 'set_transaction_id', 'get_transaction_id', uniqid() ),
+			array( 'set_status', 'get_status', 'completed' ),
 		);
 	}
 
@@ -68,11 +68,11 @@ 
 
 	public function set_provider() {
 		return array(
-			array( 'set_consumer_name',           'consumer_name',           'John Doe' ),
+			array( 'set_consumer_name', 'consumer_name', 'John Doe' ),
 			array( 'set_consumer_account_number', 'consumer_account_number', '1086.34.779' ),
-			array( 'set_consumer_iban',           'consumer_iban',           'NL56 RABO 0108 6347 79' ),
-			array( 'set_consumer_bic',            'consumer_bic',            'RABONL2U' ),
-			array( 'set_consumer_city',           'consumer_city',           'Drachten' ),
+			array( 'set_consumer_iban', 'consumer_iban', 'NL56 RABO 0108 6347 79' ),
+			array( 'set_consumer_bic', 'consumer_bic', 'RABONL2U' ),
+			array( 'set_consumer_city', 'consumer_city', 'Drachten' ),
 		);
 	}
 
@@ -91,23 +91,23 @@ 
 
 	public function get_provider() {
 		return array(
-			array( 'order_id',            'get_order_id',            1234 ),
-			array( 'method',              'get_method',              'ideal' ),
-			array( 'issuer',              'get_issuer',              'ideal_KNABNL2H' ),
-			array( 'language',            'get_language',            'nl' ),
-			array( 'locale',              'get_locale',              'nl_NL' ),
-			array( 'description',         'get_description',         'Lorem ipsum dolor sit amet, consectetur.' ),
-			array( 'email',               'get_email',               'john.doe@example.com' ),
-			array( 'first_name',          'get_first_name',          'John' ),
-			array( 'last_name',           'get_last_name',           'Doe' ),
-			array( 'customer_name',       'get_customer_name',       'John Doe' ),
-			array( 'address',             'get_address',             'Burgemeester Wuiteweg 39b' ),
-			array( 'city',                'get_city',                'Drachten' ),
-			array( 'zip',                 'get_zip',                 '9203 KA' ),
-			array( 'country',             'get_country',             'NL' ),
-			array( 'telephone_number',    'get_telephone_number',    '1234567890' ),
+			array( 'order_id', 'get_order_id', 1234 ),
+			array( 'method', 'get_method', 'ideal' ),
+			array( 'issuer', 'get_issuer', 'ideal_KNABNL2H' ),
+			array( 'language', 'get_language', 'nl' ),
+			array( 'locale', 'get_locale', 'nl_NL' ),
+			array( 'description', 'get_description', 'Lorem ipsum dolor sit amet, consectetur.' ),
+			array( 'email', 'get_email', 'john.doe@example.com' ),
+			array( 'first_name', 'get_first_name', 'John' ),
+			array( 'last_name', 'get_last_name', 'Doe' ),
+			array( 'customer_name', 'get_customer_name', 'John Doe' ),
+			array( 'address', 'get_address', 'Burgemeester Wuiteweg 39b' ),
+			array( 'city', 'get_city', 'Drachten' ),
+			array( 'zip', 'get_zip', '9203 KA' ),
+			array( 'country', 'get_country', 'NL' ),
+			array( 'telephone_number', 'get_telephone_number', '1234567890' ),
 			array( 'analytics_client_id', 'get_analytics_client_id', 'GA1.2.1234567890.1234567890' ),
-			array( 'entrance_code',       'get_entrance_code',       uniqid() ),
+			array( 'entrance_code', 'get_entrance_code', uniqid() ),
 		);
 	}
 

tests/Subscriptions/SubscriptionTest.php

Spacing +12 added lines, -12 removed lines

@@ -44,10 +44,10 @@ 
 
 	public function get_and_set_provider() {
 		return array(
-			array( 'set_amount',         'get_amount',         new Money( 89.95, 'EUR' ) ),
-			array( 'set_id',             'get_id',             uniqid()    ),
-			array( 'set_status',         'get_status',         'completed' ),
-			array( 'set_transaction_id', 'get_transaction_id', uniqid()    ),
+			array( 'set_amount', 'get_amount', new Money( 89.95, 'EUR' ) ),
+			array( 'set_id', 'get_id', uniqid() ),
+			array( 'set_status', 'get_status', 'completed' ),
+			array( 'set_transaction_id', 'get_transaction_id', uniqid() ),
 		);
 	}
 
@@ -73,7 +73,7 @@ 
 		return array(
 			array( 'set_consumer_name', 'consumer_name', 'John Doe' ),
 			array( 'set_consumer_iban', 'consumer_iban', 'NL56 RABO 0108 6347 79' ),
-			array( 'set_consumer_bic',  'consumer_bic',  'RABONL2U' ),
+			array( 'set_consumer_bic', 'consumer_bic', 'RABONL2U' ),
 		);
 	}
 
@@ -97,13 +97,13 @@ 
 	 */
 	public function get_provider() {
 		return array(
-			array( 'key',                 'get_key',             uniqid() ),
-			array( 'source',              'get_source',          'woocommerce' ),
-			array( 'source_id',           'get_source_id',       '1234' ),
-			array( 'frequency',           'get_frequency',       'daily' ),
-			array( 'interval',            'get_interval',        '1' ),
-			array( 'interval_period',     'get_interval_period', 'Y' ),
-			array( 'description',         'get_description',     'Lorem ipsum dolor sit amet, consectetur.' ),
+			array( 'key', 'get_key', uniqid() ),
+			array( 'source', 'get_source', 'woocommerce' ),
+			array( 'source_id', 'get_source_id', '1234' ),
+			array( 'frequency', 'get_frequency', 'daily' ),
+			array( 'interval', 'get_interval', '1' ),
+			array( 'interval_period', 'get_interval_period', 'Y' ),
+			array( 'description', 'get_description', 'Lorem ipsum dolor sit amet, consectetur.' ),
 		);
 	}