Passed
Push — master ( dd57fd...bbdbbd )
by Paul
04:15
created
plugin/Router.php 1 patch
Spacing   +47 added lines, -47 removed lines patch added patch discarded remove patch
@@ -14,10 +14,10 @@  discard block
 block discarded – undo
14 14
 
15 15
     public function __construct()
16 16
     {
17
-        $this->unguardedActions = apply_filters('site-reviews/router/unguarded-actions', [
17
+        $this->unguardedActions = apply_filters( 'site-reviews/router/unguarded-actions', [
18 18
             'dismiss-notice',
19 19
             'fetch-paged-reviews',
20
-        ]);
20
+        ] );
21 21
     }
22 22
 
23 23
     /**
@@ -26,11 +26,11 @@  discard block
 block discarded – undo
26 26
     public function routeAdminPostRequest()
27 27
     {
28 28
         $request = $this->getRequest();
29
-        if (!$this->isValidPostRequest($request)) {
29
+        if( !$this->isValidPostRequest( $request ) ) {
30 30
             return;
31 31
         }
32
-        check_admin_referer($request['_action']);
33
-        $this->routeRequest('admin', $request['_action'], $request);
32
+        check_admin_referer( $request['_action'] );
33
+        $this->routeRequest( 'admin', $request['_action'], $request );
34 34
     }
35 35
 
36 36
     /**
@@ -39,9 +39,9 @@  discard block
 block discarded – undo
39 39
     public function routeAjaxRequest()
40 40
     {
41 41
         $request = $this->getRequest();
42
-        $this->checkAjaxRequest($request);
43
-        $this->checkAjaxNonce($request);
44
-        $this->routeRequest('ajax', $request['_action'], $request);
42
+        $this->checkAjaxRequest( $request );
43
+        $this->checkAjaxNonce( $request );
44
+        $this->routeRequest( 'ajax', $request['_action'], $request );
45 45
         wp_die();
46 46
     }
47 47
 
@@ -50,45 +50,45 @@  discard block
 block discarded – undo
50 50
      */
51 51
     public function routePublicPostRequest()
52 52
     {
53
-        if (is_admin()) {
53
+        if( is_admin() ) {
54 54
             return;
55 55
         }
56 56
         $request = $this->getRequest();
57
-        if (!$this->isValidPostRequest($request)) {
57
+        if( !$this->isValidPostRequest( $request ) ) {
58 58
             return;
59 59
         }
60
-        if (!$this->isValidPublicNonce($request)) {
60
+        if( !$this->isValidPublicNonce( $request ) ) {
61 61
             return;
62 62
         }
63
-        $this->routeRequest('public', $request['_action'], $request);
63
+        $this->routeRequest( 'public', $request['_action'], $request );
64 64
     }
65 65
 
66 66
     /**
67 67
      * @return void
68 68
      */
69
-    protected function checkAjaxNonce(array $request)
69
+    protected function checkAjaxNonce( array $request )
70 70
     {
71
-        if (!is_user_logged_in() || in_array(Arr::get($request, '_action'), $this->unguardedActions)) {
71
+        if( !is_user_logged_in() || in_array( Arr::get( $request, '_action' ), $this->unguardedActions ) ) {
72 72
             return;
73 73
         }
74
-        if (!isset($request['_nonce'])) {
75
-            $this->sendAjaxError('request is missing a nonce', $request);
74
+        if( !isset($request['_nonce']) ) {
75
+            $this->sendAjaxError( 'request is missing a nonce', $request );
76 76
         }
77
-        if (!wp_verify_nonce($request['_nonce'], $request['_action'])) {
78
-            $this->sendAjaxError('request failed the nonce check', $request, 403);
77
+        if( !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
78
+            $this->sendAjaxError( 'request failed the nonce check', $request, 403 );
79 79
         }
80 80
     }
81 81
 
82 82
     /**
83 83
      * @return void
84 84
      */
85
-    protected function checkAjaxRequest(array $request)
85
+    protected function checkAjaxRequest( array $request )
86 86
     {
87
-        if (!isset($request['_action'])) {
88
-            $this->sendAjaxError('request must include an action', $request);
87
+        if( !isset($request['_action']) ) {
88
+            $this->sendAjaxError( 'request must include an action', $request );
89 89
         }
90
-        if (empty($request['_ajax_request'])) {
91
-            $this->sendAjaxError('request is invalid', $request);
90
+        if( empty($request['_ajax_request']) ) {
91
+            $this->sendAjaxError( 'request is invalid', $request );
92 92
         }
93 93
     }
94 94
 
@@ -99,12 +99,12 @@  discard block
 block discarded – undo
99 99
      */
100 100
     protected function getRequest()
101 101
     {
102
-        $request = Helper::filterInputArray(Application::ID);
103
-        if (Helper::filterInput('action') == Application::PREFIX.'action') {
102
+        $request = Helper::filterInputArray( Application::ID );
103
+        if( Helper::filterInput( 'action' ) == Application::PREFIX.'action' ) {
104 104
             $request['_ajax_request'] = true;
105 105
         }
106
-        if ('submit-review' == Helper::filterInput('_action', $request)) {
107
-            $request['_recaptcha-token'] = Helper::filterInput('g-recaptcha-response');
106
+        if( 'submit-review' == Helper::filterInput( '_action', $request ) ) {
107
+            $request['_recaptcha-token'] = Helper::filterInput( 'g-recaptcha-response' );
108 108
         }
109 109
         return $request;
110 110
     }
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
     /**
113 113
      * @return bool
114 114
      */
115
-    protected function isValidPostRequest(array $request = [])
115
+    protected function isValidPostRequest( array $request = [] )
116 116
     {
117 117
         return !empty($request['_action']) && empty($request['_ajax_request']);
118 118
     }
@@ -120,10 +120,10 @@  discard block
 block discarded – undo
120 120
     /**
121 121
      * @return bool
122 122
      */
123
-    protected function isValidPublicNonce(array $request)
123
+    protected function isValidPublicNonce( array $request )
124 124
     {
125
-        if (is_user_logged_in() && !wp_verify_nonce($request['_nonce'], $request['_action'])) {
126
-            glsr_log()->error('nonce check failed for public request')->debug($request);
125
+        if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
126
+            glsr_log()->error( 'nonce check failed for public request' )->debug( $request );
127 127
             return false;
128 128
         }
129 129
         return true;
@@ -134,19 +134,19 @@  discard block
 block discarded – undo
134 134
      * @param string $action
135 135
      * @return void
136 136
      */
137
-    protected function routeRequest($type, $action, array $request = [])
137
+    protected function routeRequest( $type, $action, array $request = [] )
138 138
     {
139 139
         $actionHook = 'site-reviews/route/'.$type.'/request';
140
-        $controller = glsr(Helper::buildClassName($type.'-controller', 'Controllers'));
141
-        $method = Helper::buildMethodName($action, 'router');
142
-        $request = apply_filters('site-reviews/route/request', $request, $action, $type);
143
-        do_action($actionHook, $action, $request);
144
-        if (is_callable([$controller, $method])) {
145
-            call_user_func([$controller, $method], $request);
140
+        $controller = glsr( Helper::buildClassName( $type.'-controller', 'Controllers' ) );
141
+        $method = Helper::buildMethodName( $action, 'router' );
142
+        $request = apply_filters( 'site-reviews/route/request', $request, $action, $type );
143
+        do_action( $actionHook, $action, $request );
144
+        if( is_callable( [$controller, $method] ) ) {
145
+            call_user_func( [$controller, $method], $request );
146 146
             return;
147 147
         }
148
-        if (0 === did_action($actionHook)) {
149
-            glsr_log('Unknown '.$type.' router request: '.$action);
148
+        if( 0 === did_action( $actionHook ) ) {
149
+            glsr_log( 'Unknown '.$type.' router request: '.$action );
150 150
         }
151 151
     }
152 152
 
@@ -155,14 +155,14 @@  discard block
 block discarded – undo
155 155
      * @param int $statusCode
156 156
      * @return void
157 157
      */
158
-    protected function sendAjaxError($error, array $request, $statusCode = 400)
158
+    protected function sendAjaxError( $error, array $request, $statusCode = 400 )
159 159
     {
160
-        glsr_log()->error($error)->debug($request);
161
-        glsr(Notice::class)->addError(__('There was an error (try reloading the page).', 'site-reviews').' <code>'.$error.'</code>');
162
-        wp_send_json_error([
163
-            'message' => __('The form could not be submitted. Please notify the site administrator.', 'site-reviews'),
164
-            'notices' => glsr(Notice::class)->get(),
160
+        glsr_log()->error( $error )->debug( $request );
161
+        glsr( Notice::class )->addError( __( 'There was an error (try reloading the page).', 'site-reviews' ).' <code>'.$error.'</code>' );
162
+        wp_send_json_error( [
163
+            'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ),
164
+            'notices' => glsr( Notice::class )->get(),
165 165
             'error' => $error,
166
-        ]);
166
+        ] );
167 167
     }
168 168
 }
Please login to merge, or discard this patch.
plugin/Controllers/TaxonomyController.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -36,14 +36,14 @@  discard block
 block discarded – undo
36 36
      */
37 37
     public function renderTaxonomyFilter()
38 38
     {
39
-        if (!is_object_in_taxonomy(glsr_current_screen()->post_type, Application::TAXONOMY)) {
39
+        if( !is_object_in_taxonomy( glsr_current_screen()->post_type, Application::TAXONOMY ) ) {
40 40
             return;
41 41
         }
42
-        echo glsr(Builder::class)->label(__('Filter by category', 'site-reviews'), [
42
+        echo glsr( Builder::class )->label( __( 'Filter by category', 'site-reviews' ), [
43 43
             'class' => 'screen-reader-text',
44 44
             'for' => Application::TAXONOMY,
45
-        ]);
46
-        wp_dropdown_categories([
45
+        ] );
46
+        wp_dropdown_categories( [
47 47
             'depth' => 3,
48 48
             'hide_empty' => true,
49 49
             'hide_if_empty' => true,
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
             'show_option_all' => $this->getShowOptionAll(),
56 56
             'taxonomy' => Application::TAXONOMY,
57 57
             'value_field' => 'slug',
58
-        ]);
58
+        ] );
59 59
     }
60 60
 
61 61
     /**
@@ -68,17 +68,17 @@  discard block
 block discarded – undo
68 68
      * @return void
69 69
      * @action set_object_terms
70 70
      */
71
-    public function restrictTermSelection($postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds)
71
+    public function restrictTermSelection( $postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds )
72 72
     {
73
-        if (Application::TAXONOMY != $taxonomy || count($newTTIds) <= 1) {
73
+        if( Application::TAXONOMY != $taxonomy || count( $newTTIds ) <= 1 ) {
74 74
             return;
75 75
         }
76
-        $diff = array_diff($newTTIds, $oldTTIds);
77
-        if (empty($newTerm = array_shift($diff))) {
78
-            $newTerm = array_shift($newTTIds);
76
+        $diff = array_diff( $newTTIds, $oldTTIds );
77
+        if( empty($newTerm = array_shift( $diff )) ) {
78
+            $newTerm = array_shift( $newTTIds );
79 79
         }
80
-        if ($newTerm) {
81
-            wp_set_object_terms($postId, intval($newTerm), $taxonomy);
80
+        if( $newTerm ) {
81
+            wp_set_object_terms( $postId, intval( $newTerm ), $taxonomy );
82 82
         }
83 83
     }
84 84
 
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
     protected function getSelected()
89 89
     {
90 90
         global $wp_query;
91
-        return Arr::get($wp_query->query, Application::TAXONOMY);
91
+        return Arr::get( $wp_query->query, Application::TAXONOMY );
92 92
     }
93 93
 
94 94
     /**
@@ -96,9 +96,9 @@  discard block
 block discarded – undo
96 96
      */
97 97
     protected function getShowOptionAll()
98 98
     {
99
-        $taxonomy = get_taxonomy(Application::TAXONOMY);
99
+        $taxonomy = get_taxonomy( Application::TAXONOMY );
100 100
         return $taxonomy
101
-            ? ucfirst(strtolower($taxonomy->labels->all_items))
101
+            ? ucfirst( strtolower( $taxonomy->labels->all_items ) )
102 102
             : '';
103 103
     }
104 104
 }
Please login to merge, or discard this patch.
plugin/Modules/Schema/BaseType.php 1 patch
Spacing   +45 added lines, -45 removed lines patch added patch discarded remove patch
@@ -37,18 +37,18 @@  discard block
 block discarded – undo
37 37
      * @param string $method
38 38
      * @return static
39 39
      */
40
-    public function __call($method, array $arguments)
40
+    public function __call( $method, array $arguments )
41 41
     {
42
-        return $this->setProperty($method, Arr::get($arguments, 0));
42
+        return $this->setProperty( $method, Arr::get( $arguments, 0 ) );
43 43
     }
44 44
 
45 45
     /**
46 46
      * @param string $type
47 47
      */
48
-    public function __construct($type = null)
48
+    public function __construct( $type = null )
49 49
     {
50
-        $this->type = !is_string($type)
51
-            ? (new ReflectionClass($this))->getShortName()
50
+        $this->type = !is_string( $type )
51
+            ? (new ReflectionClass( $this ))->getShortName()
52 52
             : $type;
53 53
         $this->setAllowedProperties();
54 54
     }
@@ -64,10 +64,10 @@  discard block
 block discarded – undo
64 64
     /**
65 65
      * @return static
66 66
      */
67
-    public function addProperties(array $properties)
67
+    public function addProperties( array $properties )
68 68
     {
69
-        foreach ($properties as $property => $value) {
70
-            $this->setProperty($property, $value);
69
+        foreach( $properties as $property => $value ) {
70
+            $this->setProperty( $property, $value );
71 71
         }
72 72
         return $this;
73 73
     }
@@ -93,9 +93,9 @@  discard block
 block discarded – undo
93 93
      * @param mixed $default
94 94
      * @return mixed
95 95
      */
96
-    public function getProperty($property, $default = null)
96
+    public function getProperty( $property, $default = null )
97 97
     {
98
-        return Arr::get($this->properties, $property, $default);
98
+        return Arr::get( $this->properties, $property, $default );
99 99
     }
100 100
 
101 101
     /**
@@ -111,10 +111,10 @@  discard block
 block discarded – undo
111 111
      * @param mixed $callback
112 112
      * @return static
113 113
      */
114
-    public function doIf($condition, $callback)
114
+    public function doIf( $condition, $callback )
115 115
     {
116
-        if ($condition) {
117
-            $callback($this);
116
+        if( $condition ) {
117
+            $callback( $this );
118 118
         }
119 119
         return $this;
120 120
     }
@@ -131,18 +131,18 @@  discard block
 block discarded – undo
131 131
      * @param mixed $offset
132 132
      * @return bool
133 133
      */
134
-    public function offsetExists($offset)
134
+    public function offsetExists( $offset )
135 135
     {
136
-        return array_key_exists($offset, $this->properties);
136
+        return array_key_exists( $offset, $this->properties );
137 137
     }
138 138
 
139 139
     /**
140 140
      * @param string $offset
141 141
      * @return mixed
142 142
      */
143
-    public function offsetGet($offset)
143
+    public function offsetGet( $offset )
144 144
     {
145
-        return $this->getProperty($offset);
145
+        return $this->getProperty( $offset );
146 146
     }
147 147
 
148 148
     /**
@@ -150,16 +150,16 @@  discard block
 block discarded – undo
150 150
      * @param mixed $value
151 151
      * @return void
152 152
      */
153
-    public function offsetSet($offset, $value)
153
+    public function offsetSet( $offset, $value )
154 154
     {
155
-        $this->setProperty($offset, $value);
155
+        $this->setProperty( $offset, $value );
156 156
     }
157 157
 
158 158
     /**
159 159
      * @param string $offset
160 160
      * @return void
161 161
      */
162
-    public function offsetUnset($offset)
162
+    public function offsetUnset( $offset )
163 163
     {
164 164
         unset($this->properties[$offset]);
165 165
     }
@@ -169,11 +169,11 @@  discard block
 block discarded – undo
169 169
      * @param mixed $value
170 170
      * @return static
171 171
      */
172
-    public function setProperty($property, $value)
172
+    public function setProperty( $property, $value )
173 173
     {
174
-        if (!in_array($property, $this->allowed)
175
-            && 'UnknownType' != (new ReflectionClass($this))->getShortName()) {
176
-            glsr_log()->warning($this->getType().' does not allow the "'.$property.'" property');
174
+        if( !in_array( $property, $this->allowed )
175
+            && 'UnknownType' != (new ReflectionClass( $this ))->getShortName() ) {
176
+            glsr_log()->warning( $this->getType().' does not allow the "'.$property.'" property' );
177 177
             return $this;
178 178
         }
179 179
         $this->properties[$property] = $value;
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
         return [
189 189
             '@context' => $this->getContext(),
190 190
             '@type' => $this->getType(),
191
-        ] + $this->serializeProperty($this->getProperties());
191
+        ] + $this->serializeProperty( $this->getProperties() );
192 192
     }
193 193
 
194 194
     /**
@@ -196,8 +196,8 @@  discard block
 block discarded – undo
196 196
      */
197 197
     public function toScript()
198 198
     {
199
-        return sprintf('<script type="application/ld+json">%s</script>',
200
-            json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
199
+        return sprintf( '<script type="application/ld+json">%s</script>',
200
+            json_encode( $this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES )
201 201
         );
202 202
     }
203 203
 
@@ -205,20 +205,20 @@  discard block
 block discarded – undo
205 205
      * @param array|null $parents
206 206
      * @return array
207 207
      */
208
-    protected function getParents($parents = null)
208
+    protected function getParents( $parents = null )
209 209
     {
210
-        if (!isset($parents)) {
210
+        if( !isset($parents) ) {
211 211
             $parents = $this->parents;
212 212
         }
213 213
         $newParents = $parents;
214
-        foreach ($parents as $parent) {
215
-            $parentClass = Helper::buildClassName($parent, __NAMESPACE__);
216
-            if (!class_exists($parentClass)) {
214
+        foreach( $parents as $parent ) {
215
+            $parentClass = Helper::buildClassName( $parent, __NAMESPACE__ );
216
+            if( !class_exists( $parentClass ) ) {
217 217
                 continue;
218 218
             }
219
-            $newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
219
+            $newParents = array_merge( $newParents, $this->getParents( (new $parentClass())->parents ) );
220 220
         }
221
-        return array_values(array_unique($newParents));
221
+        return array_values( array_unique( $newParents ) );
222 222
     }
223 223
 
224 224
     /**
@@ -227,12 +227,12 @@  discard block
 block discarded – undo
227 227
     protected function setAllowedProperties()
228 228
     {
229 229
         $parents = $this->getParents();
230
-        foreach ($parents as $parent) {
231
-            $parentClass = Helper::buildClassName($parent, __NAMESPACE__);
232
-            if (!class_exists($parentClass)) {
230
+        foreach( $parents as $parent ) {
231
+            $parentClass = Helper::buildClassName( $parent, __NAMESPACE__ );
232
+            if( !class_exists( $parentClass ) ) {
233 233
                 continue;
234 234
             }
235
-            $this->allowed = array_values(array_unique(array_merge((new $parentClass())->allowed, $this->allowed)));
235
+            $this->allowed = array_values( array_unique( array_merge( (new $parentClass())->allowed, $this->allowed ) ) );
236 236
         }
237 237
     }
238 238
 
@@ -240,19 +240,19 @@  discard block
 block discarded – undo
240 240
      * @param mixed $property
241 241
      * @return array|string
242 242
      */
243
-    protected function serializeProperty($property)
243
+    protected function serializeProperty( $property )
244 244
     {
245
-        if (is_array($property)) {
246
-            return array_map([$this, 'serializeProperty'], $property);
245
+        if( is_array( $property ) ) {
246
+            return array_map( [$this, 'serializeProperty'], $property );
247 247
         }
248
-        if ($property instanceof Type) {
248
+        if( $property instanceof Type ) {
249 249
             $property = $property->toArray();
250 250
             unset($property['@context']);
251 251
         }
252
-        if ($property instanceof DateTimeInterface) {
253
-            $property = $property->format(DateTime::ATOM);
252
+        if( $property instanceof DateTimeInterface ) {
253
+            $property = $property->format( DateTime::ATOM );
254 254
         }
255
-        if (is_object($property)) {
255
+        if( is_object( $property ) ) {
256 256
             throw new InvalidProperty();
257 257
         }
258 258
         return $property;
Please login to merge, or discard this patch.
plugin/Modules/Html/Settings.php 1 patch
Spacing   +67 added lines, -67 removed lines patch added patch discarded remove patch
@@ -20,31 +20,31 @@  discard block
 block discarded – undo
20 20
      * @param string $id
21 21
      * @return string
22 22
      */
23
-    public function buildFields($id)
23
+    public function buildFields( $id )
24 24
     {
25
-        $this->settings = glsr(DefaultsManager::class)->settings();
26
-        $method = Helper::buildMethodName($id, 'getTemplateDataFor');
27
-        $data = !method_exists($this, $method)
28
-            ? $this->getTemplateData($id)
29
-            : $this->$method($id);
30
-        return glsr(Template::class)->build('pages/settings/'.$id, $data);
25
+        $this->settings = glsr( DefaultsManager::class )->settings();
26
+        $method = Helper::buildMethodName( $id, 'getTemplateDataFor' );
27
+        $data = !method_exists( $this, $method )
28
+            ? $this->getTemplateData( $id )
29
+            : $this->$method( $id );
30
+        return glsr( Template::class )->build( 'pages/settings/'.$id, $data );
31 31
     }
32 32
 
33 33
     /**
34 34
      * @return string
35 35
      */
36
-    protected function getFieldDefault(array $field)
36
+    protected function getFieldDefault( array $field )
37 37
     {
38
-        return Arr::get($field, 'default');
38
+        return Arr::get( $field, 'default' );
39 39
     }
40 40
 
41 41
     /**
42 42
      * @return string
43 43
      */
44
-    protected function getFieldNameForDependsOn($path)
44
+    protected function getFieldNameForDependsOn( $path )
45 45
     {
46
-        $fieldName = Str::convertPathToName($path, OptionManager::databaseKey());
47
-        return $this->isMultiDependency($path)
46
+        $fieldName = Str::convertPathToName( $path, OptionManager::databaseKey() );
47
+        return $this->isMultiDependency( $path )
48 48
             ? $fieldName.'[]'
49 49
             : $fieldName;
50 50
     }
@@ -52,25 +52,25 @@  discard block
 block discarded – undo
52 52
     /**
53 53
      * @return array
54 54
      */
55
-    protected function getSettingFields($path)
55
+    protected function getSettingFields( $path )
56 56
     {
57
-        return array_filter($this->settings, function ($key) use ($path) {
58
-            return Str::startsWith($path, $key);
59
-        }, ARRAY_FILTER_USE_KEY);
57
+        return array_filter( $this->settings, function( $key ) use ($path) {
58
+            return Str::startsWith( $path, $key );
59
+        }, ARRAY_FILTER_USE_KEY );
60 60
     }
61 61
 
62 62
     /**
63 63
      * @return string
64 64
      */
65
-    protected function getSettingRows(array $fields)
65
+    protected function getSettingRows( array $fields )
66 66
     {
67 67
         $rows = '';
68
-        foreach ($fields as $name => $field) {
69
-            $field = wp_parse_args($field, [
68
+        foreach( $fields as $name => $field ) {
69
+            $field = wp_parse_args( $field, [
70 70
                 'is_setting' => true,
71 71
                 'name' => $name,
72
-            ]);
73
-            $rows.= new Field($this->normalize($field));
72
+            ] );
73
+            $rows .= new Field( $this->normalize( $field ) );
74 74
         }
75 75
         return $rows;
76 76
     }
@@ -79,12 +79,12 @@  discard block
 block discarded – undo
79 79
      * @param string $id
80 80
      * @return array
81 81
      */
82
-    protected function getTemplateData($id)
82
+    protected function getTemplateData( $id )
83 83
     {
84
-        $fields = $this->getSettingFields($this->normalizeSettingPath($id));
84
+        $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
85 85
         return [
86 86
             'context' => [
87
-                'rows' => $this->getSettingRows($fields),
87
+                'rows' => $this->getSettingRows( $fields ),
88 88
             ],
89 89
         ];
90 90
     }
@@ -93,19 +93,19 @@  discard block
 block discarded – undo
93 93
      * @param string $id
94 94
      * @return array
95 95
      */
96
-    protected function getTemplateDataForAddons($id)
96
+    protected function getTemplateDataForAddons( $id )
97 97
     {
98
-        $fields = $this->getSettingFields($this->normalizeSettingPath($id));
99
-        $settings = Arr::convertDotNotationArray($fields);
100
-        $settingKeys = array_keys($settings['settings']['addons']);
98
+        $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
99
+        $settings = Arr::convertDotNotationArray( $fields );
100
+        $settingKeys = array_keys( $settings['settings']['addons'] );
101 101
         $results = [];
102
-        foreach ($settingKeys as $key) {
103
-            $addonFields = array_filter($fields, function ($path) use ($key) {
104
-                return Str::startsWith('settings.addons.'.$key, $path);
105
-            }, ARRAY_FILTER_USE_KEY);
106
-            $results[$key] = $this->getSettingRows($addonFields);
102
+        foreach( $settingKeys as $key ) {
103
+            $addonFields = array_filter( $fields, function( $path ) use ($key) {
104
+                return Str::startsWith( 'settings.addons.'.$key, $path );
105
+            }, ARRAY_FILTER_USE_KEY );
106
+            $results[$key] = $this->getSettingRows( $addonFields );
107 107
         }
108
-        ksort($results);
108
+        ksort( $results );
109 109
         return [
110 110
             'settings' => $results,
111 111
         ];
@@ -115,13 +115,13 @@  discard block
 block discarded – undo
115 115
      * @param string $id
116 116
      * @return array
117 117
      */
118
-    protected function getTemplateDataForLicenses($id)
118
+    protected function getTemplateDataForLicenses( $id )
119 119
     {
120
-        $fields = $this->getSettingFields($this->normalizeSettingPath($id));
121
-        ksort($fields);
120
+        $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
121
+        ksort( $fields );
122 122
         return [
123 123
             'context' => [
124
-                'rows' => $this->getSettingRows($fields),
124
+                'rows' => $this->getSettingRows( $fields ),
125 125
             ],
126 126
         ];
127 127
     }
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
      */
132 132
     protected function getTemplateDataForTranslations()
133 133
     {
134
-        $translations = glsr(Translation::class)->renderAll();
134
+        $translations = glsr( Translation::class )->renderAll();
135 135
         $class = empty($translations)
136 136
             ? 'glsr-hidden'
137 137
             : '';
@@ -149,16 +149,16 @@  discard block
 block discarded – undo
149 149
      * @param string|array $expectedValue
150 150
      * @return bool
151 151
      */
152
-    protected function isFieldHidden($path, $expectedValue)
152
+    protected function isFieldHidden( $path, $expectedValue )
153 153
     {
154
-        $optionValue = glsr(OptionManager::class)->get(
154
+        $optionValue = glsr( OptionManager::class )->get(
155 155
             $path,
156
-            Arr::get(glsr()->defaults, $path)
156
+            Arr::get( glsr()->defaults, $path )
157 157
         );
158
-        if (is_array($expectedValue)) {
159
-            return is_array($optionValue)
160
-                ? 0 === count(array_intersect($optionValue, $expectedValue))
161
-                : !in_array($optionValue, $expectedValue);
158
+        if( is_array( $expectedValue ) ) {
159
+            return is_array( $optionValue )
160
+                ? 0 === count( array_intersect( $optionValue, $expectedValue ) )
161
+                : !in_array( $optionValue, $expectedValue );
162 162
         }
163 163
         return $optionValue != $expectedValue;
164 164
     }
@@ -166,9 +166,9 @@  discard block
 block discarded – undo
166 166
     /**
167 167
      * @return bool
168 168
      */
169
-    protected function isMultiDependency($path)
169
+    protected function isMultiDependency( $path )
170 170
     {
171
-        if (isset($this->settings[$path])) {
171
+        if( isset($this->settings[$path]) ) {
172 172
             $field = $this->settings[$path];
173 173
             return ('checkbox' == $field['type'] && !empty($field['options']))
174 174
                 || !empty($field['multiple']);
@@ -179,32 +179,32 @@  discard block
 block discarded – undo
179 179
     /**
180 180
      * @return array
181 181
      */
182
-    protected function normalize(array $field)
182
+    protected function normalize( array $field )
183 183
     {
184
-        $field = $this->normalizeDependsOn($field);
185
-        $field = $this->normalizeLabelAndLegend($field);
186
-        $field = $this->normalizeValue($field);
184
+        $field = $this->normalizeDependsOn( $field );
185
+        $field = $this->normalizeLabelAndLegend( $field );
186
+        $field = $this->normalizeValue( $field );
187 187
         return $field;
188 188
     }
189 189
 
190 190
     /**
191 191
      * @return array
192 192
      */
193
-    protected function normalizeDependsOn(array $field)
193
+    protected function normalizeDependsOn( array $field )
194 194
     {
195
-        if (!empty($field['depends_on']) && is_array($field['depends_on'])) {
195
+        if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
196 196
             $isFieldHidden = false;
197 197
             $conditions = [];
198
-            foreach ($field['depends_on'] as $path => $value) {
198
+            foreach( $field['depends_on'] as $path => $value ) {
199 199
                 $conditions[] = [
200
-                    'name' => $this->getFieldNameForDependsOn($path),
200
+                    'name' => $this->getFieldNameForDependsOn( $path ),
201 201
                     'value' => $value,
202 202
                 ];
203
-                if ($this->isFieldHidden($path, $value)) {
203
+                if( $this->isFieldHidden( $path, $value ) ) {
204 204
                     $isFieldHidden = true;
205 205
                 }
206 206
             }
207
-            $field['data-depends'] = json_encode($conditions, JSON_HEX_APOS | JSON_HEX_QUOT);
207
+            $field['data-depends'] = json_encode( $conditions, JSON_HEX_APOS | JSON_HEX_QUOT );
208 208
             $field['is_hidden'] = $isFieldHidden;
209 209
         }
210 210
         return $field;
@@ -213,14 +213,14 @@  discard block
 block discarded – undo
213 213
     /**
214 214
      * @return array
215 215
      */
216
-    protected function normalizeLabelAndLegend(array $field)
216
+    protected function normalizeLabelAndLegend( array $field )
217 217
     {
218
-        if (!empty($field['label'])) {
218
+        if( !empty($field['label']) ) {
219 219
             $field['legend'] = $field['label'];
220 220
             unset($field['label']);
221 221
         } else {
222 222
             $field['is_valid'] = false;
223
-            glsr_log()->warning('Setting field is missing a label')->debug($field);
223
+            glsr_log()->warning( 'Setting field is missing a label' )->debug( $field );
224 224
         }
225 225
         return $field;
226 226
     }
@@ -228,12 +228,12 @@  discard block
 block discarded – undo
228 228
     /**
229 229
      * @return array
230 230
      */
231
-    protected function normalizeValue(array $field)
231
+    protected function normalizeValue( array $field )
232 232
     {
233
-        if (!isset($field['value'])) {
234
-            $field['value'] = glsr(OptionManager::class)->get(
233
+        if( !isset($field['value']) ) {
234
+            $field['value'] = glsr( OptionManager::class )->get(
235 235
                 $field['name'],
236
-                $this->getFieldDefault($field)
236
+                $this->getFieldDefault( $field )
237 237
             );
238 238
         }
239 239
         return $field;
@@ -242,8 +242,8 @@  discard block
 block discarded – undo
242 242
     /**
243 243
      * @return string
244 244
      */
245
-    protected function normalizeSettingPath($path)
245
+    protected function normalizeSettingPath( $path )
246 246
     {
247
-        return Str::prefix('settings.', rtrim($path, '.'));
247
+        return Str::prefix( 'settings.', rtrim( $path, '.' ) );
248 248
     }
249 249
 }
Please login to merge, or discard this patch.
plugin/Modules/Html/Fields/Field.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
      */
13 13
     protected $builder;
14 14
 
15
-    public function __construct(Builder $builder)
15
+    public function __construct( Builder $builder )
16 16
     {
17 17
         $this->builder = $builder;
18 18
     }
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
      */
23 23
     public function build()
24 24
     {
25
-        glsr_log()->error('Build method is not implemented for '.get_class($this));
25
+        glsr_log()->error( 'Build method is not implemented for '.get_class( $this ) );
26 26
     }
27 27
 
28 28
     /**
@@ -36,14 +36,14 @@  discard block
 block discarded – undo
36 36
     /**
37 37
      * @return array
38 38
      */
39
-    public static function merge(array $args)
39
+    public static function merge( array $args )
40 40
     {
41 41
         $merged = array_merge(
42
-            wp_parse_args($args, static::defaults()),
42
+            wp_parse_args( $args, static::defaults() ),
43 43
             static::required()
44 44
         );
45
-        $merged['class'] = implode(' ', static::mergedAttribute('class', ' ', $args));
46
-        $merged['style'] = implode(';', static::mergedAttribute('style', ';', $args));
45
+        $merged['class'] = implode( ' ', static::mergedAttribute( 'class', ' ', $args ) );
46
+        $merged['style'] = implode( ';', static::mergedAttribute( 'style', ';', $args ) );
47 47
         return $merged;
48 48
     }
49 49
 
@@ -52,13 +52,13 @@  discard block
 block discarded – undo
52 52
      * @param string $key
53 53
      * @return array
54 54
      */
55
-    public static function mergedAttribute($key, $delimiter, array $args)
55
+    public static function mergedAttribute( $key, $delimiter, array $args )
56 56
     {
57
-        return array_filter(array_merge(
58
-            explode($delimiter, Arr::get($args, $key)),
59
-            explode($delimiter, Arr::get(static::defaults(), $key)),
60
-            explode($delimiter, Arr::get(static::required(), $key))
61
-        ));
57
+        return array_filter( array_merge(
58
+            explode( $delimiter, Arr::get( $args, $key ) ),
59
+            explode( $delimiter, Arr::get( static::defaults(), $key ) ),
60
+            explode( $delimiter, Arr::get( static::required(), $key ) )
61
+        ) );
62 62
     }
63 63
 
64 64
     /**
@@ -74,6 +74,6 @@  discard block
 block discarded – undo
74 74
      */
75 75
     protected function mergeFieldArgs()
76 76
     {
77
-        $this->builder->args = static::merge($this->builder->args);
77
+        $this->builder->args = static::merge( $this->builder->args );
78 78
     }
79 79
 }
Please login to merge, or discard this patch.
plugin/Modules/Style.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 
35 35
     public function __construct()
36 36
     {
37
-        $this->style = glsr(OptionManager::class)->get('settings.general.style', 'default');
37
+        $this->style = glsr( OptionManager::class )->get( 'settings.general.style', 'default' );
38 38
         $this->setConfig();
39 39
     }
40 40
 
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
      * @param string $view
43 43
      * @return string
44 44
      */
45
-    public function filterView($view)
45
+    public function filterView( $view )
46 46
     {
47 47
         $styledViews = [
48 48
             'templates/form/field',
@@ -50,15 +50,15 @@  discard block
 block discarded – undo
50 50
             'templates/form/submit-button',
51 51
             'templates/reviews-form',
52 52
         ];
53
-        if (!preg_match('('.implode('|', $styledViews).')', $view)) {
53
+        if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) {
54 54
             return $view;
55 55
         }
56
-        $views = $this->generatePossibleViews($view);
57
-        foreach ($views as $possibleView) {
58
-            if (!file_exists(glsr()->file($possibleView))) {
56
+        $views = $this->generatePossibleViews( $view );
57
+        foreach( $views as $possibleView ) {
58
+            if( !file_exists( glsr()->file( $possibleView ) ) ) {
59 59
                 continue;
60 60
             }
61
-            return Str::removePrefix('views/', $possibleView);
61
+            return Str::removePrefix( 'views/', $possibleView );
62 62
         }
63 63
         return $view;
64 64
     }
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
      */
69 69
     public function get()
70 70
     {
71
-        return apply_filters('site-reviews/style', $this->style);
71
+        return apply_filters( 'site-reviews/style', $this->style );
72 72
     }
73 73
 
74 74
     /**
@@ -77,77 +77,77 @@  discard block
 block discarded – undo
77 77
     public function setConfig()
78 78
     {
79 79
         $config = shortcode_atts(
80
-            array_fill_keys(['fields', 'pagination', 'validation'], []),
81
-            glsr()->config('styles/'.$this->style)
80
+            array_fill_keys( ['fields', 'pagination', 'validation'], [] ),
81
+            glsr()->config( 'styles/'.$this->style )
82 82
         );
83
-        $this->fields = glsr(StyleFieldsDefaults::class)->restrict($config['fields']);
84
-        $this->pagination = glsr(PaginationDefaults::class)->restrict($config['pagination']);
85
-        $this->validation = glsr(StyleValidationDefaults::class)->restrict($config['validation']);
83
+        $this->fields = glsr( StyleFieldsDefaults::class )->restrict( $config['fields'] );
84
+        $this->pagination = glsr( PaginationDefaults::class )->restrict( $config['pagination'] );
85
+        $this->validation = glsr( StyleValidationDefaults::class )->restrict( $config['validation'] );
86 86
     }
87 87
 
88 88
     /**
89 89
      * @return void
90 90
      */
91
-    public function modifyField(Builder $instance)
91
+    public function modifyField( Builder $instance )
92 92
     {
93
-        if (!$this->isPublicInstance($instance) || empty(array_filter($this->fields))) {
93
+        if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->fields )) ) {
94 94
             return;
95 95
         }
96
-        call_user_func_array([$this, 'customize'], [$instance]);
96
+        call_user_func_array( [$this, 'customize'], [$instance] );
97 97
     }
98 98
 
99 99
     /**
100 100
      * @return array
101 101
      */
102
-    public function paginationArgs(array $args)
102
+    public function paginationArgs( array $args )
103 103
     {
104
-        return wp_parse_args($args, $this->pagination);
104
+        return wp_parse_args( $args, $this->pagination );
105 105
     }
106 106
 
107 107
     /**
108 108
      * @return void
109 109
      */
110
-    protected function customize(Builder $instance)
110
+    protected function customize( Builder $instance )
111 111
     {
112
-        if (!array_key_exists($instance->tag, $this->fields)) {
112
+        if( !array_key_exists( $instance->tag, $this->fields ) ) {
113 113
             return;
114 114
         }
115
-        $args = wp_parse_args($instance->args, array_fill_keys(['class', 'type'], ''));
115
+        $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) );
116 116
         $key = $instance->tag.'_'.$args['type'];
117
-        $classes = Arr::get($this->fields, $key, Arr::get($this->fields, $instance->tag));
118
-        $instance->args['class'] = trim($args['class'].' '.$classes);
119
-        do_action_ref_array('site-reviews/customize/'.$this->style, [$instance]);
117
+        $classes = Arr::get( $this->fields, $key, Arr::get( $this->fields, $instance->tag ) );
118
+        $instance->args['class'] = trim( $args['class'].' '.$classes );
119
+        do_action_ref_array( 'site-reviews/customize/'.$this->style, [$instance] );
120 120
     }
121 121
 
122 122
     /**
123 123
      * @param string $view
124 124
      * @return array
125 125
      */
126
-    protected function generatePossibleViews($view)
126
+    protected function generatePossibleViews( $view )
127 127
     {
128
-        $basename = basename($view);
129
-        $basepath = rtrim($view, $basename);
128
+        $basename = basename( $view );
129
+        $basepath = rtrim( $view, $basename );
130 130
         $customPath = 'views/partials/styles/'.$this->style.'/';
131
-        $parts = explode('_', $basename);
131
+        $parts = explode( '_', $basename );
132 132
         $views = [
133 133
             $customPath.$basename,
134 134
             $customPath.$parts[0],
135 135
             $view,
136 136
             $basepath.$parts[0],
137 137
         ];
138
-        return array_filter($views);
138
+        return array_filter( $views );
139 139
     }
140 140
 
141 141
     /**
142 142
      * @return bool
143 143
      */
144
-    protected function isPublicInstance(Builder $instance)
144
+    protected function isPublicInstance( Builder $instance )
145 145
     {
146
-        $args = wp_parse_args($instance->args, [
146
+        $args = wp_parse_args( $instance->args, [
147 147
             'is_public' => false,
148 148
             'is_raw' => false,
149
-        ]);
150
-        if (is_admin() || !$args['is_public'] || $args['is_raw']) {
149
+        ] );
150
+        if( is_admin() || !$args['is_public'] || $args['is_raw'] ) {
151 151
             return false;
152 152
         }
153 153
         return true;
Please login to merge, or discard this patch.
plugin/Review.php 1 patch
Spacing   +40 added lines, -40 removed lines patch added patch discarded remove patch
@@ -33,27 +33,27 @@  discard block
 block discarded – undo
33 33
     public $url;
34 34
     public $user_id;
35 35
 
36
-    public function __construct(WP_Post $post)
36
+    public function __construct( WP_Post $post )
37 37
     {
38
-        if (Application::POST_TYPE != $post->post_type) {
38
+        if( Application::POST_TYPE != $post->post_type ) {
39 39
             return;
40 40
         }
41 41
         $this->content = $post->post_content;
42 42
         $this->date = $post->post_date;
43
-        $this->ID = intval($post->ID);
43
+        $this->ID = intval( $post->ID );
44 44
         $this->status = $post->post_status;
45 45
         $this->title = $post->post_title;
46
-        $this->user_id = intval($post->post_author);
47
-        $this->setProperties($post);
48
-        $this->setTermIds($post);
46
+        $this->user_id = intval( $post->post_author );
47
+        $this->setProperties( $post );
48
+        $this->setTermIds( $post );
49 49
     }
50 50
 
51 51
     /**
52 52
      * @return mixed
53 53
      */
54
-    public function __get($key)
54
+    public function __get( $key )
55 55
     {
56
-        return $this->offsetGet($key);
56
+        return $this->offsetGet( $key );
57 57
     }
58 58
 
59 59
     /**
@@ -61,41 +61,41 @@  discard block
 block discarded – undo
61 61
      */
62 62
     public function __toString()
63 63
     {
64
-        return (string) $this->build();
64
+        return (string)$this->build();
65 65
     }
66 66
 
67 67
     /**
68 68
      * @return ReviewHtml
69 69
      */
70
-    public function build(array $args = [])
70
+    public function build( array $args = [] )
71 71
     {
72
-        if (empty($this->ID)) {
73
-            return new ReviewHtml($this);
72
+        if( empty($this->ID) ) {
73
+            return new ReviewHtml( $this );
74 74
         }
75
-        $partial = glsr(SiteReviewsPartial::class);
76
-        $partial->args = glsr(SiteReviewsDefaults::class)->merge($args);
77
-        $partial->options = Arr::flattenArray(glsr(OptionManager::class)->all());
78
-        return $partial->buildReview($this);
75
+        $partial = glsr( SiteReviewsPartial::class );
76
+        $partial->args = glsr( SiteReviewsDefaults::class )->merge( $args );
77
+        $partial->options = Arr::flattenArray( glsr( OptionManager::class )->all() );
78
+        return $partial->buildReview( $this );
79 79
     }
80 80
 
81 81
     /**
82 82
      * @param mixed $key
83 83
      * @return bool
84 84
      */
85
-    public function offsetExists($key)
85
+    public function offsetExists( $key )
86 86
     {
87
-        return property_exists($this, $key) || array_key_exists($key, (array) $this->custom);
87
+        return property_exists( $this, $key ) || array_key_exists( $key, (array)$this->custom );
88 88
     }
89 89
 
90 90
     /**
91 91
      * @param mixed $key
92 92
      * @return mixed
93 93
      */
94
-    public function offsetGet($key)
94
+    public function offsetGet( $key )
95 95
     {
96
-        return property_exists($this, $key)
96
+        return property_exists( $this, $key )
97 97
             ? $this->$key
98
-            : Arr::get($this->custom, $key, null);
98
+            : Arr::get( $this->custom, $key, null );
99 99
     }
100 100
 
101 101
     /**
@@ -103,14 +103,14 @@  discard block
 block discarded – undo
103 103
      * @param mixed $value
104 104
      * @return void
105 105
      */
106
-    public function offsetSet($key, $value)
106
+    public function offsetSet( $key, $value )
107 107
     {
108
-        if (property_exists($this, $key)) {
108
+        if( property_exists( $this, $key ) ) {
109 109
             $this->$key = $value;
110 110
             return;
111 111
         }
112
-        if (!is_array($this->custom)) {
113
-            $this->custom = array_filter((array) $this->custom);
112
+        if( !is_array( $this->custom ) ) {
113
+            $this->custom = array_filter( (array)$this->custom );
114 114
         }
115 115
         $this->custom[$key] = $value;
116 116
     }
@@ -119,9 +119,9 @@  discard block
 block discarded – undo
119 119
      * @param mixed $key
120 120
      * @return void
121 121
      */
122
-    public function offsetUnset($key)
122
+    public function offsetUnset( $key )
123 123
     {
124
-        $this->offsetSet($key, null);
124
+        $this->offsetSet( $key, null );
125 125
     }
126 126
 
127 127
     /**
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
     /**
136 136
      * @return bool
137 137
      */
138
-    protected function isModified(array $properties)
138
+    protected function isModified( array $properties )
139 139
     {
140 140
         return $this->date != $properties['date']
141 141
             || $this->content != $properties['content']
@@ -145,39 +145,39 @@  discard block
 block discarded – undo
145 145
     /**
146 146
      * @return void
147 147
      */
148
-    protected function setProperties(WP_Post $post)
148
+    protected function setProperties( WP_Post $post )
149 149
     {
150 150
         $defaults = [
151
-            'author' => __('Anonymous', 'site-reviews'),
151
+            'author' => __( 'Anonymous', 'site-reviews' ),
152 152
             'date' => '',
153 153
             'review_id' => '',
154 154
             'review_type' => 'local',
155 155
         ];
156 156
         $meta = array_filter(
157
-            array_map('array_shift', array_filter((array) get_post_meta($post->ID))),
157
+            array_map( 'array_shift', array_filter( (array)get_post_meta( $post->ID ) ) ),
158 158
             'strlen'
159 159
         );
160
-        $meta = array_merge($defaults, Arr::unprefixArrayKeys($meta));
161
-        $properties = glsr(CreateReviewDefaults::class)->restrict(array_merge($defaults, $meta));
162
-        $this->modified = $this->isModified($properties);
163
-        array_walk($properties, function ($value, $key) {
164
-            if (!property_exists($this, $key) || isset($this->$key)) {
160
+        $meta = array_merge( $defaults, Arr::unprefixArrayKeys( $meta ) );
161
+        $properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ) );
162
+        $this->modified = $this->isModified( $properties );
163
+        array_walk( $properties, function( $value, $key ) {
164
+            if( !property_exists( $this, $key ) || isset($this->$key) ) {
165 165
                 return;
166 166
             }
167
-            $this->$key = maybe_unserialize($value);
167
+            $this->$key = maybe_unserialize( $value );
168 168
         });
169 169
     }
170 170
 
171 171
     /**
172 172
      * @return void
173 173
      */
174
-    protected function setTermIds(WP_Post $post)
174
+    protected function setTermIds( WP_Post $post )
175 175
     {
176 176
         $this->term_ids = [];
177
-        if (!is_array($terms = get_the_terms($post, Application::TAXONOMY))) {
177
+        if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ) ) ) {
178 178
             return;
179 179
         }
180
-        foreach ($terms as $term) {
180
+        foreach( $terms as $term ) {
181 181
             $this->term_ids[] = $term->term_id;
182 182
         }
183 183
     }
Please login to merge, or discard this patch.
plugin/Modules/Email.php 1 patch
Spacing   +52 added lines, -52 removed lines patch added patch discarded remove patch
@@ -42,15 +42,15 @@  discard block
 block discarded – undo
42 42
     /**
43 43
      * @return Email
44 44
      */
45
-    public function compose(array $email)
45
+    public function compose( array $email )
46 46
     {
47
-        $this->normalize($email);
47
+        $this->normalize( $email );
48 48
         $this->attachments = $this->email['attachments'];
49 49
         $this->headers = $this->buildHeaders();
50 50
         $this->message = $this->buildHtmlMessage();
51 51
         $this->subject = $this->email['subject'];
52 52
         $this->to = $this->email['to'];
53
-        add_action('phpmailer_init', [$this, 'buildPlainTextMessage']);
53
+        add_action( 'phpmailer_init', [$this, 'buildPlainTextMessage'] );
54 54
         return $this;
55 55
     }
56 56
 
@@ -58,11 +58,11 @@  discard block
 block discarded – undo
58 58
      * @param string $format
59 59
      * @return string|null
60 60
      */
61
-    public function read($format = '')
61
+    public function read( $format = '' )
62 62
     {
63
-        if ('plaintext' == $format) {
64
-            $message = $this->stripHtmlTags($this->message);
65
-            return apply_filters('site-reviews/email/message', $message, 'text', $this);
63
+        if( 'plaintext' == $format ) {
64
+            $message = $this->stripHtmlTags( $this->message );
65
+            return apply_filters( 'site-reviews/email/message', $message, 'text', $this );
66 66
         }
67 67
         return $this->message;
68 68
     }
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
      */
73 73
     public function send()
74 74
     {
75
-        if (!$this->message || !$this->subject || !$this->to) {
75
+        if( !$this->message || !$this->subject || !$this->to ) {
76 76
             return;
77 77
         }
78
-        add_action('wp_mail_failed', [$this, 'logMailError']);
78
+        add_action( 'wp_mail_failed', [$this, 'logMailError'] );
79 79
         $sent = wp_mail(
80 80
             $this->to,
81 81
             $this->subject,
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
             $this->headers,
84 84
             $this->attachments
85 85
         );
86
-        remove_action('wp_mail_failed', [$this, 'logMailError']);
86
+        remove_action( 'wp_mail_failed', [$this, 'logMailError'] );
87 87
         $this->reset();
88 88
         return $sent;
89 89
     }
@@ -92,16 +92,16 @@  discard block
 block discarded – undo
92 92
      * @return void
93 93
      * @action phpmailer_init
94 94
      */
95
-    public function buildPlainTextMessage(PHPMailer $phpmailer)
95
+    public function buildPlainTextMessage( PHPMailer $phpmailer )
96 96
     {
97
-        if (empty($this->email)) {
97
+        if( empty($this->email) ) {
98 98
             return;
99 99
         }
100
-        if ('text/plain' === $phpmailer->ContentType || !empty($phpmailer->AltBody)) {
100
+        if( 'text/plain' === $phpmailer->ContentType || !empty($phpmailer->AltBody) ) {
101 101
             return;
102 102
         }
103
-        $message = $this->stripHtmlTags($phpmailer->Body);
104
-        $phpmailer->AltBody = apply_filters('site-reviews/email/message', $message, 'text', $this);
103
+        $message = $this->stripHtmlTags( $phpmailer->Body );
104
+        $phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
105 105
     }
106 106
 
107 107
     /**
@@ -112,14 +112,14 @@  discard block
 block discarded – undo
112 112
         $allowed = [
113 113
             'bcc', 'cc', 'from', 'reply-to',
114 114
         ];
115
-        $headers = array_intersect_key($this->email, array_flip($allowed));
116
-        $headers = array_filter($headers);
117
-        foreach ($headers as $key => $value) {
115
+        $headers = array_intersect_key( $this->email, array_flip( $allowed ) );
116
+        $headers = array_filter( $headers );
117
+        foreach( $headers as $key => $value ) {
118 118
             unset($headers[$key]);
119 119
             $headers[] = $key.': '.$value;
120 120
         }
121 121
         $headers[] = 'Content-Type: text/html';
122
-        return apply_filters('site-reviews/email/headers', $headers, $this);
122
+        return apply_filters( 'site-reviews/email/headers', $headers, $this );
123 123
     }
124 124
 
125 125
     /**
@@ -127,54 +127,54 @@  discard block
 block discarded – undo
127 127
      */
128 128
     protected function buildHtmlMessage()
129 129
     {
130
-        $template = trim(glsr(OptionManager::class)->get('settings.general.notification_message'));
131
-        if (!empty($template)) {
132
-            $message = glsr(Template::class)->interpolate(
130
+        $template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) );
131
+        if( !empty($template) ) {
132
+            $message = glsr( Template::class )->interpolate(
133 133
                 $template, 
134 134
                 ['context' => $this->email['template-tags']], 
135 135
                 $this->email['template']
136 136
             );
137
-        } elseif ($this->email['template']) {
138
-            $message = glsr(Template::class)->build('templates/'.$this->email['template'], [
137
+        } elseif( $this->email['template'] ) {
138
+            $message = glsr( Template::class )->build( 'templates/'.$this->email['template'], [
139 139
                 'context' => $this->email['template-tags'],
140
-            ]);
140
+            ] );
141 141
         }
142
-        if (!isset($message)) {
142
+        if( !isset($message) ) {
143 143
             $message = $this->email['message'];
144 144
         }
145 145
         $message = $this->email['before'].$message.$this->email['after'];
146
-        $message = strip_shortcodes($message);
147
-        $message = wptexturize($message);
148
-        $message = wpautop($message);
149
-        $message = str_replace('&lt;&gt; ', '', $message);
150
-        $message = str_replace(']]>', ']]&gt;', $message);
151
-        $message = glsr(Template::class)->build('partials/email/index', [
146
+        $message = strip_shortcodes( $message );
147
+        $message = wptexturize( $message );
148
+        $message = wpautop( $message );
149
+        $message = str_replace( '&lt;&gt; ', '', $message );
150
+        $message = str_replace( ']]>', ']]&gt;', $message );
151
+        $message = glsr( Template::class )->build( 'partials/email/index', [
152 152
             'context' => ['message' => $message],
153
-        ]);
154
-        return apply_filters('site-reviews/email/message', stripslashes($message), 'html', $this);
153
+        ] );
154
+        return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this );
155 155
     }
156 156
 
157 157
     /**
158 158
      * @param \WP_Error $error
159 159
      * @return void
160 160
      */
161
-    protected function logMailError($error)
161
+    protected function logMailError( $error )
162 162
     {
163
-        glsr_log()->error('Email was not sent (wp_mail failed)')
164
-            ->debug($this)
165
-            ->debug($error);
163
+        glsr_log()->error( 'Email was not sent (wp_mail failed)' )
164
+            ->debug( $this )
165
+            ->debug( $error );
166 166
     }
167 167
 
168 168
     /**
169 169
      * @return void
170 170
      */
171
-    protected function normalize(array $email = [])
171
+    protected function normalize( array $email = [] )
172 172
     {
173
-        $email = shortcode_atts(glsr(EmailDefaults::class)->defaults(), $email);
174
-        if (empty($email['reply-to'])) {
173
+        $email = shortcode_atts( glsr( EmailDefaults::class )->defaults(), $email );
174
+        if( empty($email['reply-to']) ) {
175 175
             $email['reply-to'] = $email['from'];
176 176
         }
177
-        $this->email = apply_filters('site-reviews/email/compose', $email, $this);
177
+        $this->email = apply_filters( 'site-reviews/email/compose', $email, $this );
178 178
     }
179 179
 
180 180
     /**
@@ -193,21 +193,21 @@  discard block
 block discarded – undo
193 193
     /**
194 194
      * @return string
195 195
      */
196
-    protected function stripHtmlTags($string)
196
+    protected function stripHtmlTags( $string )
197 197
     {
198 198
         // remove invisible elements
199
-        $string = preg_replace('@<(embed|head|noembed|noscript|object|script|style)[^>]*?>.*?</\\1>@siu', '', $string);
199
+        $string = preg_replace( '@<(embed|head|noembed|noscript|object|script|style)[^>]*?>.*?</\\1>@siu', '', $string );
200 200
         // replace certain elements with a line-break
201
-        $string = preg_replace('@</(div|h[1-9]|p|pre|tr)@iu', "\r\n\$0", $string);
201
+        $string = preg_replace( '@</(div|h[1-9]|p|pre|tr)@iu', "\r\n\$0", $string );
202 202
         // replace other elements with a space
203
-        $string = preg_replace('@</(td|th)@iu', ' $0', $string);
203
+        $string = preg_replace( '@</(td|th)@iu', ' $0', $string );
204 204
         // add a placeholder for plain-text bullets to list elements
205
-        $string = preg_replace('@<(li)[^>]*?>@siu', '$0-o-^-o-', $string);
205
+        $string = preg_replace( '@<(li)[^>]*?>@siu', '$0-o-^-o-', $string );
206 206
         // strip all remaining HTML tags
207
-        $string = wp_strip_all_tags($string);
208
-        $string = wp_specialchars_decode($string, ENT_QUOTES);
209
-        $string = preg_replace('/\v(?:[\v\h]+){2,}/', "\r\n\r\n", $string);
210
-        $string = str_replace('-o-^-o-', ' - ', $string);
211
-        return html_entity_decode($string, ENT_QUOTES, 'UTF-8');
207
+        $string = wp_strip_all_tags( $string );
208
+        $string = wp_specialchars_decode( $string, ENT_QUOTES );
209
+        $string = preg_replace( '/\v(?:[\v\h]+){2,}/', "\r\n\r\n", $string );
210
+        $string = str_replace( '-o-^-o-', ' - ', $string );
211
+        return html_entity_decode( $string, ENT_QUOTES, 'UTF-8' );
212 212
     }
213 213
 }
Please login to merge, or discard this patch.
plugin/Shortcodes/SiteReviewsPopup.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -10,71 +10,71 @@
 block discarded – undo
10 10
     public function fields()
11 11
     {
12 12
         return [[
13
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
13
+            'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
14 14
             'minWidth' => 320,
15 15
             'type' => 'container',
16 16
         ], [
17
-            'label' => esc_html__('Title', 'site-reviews'),
17
+            'label' => esc_html__( 'Title', 'site-reviews' ),
18 18
             'name' => 'title',
19
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
19
+            'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
20 20
             'type' => 'textbox',
21 21
         ], [
22
-            'label' => esc_html__('Display', 'site-reviews'),
22
+            'label' => esc_html__( 'Display', 'site-reviews' ),
23 23
             'maxLength' => 5,
24 24
             'name' => 'display',
25 25
             'size' => 3,
26 26
             'text' => '10',
27
-            'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
27
+            'tooltip' => __( 'How many reviews would you like to display (default: 10)?', 'site-reviews' ),
28 28
             'type' => 'textbox',
29 29
         ], [
30
-            'label' => esc_html__('Rating', 'site-reviews'),
30
+            'label' => esc_html__( 'Rating', 'site-reviews' ),
31 31
             'name' => 'rating',
32 32
             'options' => [
33
-                '5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
34
-                '4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
35
-                '3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
36
-                '2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
37
-                '1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
38
-                '0' => esc_html(__('Unrated', 'site-reviews')),
33
+                '5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 ) ),
34
+                '4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 ) ),
35
+                '3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 ) ),
36
+                '2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ) ),
37
+                '1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ) ),
38
+                '0' => esc_html( __( 'Unrated', 'site-reviews' ) ),
39 39
             ],
40
-            'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
40
+            'tooltip' => __( 'What is the minimum rating to display (default: 1 star)?', 'site-reviews' ),
41 41
             'type' => 'listbox',
42 42
         ], [
43
-            'label' => esc_html__('Pagination', 'site-reviews'),
43
+            'label' => esc_html__( 'Pagination', 'site-reviews' ),
44 44
             'name' => 'pagination',
45 45
             'options' => [
46
-                'true' => esc_html__('Enable', 'site-reviews'),
47
-                'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
48
-                'false' => esc_html__('Disable', 'site-reviews'),
46
+                'true' => esc_html__( 'Enable', 'site-reviews' ),
47
+                'ajax' => esc_html__( 'Enable (using ajax)', 'site-reviews' ),
48
+                'false' => esc_html__( 'Disable', 'site-reviews' ),
49 49
             ],
50
-            'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
50
+            'tooltip' => __( 'When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews' ),
51 51
             'type' => 'listbox',
52 52
         ],
53
-        $this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
54
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
53
+        $this->getTypes( __( 'Which type of review would you like to display?', 'site-reviews' ) ),
54
+        $this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' ) ),
55 55
         [
56
-            'label' => esc_html__('Assigned To', 'site-reviews'),
56
+            'label' => esc_html__( 'Assigned To', 'site-reviews' ),
57 57
             'name' => 'assigned_to',
58
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
58
+            'tooltip' => __( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews' ),
59 59
             'type' => 'textbox',
60 60
         ], [
61
-            'label' => esc_html__('Schema', 'site-reviews'),
61
+            'label' => esc_html__( 'Schema', 'site-reviews' ),
62 62
             'name' => 'schema',
63 63
             'options' => [
64
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
65
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
64
+                'true' => esc_html__( 'Enable rich snippets', 'site-reviews' ),
65
+                'false' => esc_html__( 'Disable rich snippets', 'site-reviews' ),
66 66
             ],
67
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
67
+            'tooltip' => __( 'Rich snippets are disabled by default.', 'site-reviews' ),
68 68
             'type' => 'listbox',
69 69
         ], [
70
-            'label' => esc_html__('Classes', 'site-reviews'),
70
+            'label' => esc_html__( 'Classes', 'site-reviews' ),
71 71
             'name' => 'class',
72
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
72
+            'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
73 73
             'type' => 'textbox',
74 74
         ], [
75 75
             'columns' => 2,
76 76
             'items' => $this->getHideOptions(),
77
-            'label' => esc_html__('Hide', 'site-reviews'),
77
+            'label' => esc_html__( 'Hide', 'site-reviews' ),
78 78
             'layout' => 'grid',
79 79
             'spacing' => 5,
80 80
             'type' => 'container',
Please login to merge, or discard this patch.