Passed
Push — master ( b92003...49c8f0 )
by Paul
03:33
created
plugin/Controllers/AdminController.php 2 patches
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -108,12 +108,16 @@
 block discarded – undo
108 108
 	public function renderTinymceButton( $editorId )
109 109
 	{
110 110
 		$allowedEditors = apply_filters( 'site-reviews/tinymce/editor-ids', ['content'], $editorId );
111
-		if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors ))return;
111
+		if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors )) {
112
+			return;
113
+		}
112 114
 		$shortcodes = [];
113 115
 		foreach( glsr()->mceShortcodes as $shortcode => $values ) {
114 116
 			$shortcodes[$shortcode] = $values;
115 117
 		}
116
-		if( empty( $shortcodes ))return;
118
+		if( empty( $shortcodes )) {
119
+			return;
120
+		}
117 121
 		glsr()->render( 'partials/editor/tinymce', [
118 122
 			'shortcodes' => $shortcodes,
119 123
 		]);
Please login to merge, or discard this patch.
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 	 */
25 25
 	public function enqueueAssets()
26 26
 	{
27
-		$command = new EnqueueAdminAssets([
27
+		$command = new EnqueueAdminAssets( [
28 28
 			'pointers' => [[
29 29
 				'content' => __( 'You can pin exceptional reviews so that they are always shown first.', 'site-reviews' ),
30 30
 				'id' => 'glsr-pointer-pinned',
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 				'target' => '#misc-pub-pinned',
37 37
 				'title' => __( 'Pin Your Reviews', 'site-reviews' ),
38 38
 			]],
39
-		]);
39
+		] );
40 40
 		$this->execute( $command );
41 41
 	}
42 42
 
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 	{
49 49
 		$links['settings'] = glsr( Builder::class )->a( __( 'Settings', 'site-reviews' ), [
50 50
 			'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=settings' ),
51
-		]);
51
+		] );
52 52
 		return $links;
53 53
 	}
54 54
 
@@ -59,19 +59,19 @@  discard block
 block discarded – undo
59 59
 	public function filterDashboardGlanceItems( array $items )
60 60
 	{
61 61
 		$postCount = wp_count_posts( Application::POST_TYPE );
62
-		if( empty( $postCount->publish )) {
62
+		if( empty($postCount->publish) ) {
63 63
 			return $items;
64 64
 		}
65 65
 		$text = _n( '%s Review', '%s Reviews', $postCount->publish, 'site-reviews' );
66
-		$text = sprintf( $text, number_format_i18n( $postCount->publish ));
66
+		$text = sprintf( $text, number_format_i18n( $postCount->publish ) );
67 67
 		$items[] = current_user_can( get_post_type_object( Application::POST_TYPE )->cap->edit_posts )
68 68
 			? glsr( Builder::class )->a( $text, [
69 69
 				'class' => 'glsr-review-count',
70 70
 				'href' => 'edit.php?post_type='.Application::POST_TYPE,
71
-			])
71
+			] )
72 72
 			: glsr( Builder::class )->span( $text, [
73 73
 				'class' => 'glsr-review-count',
74
-			]);
74
+			] );
75 75
 		return $items;
76 76
 	}
77 77
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 	 */
82 82
 	public function filterTinymcePlugins( array $plugins )
83 83
 	{
84
-		if( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' )) {
84
+		if( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ) ) {
85 85
 			$plugins['glsr_shortcode'] = glsr()->url( 'assets/scripts/mce-plugin.js' );
86 86
 		}
87 87
 		return $plugins;
@@ -93,11 +93,11 @@  discard block
 block discarded – undo
93 93
 	 */
94 94
 	public function registerTinymcePopups()
95 95
 	{
96
-		$command = new RegisterTinymcePopups([
96
+		$command = new RegisterTinymcePopups( [
97 97
 			'site_reviews' => esc_html__( 'Recent Reviews', 'site-reviews' ),
98 98
 			'site_reviews_form' => esc_html__( 'Submit a Review', 'site-reviews' ),
99 99
 			'site_reviews_summary' => esc_html__( 'Summary of Reviews', 'site-reviews' ),
100
-		]);
100
+		] );
101 101
 		$this->execute( $command );
102 102
 	}
103 103
 
@@ -109,15 +109,15 @@  discard block
 block discarded – undo
109 109
 	public function renderTinymceButton( $editorId )
110 110
 	{
111 111
 		$allowedEditors = apply_filters( 'site-reviews/tinymce/editor-ids', ['content'], $editorId );
112
-		if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors ))return;
112
+		if( glsr_current_screen()->base != 'post' || !in_array( $editorId, $allowedEditors ) )return;
113 113
 		$shortcodes = [];
114 114
 		foreach( glsr()->mceShortcodes as $shortcode => $values ) {
115 115
 			$shortcodes[$shortcode] = $values;
116 116
 		}
117
-		if( empty( $shortcodes ))return;
117
+		if( empty($shortcodes) )return;
118 118
 		glsr()->render( 'partials/editor/tinymce', [
119 119
 			'shortcodes' => $shortcodes,
120
-		]);
120
+		] );
121 121
 	}
122 122
 
123 123
 	/**
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 	public function routerClearConsole()
127 127
 	{
128 128
 		glsr( Console::class )->clear();
129
-		glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' ));
129
+		glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' ) );
130 130
 	}
131 131
 
132 132
 	/**
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 	 */
135 135
 	public function routerFetchConsole()
136 136
 	{
137
-		glsr( Notice::class )->addSuccess( __( 'Console reloaded.', 'site-reviews' ));
137
+		glsr( Notice::class )->addSuccess( __( 'Console reloaded.', 'site-reviews' ) );
138 138
 	}
139 139
 
140 140
 	/**
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
 		}
154 154
 		$postIds = glsr( SqlQueries::class )->getReviewsMeta( 'assigned_to' );
155 155
 		foreach( $postIds as $postId ) {
156
-			$countManager->setPostCounts( $postId, $countManager->buildPostCounts( $postId ));
156
+			$countManager->setPostCounts( $postId, $countManager->buildPostCounts( $postId ) );
157 157
 		}
158 158
 		$countManager->setCounts( $countManager->buildCounts() );
159 159
 		if( $showNotice ) {
160
-			glsr( Notice::class )->clear()->addSuccess( __( 'Recalculated rating counts.', 'site-reviews' ));
160
+			glsr( Notice::class )->clear()->addSuccess( __( 'Recalculated rating counts.', 'site-reviews' ) );
161 161
 		}
162
-		glsr( OptionManager::class )->set( 'last_review_count', current_time( 'timestamp' ));
162
+		glsr( OptionManager::class )->set( 'last_review_count', current_time( 'timestamp' ) );
163 163
 	}
164 164
 
165 165
 	/**
@@ -193,17 +193,17 @@  discard block
 block discarded – undo
193 193
 	{
194 194
 		$file = $_FILES['import-file'];
195 195
 		if( $file['error'] !== UPLOAD_ERR_OK ) {
196
-			return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] ));
196
+			return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] ) );
197 197
 		}
198
-		if( $file['type'] !== 'application/json' || !glsr( Helper::class )->endsWith( '.json', $file['name'] )) {
199
-			return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' ));
198
+		if( $file['type'] !== 'application/json' || !glsr( Helper::class )->endsWith( '.json', $file['name'] ) ) {
199
+			return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' ) );
200 200
 		}
201 201
 		$settings = json_decode( file_get_contents( $file['tmp_name'] ), true );
202
-		if( empty( $settings )) {
203
-			return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' ));
202
+		if( empty($settings) ) {
203
+			return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' ) );
204 204
 		}
205
-		glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings ));
206
-		glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' ));
205
+		glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings ) );
206
+		glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' ) );
207 207
 	}
208 208
 
209 209
 	/**
@@ -221,7 +221,7 @@  discard block
 block discarded – undo
221 221
 			UPLOAD_ERR_CANT_WRITE => __( 'Failed to write file to disk.', 'site-reviews' ),
222 222
 			UPLOAD_ERR_EXTENSION => __( 'A PHP extension stopped the file upload.', 'site-reviews' ),
223 223
 		];
224
-		return !isset( $errors[$errorCode] )
224
+		return !isset($errors[$errorCode])
225 225
 			? __( 'Unknown upload error.', 'site-reviews' )
226 226
 			: $errors[$errorCode];
227 227
 	}
Please login to merge, or discard this patch.
plugin/Handlers/ChangeStatus.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -13,11 +13,11 @@  discard block
 block discarded – undo
13 13
 	 */
14 14
 	public function handle( Command $command )
15 15
 	{
16
-		$postId = wp_update_post([
16
+		$postId = wp_update_post( [
17 17
 			'ID' => $command->id,
18 18
 			'post_status' => $command->status,
19
-		]);
20
-		if( is_wp_error( $postId )) {
19
+		] );
20
+		if( is_wp_error( $postId ) ) {
21 21
 			glsr_log()->error( $postId->get_error_message() );
22 22
 			return [];
23 23
 		}
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 			'aria-label' => '“'.esc_attr( $title ).'” ('.__( 'Edit', 'site-reviews' ).')',
40 40
 			'class' => 'row-title',
41 41
 			'href' => get_edit_post_link( $postId ),
42
-		]);
42
+		] );
43 43
 	}
44 44
 
45 45
 	/**
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 	protected function getPostState( $postId )
50 50
 	{
51 51
 		ob_start();
52
-		_post_states( get_post( $postId ));
52
+		_post_states( get_post( $postId ) );
53 53
 		return ob_get_clean();
54 54
 	}
55 55
 
@@ -59,13 +59,13 @@  discard block
 block discarded – undo
59 59
 	protected function getStatusLinks()
60 60
 	{
61 61
 		global $avail_post_stati;
62
-		require_once( ABSPATH.'wp-admin/includes/class-wp-posts-list-table.php' );
62
+		require_once(ABSPATH.'wp-admin/includes/class-wp-posts-list-table.php');
63 63
 		$hookName = 'edit-'.Application::POST_TYPE;
64 64
 		set_current_screen( $hookName );
65 65
 		$avail_post_stati = get_available_post_statuses( Application::POST_TYPE );
66 66
 		$table = new \WP_Posts_List_Table( ['screen' => $hookName] );
67 67
 		$views = apply_filters( 'views_'.$hookName, $table->get_views() ); // uses compat get_views()
68
-		if( empty( $views ))return;
68
+		if( empty($views) )return;
69 69
 		foreach( $views as $class => $view ) {
70 70
 			$views[$class] = "\t<li class='$class'>$view";
71 71
 		}
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,9 @@
 block discarded – undo
65 65
 		$avail_post_stati = get_available_post_statuses( Application::POST_TYPE );
66 66
 		$table = new \WP_Posts_List_Table( ['screen' => $hookName] );
67 67
 		$views = apply_filters( 'views_'.$hookName, $table->get_views() ); // uses compat get_views()
68
-		if( empty( $views ))return;
68
+		if( empty( $views )) {
69
+			return;
70
+		}
69 71
 		foreach( $views as $class => $view ) {
70 72
 			$views[$class] = "\t<li class='$class'>$view";
71 73
 		}
Please login to merge, or discard this patch.
compatibility.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -20,9 +20,9 @@  discard block
 block discarded – undo
20 20
  * @see https://wordpress.org/plugins/wp-super-cache/
21 21
  */
22 22
 add_action( 'site-reviews/review/created', function( $review, $request ) {
23
-	if( !function_exists( 'wp_cache_post_change' ))return;
23
+	if( !function_exists( 'wp_cache_post_change' ) )return;
24 24
 	wp_cache_post_change( $request->post_id );
25
-	if( empty( $review->assigned_to ) || $review->assigned_to == $request->post_id )return;
25
+	if( empty($review->assigned_to) || $review->assigned_to == $request->post_id )return;
26 26
 	wp_cache_post_change( $review->assigned_to );
27 27
 }, 10, 2 );
28 28
 
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
  */
34 34
 add_filter( 'sbp_exclude_defer_scripts', function( $scriptHandles ) {
35 35
 	$scriptHandles[] = 'site-reviews/google-recaptcha';
36
-	return array_keys( array_flip( $scriptHandles ));
36
+	return array_keys( array_flip( $scriptHandles ) );
37 37
 });
38 38
 
39 39
 /**
@@ -43,8 +43,8 @@  discard block
 block discarded – undo
43 43
  * @see https://searchandfilter.com/
44 44
  */
45 45
 add_filter( 'sf_edit_query_args', function( $query ) {
46
-	if( !empty( $query['meta_key'] ) && $query['meta_key'] == '_glsr_ranking' ) {
47
-		unset( $query['meta_key'] );
46
+	if( !empty($query['meta_key']) && $query['meta_key'] == '_glsr_ranking' ) {
47
+		unset($query['meta_key']);
48 48
 		$query['meta_query'] = [
49 49
 			'relation' => 'OR',
50 50
 			['key' => '_glsr_ranking', 'compare' => 'EXISTS'],
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -8,7 +8,9 @@  discard block
 block discarded – undo
8 8
  * @see https://www.elegantthemes.com/gallery/divi/
9 9
  */
10 10
 add_action( 'site-reviews/customize/divi', function( $instance ) {
11
-	if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' )return;
11
+	if( $instance->tag != 'label' || $instance->args['type'] != 'checkbox' ) {
12
+		return;
13
+	}
12 14
 	$instance->args['text'] = '<i></i>'.$instance->args['text'];
13 15
 });
14 16
 
@@ -20,9 +22,13 @@  discard block
 block discarded – undo
20 22
  * @see https://wordpress.org/plugins/wp-super-cache/
21 23
  */
22 24
 add_action( 'site-reviews/review/created', function( $review, $request ) {
23
-	if( !function_exists( 'wp_cache_post_change' ))return;
25
+	if( !function_exists( 'wp_cache_post_change' )) {
26
+		return;
27
+	}
24 28
 	wp_cache_post_change( $request->post_id );
25
-	if( empty( $review->assigned_to ) || $review->assigned_to == $request->post_id )return;
29
+	if( empty( $review->assigned_to ) || $review->assigned_to == $request->post_id ) {
30
+		return;
31
+	}
26 32
 	wp_cache_post_change( $review->assigned_to );
27 33
 }, 10, 2 );
28 34
 
Please login to merge, or discard this patch.
plugin/Shortcodes/TinymcePopupGenerator.php 2 patches
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -56,9 +56,13 @@  discard block
 block discarded – undo
56 56
 	protected function generateFields( array $fields )
57 57
 	{
58 58
 		$generatedFields = array_map( function( $field ) {
59
-			if( empty( $field ))return;
59
+			if( empty( $field )) {
60
+				return;
61
+			}
60 62
 			$field = $this->normalize( $field );
61
-			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] )))return;
63
+			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] ))) {
64
+				return;
65
+			}
62 66
 			return $this->$method( $field );
63 67
 		}, $fields );
64 68
 		return array_values( array_filter( $generatedFields ));
@@ -73,7 +77,9 @@  discard block
 block discarded – undo
73 77
 		if( !empty( $this->errors )) {
74 78
 			$errors = [];
75 79
 			foreach( $this->required as $name => $alert ) {
76
-				if( false !== array_search( $name, glsr_array_column( $fields, 'name' )))continue;
80
+				if( false !== array_search( $name, glsr_array_column( $fields, 'name' ))) {
81
+					continue;
82
+				}
77 83
 				$errors[] = $this->errors[$name];
78 84
 			}
79 85
 			$this->errors = $errors;
@@ -117,7 +123,9 @@  discard block
 block discarded – undo
117 123
 	 */
118 124
 	protected function normalizeContainer( array $field )
119 125
 	{
120
-		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ))return;
126
+		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field )) {
127
+			return;
128
+		}
121 129
 		$field['items'] = $this->generateFields( $field['items'] );
122 130
 		return $field;
123 131
 	}
@@ -127,7 +135,9 @@  discard block
 block discarded – undo
127 135
 	 */
128 136
 	protected function normalizeField( array $field, array $defaults )
129 137
 	{
130
-		if( !$this->validate( $field ))return;
138
+		if( !$this->validate( $field )) {
139
+			return;
140
+		}
131 141
 		return array_filter( shortcode_atts( $defaults, $field ), function( $value ) {
132 142
 			return $value !== '';
133 143
 		});
@@ -148,7 +158,9 @@  discard block
 block discarded – undo
148 158
 			'type' => '',
149 159
 			'value' => '',
150 160
 		]);
151
-		if( !is_array( $listbox ))return;
161
+		if( !is_array( $listbox )) {
162
+			return;
163
+		}
152 164
 		if( !array_key_exists( '', $listbox['options'] )) {
153 165
 			$listbox['options'] = ['' => $listbox['placeholder']] + $listbox['options'];
154 166
 		}
Please login to merge, or discard this patch.
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 			'label' => '['.$tag.']',
49 49
 			'required' => $this->required,
50 50
 			'title' => esc_html__( 'Shortcode', 'site-reviews' ),
51
-		]);
51
+		] );
52 52
 		return $this;
53 53
 	}
54 54
 
@@ -58,12 +58,12 @@  discard block
 block discarded – undo
58 58
 	protected function generateFields( array $fields )
59 59
 	{
60 60
 		$generatedFields = array_map( function( $field ) {
61
-			if( empty( $field ))return;
61
+			if( empty($field) )return;
62 62
 			$field = $this->normalize( $field );
63
-			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] )))return;
63
+			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] ) ) )return;
64 64
 			return $this->$method( $field );
65 65
 		}, $fields );
66
-		return array_values( array_filter( $generatedFields ));
66
+		return array_values( array_filter( $generatedFields ) );
67 67
 	}
68 68
 
69 69
 	/**
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 	protected function getCategories( $tooltip = '' )
74 74
 	{
75 75
 		$terms = glsr( Database::class )->getTerms();
76
-		if( empty( $terms )) {
76
+		if( empty($terms) ) {
77 77
 			return [];
78 78
 		}
79 79
 		return [
@@ -91,15 +91,15 @@  discard block
 block discarded – undo
91 91
 	protected function getFields()
92 92
 	{
93 93
 		$fields = $this->generateFields( $this->fields() );
94
-		if( !empty( $this->errors )) {
94
+		if( !empty($this->errors) ) {
95 95
 			$errors = [];
96 96
 			foreach( $this->required as $name => $alert ) {
97
-				if( false !== array_search( $name, glsr_array_column( $fields, 'name' )))continue;
97
+				if( false !== array_search( $name, glsr_array_column( $fields, 'name' ) ) )continue;
98 98
 				$errors[] = $this->errors[$name];
99 99
 			}
100 100
 			$this->errors = $errors;
101 101
 		}
102
-		return empty( $this->errors )
102
+		return empty($this->errors)
103 103
 			? $fields
104 104
 			: $this->errors;
105 105
 	}
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 	 */
110 110
 	protected function getHideOptions()
111 111
 	{
112
-		$classname = str_replace( 'Popup', 'Shortcode', get_class( $this ));
112
+		$classname = str_replace( 'Popup', 'Shortcode', get_class( $this ) );
113 113
 		$hideOptions = glsr( $classname )->getHideOptions();
114 114
 		$options = [];
115 115
 		foreach( $hideOptions as $name => $tooltip ) {
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 		return wp_parse_args( $field, [
150 150
 			'items' => [],
151 151
 			'type' => '',
152
-		]);
152
+		] );
153 153
 	}
154 154
 
155 155
 	/**
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 			'tooltip' => '',
168 168
 			'type' => '',
169 169
 			'value' => '',
170
-		]);
170
+		] );
171 171
 	}
172 172
 
173 173
 	/**
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	 */
176 176
 	protected function normalizeContainer( array $field )
177 177
 	{
178
-		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ))return;
178
+		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ) )return;
179 179
 		$field['items'] = $this->generateFields( $field['items'] );
180 180
 		return $field;
181 181
 	}
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
 	 */
186 186
 	protected function normalizeField( array $field, array $defaults )
187 187
 	{
188
-		if( !$this->validate( $field ))return;
188
+		if( !$this->validate( $field ) )return;
189 189
 		return array_filter( shortcode_atts( $defaults, $field ), function( $value ) {
190 190
 			return $value !== '';
191 191
 		});
@@ -205,9 +205,9 @@  discard block
 block discarded – undo
205 205
 			'tooltip' => '',
206 206
 			'type' => '',
207 207
 			'value' => '',
208
-		]);
209
-		if( !is_array( $listbox ))return;
210
-		if( !array_key_exists( '', $listbox['options'] )) {
208
+		] );
209
+		if( !is_array( $listbox ) )return;
210
+		if( !array_key_exists( '', $listbox['options'] ) ) {
211 211
 			$listbox['options'] = ['' => $listbox['placeholder']] + $listbox['options'];
212 212
 		}
213 213
 		foreach( $listbox['options'] as $value => $text ) {
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
 	 */
225 225
 	protected function normalizePost( array $field )
226 226
 	{
227
-		if( !is_array( $field['query_args'] )) {
227
+		if( !is_array( $field['query_args'] ) ) {
228 228
 			$field['query_args'] = [];
229 229
 		}
230 230
 		$posts = get_posts( wp_parse_args( $field['query_args'], [
@@ -232,8 +232,8 @@  discard block
 block discarded – undo
232 232
 			'orderby' => 'title',
233 233
 			'post_type' => 'post',
234 234
 			'posts_per_page' => 30,
235
-		]));
236
-		if( !empty( $posts )) {
235
+		] ) );
236
+		if( !empty($posts) ) {
237 237
 			$options = [];
238 238
 			foreach( $posts as $post ) {
239 239
 				$options[$post->ID] = esc_html( $post->post_title );
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
 			'tooltip' => '',
264 264
 			'type' => '',
265 265
 			'value' => '',
266
-		]);
266
+		] );
267 267
 	}
268 268
 
269 269
 	/**
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
 	 */
272 272
 	protected function validate( array $field )
273 273
 	{
274
-		$args = shortcode_atts([
274
+		$args = shortcode_atts( [
275 275
 			'label' => '',
276 276
 			'name' => false,
277 277
 			'required' => false,
@@ -287,13 +287,13 @@  discard block
 block discarded – undo
287 287
 	 */
288 288
 	protected function validateErrors( array $args )
289 289
 	{
290
-		if( !isset( $args['required']['error'] )) {
290
+		if( !isset($args['required']['error']) ) {
291 291
 			return true;
292 292
 		}
293
-		$this->errors[$args['name']] = $this->normalizeContainer([
293
+		$this->errors[$args['name']] = $this->normalizeContainer( [
294 294
 			'html' => $args['required']['error'],
295 295
 			'type' => 'container',
296
-		]);
296
+		] );
297 297
 		return false;
298 298
 	}
299 299
 
@@ -306,10 +306,10 @@  discard block
 block discarded – undo
306 306
 			return true;
307 307
 		}
308 308
 		$alert = esc_html__( 'Some of the shortcode options are required.', 'site-reviews' );
309
-		if( isset( $args['required']['alert'] )) {
309
+		if( isset($args['required']['alert']) ) {
310 310
 			$alert = $args['required']['alert'];
311 311
 		}
312
-		else if( !empty( $args['label'] )) {
312
+		else if( !empty($args['label']) ) {
313 313
 			$alert = sprintf(
314 314
 				esc_html_x( 'The "%s" option is required.', 'the option label', 'site-reviews' ),
315 315
 				str_replace( ':', '', $args['label'] )
Please login to merge, or discard this patch.
plugin/Modules/Translation.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	{
31 31
 		$translations = $this->translations();
32 32
 		$entries = $this->filter( $translations, $this->entries() )->results();
33
-		array_walk( $translations, function( &$entry ) use( $entries ) {
33
+		array_walk( $translations, function( &$entry ) use($entries) {
34 34
 			$entry['desc'] = array_key_exists( $entry['id'], $entries )
35 35
 				? $this->getEntryString( $entries[$entry['id']], 'msgctxt' )
36 36
 				: '';
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 	 */
44 44
 	public function entries()
45 45
 	{
46
-		if( !isset( $this->entries )) {
46
+		if( !isset($this->entries) ) {
47 47
 			$potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' );
48 48
 			$entries = $this->extractEntriesFromPotFile( $potFile );
49 49
 			$entries = apply_filters( 'site-reviews/translation/entries', $entries );
@@ -88,13 +88,13 @@  discard block
 block discarded – undo
88 88
 	 */
89 89
 	public function filter( $filterWith = null, $entries = null, $intersect = true )
90 90
 	{
91
-		if( !is_array( $entries )) {
91
+		if( !is_array( $entries ) ) {
92 92
 			$entries = $this->results;
93 93
 		}
94
-		if( !is_array( $filterWith )) {
94
+		if( !is_array( $filterWith ) ) {
95 95
 			$filterWith = $this->translations();
96 96
 		}
97
-		$keys = array_flip( glsr_array_column( $filterWith, 'id' ));
97
+		$keys = array_flip( glsr_array_column( $filterWith, 'id' ) );
98 98
 		$this->results = $intersect
99 99
 			? array_intersect_key( $entries, $keys )
100 100
 			: array_diff_key( $entries, $keys );
@@ -108,17 +108,17 @@  discard block
 block discarded – undo
108 108
 	public function render( $template, array $entry )
109 109
 	{
110 110
 		$data = array_combine(
111
-			array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry )),
111
+			array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry ) ),
112 112
 			$entry
113 113
 		);
114 114
 		$data['data.class'] = $data['data.error'] = '';
115
-		if( array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' )) === false ) {
115
+		if( array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' ) ) === false ) {
116 116
 			$data['data.class'] = 'is-invalid';
117 117
 			$data['data.error'] = __( 'This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews' );
118 118
 		}
119 119
 		return glsr( Template::class )->build( 'partials/translations/'.$template, [
120 120
 			'context' => $data,
121
-		]);
121
+		] );
122 122
 	}
123 123
 
124 124
 	/**
@@ -150,13 +150,13 @@  discard block
 block discarded – undo
150 150
 				'p1' => $this->getEntryString( $entry, 'msgid_plural' ),
151 151
 				's1' => $this->getEntryString( $entry, 'msgid' ),
152 152
 			];
153
-			$text = !empty( $data['p1'] )
153
+			$text = !empty($data['p1'])
154 154
 				? sprintf( '%s | %s', $data['s1'], $data['p1'] )
155 155
 				: $data['s1'];
156 156
 			$rendered .= $this->render( 'result', [
157
-				'entry' => json_encode( $data, JSON_HEX_APOS|JSON_HEX_QUOT|JSON_HEX_TAG|JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE ),
157
+				'entry' => json_encode( $data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ),
158 158
 				'text' => wp_strip_all_tags( $text ),
159
-			]);
159
+			] );
160 160
 		}
161 161
 		if( $resetAfterRender ) {
162 162
 			$this->reset();
@@ -189,12 +189,12 @@  discard block
 block discarded – undo
189 189
 	public function search( $needle = '' )
190 190
 	{
191 191
 		$this->reset();
192
-		$needle = trim( strtolower( $needle ));
192
+		$needle = trim( strtolower( $needle ) );
193 193
 		foreach( $this->entries() as $key => $entry ) {
194
-			$single = strtolower( $this->getEntryString( $entry, 'msgid' ));
195
-			$plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ));
194
+			$single = strtolower( $this->getEntryString( $entry, 'msgid' ) );
195
+			$plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ) );
196 196
 			if( strlen( $needle ) < static::SEARCH_THRESHOLD ) {
197
-				if( in_array( $needle, [$single, $plural] )) {
197
+				if( in_array( $needle, [$single, $plural] ) ) {
198 198
 					$this->results[$key] = $entry;
199 199
 				}
200 200
 			}
@@ -212,10 +212,10 @@  discard block
 block discarded – undo
212 212
 	public function translations()
213 213
 	{
214 214
 		static $translations;
215
-		if( empty( $translations )) {
215
+		if( empty($translations) ) {
216 216
 			$settings = glsr( OptionManager::class )->get( 'settings' );
217
-			$translations = isset( $settings['strings'] )
218
-				? $this->normalizeSettings( (array) $settings['strings'] )
217
+			$translations = isset($settings['strings'])
218
+				? $this->normalizeSettings( (array)$settings['strings'] )
219 219
 				: [];
220 220
 		}
221 221
 		return $translations;
@@ -227,8 +227,8 @@  discard block
 block discarded – undo
227 227
 	 */
228 228
 	protected function getEntryString( array $entry, $key )
229 229
 	{
230
-		return isset( $entry[$key] )
231
-			? implode( '', (array) $entry[$key] )
230
+		return isset($entry[$key])
231
+			? implode( '', (array)$entry[$key] )
232 232
 			: '';
233 233
 	}
234 234
 
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
 		$keys = [
241 241
 			'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]',
242 242
 		];
243
-		array_walk( $entries, function( &$entry ) use( $keys ) {
243
+		array_walk( $entries, function( &$entry ) use($keys) {
244 244
 			foreach( $keys as $key ) {
245 245
 				$entry = $this->normalizeEntryString( $entry, $key );
246 246
 			}
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
 	 */
255 255
 	protected function normalizeEntryString( array $entry, $key )
256 256
 	{
257
-		if( isset( $entry[$key] )) {
257
+		if( isset($entry[$key]) ) {
258 258
 			$entry[$key] = $this->getEntryString( $entry, $key );
259 259
 		}
260 260
 		return $entry;
@@ -268,11 +268,11 @@  discard block
 block discarded – undo
268 268
 		$defaultString = array_fill_keys( ['id', 's1', 's2', 'p1', 'p2'], '' );
269 269
 		$strings = array_filter( $strings, 'is_array' );
270 270
 		foreach( $strings as &$string ) {
271
-			$string['type'] = isset( $string['p1'] ) ? 'plural' : 'single';
271
+			$string['type'] = isset($string['p1']) ? 'plural' : 'single';
272 272
 			$string = wp_parse_args( $string, $defaultString );
273 273
 		}
274 274
 		return array_filter( $strings, function( $string ) {
275
-			return !empty( $string['id'] );
275
+			return !empty($string['id']);
276 276
 		});
277 277
 	}
278 278
 }
Please login to merge, or discard this patch.
helpers.php 2 patches
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -19,8 +19,8 @@  discard block
 block discarded – undo
19 19
 		'glsr_get_review', 'glsr_get_reviews',
20 20
 		'glsr_log',
21 21
 	);
22
-	if( !in_array( $hook, $hooks ) || !function_exists( $hook ))return;
23
-	add_filter( $hook, function() use( $hook, $args ) {
22
+	if( !in_array( $hook, $hooks ) || !function_exists( $hook ) )return;
23
+	add_filter( $hook, function() use($hook, $args) {
24 24
 		array_shift( $args ); // remove the fallback value
25 25
 		return call_user_func_array( $hook, $args );
26 26
 	});
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
  */
32 32
 function glsr( $alias = null ) {
33 33
 	$app = \GeminiLabs\SiteReviews\Application::load();
34
-	return !empty( $alias )
34
+	return !empty($alias)
35 35
 		? $app->make( $alias )
36 36
 		: $app;
37 37
 }
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 	$result = array();
46 46
 	foreach( $array as $subarray ) {
47 47
 		$subarray = (array)$subarray;
48
-		if( !isset( $subarray[$column] ))continue;
48
+		if( !isset($subarray[$column]) )continue;
49 49
 		$result[] = $subarray[$column];
50 50
 	}
51 51
 	return $result;
@@ -56,19 +56,19 @@  discard block
 block discarded – undo
56 56
  */
57 57
 function glsr_calculate_ratings() {
58 58
 	glsr( 'Controllers\AdminController' )->routerCountReviews( false );
59
-	glsr_log()->info( __( 'Recalculated rating counts.', 'site-reviews' ));
59
+	glsr_log()->info( __( 'Recalculated rating counts.', 'site-reviews' ) );
60 60
 }
61 61
 
62 62
 /**
63 63
  * @return null|\GeminiLabs\SiteReviews\Review
64 64
  */
65 65
 function glsr_create_review( $reviewValues = array() ) {
66
-	if( !is_array( $reviewValues )) {
66
+	if( !is_array( $reviewValues ) ) {
67 67
 		$reviewValues = array();
68 68
 	}
69 69
 	$review = new \GeminiLabs\SiteReviews\Commands\CreateReview( $reviewValues );
70 70
 	$result = glsr( 'Database\ReviewManager' )->create( $review );
71
-	return !empty( $result )
71
+	return !empty($result)
72 72
 		? $result
73 73
 		: null;
74 74
 }
@@ -77,10 +77,10 @@  discard block
 block discarded – undo
77 77
  * @return \WP_Screen|object
78 78
  */
79 79
 function glsr_current_screen() {
80
-	if( function_exists( 'get_current_screen' )) {
80
+	if( function_exists( 'get_current_screen' ) ) {
81 81
 		$screen = get_current_screen();
82 82
 	}
83
-	return empty( $screen )
83
+	return empty($screen)
84 84
 		? (object)array_fill_keys( ['base', 'id', 'post_type'], null )
85 85
 		: $screen;
86 86
 }
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
  * @return \GeminiLabs\SiteReviews\Review|void
127 127
  */
128 128
 function glsr_get_review( $post_id ) {
129
-	if( !is_numeric( $post_id ))return;
129
+	if( !is_numeric( $post_id ) )return;
130 130
 	$post = get_post( $post_id );
131 131
 	if( $post instanceof WP_Post ) {
132 132
 		return glsr( 'Database\ReviewManager' )->single( $post );
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
  * @todo document change of $reviews->reviews to $reviews->results
139 139
  */
140 140
 function glsr_get_reviews( $args = array() ) {
141
-	if( !is_array( $args )) {
141
+	if( !is_array( $args ) ) {
142 142
 		$args = [];
143 143
 	}
144 144
 	return glsr( 'Database\ReviewManager' )->get( $args );
@@ -149,11 +149,11 @@  discard block
 block discarded – undo
149 149
  */
150 150
 function glsr_log() {
151 151
 	$args = func_get_args();
152
-	$context = isset( $args[1] )
152
+	$context = isset($args[1])
153 153
 		? $args[1]
154 154
 		: [];
155 155
 	$console = glsr( 'Modules\Console' );
156
-	return !empty( $args )
156
+	return !empty($args)
157 157
 		? $console->log( 'debug', $args[0], $context )
158 158
 		: $console;
159 159
 }
Please login to merge, or discard this patch.
Braces   +31 added lines, -14 removed lines patch added patch discarded remove patch
@@ -19,7 +19,9 @@  discard block
 block discarded – undo
19 19
 		'glsr_get_review', 'glsr_get_reviews',
20 20
 		'glsr_log',
21 21
 	);
22
-	if( !in_array( $hook, $hooks ) || !function_exists( $hook ))return;
22
+	if( !in_array( $hook, $hooks ) || !function_exists( $hook )) {
23
+		return;
24
+	}
23 25
 	add_filter( $hook, function() use( $hook, $args ) {
24 26
 		array_shift( $args ); // remove the fallback value
25 27
 		return call_user_func_array( $hook, $args );
@@ -29,7 +31,8 @@  discard block
 block discarded – undo
29 31
 /**
30 32
  * @return mixed
31 33
  */
32
-function glsr( $alias = null ) {
34
+function glsr( $alias = null )
35
+{
33 36
 	$app = \GeminiLabs\SiteReviews\Application::load();
34 37
 	return !empty( $alias )
35 38
 		? $app->make( $alias )
@@ -41,11 +44,14 @@  discard block
 block discarded – undo
41 44
  * @param $column string
42 45
  * @return array
43 46
  */
44
-function glsr_array_column( array $array, $column ) {
47
+function glsr_array_column( array $array, $column )
48
+{
45 49
 	$result = array();
46 50
 	foreach( $array as $subarray ) {
47 51
 		$subarray = (array)$subarray;
48
-		if( !isset( $subarray[$column] ))continue;
52
+		if( !isset( $subarray[$column] )) {
53
+			continue;
54
+		}
49 55
 		$result[] = $subarray[$column];
50 56
 	}
51 57
 	return $result;
@@ -54,7 +60,8 @@  discard block
 block discarded – undo
54 60
 /**
55 61
  * @return void
56 62
  */
57
-function glsr_calculate_ratings() {
63
+function glsr_calculate_ratings()
64
+{
58 65
 	glsr( 'Controllers\AdminController' )->routerCountReviews( false );
59 66
 	glsr_log()->info( __( 'Recalculated rating counts.', 'site-reviews' ));
60 67
 }
@@ -62,7 +69,8 @@  discard block
 block discarded – undo
62 69
 /**
63 70
  * @return null|\GeminiLabs\SiteReviews\Review
64 71
  */
65
-function glsr_create_review( $reviewValues = array() ) {
72
+function glsr_create_review( $reviewValues = array() )
73
+{
66 74
 	if( !is_array( $reviewValues )) {
67 75
 		$reviewValues = array();
68 76
 	}
@@ -76,7 +84,8 @@  discard block
 block discarded – undo
76 84
 /**
77 85
  * @return \WP_Screen|object
78 86
  */
79
-function glsr_current_screen() {
87
+function glsr_current_screen()
88
+{
80 89
 	if( function_exists( 'get_current_screen' )) {
81 90
 		$screen = get_current_screen();
82 91
 	}
@@ -89,7 +98,8 @@  discard block
 block discarded – undo
89 98
  * @param mixed ...$vars
90 99
  * @return void
91 100
  */
92
-function glsr_debug( ...$vars ) {
101
+function glsr_debug( ...$vars )
102
+{
93 103
 	if( count( $vars ) == 1 ) {
94 104
 		$value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' );
95 105
 		printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value );
@@ -108,7 +118,8 @@  discard block
 block discarded – undo
108 118
  * @param mixed $fallback
109 119
  * @return string|array
110 120
  */
111
-function glsr_get_option( $path = '', $fallback = '' ) {
121
+function glsr_get_option( $path = '', $fallback = '' )
122
+{
112 123
 	return is_string( $path )
113 124
 		? glsr( 'Database\OptionManager' )->get( 'settings.'.$path, $fallback )
114 125
 		: $fallback;
@@ -117,7 +128,8 @@  discard block
 block discarded – undo
117 128
 /**
118 129
  * @return array
119 130
  */
120
-function glsr_get_options() {
131
+function glsr_get_options()
132
+{
121 133
 	return glsr( 'Database\OptionManager' )->get( 'settings' );
122 134
 }
123 135
 
@@ -125,8 +137,11 @@  discard block
 block discarded – undo
125 137
  * @param int $post_id
126 138
  * @return \GeminiLabs\SiteReviews\Review|void
127 139
  */
128
-function glsr_get_review( $post_id ) {
129
-	if( !is_numeric( $post_id ))return;
140
+function glsr_get_review( $post_id )
141
+{
142
+	if( !is_numeric( $post_id )) {
143
+		return;
144
+	}
130 145
 	$post = get_post( $post_id );
131 146
 	if( $post instanceof WP_Post ) {
132 147
 		return glsr( 'Database\ReviewManager' )->single( $post );
@@ -137,7 +152,8 @@  discard block
 block discarded – undo
137 152
  * @return array
138 153
  * @todo document change of $reviews->reviews to $reviews->results
139 154
  */
140
-function glsr_get_reviews( $args = array() ) {
155
+function glsr_get_reviews( $args = array() )
156
+{
141 157
 	if( !is_array( $args )) {
142 158
 		$args = [];
143 159
 	}
@@ -147,7 +163,8 @@  discard block
 block discarded – undo
147 163
 /**
148 164
  * @return \GeminiLabs\SiteReviews\Modules\Console
149 165
  */
150
-function glsr_log() {
166
+function glsr_log()
167
+{
151 168
 	$args = func_get_args();
152 169
 	$context = isset( $args[1] )
153 170
 		? $args[1]
Please login to merge, or discard this patch.
plugin/Database/SqlQueries.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -23,11 +23,11 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public function deleteAllSessions( $sessionCookiePrefix )
25 25
 	{
26
-		return $this->db->query("
26
+		return $this->db->query( "
27 27
 			DELETE
28 28
 			FROM {$this->db->options}
29 29
 			WHERE option_name LIKE '{$sessionCookiePrefix}_%'
30
-		");
30
+		" );
31 31
 	}
32 32
 
33 33
 	/**
@@ -36,11 +36,11 @@  discard block
 block discarded – undo
36 36
 	 */
37 37
 	public function deleteExpiredSessions( $expiredSessions )
38 38
 	{
39
-		return $this->db->query("
39
+		return $this->db->query( "
40 40
 			DELETE
41 41
 			FROM {$this->db->options}
42 42
 			WHERE option_name IN ('{$expiredSessions}')
43
-		");
43
+		" );
44 44
 	}
45 45
 
46 46
 	/**
@@ -49,14 +49,14 @@  discard block
 block discarded – undo
49 49
 	 */
50 50
 	public function getReviewCountsFor( $metaKey )
51 51
 	{
52
-		return (array) $this->db->get_results("
52
+		return (array)$this->db->get_results( "
53 53
 			SELECT m.meta_value AS name, COUNT(*) num_posts
54 54
 			FROM {$this->db->posts} AS p
55 55
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
56 56
 			WHERE p.post_type = '{$this->postType}'
57 57
 			AND m.meta_key = '{$metaKey}'
58 58
 			GROUP BY name
59
-		");
59
+		" );
60 60
 	}
61 61
 
62 62
 	/**
@@ -66,13 +66,13 @@  discard block
 block discarded – undo
66 66
 	 */
67 67
 	public function getExpiredSessions( $sessionCookiePrefix, $limit )
68 68
 	{
69
-		return $this->db->get_results("
69
+		return $this->db->get_results( "
70 70
 			SELECT option_name AS name, option_value AS expiration
71 71
 			FROM {$this->db->options}
72 72
 			WHERE option_name LIKE '{$sessionCookiePrefix}_expires_%'
73 73
 			ORDER BY option_value ASC
74 74
 			LIMIT 0, {$limit}
75
-		");
75
+		" );
76 76
 	}
77 77
 
78 78
 	/**
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	 */
83 83
 	public function getReviewCounts( $lastPostId = 0, $limit = 500 )
84 84
 	{
85
-		return $this->db->get_results("
85
+		return $this->db->get_results( "
86 86
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
87 87
 			FROM {$this->db->posts} AS p
88 88
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -103,14 +103,14 @@  discard block
 block discarded – undo
103 103
 	 */
104 104
 	public function getReviewPostId( $metaReviewId )
105 105
 	{
106
-		$postId = $this->db->get_var("
106
+		$postId = $this->db->get_var( "
107 107
 			SELECT p.ID
108 108
 			FROM {$this->db->posts} AS p
109 109
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
110 110
 			WHERE p.post_type = '{$this->postType}'
111 111
 			AND m.meta_key = 'review_id'
112 112
 			AND m.meta_value = '{$metaReviewId}'
113
-		");
113
+		" );
114 114
 		return intval( $postId );
115 115
 	}
116 116
 
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	public function getReviewIdsByType( $reviewType )
122 122
 	{
123
-		$query = $this->db->get_col("
123
+		$query = $this->db->get_col( "
124 124
 			SELECT m1.meta_value AS review_id
125 125
 			FROM {$this->db->posts} AS p
126 126
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -129,8 +129,8 @@  discard block
 block discarded – undo
129 129
 			AND m1.meta_key = 'review_id'
130 130
 			AND m2.meta_key = 'review_type'
131 131
 			AND m2.meta_value = '{$reviewType}'
132
-		");
133
-		return array_keys( array_flip( $query ));
132
+		" );
133
+		return array_keys( array_flip( $query ) );
134 134
 	}
135 135
 
136 136
 	/**
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 	 */
142 142
 	public function getReviewPostCounts( $postId, $lastPostId = 0, $limit = 500 )
143 143
 	{
144
-		return $this->db->get_results("
144
+		return $this->db->get_results( "
145 145
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
146 146
 			FROM {$this->db->posts} AS p
147 147
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -167,9 +167,9 @@  discard block
 block discarded – undo
167 167
 	public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 )
168 168
 	{
169 169
 		sort( $postIds );
170
-		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds )), $limit );
170
+		$postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit );
171 171
 		$postIds = implode( ',', $postIds );
172
-		return $this->db->get_results("
172
+		return $this->db->get_results( "
173 173
 			SELECT p.ID, m.meta_value AS rating
174 174
 			FROM {$this->db->posts} AS p
175 175
 			INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 		$queryBuilder = glsr( QueryBuilder::class );
194 194
 		$key = $queryBuilder->buildSqlOr( $key, "m.meta_key = '%s'" );
195 195
 		$status = $queryBuilder->buildSqlOr( $status, "p.post_status = '%s'" );
196
-		return $this->db->get_col("
196
+		return $this->db->get_col( "
197 197
 			SELECT DISTINCT m.meta_value
198 198
 			FROM {$this->db->postmeta} m
199 199
 			LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 			AND ({$key})
202 202
 			AND ({$status})
203 203
 			ORDER BY m.meta_value
204
-		");
204
+		" );
205 205
 	}
206 206
 
207 207
 	/**
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
 	 */
213 213
 	public function getReviewTermCounts( $termTaxonomyId, $lastPostId = 0, $limit = 500 )
214 214
 	{
215
-		return $this->db->get_results("
215
+		return $this->db->get_results( "
216 216
 			SELECT p.ID, m1.meta_value AS rating, m2.meta_value AS type
217 217
 			FROM {$this->db->posts} AS p
218 218
 			INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
Please login to merge, or discard this patch.
plugin/Database/ReviewManager.php 2 patches
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -35,12 +35,12 @@  discard block
 block discarded – undo
35 35
 			'post_type' => Application::POST_TYPE,
36 36
 		];
37 37
 		$postId = wp_insert_post( $postValues, true );
38
-		if( is_wp_error( $postId )) {
38
+		if( is_wp_error( $postId ) ) {
39 39
 			glsr_log()->error( $postId->get_error_message() )->debug( $postValues );
40 40
 			return false;
41 41
 		}
42 42
 		$this->setTerms( $postId, $command->category );
43
-		$review = $this->single( get_post( $postId ));
43
+		$review = $this->single( get_post( $postId ) );
44 44
 		do_action( 'site-reviews/review/created', $review, $command );
45 45
 		return $review;
46 46
 	}
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 	 */
52 52
 	public function delete( $metaReviewId )
53 53
 	{
54
-		if( $postId = $this->getPostId( $metaReviewId )) {
54
+		if( $postId = $this->getPostId( $metaReviewId ) ) {
55 55
 			wp_delete_post( $postId, true );
56 56
 		}
57 57
 	}
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 		$paged = glsr( QueryBuilder::class )->getPaged(
74 74
 			wp_validate_boolean( $args['pagination'] )
75 75
 		);
76
-		$reviews = new WP_Query([
76
+		$reviews = new WP_Query( [
77 77
 			'meta_key' => 'pinned',
78 78
 			'meta_query' => $metaQuery,
79 79
 			'offset' => $args['offset'],
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 			'post_type' => Application::POST_TYPE,
87 87
 			'posts_per_page' => $args['count'],
88 88
 			'tax_query' => $taxQuery,
89
-		]);
89
+		] );
90 90
 		return (object)[
91 91
 			'results' => array_map( [$this, 'single'], $reviews->posts ),
92 92
 			'max_num_pages' => $reviews->max_num_pages,
@@ -120,11 +120,11 @@  discard block
 block discarded – undo
120 120
 		$terms = [];
121 121
 		$termIds = glsr( Helper::class )->convertStringToArray( $commaSeparatedTermIds );
122 122
 		foreach( $termIds as $termId ) {
123
-			if( is_numeric( $termId )) {
123
+			if( is_numeric( $termId ) ) {
124 124
 				$termId = intval( $termId );
125 125
 			}
126 126
 			$term = term_exists( $termId, Application::TAXONOMY );
127
-			if( !isset( $term['term_id'] ))continue;
127
+			if( !isset($term['term_id']) )continue;
128 128
 			$terms[] = $term;
129 129
 		}
130 130
 		return $terms;
@@ -138,13 +138,13 @@  discard block
 block discarded – undo
138 138
 	{
139 139
 		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return;
140 140
 		delete_post_meta( $postId, '_edit_last' );
141
-		$result = wp_update_post([
141
+		$result = wp_update_post( [
142 142
 			'ID' => $postId,
143 143
 			'post_content' => get_post_meta( $postId, 'content', true ),
144 144
 			'post_date' => get_post_meta( $postId, 'date', true ),
145 145
 			'post_title' => get_post_meta( $postId, 'title', true ),
146
-		]);
147
-		if( is_wp_error( $result )) {
146
+		] );
147
+		if( is_wp_error( $result ) ) {
148 148
 			glsr_log()->error( $result->get_error_message() );
149 149
 		}
150 150
 	}
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 	protected function getNewPostStatus( array $review, $isBlacklisted )
167 167
 	{
168 168
 		$requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' );
169
-		return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted )
169
+		return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted)
170 170
 			? 'pending'
171 171
 			: 'publish';
172 172
 	}
@@ -179,9 +179,9 @@  discard block
 block discarded – undo
179 179
 	protected function setTerms( $postId, $termIds )
180 180
 	{
181 181
 		$terms = $this->normalizeTermIds( $termIds );
182
-		if( empty( $terms ))return;
182
+		if( empty($terms) )return;
183 183
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
184
-		if( is_wp_error( $result )) {
184
+		if( is_wp_error( $result ) ) {
185 185
 			glsr_log()->error( $result->get_error_message() );
186 186
 		}
187 187
 	}
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -124,7 +124,9 @@  discard block
 block discarded – undo
124 124
 				$termId = intval( $termId );
125 125
 			}
126 126
 			$term = term_exists( $termId, Application::TAXONOMY );
127
-			if( !isset( $term['term_id'] ))continue;
127
+			if( !isset( $term['term_id'] )) {
128
+				continue;
129
+			}
128 130
 			$terms[] = $term;
129 131
 		}
130 132
 		return $terms;
@@ -136,7 +138,9 @@  discard block
 block discarded – undo
136 138
 	 */
137 139
 	public function revert( $postId )
138 140
 	{
139
-		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return;
141
+		if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE ) {
142
+			return;
143
+		}
140 144
 		delete_post_meta( $postId, '_edit_last' );
141 145
 		$result = wp_update_post([
142 146
 			'ID' => $postId,
@@ -154,7 +158,9 @@  discard block
 block discarded – undo
154 158
 	 */
155 159
 	public function single( WP_Post $post )
156 160
 	{
157
-		if( $post->post_type != Application::POST_TYPE )return;
161
+		if( $post->post_type != Application::POST_TYPE ) {
162
+			return;
163
+		}
158 164
 		$review = new Review( $post );
159 165
 		return apply_filters( 'site-reviews/get/review', $review, $post );
160 166
 	}
@@ -179,7 +185,9 @@  discard block
 block discarded – undo
179 185
 	protected function setTerms( $postId, $termIds )
180 186
 	{
181 187
 		$terms = $this->normalizeTermIds( $termIds );
182
-		if( empty( $terms ))return;
188
+		if( empty( $terms )) {
189
+			return;
190
+		}
183 191
 		$result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY );
184 192
 		if( is_wp_error( $result )) {
185 193
 			glsr_log()->error( $result->get_error_message() );
Please login to merge, or discard this patch.
plugin/Controllers/ReviewController.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -26,15 +26,15 @@  discard block
 block discarded – undo
26 26
 	{
27 27
 		sort( $termTaxonomyIds );
28 28
 		sort( $oldTermTaxonomyIds );
29
-		if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ))return;
30
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
29
+		if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ) )return;
30
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
31 31
 		$ignoredIds = array_intersect( $oldTermTaxonomyIds, $termTaxonomyIds );
32 32
 		$decreasedIds = array_diff( $oldTermTaxonomyIds, $ignoredIds );
33 33
 		$increasedIds = array_diff( $termTaxonomyIds, $ignoredIds );
34
-		if( $review->term_ids = glsr( Database::class )->getTermIds( $decreasedIds, 'term_taxonomy_id' )) {
34
+		if( $review->term_ids = glsr( Database::class )->getTermIds( $decreasedIds, 'term_taxonomy_id' ) ) {
35 35
 			glsr( CountsManager::class )->decreaseTermCounts( $review );
36 36
 		}
37
-		if( $review->term_ids = glsr( Database::class )->getTermIds( $increasedIds, 'term_taxonomy_id' )) {
37
+		if( $review->term_ids = glsr( Database::class )->getTermIds( $increasedIds, 'term_taxonomy_id' ) ) {
38 38
 			glsr( CountsManager::class )->increaseTermCounts( $review );
39 39
 		}
40 40
 	}
@@ -47,8 +47,8 @@  discard block
 block discarded – undo
47 47
 	 */
48 48
 	public function onAfterChangeStatus( $newStatus, $oldStatus, WP_Post $post )
49 49
 	{
50
-		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return;
51
-		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ));
50
+		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ) )return;
51
+		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ) );
52 52
 		if( $post->post_status == 'publish' ) {
53 53
 			glsr( CountsManager::class )->increase( $review );
54 54
 		}
@@ -74,8 +74,8 @@  discard block
 block discarded – undo
74 74
 	 */
75 75
 	public function onBeforeDelete( $postId )
76 76
 	{
77
-		if( !$this->isReviewPostId( $postId ))return;
78
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
77
+		if( !$this->isReviewPostId( $postId ) )return;
78
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
79 79
 		glsr( CountsManager::class )->decrease( $review );
80 80
 	}
81 81
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 		if( !$this->isReviewPostId( $postId )
93 93
 			|| !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] )
94 94
 		)return;
95
-		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
95
+		$review = glsr( ReviewManager::class )->single( get_post( $postId ) );
96 96
 		if( $review->$metaKey == $metaValue )return;
97 97
 		$method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' );
98 98
 		call_user_func( [$this, $method], $review, $metaValue );
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -26,7 +26,9 @@  discard block
 block discarded – undo
26 26
 	{
27 27
 		sort( $termTaxonomyIds );
28 28
 		sort( $oldTermTaxonomyIds );
29
-		if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId ))return;
29
+		if( $termTaxonomyIds === $oldTermTaxonomyIds || !$this->isReviewPostId( $postId )) {
30
+			return;
31
+		}
30 32
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
31 33
 		$ignoredIds = array_intersect( $oldTermTaxonomyIds, $termTaxonomyIds );
32 34
 		$decreasedIds = array_diff( $oldTermTaxonomyIds, $ignoredIds );
@@ -47,7 +49,9 @@  discard block
 block discarded – undo
47 49
 	 */
48 50
 	public function onAfterChangeStatus( $newStatus, $oldStatus, WP_Post $post )
49 51
 	{
50
-		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] ))return;
52
+		if( $post->post_type != Application::POST_TYPE || in_array( $oldStatus, ['new', $newStatus] )) {
53
+			return;
54
+		}
51 55
 		$review = glsr( ReviewManager::class )->single( get_post( $post->ID ));
52 56
 		if( $post->post_status == 'publish' ) {
53 57
 			glsr( CountsManager::class )->increase( $review );
@@ -63,7 +67,9 @@  discard block
 block discarded – undo
63 67
 	 */
64 68
 	public function onAfterCreate( Review $review )
65 69
 	{
66
-		if( $review->status !== 'publish' )return;
70
+		if( $review->status !== 'publish' ) {
71
+			return;
72
+		}
67 73
 		glsr( CountsManager::class )->increase( $review );
68 74
 	}
69 75
 
@@ -74,7 +80,9 @@  discard block
 block discarded – undo
74 80
 	 */
75 81
 	public function onBeforeDelete( $postId )
76 82
 	{
77
-		if( !$this->isReviewPostId( $postId ))return;
83
+		if( !$this->isReviewPostId( $postId )) {
84
+			return;
85
+		}
78 86
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
79 87
 		glsr( CountsManager::class )->decrease( $review );
80 88
 	}
@@ -91,9 +99,13 @@  discard block
 block discarded – undo
91 99
 	{
92 100
 		if( !$this->isReviewPostId( $postId )
93 101
 			|| !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] )
94
-		)return;
102
+		) {
103
+			return;
104
+		}
95 105
 		$review = glsr( ReviewManager::class )->single( get_post( $postId ));
96
-		if( $review->$metaKey == $metaValue )return;
106
+		if( $review->$metaKey == $metaValue ) {
107
+			return;
108
+		}
97 109
 		$method = glsr( Helper::class )->buildMethodName( $metaKey, 'onBeforeChange' );
98 110
 		call_user_func( [$this, $method], $review, $metaValue );
99 111
 	}
Please login to merge, or discard this patch.