@@ -16,7 +16,7 @@ discard block |
||
| 16 | 16 | */ |
| 17 | 17 | public function callbackRegisterSettings( $input ) |
| 18 | 18 | { |
| 19 | - if( !is_array( $input )) { |
|
| 19 | + if( !is_array( $input ) ) { |
|
| 20 | 20 | $input = ['settings' => []]; |
| 21 | 21 | } |
| 22 | 22 | if( key( $input ) == 'settings' ) { |
@@ -24,7 +24,7 @@ discard block |
||
| 24 | 24 | $options = $this->sanitizeGeneral( $input, $options ); |
| 25 | 25 | $options = $this->sanitizeSubmissions( $input, $options ); |
| 26 | 26 | $options = $this->sanitizeTranslations( $input, $options ); |
| 27 | - glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' )); |
|
| 27 | + glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) ); |
|
| 28 | 28 | return $options; |
| 29 | 29 | } |
| 30 | 30 | return $input; |
@@ -38,7 +38,7 @@ discard block |
||
| 38 | 38 | { |
| 39 | 39 | register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [ |
| 40 | 40 | 'sanitize_callback' => [$this, 'callbackRegisterSettings'], |
| 41 | - ]); |
|
| 41 | + ] ); |
|
| 42 | 42 | } |
| 43 | 43 | |
| 44 | 44 | /** |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | protected function sanitizeGeneral( array $input, array $options ) |
| 48 | 48 | { |
| 49 | 49 | $inputForm = $input['settings']['general']; |
| 50 | - if( !isset( $inputForm['notifications'] )) { |
|
| 50 | + if( !isset($inputForm['notifications']) ) { |
|
| 51 | 51 | $options['settings']['general']['notifications'] = []; |
| 52 | 52 | } |
| 53 | 53 | if( trim( $inputForm['notification_message'] ) == '' ) { |
@@ -62,7 +62,7 @@ discard block |
||
| 62 | 62 | protected function sanitizeSubmissions( array $input, array $options ) |
| 63 | 63 | { |
| 64 | 64 | $inputForm = $input['settings']['submissions']; |
| 65 | - if( !isset( $inputForm['required'] )) { |
|
| 65 | + if( !isset($inputForm['required']) ) { |
|
| 66 | 66 | $options['settings']['submissions']['required'] = []; |
| 67 | 67 | } |
| 68 | 68 | return $options; |
@@ -73,13 +73,13 @@ discard block |
||
| 73 | 73 | */ |
| 74 | 74 | protected function sanitizeTranslations( array $input, array $options ) |
| 75 | 75 | { |
| 76 | - if( isset( $input['settings']['strings'] )) { |
|
| 77 | - $options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] )); |
|
| 76 | + if( isset($input['settings']['strings']) ) { |
|
| 77 | + $options['settings']['strings'] = array_values( array_filter( $input['settings']['strings'] ) ); |
|
| 78 | 78 | array_walk( $options['settings']['strings'], function( &$string ) { |
| 79 | - if( isset( $string['s2'] )) { |
|
| 79 | + if( isset($string['s2']) ) { |
|
| 80 | 80 | $string['s2'] = wp_strip_all_tags( $string['s2'] ); |
| 81 | 81 | } |
| 82 | - if( isset( $string['p2'] )) { |
|
| 82 | + if( isset($string['p2']) ) { |
|
| 83 | 83 | $string['p2'] = wp_strip_all_tags( $string['p2'] ); |
| 84 | 84 | } |
| 85 | 85 | }); |
@@ -35,13 +35,13 @@ discard block |
||
| 35 | 35 | 'post_type' => Application::POST_TYPE, |
| 36 | 36 | ]; |
| 37 | 37 | $postId = wp_insert_post( $post, true ); |
| 38 | - if( is_wp_error( $postId )) { |
|
| 38 | + if( is_wp_error( $postId ) ) { |
|
| 39 | 39 | glsr_log()->error( $postId->get_error_message() )->info( $post ); |
| 40 | 40 | return false; |
| 41 | 41 | } |
| 42 | 42 | $this->setTerms( $postId, $command->category ); |
| 43 | 43 | do_action( 'site-reviews/create/review', $post, $review, $postId ); |
| 44 | - return $this->single( get_post( $postId )); |
|
| 44 | + return $this->single( get_post( $postId ) ); |
|
| 45 | 45 | } |
| 46 | 46 | |
| 47 | 47 | /** |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | */ |
| 51 | 51 | public function delete( $metaReviewId ) |
| 52 | 52 | { |
| 53 | - if( $postId = $this->getPostId( $metaReviewId )) { |
|
| 53 | + if( $postId = $this->getPostId( $metaReviewId ) ) { |
|
| 54 | 54 | wp_delete_post( $postId, true ); |
| 55 | 55 | } |
| 56 | 56 | } |
@@ -72,7 +72,7 @@ discard block |
||
| 72 | 72 | $paged = glsr( QueryBuilder::class )->getPaged( |
| 73 | 73 | wp_validate_boolean( $args['pagination'] ) |
| 74 | 74 | ); |
| 75 | - $reviews = new WP_Query([ |
|
| 75 | + $reviews = new WP_Query( [ |
|
| 76 | 76 | 'meta_key' => 'pinned', |
| 77 | 77 | 'meta_query' => $metaQuery, |
| 78 | 78 | 'offset' => $args['offset'], |
@@ -85,7 +85,7 @@ discard block |
||
| 85 | 85 | 'post_type' => Application::POST_TYPE, |
| 86 | 86 | 'posts_per_page' => $args['count'], |
| 87 | 87 | 'tax_query' => $taxQuery, |
| 88 | - ]); |
|
| 88 | + ] ); |
|
| 89 | 89 | return (object)[ |
| 90 | 90 | 'results' => array_map( [$this, 'single'], $reviews->posts ), |
| 91 | 91 | 'max_num_pages' => $reviews->max_num_pages, |
@@ -108,10 +108,10 @@ discard block |
||
| 108 | 108 | public function normalizeTerms( $commaSeparatedTermIds ) |
| 109 | 109 | { |
| 110 | 110 | $terms = []; |
| 111 | - $termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds ))); |
|
| 111 | + $termIds = array_filter( array_map( 'trim', explode( ',', $commaSeparatedTermIds ) ) ); |
|
| 112 | 112 | foreach( $termIds as $termId ) { |
| 113 | 113 | $term = get_term( $termId, Application::TAXONOMY ); |
| 114 | - if( !isset( $term->term_id ))continue; |
|
| 114 | + if( !isset($term->term_id) )continue; |
|
| 115 | 115 | $terms[] = $term->term_id; |
| 116 | 116 | } |
| 117 | 117 | return $terms; |
@@ -125,13 +125,13 @@ discard block |
||
| 125 | 125 | { |
| 126 | 126 | if( get_post_field( 'post_type', $postId ) != Application::POST_TYPE )return; |
| 127 | 127 | delete_post_meta( $postId, '_edit_last' ); |
| 128 | - $result = wp_update_post([ |
|
| 128 | + $result = wp_update_post( [ |
|
| 129 | 129 | 'ID' => $postId, |
| 130 | 130 | 'post_content' => get_post_meta( $postId, 'content', true ), |
| 131 | 131 | 'post_date' => get_post_meta( $postId, 'date', true ), |
| 132 | 132 | 'post_title' => get_post_meta( $postId, 'title', true ), |
| 133 | - ]); |
|
| 134 | - if( is_wp_error( $result )) { |
|
| 133 | + ] ); |
|
| 134 | + if( is_wp_error( $result ) ) { |
|
| 135 | 135 | glsr_log()->error( $result->get_error_message() ); |
| 136 | 136 | } |
| 137 | 137 | } |
@@ -153,7 +153,7 @@ discard block |
||
| 153 | 153 | protected function getNewPostStatus( array $review, $isBlacklisted ) |
| 154 | 154 | { |
| 155 | 155 | $requireApprovalOption = glsr( OptionManager::class )->get( 'settings.general.require.approval' ); |
| 156 | - return $review['review_type'] == 'local' && ( $requireApprovalOption == 'yes' || $isBlacklisted ) |
|
| 156 | + return $review['review_type'] == 'local' && ($requireApprovalOption == 'yes' || $isBlacklisted) |
|
| 157 | 157 | ? 'pending' |
| 158 | 158 | : 'publish'; |
| 159 | 159 | } |
@@ -166,9 +166,9 @@ discard block |
||
| 166 | 166 | protected function setTerms( $postId, $termIds ) |
| 167 | 167 | { |
| 168 | 168 | $terms = $this->normalizeTerms( $termIds ); |
| 169 | - if( empty( $terms ))return; |
|
| 169 | + if( empty($terms) )return; |
|
| 170 | 170 | $result = wp_set_object_terms( $postId, $terms, Application::TAXONOMY ); |
| 171 | - if( is_wp_error( $result )) { |
|
| 171 | + if( is_wp_error( $result ) ) { |
|
| 172 | 172 | glsr_log()->error( $result->get_error_message() ); |
| 173 | 173 | } |
| 174 | 174 | } |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | { |
| 67 | 67 | $instance = new static; |
| 68 | 68 | $instance->setTagFromMethod( $method ); |
| 69 | - call_user_func_array( [$instance, 'normalize'], $args += ['',''] ); |
|
| 69 | + call_user_func_array( [$instance, 'normalize'], $args += ['', ''] ); |
|
| 70 | 70 | $tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT ); |
| 71 | 71 | do_action_ref_array( 'site-reviews/builder', [&$instance] ); |
| 72 | 72 | $generatedTag = in_array( $instance->tag, $tags ) |
@@ -90,8 +90,8 @@ discard block |
||
| 90 | 90 | 'render' => 'is_bool', |
| 91 | 91 | 'tag' => 'is_string', |
| 92 | 92 | ]; |
| 93 | - if( !isset( $properties[$property] ) |
|
| 94 | - || empty( array_filter( [$value], $properties[$property] )) |
|
| 93 | + if( !isset($properties[$property]) |
|
| 94 | + || empty(array_filter( [$value], $properties[$property] )) |
|
| 95 | 95 | )return; |
| 96 | 96 | $this->$property = $value; |
| 97 | 97 | } |
@@ -101,7 +101,7 @@ discard block |
||
| 101 | 101 | */ |
| 102 | 102 | public function getClosingTag() |
| 103 | 103 | { |
| 104 | - if( empty( $this->tag ))return; |
|
| 104 | + if( empty($this->tag) )return; |
|
| 105 | 105 | return '</'.$this->tag.'>'; |
| 106 | 106 | } |
| 107 | 107 | |
@@ -110,8 +110,8 @@ discard block |
||
| 110 | 110 | */ |
| 111 | 111 | public function getOpeningTag() |
| 112 | 112 | { |
| 113 | - if( empty( $this->tag ))return; |
|
| 114 | - $attributes = glsr( Attributes::class )->{$this->tag}( $this->args )->toString(); |
|
| 113 | + if( empty($this->tag) )return; |
|
| 114 | + $attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString(); |
|
| 115 | 115 | return '<'.trim( $this->tag.' '.$attributes ).'>'; |
| 116 | 116 | } |
| 117 | 117 | |
@@ -120,8 +120,8 @@ discard block |
||
| 120 | 120 | */ |
| 121 | 121 | public function raw( array $field ) |
| 122 | 122 | { |
| 123 | - unset( $field['label'] ); |
|
| 124 | - return $this->{$field['type']}( $field ); |
|
| 123 | + unset($field['label']); |
|
| 124 | + return $this->{$field['type']}($field); |
|
| 125 | 125 | } |
| 126 | 126 | |
| 127 | 127 | /** |
@@ -130,7 +130,7 @@ discard block |
||
| 130 | 130 | protected function buildCustomField() |
| 131 | 131 | { |
| 132 | 132 | $className = $this->getCustomFieldClassName(); |
| 133 | - if( class_exists( $className )) { |
|
| 133 | + if( class_exists( $className ) ) { |
|
| 134 | 134 | return (new $className( $this ))->build(); |
| 135 | 135 | } |
| 136 | 136 | glsr_log()->error( 'Field missing: '.$className ); |
@@ -141,7 +141,7 @@ discard block |
||
| 141 | 141 | */ |
| 142 | 142 | protected function buildDefaultTag( $text = '' ) |
| 143 | 143 | { |
| 144 | - if( empty( $text )) { |
|
| 144 | + if( empty($text) ) { |
|
| 145 | 145 | $text = $this->args['text']; |
| 146 | 146 | } |
| 147 | 147 | return $this->getOpeningTag().$text.$this->getClosingTag(); |
@@ -152,7 +152,7 @@ discard block |
||
| 152 | 152 | */ |
| 153 | 153 | protected function buildFieldDescription() |
| 154 | 154 | { |
| 155 | - if( empty( $this->args['description'] ))return; |
|
| 155 | + if( empty($this->args['description']) )return; |
|
| 156 | 156 | if( $this->args['is_widget'] ) { |
| 157 | 157 | return $this->small( $this->args['description'] ); |
| 158 | 158 | } |
@@ -164,13 +164,13 @@ discard block |
||
| 164 | 164 | */ |
| 165 | 165 | protected function buildFormInput() |
| 166 | 166 | { |
| 167 | - if( !in_array( $this->args['type'], ['checkbox', 'radio'] )) { |
|
| 168 | - if( isset( $this->args['multiple'] )) { |
|
| 169 | - $this->args['name'].= '[]'; |
|
| 167 | + if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) { |
|
| 168 | + if( isset($this->args['multiple']) ) { |
|
| 169 | + $this->args['name'] .= '[]'; |
|
| 170 | 170 | } |
| 171 | 171 | return $this->buildFormLabel().$this->getOpeningTag(); |
| 172 | 172 | } |
| 173 | - return empty( $this->args['options'] ) |
|
| 173 | + return empty($this->args['options']) |
|
| 174 | 174 | ? $this->buildFormInputChoice() |
| 175 | 175 | : $this->buildFormInputMultiChoice(); |
| 176 | 176 | } |
@@ -180,19 +180,19 @@ discard block |
||
| 180 | 180 | */ |
| 181 | 181 | protected function buildFormInputChoice() |
| 182 | 182 | { |
| 183 | - if( !empty( $this->args['text'] )) { |
|
| 183 | + if( !empty($this->args['text']) ) { |
|
| 184 | 184 | $this->args['label'] = $this->args['text']; |
| 185 | 185 | } |
| 186 | 186 | if( !$this->args['is_public'] ) { |
| 187 | - return $this->buildFormLabel([ |
|
| 187 | + return $this->buildFormLabel( [ |
|
| 188 | 188 | 'class' => 'glsr-'.$this->args['type'].'-label', |
| 189 | 189 | 'text' => $this->getOpeningTag().' '.$this->args['label'].'<span></span>', |
| 190 | - ]); |
|
| 190 | + ] ); |
|
| 191 | 191 | } |
| 192 | - return $this->getOpeningTag().$this->buildFormLabel([ |
|
| 192 | + return $this->getOpeningTag().$this->buildFormLabel( [ |
|
| 193 | 193 | 'class' => 'glsr-'.$this->args['type'].'-label', |
| 194 | 194 | 'text' => $this->args['label'].'<span></span>', |
| 195 | - ]); |
|
| 195 | + ] ); |
|
| 196 | 196 | } |
| 197 | 197 | |
| 198 | 198 | /** |
@@ -201,22 +201,22 @@ discard block |
||
| 201 | 201 | protected function buildFormInputMultiChoice() |
| 202 | 202 | { |
| 203 | 203 | if( $this->args['type'] == 'checkbox' ) { |
| 204 | - $this->args['name'].= '[]'; |
|
| 204 | + $this->args['name'] .= '[]'; |
|
| 205 | 205 | } |
| 206 | 206 | $index = 0; |
| 207 | - $options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use( &$index ) { |
|
| 207 | + $options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use(&$index) { |
|
| 208 | 208 | return $carry.$this->li( $this->{$this->args['type']}([ |
| 209 | 209 | 'checked' => in_array( $key, (array)$this->args['value'] ), |
| 210 | 210 | 'id' => $this->args['id'].'-'.$index++, |
| 211 | 211 | 'name' => $this->args['name'], |
| 212 | 212 | 'text' => $this->args['options'][$key], |
| 213 | 213 | 'value' => $key, |
| 214 | - ])); |
|
| 214 | + ]) ); |
|
| 215 | 215 | }); |
| 216 | 216 | return $this->ul( $options, [ |
| 217 | 217 | 'class' => $this->args['class'], |
| 218 | 218 | 'id' => $this->args['id'], |
| 219 | - ]); |
|
| 219 | + ] ); |
|
| 220 | 220 | } |
| 221 | 221 | |
| 222 | 222 | /** |
@@ -224,13 +224,13 @@ discard block |
||
| 224 | 224 | */ |
| 225 | 225 | protected function buildFormLabel( array $customArgs = [] ) |
| 226 | 226 | { |
| 227 | - if( empty( $this->args['label'] ) || $this->args['type'] == 'hidden' )return; |
|
| 227 | + if( empty($this->args['label']) || $this->args['type'] == 'hidden' )return; |
|
| 228 | 228 | return $this->label( wp_parse_args( $customArgs, [ |
| 229 | 229 | 'for' => $this->args['id'], |
| 230 | 230 | 'is_public' => $this->args['is_public'], |
| 231 | 231 | 'text' => $this->args['label'], |
| 232 | 232 | 'type' => $this->args['type'], |
| 233 | - ])); |
|
| 233 | + ] ) ); |
|
| 234 | 234 | } |
| 235 | 235 | |
| 236 | 236 | /** |
@@ -247,11 +247,11 @@ discard block |
||
| 247 | 247 | protected function buildFormSelectOptions() |
| 248 | 248 | { |
| 249 | 249 | return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) { |
| 250 | - return $carry.$this->option([ |
|
| 250 | + return $carry.$this->option( [ |
|
| 251 | 251 | 'selected' => $this->args['value'] == $key, |
| 252 | 252 | 'text' => $this->args['options'][$key], |
| 253 | 253 | 'value' => $key, |
| 254 | - ]); |
|
| 254 | + ] ); |
|
| 255 | 255 | }); |
| 256 | 256 | } |
| 257 | 257 | |
@@ -269,10 +269,10 @@ discard block |
||
| 269 | 269 | protected function buildTag() |
| 270 | 270 | { |
| 271 | 271 | $this->mergeArgsWithRequiredDefaults(); |
| 272 | - if( in_array( $this->tag, static::TAGS_SINGLE )) { |
|
| 272 | + if( in_array( $this->tag, static::TAGS_SINGLE ) ) { |
|
| 273 | 273 | return $this->getOpeningTag(); |
| 274 | 274 | } |
| 275 | - if( !in_array( $this->tag, static::TAGS_FORM )) { |
|
| 275 | + if( !in_array( $this->tag, static::TAGS_FORM ) ) { |
|
| 276 | 276 | return $this->buildDefaultTag(); |
| 277 | 277 | } |
| 278 | 278 | return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription(); |
@@ -292,7 +292,7 @@ discard block |
||
| 292 | 292 | protected function mergeArgsWithRequiredDefaults() |
| 293 | 293 | { |
| 294 | 294 | $className = $this->getCustomFieldClassName(); |
| 295 | - if( class_exists( $className )) { |
|
| 295 | + if( class_exists( $className ) ) { |
|
| 296 | 296 | $this->args = array_merge( |
| 297 | 297 | wp_parse_args( $this->args, $className::defaults() ), |
| 298 | 298 | $className::required() |
@@ -307,16 +307,16 @@ discard block |
||
| 307 | 307 | */ |
| 308 | 308 | protected function normalize( ...$params ) |
| 309 | 309 | { |
| 310 | - if( is_string( $params[0] ) || is_numeric( $params[0] )) { |
|
| 310 | + if( is_string( $params[0] ) || is_numeric( $params[0] ) ) { |
|
| 311 | 311 | $this->setNameOrTextAttributeForTag( $params[0] ); |
| 312 | 312 | } |
| 313 | - if( is_array( $params[0] )) { |
|
| 313 | + if( is_array( $params[0] ) ) { |
|
| 314 | 314 | $this->args += $params[0]; |
| 315 | 315 | } |
| 316 | - else if( is_array( $params[1] )) { |
|
| 316 | + else if( is_array( $params[1] ) ) { |
|
| 317 | 317 | $this->args += $params[1]; |
| 318 | 318 | } |
| 319 | - if( !isset( $this->args['is_public'] )) { |
|
| 319 | + if( !isset($this->args['is_public']) ) { |
|
| 320 | 320 | $this->args['is_public'] = false; |
| 321 | 321 | } |
| 322 | 322 | } |
@@ -340,7 +340,7 @@ discard block |
||
| 340 | 340 | protected function setTagFromMethod( $method ) |
| 341 | 341 | { |
| 342 | 342 | $this->tag = strtolower( $method ); |
| 343 | - if( in_array( $this->tag, static::INPUT_TYPES )) { |
|
| 343 | + if( in_array( $this->tag, static::INPUT_TYPES ) ) { |
|
| 344 | 344 | $this->args['type'] = $this->tag; |
| 345 | 345 | $this->tag = 'input'; |
| 346 | 346 | } |
@@ -29,7 +29,7 @@ discard block |
||
| 29 | 29 | */ |
| 30 | 30 | protected function getFieldDefault( array $field ) |
| 31 | 31 | { |
| 32 | - return isset( $field['default'] ) |
|
| 32 | + return isset($field['default']) |
|
| 33 | 33 | ? $field['default'] |
| 34 | 34 | : ''; |
| 35 | 35 | } |
@@ -40,7 +40,7 @@ discard block |
||
| 40 | 40 | protected function getSettingFields( $path ) |
| 41 | 41 | { |
| 42 | 42 | $settings = glsr( DefaultsManager::class )->settings(); |
| 43 | - return array_filter( $settings, function( $key ) use( $path ) { |
|
| 43 | + return array_filter( $settings, function( $key ) use($path) { |
|
| 44 | 44 | return glsr( Helper::class )->startsWith( $path, $key ); |
| 45 | 45 | }, ARRAY_FILTER_USE_KEY ); |
| 46 | 46 | } |
@@ -55,8 +55,8 @@ discard block |
||
| 55 | 55 | $field = wp_parse_args( $field, [ |
| 56 | 56 | 'is_setting' => true, |
| 57 | 57 | 'name' => $name, |
| 58 | - ]); |
|
| 59 | - $rows.= new Field( $this->normalize( $field )); |
|
| 58 | + ] ); |
|
| 59 | + $rows .= new Field( $this->normalize( $field ) ); |
|
| 60 | 60 | } |
| 61 | 61 | return $rows; |
| 62 | 62 | } |
@@ -67,7 +67,7 @@ discard block |
||
| 67 | 67 | */ |
| 68 | 68 | protected function getTemplateData( $id ) |
| 69 | 69 | { |
| 70 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
| 70 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
| 71 | 71 | return [ |
| 72 | 72 | 'context' => [ |
| 73 | 73 | 'rows' => $this->getSettingRows( $fields ), |
@@ -81,12 +81,12 @@ discard block |
||
| 81 | 81 | */ |
| 82 | 82 | protected function getTemplateDataForAddons( $id ) |
| 83 | 83 | { |
| 84 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
| 84 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
| 85 | 85 | $settings = glsr( Helper::class )->convertDotNotationArray( $fields ); |
| 86 | 86 | $settingKeys = array_keys( $settings['settings']['addons'] ); |
| 87 | 87 | $results = []; |
| 88 | 88 | foreach( $settingKeys as $key ) { |
| 89 | - $addonFields = array_filter( $fields, function( $path ) use( $key ) { |
|
| 89 | + $addonFields = array_filter( $fields, function( $path ) use($key) { |
|
| 90 | 90 | return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path ); |
| 91 | 91 | }, ARRAY_FILTER_USE_KEY ); |
| 92 | 92 | $results[$key] = $this->getSettingRows( $addonFields ); |
@@ -102,7 +102,7 @@ discard block |
||
| 102 | 102 | protected function getTemplateDataForTranslations() |
| 103 | 103 | { |
| 104 | 104 | $translations = glsr( Translator::class )->renderAll(); |
| 105 | - $class = empty( $translations ) |
|
| 105 | + $class = empty($translations) |
|
| 106 | 106 | ? 'glsr-hidden' |
| 107 | 107 | : ''; |
| 108 | 108 | return [ |
@@ -125,9 +125,9 @@ discard block |
||
| 125 | 125 | $path, |
| 126 | 126 | glsr( Helper::class )->getPathValue( $path, glsr()->defaults ) |
| 127 | 127 | ); |
| 128 | - if( is_array( $expectedValue )) { |
|
| 128 | + if( is_array( $expectedValue ) ) { |
|
| 129 | 129 | return is_array( $optionValue ) |
| 130 | - ? count( array_intersect( $optionValue, $expectedValue )) === 0 |
|
| 130 | + ? count( array_intersect( $optionValue, $expectedValue ) ) === 0 |
|
| 131 | 131 | : !in_array( $optionValue, $expectedValue ); |
| 132 | 132 | } |
| 133 | 133 | return $optionValue != $expectedValue; |
@@ -149,17 +149,17 @@ discard block |
||
| 149 | 149 | */ |
| 150 | 150 | protected function normalizeDependsOn( array $field ) |
| 151 | 151 | { |
| 152 | - if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) { |
|
| 152 | + if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) { |
|
| 153 | 153 | $path = key( $field['depends_on'] ); |
| 154 | 154 | $expectedValue = $field['depends_on'][$path]; |
| 155 | 155 | $fieldName = glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ); |
| 156 | - if( is_array( $expectedValue )) { |
|
| 157 | - $fieldName.= '[]'; |
|
| 156 | + if( is_array( $expectedValue ) ) { |
|
| 157 | + $fieldName .= '[]'; |
|
| 158 | 158 | } |
| 159 | - $field['data-depends'] = json_encode([ |
|
| 159 | + $field['data-depends'] = json_encode( [ |
|
| 160 | 160 | 'name' => $fieldName, |
| 161 | 161 | 'value' => $expectedValue, |
| 162 | - ], JSON_HEX_APOS|JSON_HEX_QUOT ); |
|
| 162 | + ], JSON_HEX_APOS | JSON_HEX_QUOT ); |
|
| 163 | 163 | $field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue ); |
| 164 | 164 | } |
| 165 | 165 | return $field; |
@@ -170,9 +170,9 @@ discard block |
||
| 170 | 170 | */ |
| 171 | 171 | protected function normalizeLabelAndLegend( array $field ) |
| 172 | 172 | { |
| 173 | - if( !empty( $field['label'] )) { |
|
| 173 | + if( !empty($field['label']) ) { |
|
| 174 | 174 | $field['legend'] = $field['label']; |
| 175 | - unset( $field['label'] ); |
|
| 175 | + unset($field['label']); |
|
| 176 | 176 | } |
| 177 | 177 | else { |
| 178 | 178 | $field['is_valid'] = false; |
@@ -186,7 +186,7 @@ discard block |
||
| 186 | 186 | */ |
| 187 | 187 | protected function normalizeValue( array $field ) |
| 188 | 188 | { |
| 189 | - if( !isset( $field['value'] )) { |
|
| 189 | + if( !isset($field['value']) ) { |
|
| 190 | 190 | $field['value'] = glsr( OptionManager::class )->get( |
| 191 | 191 | $field['name'], |
| 192 | 192 | $this->getFieldDefault( $field ) |
@@ -11,13 +11,13 @@ discard block |
||
| 11 | 11 | */ |
| 12 | 12 | public function isBlacklisted( array $review ) |
| 13 | 13 | { |
| 14 | - $target = implode( "\n", array_filter([ |
|
| 14 | + $target = implode( "\n", array_filter( [ |
|
| 15 | 15 | $review['name'], |
| 16 | 16 | $review['content'], |
| 17 | 17 | $review['email'], |
| 18 | 18 | $review['ip_address'], |
| 19 | 19 | $review['title'], |
| 20 | - ])); |
|
| 20 | + ] ) ); |
|
| 21 | 21 | return (bool)apply_filters( 'site-reviews/blacklist/is-blacklisted', |
| 22 | 22 | $this->check( $target ), |
| 23 | 23 | $review |
@@ -30,16 +30,16 @@ discard block |
||
| 30 | 30 | */ |
| 31 | 31 | protected function check( $target ) |
| 32 | 32 | { |
| 33 | - $blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' )); |
|
| 34 | - if( empty( $blacklist )) { |
|
| 33 | + $blacklist = trim( glsr( OptionManager::class )->get( 'settings.submissions.blacklist.entries' ) ); |
|
| 34 | + if( empty($blacklist) ) { |
|
| 35 | 35 | return false; |
| 36 | 36 | } |
| 37 | 37 | $lines = explode( "\n", $blacklist ); |
| 38 | - foreach( (array) $lines as $line ) { |
|
| 38 | + foreach( (array)$lines as $line ) { |
|
| 39 | 39 | $line = trim( $line ); |
| 40 | - if( empty( $line ) || 256 < strlen( $line ))continue; |
|
| 41 | - $pattern = sprintf( '#%s#i', preg_quote( $line, '#' )); |
|
| 42 | - if( preg_match( $pattern, $target )) { |
|
| 40 | + if( empty($line) || 256 < strlen( $line ) )continue; |
|
| 41 | + $pattern = sprintf( '#%s#i', preg_quote( $line, '#' ) ); |
|
| 42 | + if( preg_match( $pattern, $target ) ) { |
|
| 43 | 43 | return true; |
| 44 | 44 | } |
| 45 | 45 | } |
@@ -32,7 +32,7 @@ discard block |
||
| 32 | 32 | public function __construct() |
| 33 | 33 | { |
| 34 | 34 | $this->types = glsr( OptionManager::class )->get( 'settings.general.notification', [] ); |
| 35 | - $this->email = count( array_intersect( ['admin', 'author', 'custom'], $types )) > 0; |
|
| 35 | + $this->email = count( array_intersect( ['admin', 'author', 'custom'], $types ) ) > 0; |
|
| 36 | 36 | $this->slack = in_array( 'slack', $types ); |
| 37 | 37 | } |
| 38 | 38 | |
@@ -41,7 +41,7 @@ discard block |
||
| 41 | 41 | */ |
| 42 | 42 | public function send( Review $review ) |
| 43 | 43 | { |
| 44 | - if( empty( $this->types ))return; |
|
| 44 | + if( empty($this->types) )return; |
|
| 45 | 45 | $this->review = $review; |
| 46 | 46 | $args = [ |
| 47 | 47 | 'link' => $this->getLink(), |
@@ -60,7 +60,7 @@ discard block |
||
| 60 | 60 | */ |
| 61 | 61 | protected function buildEmail( array $args ) |
| 62 | 62 | { |
| 63 | - return glsr( Email::class )->compose([ |
|
| 63 | + return glsr( Email::class )->compose( [ |
|
| 64 | 64 | 'to' => $this->getEmailAddresses(), |
| 65 | 65 | 'subject' => $args['title'], |
| 66 | 66 | 'template' => 'email-notification', |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | 'review_rating' => $this->review->rating, |
| 74 | 74 | 'review_title' => $this->review->title, |
| 75 | 75 | ], |
| 76 | - ]); |
|
| 76 | + ] ); |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | /** |
@@ -83,9 +83,9 @@ discard block |
||
| 83 | 83 | { |
| 84 | 84 | return glsr( Slack::class )->compose( $review, [ |
| 85 | 85 | 'fallback' => $this->buildEmail( $args )->read( 'plaintext' ), |
| 86 | - 'link' => sprintf( '<%s|%s>', $args['link'], __( 'View Review', 'site-reviews' )), |
|
| 86 | + 'link' => sprintf( '<%s|%s>', $args['link'], __( 'View Review', 'site-reviews' ) ), |
|
| 87 | 87 | 'pretext' => $args['title'], |
| 88 | - ]); |
|
| 88 | + ] ); |
|
| 89 | 89 | } |
| 90 | 90 | |
| 91 | 91 | /** |
@@ -94,22 +94,22 @@ discard block |
||
| 94 | 94 | protected function getEmailAddresses() |
| 95 | 95 | { |
| 96 | 96 | $emails = []; |
| 97 | - if( in_array( 'admin', $this->types )) { |
|
| 97 | + if( in_array( 'admin', $this->types ) ) { |
|
| 98 | 98 | $emails[] = get_option( 'admin_email' ); |
| 99 | 99 | } |
| 100 | - if( in_array( 'author', $this->types )) { |
|
| 101 | - $assignedPost = get_post( intval( $this->review->assigned_to )); |
|
| 100 | + if( in_array( 'author', $this->types ) ) { |
|
| 101 | + $assignedPost = get_post( intval( $this->review->assigned_to ) ); |
|
| 102 | 102 | if( $assignedPost instanceof WP_Post ) { |
| 103 | 103 | $emails[] = get_the_author_meta( 'user_email', $assignedPost->ID ); |
| 104 | 104 | } |
| 105 | 105 | } |
| 106 | - if( in_array( 'custom', $this->types )) { |
|
| 106 | + if( in_array( 'custom', $this->types ) ) { |
|
| 107 | 107 | $customEmails = glsr( OptionManager::class )->get( 'settings.general.notification_email' ); |
| 108 | 108 | $customEmails = str_replace( [' ', ',', ';'], ',', $customEmails ); |
| 109 | 109 | $customEmails = explode( ',', $customEmails ); |
| 110 | 110 | $emails = array_merge( $emails, $customEmails ); |
| 111 | 111 | } |
| 112 | - $emails = array_filter( array_keys( array_flip( $emails ))); |
|
| 112 | + $emails = array_filter( array_keys( array_flip( $emails ) ) ); |
|
| 113 | 113 | return apply_filters( 'site-reviews/notification/emails', $emails, $this->review ); |
| 114 | 114 | } |
| 115 | 115 | |
@@ -126,16 +126,16 @@ discard block |
||
| 126 | 126 | */ |
| 127 | 127 | protected function getTitle() |
| 128 | 128 | { |
| 129 | - $assignedTitle = get_the_title( intval( $this->review->assigned_to )); |
|
| 129 | + $assignedTitle = get_the_title( intval( $this->review->assigned_to ) ); |
|
| 130 | 130 | $title = _nx( |
| 131 | 131 | 'New %s-star review', |
| 132 | 132 | 'New %s-star review of: %s', |
| 133 | - intval( empty( $assignedTitle )), |
|
| 133 | + intval( empty($assignedTitle) ), |
|
| 134 | 134 | 'This string differs depending on whether or not the review has been assigned to a post.', |
| 135 | 135 | 'site-reviews' |
| 136 | 136 | ); |
| 137 | 137 | $title = sprintf( '[%s] %s', |
| 138 | - wp_specialchars_decode( strval( get_option( 'blogname' )), ENT_QUOTES ), |
|
| 138 | + wp_specialchars_decode( strval( get_option( 'blogname' ) ), ENT_QUOTES ), |
|
| 139 | 139 | sprintf( $title, $this->review->rating, $assignedTitle ) |
| 140 | 140 | ); |
| 141 | 141 | return apply_filters( 'site-reviews/notification/title', $title, $this->review ); |
@@ -147,7 +147,7 @@ discard block |
||
| 147 | 147 | protected function sendToEmail( array $args ) |
| 148 | 148 | { |
| 149 | 149 | $email = $this->buildEmail( $args ); |
| 150 | - if( empty( $email->to )) { |
|
| 150 | + if( empty($email->to) ) { |
|
| 151 | 151 | glsr_log()->error( 'Email notification was not sent: missing email address' ); |
| 152 | 152 | return; |
| 153 | 153 | } |
@@ -163,7 +163,7 @@ discard block |
||
| 163 | 163 | { |
| 164 | 164 | $notification = $this->buildSlackNotification( $args ); |
| 165 | 165 | $result = $notification->send(); |
| 166 | - if( is_wp_error( $result )) { |
|
| 166 | + if( is_wp_error( $result ) ) { |
|
| 167 | 167 | $notification->review = null; |
| 168 | 168 | glsr_log()->error( $result->get_error_message() )->debug( $notification ); |
| 169 | 169 | } |
@@ -41,7 +41,9 @@ |
||
| 41 | 41 | */ |
| 42 | 42 | public function send( Review $review ) |
| 43 | 43 | { |
| 44 | - if( empty( $this->types ))return; |
|
| 44 | + if( empty( $this->types )) { |
|
| 45 | + return; |
|
| 46 | + } |
|
| 45 | 47 | $this->review = $review; |
| 46 | 48 | $args = [ |
| 47 | 49 | 'link' => $this->getLink(), |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | */ |
| 36 | 36 | public function compose( Review $review, array $notification ) |
| 37 | 37 | { |
| 38 | - if( empty( $this->endpoint ))return; |
|
| 38 | + if( empty($this->endpoint) )return; |
|
| 39 | 39 | $this->review = $review; |
| 40 | 40 | $args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification ); |
| 41 | 41 | $notification = [ |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | */ |
| 58 | 58 | public function send() |
| 59 | 59 | { |
| 60 | - if( empty( $this->endpoint )) { |
|
| 60 | + if( empty($this->endpoint) ) { |
|
| 61 | 61 | return new WP_Error( 'slack', 'Slack notification was not sent: missing endpoint' ); |
| 62 | 62 | } |
| 63 | 63 | return wp_remote_post( $endpoint, [ |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | 'redirection' => 5, |
| 70 | 70 | 'sslverify' => false, |
| 71 | 71 | 'timeout' => 45, |
| 72 | - ]); |
|
| 72 | + ] ); |
|
| 73 | 73 | } |
| 74 | 74 | |
| 75 | 75 | /** |
@@ -77,11 +77,11 @@ discard block |
||
| 77 | 77 | */ |
| 78 | 78 | protected function buildAuthorField() |
| 79 | 79 | { |
| 80 | - $email = !empty( $this->review->email ) |
|
| 80 | + $email = !empty($this->review->email) |
|
| 81 | 81 | ? '<'.$this->review->email.'>' |
| 82 | 82 | : ''; |
| 83 | 83 | $author = trim( rtrim( $this->review->author ).' '.$email ); |
| 84 | - return ['value' => implode( ' - ', array_filter( [$author, $this->review->ip_address] ))]; |
|
| 84 | + return ['value' => implode( ' - ', array_filter( [$author, $this->review->ip_address] ) )]; |
|
| 85 | 85 | } |
| 86 | 86 | |
| 87 | 87 | /** |
@@ -89,7 +89,7 @@ discard block |
||
| 89 | 89 | */ |
| 90 | 90 | protected function buildContentField() |
| 91 | 91 | { |
| 92 | - return !empty( $this->review->content ) |
|
| 92 | + return !empty($this->review->content) |
|
| 93 | 93 | ? ['value' => $this->review->content] |
| 94 | 94 | : []; |
| 95 | 95 | } |
@@ -99,7 +99,7 @@ discard block |
||
| 99 | 99 | */ |
| 100 | 100 | protected function buildLinkField() |
| 101 | 101 | { |
| 102 | - return !empty( $this->link ) |
|
| 102 | + return !empty($this->link) |
|
| 103 | 103 | ? ['value' => $this->link] |
| 104 | 104 | : []; |
| 105 | 105 | } |
@@ -125,7 +125,7 @@ discard block |
||
| 125 | 125 | protected function buildStarsField() |
| 126 | 126 | { |
| 127 | 127 | $solidStars = str_repeat( '★', $this->review->rating ); |
| 128 | - $emptyStars = str_repeat( '☆', max( 0, Rating::MAX_RATING - $this->review->rating )); |
|
| 128 | + $emptyStars = str_repeat( '☆', max( 0, Rating::MAX_RATING - $this->review->rating ) ); |
|
| 129 | 129 | $stars = $solidStars.$emptyStars; |
| 130 | 130 | $stars = apply_filters( 'site-reviews/slack/stars', $stars, $this->review->rating, Rating::MAX_RATING ); |
| 131 | 131 | return ['title' => $stars]; |
@@ -136,7 +136,7 @@ discard block |
||
| 136 | 136 | */ |
| 137 | 137 | protected function buildTitleField() |
| 138 | 138 | { |
| 139 | - return !empty( $this->review->title ) |
|
| 139 | + return !empty($this->review->title) |
|
| 140 | 140 | ? ['title' => $this->review->title] |
| 141 | 141 | : []; |
| 142 | 142 | } |
@@ -35,7 +35,9 @@ |
||
| 35 | 35 | */ |
| 36 | 36 | public function compose( Review $review, array $notification ) |
| 37 | 37 | { |
| 38 | - if( empty( $this->endpoint ))return; |
|
| 38 | + if( empty( $this->endpoint )) { |
|
| 39 | + return; |
|
| 40 | + } |
|
| 39 | 41 | $this->review = $review; |
| 40 | 42 | $args = shortcode_atts( glsr( SlackDefaults::class )->defaults(), $notification ); |
| 41 | 43 | $notification = [ |
@@ -17,14 +17,14 @@ |
||
| 17 | 17 | $review = glsr( ReviewManager::class )->create( $command ); |
| 18 | 18 | if( !$review ) { |
| 19 | 19 | glsr( Session::class )->set( $command->form_id.'errors', [] ); |
| 20 | - glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' )); |
|
| 20 | + glsr( Session::class )->set( $command->form_id.'message', __( 'Your review could not be submitted and the error has been logged. Please notify the site admin.', 'site-reviews' ) ); |
|
| 21 | 21 | return; |
| 22 | 22 | } |
| 23 | - glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' )); |
|
| 23 | + glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' ) ); |
|
| 24 | 24 | glsr( Notification::class )->send( $review ); |
| 25 | 25 | do_action( 'site-reviews/local/review/submitted', $review ); |
| 26 | 26 | if( $command->ajax_request )return; |
| 27 | - if( empty( $command->referer )) { |
|
| 27 | + if( empty($command->referer) ) { |
|
| 28 | 28 | glsr_log()->warning( 'The form referer ($_SERVER[REQUEST_URI]) was empty.' )->info( $command ); |
| 29 | 29 | $command->referer = home_url(); |
| 30 | 30 | } |
@@ -23,7 +23,9 @@ |
||
| 23 | 23 | glsr( Session::class )->set( $command->form_id.'message', __( 'Your review has been submitted!', 'site-reviews' )); |
| 24 | 24 | glsr( Notification::class )->send( $review ); |
| 25 | 25 | do_action( 'site-reviews/local/review/submitted', $review ); |
| 26 | - if( $command->ajax_request )return; |
|
| 26 | + if( $command->ajax_request ) { |
|
| 27 | + return; |
|
| 28 | + } |
|
| 27 | 29 | if( empty( $command->referer )) { |
| 28 | 30 | glsr_log()->warning( 'The form referer ($_SERVER[REQUEST_URI]) was empty.' )->info( $command ); |
| 29 | 31 | $command->referer = home_url(); |