Passed
Push — develop ( 0321f9...73d743 )
by Reüel
09:16 queued 04:10
created
classes/Admin/AdminSubscriptionPostType.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
 		$screen = get_current_screen();
73 73
 
74 74
 		if ( self::POST_TYPE === $screen->post_type ) {
75
-			if ( ! isset( $vars['post_status'] ) ) {
76
-				$vars['post_status'] = array_keys( SubscriptionPostType::get_states() );
75
+			if ( ! isset( $vars[ 'post_status' ] ) ) {
76
+				$vars[ 'post_status' ] = array_keys( SubscriptionPostType::get_states() );
77 77
 
78
-				$vars['post_status'][] = 'publish';
78
+				$vars[ 'post_status' ][ ] = 'publish';
79 79
 			}
80 80
 		}
81 81
 
@@ -113,8 +113,8 @@  discard block
 block discarded – undo
113 113
 	 * @return array
114 114
 	 */
115 115
 	public function sortable_columns( $sortable_columns ) {
116
-		$sortable_columns['pronamic_subscription_title'] = 'ID';
117
-		$sortable_columns['pronamic_subscription_date']  = 'date';
116
+		$sortable_columns[ 'pronamic_subscription_title' ] = 'ID';
117
+		$sortable_columns[ 'pronamic_subscription_date' ]  = 'date';
118 118
 
119 119
 		return $sortable_columns;
120 120
 	}
Please login to merge, or discard this patch.
classes/Admin/AdminPaymentPostType.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -93,10 +93,10 @@  discard block
 block discarded – undo
93 93
 		$screen = get_current_screen();
94 94
 
95 95
 		if ( self::POST_TYPE === $screen->post_type ) {
96
-			if ( ! isset( $vars['post_status'] ) ) {
97
-				$vars['post_status'] = array_keys( PaymentPostType::get_payment_states() );
96
+			if ( ! isset( $vars[ 'post_status' ] ) ) {
97
+				$vars[ 'post_status' ] = array_keys( PaymentPostType::get_payment_states() );
98 98
 
99
-				$vars['post_status'][] = 'publish';
99
+				$vars[ 'post_status' ][ ] = 'publish';
100 100
 			}
101 101
 		}
102 102
 
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 
127 127
 			Plugin::update_payment( $payment, false );
128 128
 
129
-			$this->admin_notices[] = array(
129
+			$this->admin_notices[ ] = array(
130 130
 				'type'    => 'info',
131 131
 				'message' => __( 'Payment status updated.', 'pronamic_ideal' ),
132 132
 			);
@@ -140,8 +140,8 @@  discard block
 block discarded – undo
140 140
 		foreach ( $this->admin_notices as $notice ) {
141 141
 			printf(
142 142
 				'<div class="notice notice-%1$s"><p>%2$s</p></div>',
143
-				esc_attr( $notice['type'] ),
144
-				esc_html( $notice['message'] )
143
+				esc_attr( $notice[ 'type' ] ),
144
+				esc_html( $notice[ 'message' ] )
145 145
 			);
146 146
 		}
147 147
 	}
@@ -196,8 +196,8 @@  discard block
 block discarded – undo
196 196
 	 * @return array
197 197
 	 */
198 198
 	public function default_hidden_columns( $hidden ) {
199
-		$hidden[] = 'pronamic_payment_gateway';
200
-		$hidden[] = 'pronamic_payment_description';
199
+		$hidden[ ] = 'pronamic_payment_gateway';
200
+		$hidden[ ] = 'pronamic_payment_description';
201 201
 
202 202
 		return $hidden;
203 203
 	}
@@ -209,9 +209,9 @@  discard block
 block discarded – undo
209 209
 	 * @return array
210 210
 	 */
211 211
 	public function sortable_columns( $sortable_columns ) {
212
-		$sortable_columns['pronamic_payment_title']  = 'ID';
213
-		$sortable_columns['pronamic_payment_amount'] = 'pronamic_payment_amount';
214
-		$sortable_columns['pronamic_payment_date']   = 'date';
212
+		$sortable_columns[ 'pronamic_payment_title' ]  = 'ID';
213
+		$sortable_columns[ 'pronamic_payment_amount' ] = 'pronamic_payment_amount';
214
+		$sortable_columns[ 'pronamic_payment_date' ]   = 'date';
215 215
 
216 216
 		return $sortable_columns;
217 217
 	}
@@ -551,14 +551,14 @@  discard block
 block discarded – undo
551 551
 			0  => '', // Unused. Messages start at index 1.
552 552
 			1  => __( 'Payment updated.', 'pronamic_ideal' ),
553 553
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352799&filters[translation_id]=37947229.
554
-			2  => $messages['post'][2],
554
+			2  => $messages[ 'post' ][ 2 ],
555 555
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352800&filters[translation_id]=37947870.
556
-			3  => $messages['post'][3],
556
+			3  => $messages[ 'post' ][ 3 ],
557 557
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352798&filters[translation_id]=37947230.
558 558
 			4  => __( 'Payment updated.', 'pronamic_ideal' ),
559 559
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352801&filters[translation_id]=37947231.
560 560
 			// translators: %s: date and time of the revision
561
-			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.
561
+			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.
562 562
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352802&filters[translation_id]=37949178.
563 563
 			6  => __( 'Payment published.', 'pronamic_ideal' ),
564 564
 			// @see https://translate.wordpress.org/projects/wp/4.4.x/admin/nl/default?filters[status]=either&filters[original_id]=2352803&filters[translation_id]=37947232.
Please login to merge, or discard this patch.
classes/Subscriptions/SubscriptionsModule.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -261,7 +261,7 @@  discard block
 block discarded – undo
261 261
 		// Handle renewals.
262 262
 		if ( is_array( $renewal ) ) {
263 263
 			$payment->recurring = false;
264
-			$payment->issuer    = $renewal['issuer'];
264
+			$payment->issuer    = $renewal[ 'issuer' ];
265 265
 		}
266 266
 
267 267
 		// Start payment.
@@ -305,11 +305,11 @@  discard block
 block discarded – undo
305 305
 	 * @return array
306 306
 	 */
307 307
 	public function exclude_subscription_comment_notes( $clauses, $query ) {
308
-		$type = $query->query_vars['type'];
308
+		$type = $query->query_vars[ 'type' ];
309 309
 
310 310
 		// Ignore subscription notes comments if it's not specifically requested.
311 311
 		if ( 'subscription_note' !== $type ) {
312
-			$clauses['where'] .= " AND comment_type != 'subscription_note'";
312
+			$clauses[ 'where' ] .= " AND comment_type != 'subscription_note'";
313 313
 		}
314 314
 
315 315
 		return $clauses;
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
 		);
597 597
 
598 598
 		if ( ! $cli_test ) {
599
-			$args['meta_query'][] = array(
599
+			$args[ 'meta_query' ][ ] = array(
600 600
 				'key'     => '_pronamic_subscription_next_payment',
601 601
 				'compare' => '<=',
602 602
 				'value'   => current_time( 'mysql', true ),
Please login to merge, or discard this patch.
src/repositories.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -75,11 +75,11 @@  discard block
 block discarded – undo
75 75
 		// Git flow
76 76
 		chdir( $git_dir );
77 77
 
78
-		if ( isset( $argv[1] ) && 'develop' === $argv[1] ) {
78
+		if ( isset( $argv[ 1 ] ) && 'develop' === $argv[ 1 ] ) {
79 79
 			`git checkout develop`;
80 80
 		}
81 81
 
82
-		if ( isset( $argv[1] ) && 'pull' === $argv[1] ) {
82
+		if ( isset( $argv[ 1 ] ) && 'pull' === $argv[ 1 ] ) {
83 83
 			$command = 'git pull';
84 84
 
85 85
 			echo $command, PHP_EOL;
@@ -87,8 +87,8 @@  discard block
 block discarded – undo
87 87
 			echo shell_exec( $command ), PHP_EOL;
88 88
 		}
89 89
 
90
-		if ( isset( $argv[1], $argv[2] ) && in_array( $argv[1], array( 'git', 'composer', 'yarn' ) ) ) {
91
-			$command = sprintf( '%s %s', $argv[1], $argv[2] );
90
+		if ( isset( $argv[ 1 ], $argv[ 2 ] ) && in_array( $argv[ 1 ], array( 'git', 'composer', 'yarn' ) ) ) {
91
+			$command = sprintf( '%s %s', $argv[ 1 ], $argv[ 2 ] );
92 92
 		}
93 93
 
94 94
 		if ( null !== $command ) {
Please login to merge, or discard this patch.
tests/Payments/PaymentTest.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -46,10 +46,10 @@  discard block
 block discarded – undo
46 46
 
47 47
 	public function get_and_set_provider() {
48 48
 		return array(
49
-			array( 'set_amount',         'get_amount',         new Money( 89.95, 'EUR' ) ),
50
-			array( 'set_id',             'get_id',             uniqid()    ),
51
-			array( 'set_transaction_id', 'get_transaction_id', uniqid()    ),
52
-			array( 'set_status',         'get_status',         'completed' ),
49
+			array( 'set_amount', 'get_amount', new Money( 89.95, 'EUR' ) ),
50
+			array( 'set_id', 'get_id', uniqid() ),
51
+			array( 'set_transaction_id', 'get_transaction_id', uniqid() ),
52
+			array( 'set_status', 'get_status', 'completed' ),
53 53
 		);
54 54
 	}
55 55
 
@@ -68,11 +68,11 @@  discard block
 block discarded – undo
68 68
 
69 69
 	public function set_provider() {
70 70
 		return array(
71
-			array( 'set_consumer_name',           'consumer_name',           'John Doe' ),
71
+			array( 'set_consumer_name', 'consumer_name', 'John Doe' ),
72 72
 			array( 'set_consumer_account_number', 'consumer_account_number', '1086.34.779' ),
73
-			array( 'set_consumer_iban',           'consumer_iban',           'NL56 RABO 0108 6347 79' ),
74
-			array( 'set_consumer_bic',            'consumer_bic',            'RABONL2U' ),
75
-			array( 'set_consumer_city',           'consumer_city',           'Drachten' ),
73
+			array( 'set_consumer_iban', 'consumer_iban', 'NL56 RABO 0108 6347 79' ),
74
+			array( 'set_consumer_bic', 'consumer_bic', 'RABONL2U' ),
75
+			array( 'set_consumer_city', 'consumer_city', 'Drachten' ),
76 76
 		);
77 77
 	}
78 78
 
@@ -91,23 +91,23 @@  discard block
 block discarded – undo
91 91
 
92 92
 	public function get_provider() {
93 93
 		return array(
94
-			array( 'order_id',            'get_order_id',            1234 ),
95
-			array( 'method',              'get_method',              'ideal' ),
96
-			array( 'issuer',              'get_issuer',              'ideal_KNABNL2H' ),
97
-			array( 'language',            'get_language',            'nl' ),
98
-			array( 'locale',              'get_locale',              'nl_NL' ),
99
-			array( 'description',         'get_description',         'Lorem ipsum dolor sit amet, consectetur.' ),
100
-			array( 'email',               'get_email',               '[email protected]' ),
101
-			array( 'first_name',          'get_first_name',          'John' ),
102
-			array( 'last_name',           'get_last_name',           'Doe' ),
103
-			array( 'customer_name',       'get_customer_name',       'John Doe' ),
104
-			array( 'address',             'get_address',             'Burgemeester Wuiteweg 39b' ),
105
-			array( 'city',                'get_city',                'Drachten' ),
106
-			array( 'zip',                 'get_zip',                 '9203 KA' ),
107
-			array( 'country',             'get_country',             'NL' ),
108
-			array( 'telephone_number',    'get_telephone_number',    '1234567890' ),
94
+			array( 'order_id', 'get_order_id', 1234 ),
95
+			array( 'method', 'get_method', 'ideal' ),
96
+			array( 'issuer', 'get_issuer', 'ideal_KNABNL2H' ),
97
+			array( 'language', 'get_language', 'nl' ),
98
+			array( 'locale', 'get_locale', 'nl_NL' ),
99
+			array( 'description', 'get_description', 'Lorem ipsum dolor sit amet, consectetur.' ),
100
+			array( 'email', 'get_email', '[email protected]' ),
101
+			array( 'first_name', 'get_first_name', 'John' ),
102
+			array( 'last_name', 'get_last_name', 'Doe' ),
103
+			array( 'customer_name', 'get_customer_name', 'John Doe' ),
104
+			array( 'address', 'get_address', 'Burgemeester Wuiteweg 39b' ),
105
+			array( 'city', 'get_city', 'Drachten' ),
106
+			array( 'zip', 'get_zip', '9203 KA' ),
107
+			array( 'country', 'get_country', 'NL' ),
108
+			array( 'telephone_number', 'get_telephone_number', '1234567890' ),
109 109
 			array( 'analytics_client_id', 'get_analytics_client_id', 'GA1.2.1234567890.1234567890' ),
110
-			array( 'entrance_code',       'get_entrance_code',       uniqid() ),
110
+			array( 'entrance_code', 'get_entrance_code', uniqid() ),
111 111
 		);
112 112
 	}
113 113
 
Please login to merge, or discard this patch.
tests/Subscriptions/SubscriptionTest.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -44,10 +44,10 @@  discard block
 block discarded – undo
44 44
 
45 45
 	public function get_and_set_provider() {
46 46
 		return array(
47
-			array( 'set_amount',         'get_amount',         new Money( 89.95, 'EUR' ) ),
48
-			array( 'set_id',             'get_id',             uniqid()    ),
49
-			array( 'set_status',         'get_status',         'completed' ),
50
-			array( 'set_transaction_id', 'get_transaction_id', uniqid()    ),
47
+			array( 'set_amount', 'get_amount', new Money( 89.95, 'EUR' ) ),
48
+			array( 'set_id', 'get_id', uniqid() ),
49
+			array( 'set_status', 'get_status', 'completed' ),
50
+			array( 'set_transaction_id', 'get_transaction_id', uniqid() ),
51 51
 		);
52 52
 	}
53 53
 
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 		return array(
74 74
 			array( 'set_consumer_name', 'consumer_name', 'John Doe' ),
75 75
 			array( 'set_consumer_iban', 'consumer_iban', 'NL56 RABO 0108 6347 79' ),
76
-			array( 'set_consumer_bic',  'consumer_bic',  'RABONL2U' ),
76
+			array( 'set_consumer_bic', 'consumer_bic', 'RABONL2U' ),
77 77
 		);
78 78
 	}
79 79
 
@@ -97,13 +97,13 @@  discard block
 block discarded – undo
97 97
 	 */
98 98
 	public function get_provider() {
99 99
 		return array(
100
-			array( 'key',                 'get_key',             uniqid() ),
101
-			array( 'source',              'get_source',          'woocommerce' ),
102
-			array( 'source_id',           'get_source_id',       '1234' ),
103
-			array( 'frequency',           'get_frequency',       'daily' ),
104
-			array( 'interval',            'get_interval',        '1' ),
105
-			array( 'interval_period',     'get_interval_period', 'Y' ),
106
-			array( 'description',         'get_description',     'Lorem ipsum dolor sit amet, consectetur.' ),
100
+			array( 'key', 'get_key', uniqid() ),
101
+			array( 'source', 'get_source', 'woocommerce' ),
102
+			array( 'source_id', 'get_source_id', '1234' ),
103
+			array( 'frequency', 'get_frequency', 'daily' ),
104
+			array( 'interval', 'get_interval', '1' ),
105
+			array( 'interval_period', 'get_interval_period', 'Y' ),
106
+			array( 'description', 'get_description', 'Lorem ipsum dolor sit amet, consectetur.' ),
107 107
 		);
108 108
 	}
109 109
 
Please login to merge, or discard this patch.