@@ -17,7 +17,7 @@ discard block |
||
| 17 | 17 | public function routeAdminPostRequest() |
| 18 | 18 | { |
| 19 | 19 | $request = $this->getRequest(); |
| 20 | - if( !$this->isValidPostRequest( $request ))return; |
|
| 20 | + if( !$this->isValidPostRequest( $request ) )return; |
|
| 21 | 21 | check_admin_referer( $request['_action'] ); |
| 22 | 22 | $this->routeRequest( 'admin', $request['_action'], $request ); |
| 23 | 23 | } |
@@ -41,8 +41,8 @@ discard block |
||
| 41 | 41 | { |
| 42 | 42 | if( is_admin() )return; |
| 43 | 43 | $request = $this->getRequest(); |
| 44 | - if( !$this->isValidPostRequest( $request ))return; |
|
| 45 | - if( !$this->isValidPublicNonce( $request ))return; |
|
| 44 | + if( !$this->isValidPostRequest( $request ) )return; |
|
| 45 | + if( !$this->isValidPublicNonce( $request ) )return; |
|
| 46 | 46 | $this->routeRequest( 'public', $request['_action'], $request ); |
| 47 | 47 | } |
| 48 | 48 | |
@@ -52,10 +52,10 @@ discard block |
||
| 52 | 52 | protected function checkAjaxNonce( array $request ) |
| 53 | 53 | { |
| 54 | 54 | if( !is_user_logged_in() )return; |
| 55 | - if( !isset( $request['_nonce'] )) { |
|
| 55 | + if( !isset($request['_nonce']) ) { |
|
| 56 | 56 | $this->sendAjaxError( 'request is missing a nonce', $request ); |
| 57 | 57 | } |
| 58 | - if( !wp_verify_nonce( $request['_nonce'], $request['_action'] )) { |
|
| 58 | + if( !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) { |
|
| 59 | 59 | $this->sendAjaxError( 'request failed the nonce check', $request, 403 ); |
| 60 | 60 | } |
| 61 | 61 | } |
@@ -65,10 +65,10 @@ discard block |
||
| 65 | 65 | */ |
| 66 | 66 | protected function checkAjaxRequest( array $request ) |
| 67 | 67 | { |
| 68 | - if( !isset( $request['_action'] )) { |
|
| 68 | + if( !isset($request['_action']) ) { |
|
| 69 | 69 | $this->sendAjaxError( 'request must include an action', $request ); |
| 70 | 70 | } |
| 71 | - if( empty( $request['_ajax_request'] )) { |
|
| 71 | + if( empty($request['_ajax_request']) ) { |
|
| 72 | 72 | $this->sendAjaxError( 'request is invalid', $request ); |
| 73 | 73 | } |
| 74 | 74 | } |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | */ |
| 96 | 96 | protected function isValidPostRequest( array $request = [] ) |
| 97 | 97 | { |
| 98 | - return !empty( $request['_action'] ) && empty( $request['_ajax_request'] ); |
|
| 98 | + return !empty($request['_action']) && empty($request['_ajax_request']); |
|
| 99 | 99 | } |
| 100 | 100 | |
| 101 | 101 | /** |
@@ -103,7 +103,7 @@ discard block |
||
| 103 | 103 | */ |
| 104 | 104 | protected function isValidPublicNonce( array $request ) |
| 105 | 105 | { |
| 106 | - if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] )) { |
|
| 106 | + if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) { |
|
| 107 | 107 | glsr_log()->error( 'nonce check failed for public request' )->debug( $request ); |
| 108 | 108 | return false; |
| 109 | 109 | } |
@@ -118,11 +118,11 @@ discard block |
||
| 118 | 118 | protected function routeRequest( $type, $action, array $request = [] ) |
| 119 | 119 | { |
| 120 | 120 | $actionHook = 'site-reviews/route/'.$type.'/request'; |
| 121 | - $controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' )); |
|
| 121 | + $controller = glsr( glsr( Helper::class )->buildClassName( $type.'-controller', 'Controllers' ) ); |
|
| 122 | 122 | $method = glsr( Helper::class )->buildMethodName( $action, 'router' ); |
| 123 | 123 | $request = apply_filters( 'site-reviews/route/request', $request, $action, $type ); |
| 124 | 124 | do_action( $actionHook, $action, $request ); |
| 125 | - if( is_callable( [$controller, $method] )) { |
|
| 125 | + if( is_callable( [$controller, $method] ) ) { |
|
| 126 | 126 | call_user_func( [$controller, $method], $request ); |
| 127 | 127 | return; |
| 128 | 128 | } |
@@ -140,10 +140,10 @@ discard block |
||
| 140 | 140 | { |
| 141 | 141 | glsr_log()->error( $error )->debug( $request ); |
| 142 | 142 | glsr( Notice::class )->addError( __( 'There was an error (try refreshing the page).', 'site-reviews' ).' <code>'.$error.'</code>' ); |
| 143 | - wp_send_json_error([ |
|
| 143 | + wp_send_json_error( [ |
|
| 144 | 144 | 'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ), |
| 145 | 145 | 'notices' => glsr( Notice::class )->get(), |
| 146 | 146 | 'error' => $error, |
| 147 | - ]); |
|
| 147 | + ] ); |
|
| 148 | 148 | } |
| 149 | 149 | } |
@@ -27,7 +27,7 @@ discard block |
||
| 27 | 27 | */ |
| 28 | 28 | public function enqueueAssets() |
| 29 | 29 | { |
| 30 | - if( apply_filters( 'site-reviews/assets/css', true )) { |
|
| 30 | + if( apply_filters( 'site-reviews/assets/css', true ) ) { |
|
| 31 | 31 | wp_enqueue_style( |
| 32 | 32 | Application::ID, |
| 33 | 33 | $this->getStylesheet(), |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | glsr()->version |
| 36 | 36 | ); |
| 37 | 37 | } |
| 38 | - if( apply_filters( 'site-reviews/assets/js', true )) { |
|
| 38 | + if( apply_filters( 'site-reviews/assets/js', true ) ) { |
|
| 39 | 39 | $dependencies = apply_filters( 'site-reviews/assets/polyfill', true ) |
| 40 | 40 | ? [Application::ID.'/polyfill'] |
| 41 | 41 | : []; |
@@ -55,11 +55,11 @@ discard block |
||
| 55 | 55 | */ |
| 56 | 56 | public function enqueuePolyfillService() |
| 57 | 57 | { |
| 58 | - if( !apply_filters( 'site-reviews/assets/polyfill', true ))return; |
|
| 59 | - wp_enqueue_script( Application::ID.'/polyfill', add_query_arg([ |
|
| 58 | + if( !apply_filters( 'site-reviews/assets/polyfill', true ) )return; |
|
| 59 | + wp_enqueue_script( Application::ID.'/polyfill', add_query_arg( [ |
|
| 60 | 60 | 'features' => 'CustomEvent,Element.prototype.closest,Element.prototype.dataset,Event,MutationObserver', |
| 61 | 61 | 'flags' => 'gated', |
| 62 | - ], 'https://cdn.polyfill.io/v2/polyfill.min.js' )); |
|
| 62 | + ], 'https://cdn.polyfill.io/v2/polyfill.min.js' ) ); |
|
| 63 | 63 | } |
| 64 | 64 | |
| 65 | 65 | /** |
@@ -72,10 +72,10 @@ discard block |
||
| 72 | 72 | // nf-google-recaptcha |
| 73 | 73 | if( !glsr( OptionManager::class )->isRecaptchaEnabled() )return; |
| 74 | 74 | $language = apply_filters( 'site-reviews/recaptcha/language', get_locale() ); |
| 75 | - wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg([ |
|
| 75 | + wp_enqueue_script( Application::ID.'/google-recaptcha', add_query_arg( [ |
|
| 76 | 76 | 'hl' => $language, |
| 77 | 77 | 'render' => 'explicit', |
| 78 | - ], 'https://www.google.com/recaptcha/api.js' )); |
|
| 78 | + ], 'https://www.google.com/recaptcha/api.js' ) ); |
|
| 79 | 79 | } |
| 80 | 80 | |
| 81 | 81 | /** |
@@ -84,8 +84,8 @@ discard block |
||
| 84 | 84 | public function inlineStyles() |
| 85 | 85 | { |
| 86 | 86 | $inlineStylesheetPath = glsr()->path( 'assets/styles/inline-styles.css' ); |
| 87 | - if( !apply_filters( 'site-reviews/assets/css', true ))return; |
|
| 88 | - if( !file_exists( $inlineStylesheetPath )) { |
|
| 87 | + if( !apply_filters( 'site-reviews/assets/css', true ) )return; |
|
| 88 | + if( !file_exists( $inlineStylesheetPath ) ) { |
|
| 89 | 89 | glsr_log()->error( 'Inline stylesheet is missing: '.$inlineStylesheetPath ); |
| 90 | 90 | return; |
| 91 | 91 | } |
@@ -120,7 +120,7 @@ discard block |
||
| 120 | 120 | */ |
| 121 | 121 | protected function getFixedSelectorsForPagination() |
| 122 | 122 | { |
| 123 | - $selectors = ['#wpadminbar','.site-navigation-fixed']; |
|
| 123 | + $selectors = ['#wpadminbar', '.site-navigation-fixed']; |
|
| 124 | 124 | return apply_filters( 'site-reviews/enqueue/public/localize/ajax-pagination', $selectors ); |
| 125 | 125 | } |
| 126 | 126 | |
@@ -130,7 +130,7 @@ discard block |
||
| 130 | 130 | protected function getStylesheet() |
| 131 | 131 | { |
| 132 | 132 | $currentStyle = glsr( Style::class )->style; |
| 133 | - return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' )) |
|
| 133 | + return file_exists( glsr()->path( 'assets/styles/custom/'.$currentStyle.'.css' ) ) |
|
| 134 | 134 | ? glsr()->url( 'assets/styles/custom/'.$currentStyle.'.css' ) |
| 135 | 135 | : glsr()->url( 'assets/styles/'.Application::ID.'.css' ); |
| 136 | 136 | } |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | */ |
| 36 | 36 | protected function getFieldDefault( array $field ) |
| 37 | 37 | { |
| 38 | - return isset( $field['default'] ) |
|
| 38 | + return isset($field['default']) |
|
| 39 | 39 | ? $field['default'] |
| 40 | 40 | : ''; |
| 41 | 41 | } |
@@ -45,7 +45,7 @@ discard block |
||
| 45 | 45 | */ |
| 46 | 46 | protected function getSettingFields( $path ) |
| 47 | 47 | { |
| 48 | - return array_filter( $this->settings, function( $key ) use( $path ) { |
|
| 48 | + return array_filter( $this->settings, function( $key ) use($path) { |
|
| 49 | 49 | return glsr( Helper::class )->startsWith( $path, $key ); |
| 50 | 50 | }, ARRAY_FILTER_USE_KEY ); |
| 51 | 51 | } |
@@ -60,8 +60,8 @@ discard block |
||
| 60 | 60 | $field = wp_parse_args( $field, [ |
| 61 | 61 | 'is_setting' => true, |
| 62 | 62 | 'name' => $name, |
| 63 | - ]); |
|
| 64 | - $rows.= new Field( $this->normalize( $field )); |
|
| 63 | + ] ); |
|
| 64 | + $rows .= new Field( $this->normalize( $field ) ); |
|
| 65 | 65 | } |
| 66 | 66 | return $rows; |
| 67 | 67 | } |
@@ -72,7 +72,7 @@ discard block |
||
| 72 | 72 | */ |
| 73 | 73 | protected function getTemplateData( $id ) |
| 74 | 74 | { |
| 75 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
| 75 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
| 76 | 76 | return [ |
| 77 | 77 | 'context' => [ |
| 78 | 78 | 'rows' => $this->getSettingRows( $fields ), |
@@ -86,12 +86,12 @@ discard block |
||
| 86 | 86 | */ |
| 87 | 87 | protected function getTemplateDataForAddons( $id ) |
| 88 | 88 | { |
| 89 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
| 89 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
| 90 | 90 | $settings = glsr( Helper::class )->convertDotNotationArray( $fields ); |
| 91 | 91 | $settingKeys = array_keys( $settings['settings']['addons'] ); |
| 92 | 92 | $results = []; |
| 93 | 93 | foreach( $settingKeys as $key ) { |
| 94 | - $addonFields = array_filter( $fields, function( $path ) use( $key ) { |
|
| 94 | + $addonFields = array_filter( $fields, function( $path ) use($key) { |
|
| 95 | 95 | return glsr( Helper::class )->startsWith( 'settings.addons.'.$key, $path ); |
| 96 | 96 | }, ARRAY_FILTER_USE_KEY ); |
| 97 | 97 | $results[$key] = $this->getSettingRows( $addonFields ); |
@@ -108,7 +108,7 @@ discard block |
||
| 108 | 108 | */ |
| 109 | 109 | protected function getTemplateDataForLicenses( $id ) |
| 110 | 110 | { |
| 111 | - $fields = $this->getSettingFields( $this->normalizeSettingPath( $id )); |
|
| 111 | + $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) ); |
|
| 112 | 112 | ksort( $fields ); |
| 113 | 113 | return [ |
| 114 | 114 | 'context' => [ |
@@ -123,7 +123,7 @@ discard block |
||
| 123 | 123 | protected function getTemplateDataForTranslations() |
| 124 | 124 | { |
| 125 | 125 | $translations = glsr( Translation::class )->renderAll(); |
| 126 | - $class = empty( $translations ) |
|
| 126 | + $class = empty($translations) |
|
| 127 | 127 | ? 'glsr-hidden' |
| 128 | 128 | : ''; |
| 129 | 129 | return [ |
@@ -146,9 +146,9 @@ discard block |
||
| 146 | 146 | $path, |
| 147 | 147 | glsr( Helper::class )->getPathValue( $path, glsr()->defaults ) |
| 148 | 148 | ); |
| 149 | - if( is_array( $expectedValue )) { |
|
| 149 | + if( is_array( $expectedValue ) ) { |
|
| 150 | 150 | return is_array( $optionValue ) |
| 151 | - ? count( array_intersect( $optionValue, $expectedValue )) === 0 |
|
| 151 | + ? count( array_intersect( $optionValue, $expectedValue ) ) === 0 |
|
| 152 | 152 | : !in_array( $optionValue, $expectedValue ); |
| 153 | 153 | } |
| 154 | 154 | return $optionValue != $expectedValue; |
@@ -159,10 +159,10 @@ discard block |
||
| 159 | 159 | */ |
| 160 | 160 | protected function isMultiDependency( $path ) |
| 161 | 161 | { |
| 162 | - if( isset( $this->settings[$path] )) { |
|
| 162 | + if( isset($this->settings[$path]) ) { |
|
| 163 | 163 | $field = $this->settings[$path]; |
| 164 | - return ( $field['type'] == 'checkbox' && !empty( $field['options'] )) |
|
| 165 | - || !empty( $field['multiple'] ); |
|
| 164 | + return ($field['type'] == 'checkbox' && !empty($field['options'])) |
|
| 165 | + || !empty($field['multiple']); |
|
| 166 | 166 | } |
| 167 | 167 | return false; |
| 168 | 168 | } |
@@ -183,17 +183,17 @@ discard block |
||
| 183 | 183 | */ |
| 184 | 184 | protected function normalizeDependsOn( array $field ) |
| 185 | 185 | { |
| 186 | - if( !empty( $field['depends_on'] ) && is_array( $field['depends_on'] )) { |
|
| 186 | + if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) { |
|
| 187 | 187 | $path = key( $field['depends_on'] ); |
| 188 | 188 | $expectedValue = $field['depends_on'][$path]; |
| 189 | 189 | $fieldName = glsr( Helper::class )->convertPathToName( $path, OptionManager::databaseKey() ); |
| 190 | - if( $this->isMultiDependency( $path )) { |
|
| 191 | - $fieldName.= '[]'; |
|
| 190 | + if( $this->isMultiDependency( $path ) ) { |
|
| 191 | + $fieldName .= '[]'; |
|
| 192 | 192 | } |
| 193 | - $field['data-depends'] = json_encode([ |
|
| 193 | + $field['data-depends'] = json_encode( [ |
|
| 194 | 194 | 'name' => $fieldName, |
| 195 | 195 | 'value' => $expectedValue, |
| 196 | - ], JSON_HEX_APOS|JSON_HEX_QUOT ); |
|
| 196 | + ], JSON_HEX_APOS | JSON_HEX_QUOT ); |
|
| 197 | 197 | $field['is_hidden'] = $this->isFieldHidden( $path, $expectedValue ); |
| 198 | 198 | } |
| 199 | 199 | return $field; |
@@ -204,9 +204,9 @@ discard block |
||
| 204 | 204 | */ |
| 205 | 205 | protected function normalizeLabelAndLegend( array $field ) |
| 206 | 206 | { |
| 207 | - if( !empty( $field['label'] )) { |
|
| 207 | + if( !empty($field['label']) ) { |
|
| 208 | 208 | $field['legend'] = $field['label']; |
| 209 | - unset( $field['label'] ); |
|
| 209 | + unset($field['label']); |
|
| 210 | 210 | } |
| 211 | 211 | else { |
| 212 | 212 | $field['is_valid'] = false; |
@@ -220,7 +220,7 @@ discard block |
||
| 220 | 220 | */ |
| 221 | 221 | protected function normalizeValue( array $field ) |
| 222 | 222 | { |
| 223 | - if( !isset( $field['value'] )) { |
|
| 223 | + if( !isset($field['value']) ) { |
|
| 224 | 224 | $field['value'] = glsr( OptionManager::class )->get( |
| 225 | 225 | $field['name'], |
| 226 | 226 | $this->getFieldDefault( $field ) |
@@ -4,7 +4,7 @@ discard block |
||
| 4 | 4 | |
| 5 | 5 | // Database/ReviewManager.php |
| 6 | 6 | add_action( 'site-reviews/review/created', function( $review ) { |
| 7 | - if( has_action( 'site-reviews/local/review/create' )) { |
|
| 7 | + if( has_action( 'site-reviews/local/review/create' ) ) { |
|
| 8 | 8 | glsr()->deprecated[] = 'The "site-reviews/local/review/create" hook has been deprecated. Please use the "site-reviews/review/created" hook instead.'; |
| 9 | 9 | do_action( 'site-reviews/local/review/create', (array)get_post( $review->ID ), (array)$review, $review->ID ); |
| 10 | 10 | } |
@@ -12,18 +12,18 @@ discard block |
||
| 12 | 12 | |
| 13 | 13 | // Handlers/CreateReview.php |
| 14 | 14 | add_action( 'site-reviews/review/submitted', function( $review ) { |
| 15 | - if( has_action( 'site-reviews/local/review/submitted' )) { |
|
| 15 | + if( has_action( 'site-reviews/local/review/submitted' ) ) { |
|
| 16 | 16 | glsr()->deprecated[] = 'The "site-reviews/local/review/submitted" hook has been deprecated. Please use the "site-reviews/review/submitted" hook instead.'; |
| 17 | 17 | do_action( 'site-reviews/local/review/submitted', null, $review ); |
| 18 | 18 | } |
| 19 | - if( has_filter( 'site-reviews/local/review/submitted/message' )) { |
|
| 19 | + if( has_filter( 'site-reviews/local/review/submitted/message' ) ) { |
|
| 20 | 20 | glsr()->deprecated[] = 'The "site-reviews/local/review/submitted/message" hook has been deprecated.'; |
| 21 | 21 | } |
| 22 | 22 | }, 9 ); |
| 23 | 23 | |
| 24 | 24 | // Database/ReviewManager.php |
| 25 | 25 | add_filter( 'site-reviews/create/review-values', function( $values, $command ) { |
| 26 | - if( has_filter( 'site-reviews/local/review' )) { |
|
| 26 | + if( has_filter( 'site-reviews/local/review' ) ) { |
|
| 27 | 27 | glsr()->deprecated[] = 'The "site-reviews/local/review" hook has been deprecated. Please use the "site-reviews/create/review-values" hook instead.'; |
| 28 | 28 | return apply_filters( 'site-reviews/local/review', $values, $command ); |
| 29 | 29 | } |
@@ -32,7 +32,7 @@ discard block |
||
| 32 | 32 | |
| 33 | 33 | // Handlers/EnqueuePublicAssets.php |
| 34 | 34 | add_filter( 'site-reviews/enqueue/public/localize', function( $variables ) { |
| 35 | - if( has_filter( 'site-reviews/enqueue/localize' )) { |
|
| 35 | + if( has_filter( 'site-reviews/enqueue/localize' ) ) { |
|
| 36 | 36 | glsr()->deprecated[] = 'The "site-reviews/enqueue/localize" hook has been deprecated. Please use the "site-reviews/enqueue/public/localize" hook instead.'; |
| 37 | 37 | return apply_filters( 'site-reviews/enqueue/localize', $variables ); |
| 38 | 38 | } |
@@ -41,7 +41,7 @@ discard block |
||
| 41 | 41 | |
| 42 | 42 | // Modules/Rating.php |
| 43 | 43 | add_filter( 'site-reviews/rating/average', function( $average ) { |
| 44 | - if( has_filter( 'site-reviews/average/rating' )) { |
|
| 44 | + if( has_filter( 'site-reviews/average/rating' ) ) { |
|
| 45 | 45 | glsr()->deprecated[] = 'The "site-reviews/average/rating" hook has been deprecated. Please use the "site-reviews/rating/average" hook instead.'; |
| 46 | 46 | } |
| 47 | 47 | return $average; |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | |
| 50 | 50 | // Modules/Rating.php |
| 51 | 51 | add_filter( 'site-reviews/rating/ranking', function( $ranking ) { |
| 52 | - if( has_filter( 'site-reviews/bayesian/ranking' )) { |
|
| 52 | + if( has_filter( 'site-reviews/bayesian/ranking' ) ) { |
|
| 53 | 53 | glsr()->deprecated[] = 'The "site-reviews/bayesian/ranking" hook has been deprecated. Please use the "site-reviews/rating/ranking" hook instead.'; |
| 54 | 54 | } |
| 55 | 55 | return $ranking; |
@@ -57,10 +57,10 @@ discard block |
||
| 57 | 57 | |
| 58 | 58 | // Modules/Html/Partials/SiteReviews.php |
| 59 | 59 | add_filter( 'site-reviews/review/build/after', function( $renderedFields ) { |
| 60 | - if( has_filter( 'site-reviews/reviews/review/text' )) { |
|
| 60 | + if( has_filter( 'site-reviews/reviews/review/text' ) ) { |
|
| 61 | 61 | glsr()->deprecated[] = 'The "site-reviews/reviews/review/text" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.'; |
| 62 | 62 | } |
| 63 | - if( has_filter( 'site-reviews/reviews/review/title' )) { |
|
| 63 | + if( has_filter( 'site-reviews/reviews/review/title' ) ) { |
|
| 64 | 64 | glsr()->deprecated[] = 'The "site-reviews/reviews/review/title" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.'; |
| 65 | 65 | } |
| 66 | 66 | return $renderedFields; |
@@ -68,26 +68,26 @@ discard block |
||
| 68 | 68 | |
| 69 | 69 | // Modules/Html/Partials/SiteReviews.php |
| 70 | 70 | add_filter( 'site-reviews/review/build/before', function( $review ) { |
| 71 | - if( has_filter( 'site-reviews/rendered/review' )) { |
|
| 71 | + if( has_filter( 'site-reviews/rendered/review' ) ) { |
|
| 72 | 72 | glsr()->deprecated[] = 'The "site-reviews/rendered/review" hook has been deprecated. Please either use a custom "review.php" template (refer to the documentation), or use the "site-reviews/review/build/after" hook instead.'; |
| 73 | 73 | } |
| 74 | - if( has_filter( 'site-reviews/rendered/review/meta/order' )) { |
|
| 74 | + if( has_filter( 'site-reviews/rendered/review/meta/order' ) ) { |
|
| 75 | 75 | glsr()->deprecated[] = 'The "site-reviews/rendered/review/meta/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).'; |
| 76 | 76 | } |
| 77 | - if( has_filter( 'site-reviews/rendered/review/order' )) { |
|
| 77 | + if( has_filter( 'site-reviews/rendered/review/order' ) ) { |
|
| 78 | 78 | glsr()->deprecated[] = 'The "site-reviews/rendered/review/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).'; |
| 79 | 79 | } |
| 80 | - if( has_filter( 'site-reviews/rendered/review-form/login-register' )) { |
|
| 80 | + if( has_filter( 'site-reviews/rendered/review-form/login-register' ) ) { |
|
| 81 | 81 | glsr()->deprecated[] = 'The "site-reviews/rendered/review-form/login-register" hook has been deprecated. Please use a custom "login-register.php" template instead (refer to the documentation).'; |
| 82 | 82 | } |
| 83 | - if( has_filter( 'site-reviews/reviews/navigation_links' )) { |
|
| 83 | + if( has_filter( 'site-reviews/reviews/navigation_links' ) ) { |
|
| 84 | 84 | glsr()->deprecated[] = 'The "site-reviews/reviews/navigation_links" hook has been deprecated. Please use a custom "pagination.php" template instead (refer to the documentation).'; |
| 85 | 85 | } |
| 86 | 86 | return $review; |
| 87 | 87 | }, 9 ); |
| 88 | 88 | |
| 89 | 89 | add_filter( 'site-reviews/validate/custom', function( $result, $request ) { |
| 90 | - if( has_filter( 'site-reviews/validate/review/submission' )) { |
|
| 90 | + if( has_filter( 'site-reviews/validate/review/submission' ) ) { |
|
| 91 | 91 | glsr_log()->notice( 'The "site-reviews/validate/review/submission" hook has been deprecated. Please use the "site-reviews/validate/custom" hook instead.' ); |
| 92 | 92 | return apply_filters( 'site-reviews/validate/review/submission', $result, $request ); |
| 93 | 93 | } |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | }, 9, 2 ); |
| 96 | 96 | |
| 97 | 97 | add_filter( 'site-reviews/views/file', function( $file, $view, $data ) { |
| 98 | - if( has_filter( 'site-reviews/addon/views/file' )) { |
|
| 98 | + if( has_filter( 'site-reviews/addon/views/file' ) ) { |
|
| 99 | 99 | glsr()->deprecated[] = 'The "site-reviews/addon/views/file" hook has been deprecated. Please use the "site-reviews/views/file" hook instead.'; |
| 100 | 100 | $file = apply_filters( 'site-reviews/addon/views/file', $file, $view, $data ); |
| 101 | 101 | } |
@@ -103,7 +103,7 @@ discard block |
||
| 103 | 103 | }, 9, 3 ); |
| 104 | 104 | |
| 105 | 105 | add_action( 'wp_footer', function() { |
| 106 | - $notices = array_keys( array_flip( glsr()->deprecated )); |
|
| 106 | + $notices = array_keys( array_flip( glsr()->deprecated ) ); |
|
| 107 | 107 | natsort( $notices ); |
| 108 | 108 | foreach( $notices as $notice ) { |
| 109 | 109 | glsr_log()->notice( $notice ); |
@@ -1,14 +1,14 @@ discard block |
||
| 1 | 1 | <?php defined( 'WPINC' ) || die; ?> |
| 2 | 2 | |
| 3 | 3 | <div id="titlediv"> |
| 4 | - <input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' )); ?>" readonly> |
|
| 4 | + <input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' ) ); ?>" readonly> |
|
| 5 | 5 | </div> |
| 6 | 6 | |
| 7 | 7 | <div id="contentdiv"> |
| 8 | 8 | <textarea readonly><?= esc_attr( $post->post_content ); ?></textarea> |
| 9 | 9 | </div> |
| 10 | 10 | |
| 11 | -<?php if( empty( $response ))return; ?> |
|
| 11 | +<?php if( empty($response) )return; ?> |
|
| 12 | 12 | |
| 13 | 13 | <div class="postbox glsr-response-postbox"> |
| 14 | 14 | <button type="button" class="handlediv" aria-expanded="true"> |
@@ -19,6 +19,6 @@ discard block |
||
| 19 | 19 | <span><?= __( 'Public Response', 'site-reviews' ); ?></span> |
| 20 | 20 | </h2> |
| 21 | 21 | <div class="inside"> |
| 22 | - <?= wpautop( esc_attr( $response )); ?> |
|
| 22 | + <?= wpautop( esc_attr( $response ) ); ?> |
|
| 23 | 23 | </div> |
| 24 | 24 | </div> |
@@ -13,14 +13,14 @@ discard block |
||
| 13 | 13 | public function getPost( $postId ) |
| 14 | 14 | { |
| 15 | 15 | $postId = trim( $postId ); |
| 16 | - if( empty( $postId ) || !is_numeric( $postId ))return; |
|
| 16 | + if( empty($postId) || !is_numeric( $postId ) )return; |
|
| 17 | 17 | if( $this->isEnabled() ) { |
| 18 | - $polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() )); |
|
| 18 | + $polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ) ); |
|
| 19 | 19 | } |
| 20 | - if( !empty( $polylangPostId )) { |
|
| 20 | + if( !empty($polylangPostId) ) { |
|
| 21 | 21 | $postId = $polylangPostId; |
| 22 | 22 | } |
| 23 | - return get_post( intval( $postId )); |
|
| 23 | + return get_post( intval( $postId ) ); |
|
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | /** |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | foreach( $this->cleanIds( $postIds ) as $postId ) { |
| 36 | 36 | $newPostIds = array_merge( |
| 37 | 37 | $newPostIds, |
| 38 | - array_values( pll_get_post_translations( $postId )) |
|
| 38 | + array_values( pll_get_post_translations( $postId ) ) |
|
| 39 | 39 | ); |
| 40 | 40 | } |
| 41 | 41 | return $this->cleanIds( $newPostIds ); |
@@ -75,7 +75,7 @@ discard block |
||
| 75 | 75 | */ |
| 76 | 76 | protected function cleanIds( array $postIds ) |
| 77 | 77 | { |
| 78 | - return array_filter( array_unique( $postIds )); |
|
| 78 | + return array_filter( array_unique( $postIds ) ); |
|
| 79 | 79 | } |
| 80 | 80 | } |
| 81 | 81 | |
@@ -15,7 +15,7 @@ discard block |
||
| 15 | 15 | */ |
| 16 | 16 | public function download( $filename, $content ) |
| 17 | 17 | { |
| 18 | - if( !current_user_can( glsr()->constant( 'CAPABILITY' )))return; |
|
| 18 | + if( !current_user_can( glsr()->constant( 'CAPABILITY' ) ) )return; |
|
| 19 | 19 | nocache_headers(); |
| 20 | 20 | header( 'Content-Type: text/plain' ); |
| 21 | 21 | header( 'Content-Disposition: attachment; filename="'.$filename.'"' ); |
@@ -30,8 +30,8 @@ discard block |
||
| 30 | 30 | */ |
| 31 | 31 | public function execute( $command ) |
| 32 | 32 | { |
| 33 | - $handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command )); |
|
| 34 | - if( !class_exists( $handlerClass )) { |
|
| 33 | + $handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ) ); |
|
| 34 | + if( !class_exists( $handlerClass ) ) { |
|
| 35 | 35 | throw new InvalidArgumentException( 'Handler '.$handlerClass.' not found.' ); |
| 36 | 36 | } |
| 37 | 37 | try { |
@@ -39,7 +39,7 @@ discard block |
||
| 39 | 39 | } |
| 40 | 40 | catch( Exception $e ) { |
| 41 | 41 | status_header( 400 ); |
| 42 | - glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() )); |
|
| 42 | + glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ) ); |
|
| 43 | 43 | glsr_log()->error( $e->getMessage() ); |
| 44 | 44 | } |
| 45 | 45 | } |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | */ |
| 50 | 50 | protected function getPostId() |
| 51 | 51 | { |
| 52 | - return intval( filter_input( INPUT_GET, 'post' )); |
|
| 52 | + return intval( filter_input( INPUT_GET, 'post' ) ); |
|
| 53 | 53 | } |
| 54 | 54 | |
| 55 | 55 | /** |
@@ -17,11 +17,11 @@ discard block |
||
| 17 | 17 | { |
| 18 | 18 | $queries = []; |
| 19 | 19 | foreach( $keys as $key ) { |
| 20 | - if( !array_key_exists( $key, $values ))continue; |
|
| 20 | + if( !array_key_exists( $key, $values ) )continue; |
|
| 21 | 21 | $methodName = glsr( Helper::class )->buildMethodName( $key, __FUNCTION__ ); |
| 22 | - if( !method_exists( $this, $methodName ))continue; |
|
| 22 | + if( !method_exists( $this, $methodName ) )continue; |
|
| 23 | 23 | $query = call_user_func( [$this, $methodName], $values[$key] ); |
| 24 | - if( is_array( $query )) { |
|
| 24 | + if( is_array( $query ) ) { |
|
| 25 | 25 | $queries[] = $query; |
| 26 | 26 | } |
| 27 | 27 | } |
@@ -36,10 +36,10 @@ discard block |
||
| 36 | 36 | $string = ''; |
| 37 | 37 | $values = array_filter( $values ); |
| 38 | 38 | foreach( $conditions as $key => $value ) { |
| 39 | - if( !isset( $values[$key] ))continue; |
|
| 39 | + if( !isset($values[$key]) )continue; |
|
| 40 | 40 | $values[$key] = implode( ',', (array)$values[$key] ); |
| 41 | 41 | $string .= strpos( $value, '%s' ) !== false |
| 42 | - ? sprintf( $value, strval( $values[$key] )) |
|
| 42 | + ? sprintf( $value, strval( $values[$key] ) ) |
|
| 43 | 43 | : $value; |
| 44 | 44 | } |
| 45 | 45 | return $string; |
@@ -53,11 +53,11 @@ discard block |
||
| 53 | 53 | */ |
| 54 | 54 | public function buildSqlOr( $values, $sprintfFormat ) |
| 55 | 55 | { |
| 56 | - if( !is_array( $values )) { |
|
| 56 | + if( !is_array( $values ) ) { |
|
| 57 | 57 | $values = explode( ',', $values ); |
| 58 | 58 | } |
| 59 | - $values = array_filter( array_map( 'trim', (array)$values )); |
|
| 60 | - $values = array_map( function( $value ) use( $sprintfFormat ) { |
|
| 59 | + $values = array_filter( array_map( 'trim', (array)$values ) ); |
|
| 60 | + $values = array_map( function( $value ) use($sprintfFormat) { |
|
| 61 | 61 | return sprintf( $sprintfFormat, $value ); |
| 62 | 62 | }, $values ); |
| 63 | 63 | return implode( ' OR ', $values ); |
@@ -72,11 +72,11 @@ discard block |
||
| 72 | 72 | */ |
| 73 | 73 | public function filterSearchByTitle( $search, WP_Query $query ) |
| 74 | 74 | { |
| 75 | - if( empty( $search ) || empty( $query->get( 'search_terms' ))) { |
|
| 75 | + if( empty($search) || empty($query->get( 'search_terms' )) ) { |
|
| 76 | 76 | return $search; |
| 77 | 77 | } |
| 78 | 78 | global $wpdb; |
| 79 | - $n = empty( $query->get( 'exact' )) |
|
| 79 | + $n = empty($query->get( 'exact' )) |
|
| 80 | 80 | ? '%' |
| 81 | 81 | : ''; |
| 82 | 82 | $search = []; |
@@ -100,7 +100,7 @@ discard block |
||
| 100 | 100 | ? glsr()->constant( 'PAGED_QUERY_VAR' ) |
| 101 | 101 | : 'page'; |
| 102 | 102 | return $isEnabled |
| 103 | - ? max( 1, intval( get_query_var( $pagedQuery ))) |
|
| 103 | + ? max( 1, intval( get_query_var( $pagedQuery ) ) ) |
|
| 104 | 104 | : 1; |
| 105 | 105 | } |
| 106 | 106 | |
@@ -110,7 +110,7 @@ discard block |
||
| 110 | 110 | */ |
| 111 | 111 | protected function buildQueryAssignedTo( $value ) |
| 112 | 112 | { |
| 113 | - if( empty( $value ))return; |
|
| 113 | + if( empty($value) )return; |
|
| 114 | 114 | $postIds = glsr( Helper::class )->convertStringToArray( $value, 'is_numeric' ); |
| 115 | 115 | return [ |
| 116 | 116 | 'compare' => 'IN', |
@@ -125,7 +125,7 @@ discard block |
||
| 125 | 125 | */ |
| 126 | 126 | protected function buildQueryCategory( $value ) |
| 127 | 127 | { |
| 128 | - if( empty( $value ))return; |
|
| 128 | + if( empty($value) )return; |
|
| 129 | 129 | return [ |
| 130 | 130 | 'field' => 'term_id', |
| 131 | 131 | 'taxonomy' => Application::TAXONOMY, |
@@ -139,7 +139,7 @@ discard block |
||
| 139 | 139 | */ |
| 140 | 140 | protected function buildQueryRating( $value ) |
| 141 | 141 | { |
| 142 | - if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 )))return; |
|
| 142 | + if( !is_numeric( $value ) || !in_array( intval( $value ), range( 1, 5 ) ) )return; |
|
| 143 | 143 | return [ |
| 144 | 144 | 'compare' => '>=', |
| 145 | 145 | 'key' => 'rating', |
@@ -153,7 +153,7 @@ discard block |
||
| 153 | 153 | */ |
| 154 | 154 | protected function buildQueryType( $value ) |
| 155 | 155 | { |
| 156 | - if( in_array( $value, ['','all'] ))return; |
|
| 156 | + if( in_array( $value, ['', 'all'] ) )return; |
|
| 157 | 157 | return [ |
| 158 | 158 | 'key' => 'review_type', |
| 159 | 159 | 'value' => $value, |
@@ -18,7 +18,7 @@ discard block |
||
| 18 | 18 | { |
| 19 | 19 | return $this->translate( $translation, $domain, [ |
| 20 | 20 | 'single' => $text, |
| 21 | - ]); |
|
| 21 | + ] ); |
|
| 22 | 22 | } |
| 23 | 23 | |
| 24 | 24 | /** |
@@ -33,7 +33,7 @@ discard block |
||
| 33 | 33 | return $this->translate( $translation, $domain, [ |
| 34 | 34 | 'context' => $context, |
| 35 | 35 | 'single' => $text, |
| 36 | - ]); |
|
| 36 | + ] ); |
|
| 37 | 37 | } |
| 38 | 38 | |
| 39 | 39 | /** |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | 'number' => $number, |
| 51 | 51 | 'plural' => $plural, |
| 52 | 52 | 'single' => $single, |
| 53 | - ]); |
|
| 53 | + ] ); |
|
| 54 | 54 | } |
| 55 | 55 | |
| 56 | 56 | /** |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | 'number' => $number, |
| 70 | 70 | 'plural' => $plural, |
| 71 | 71 | 'single' => $single, |
| 72 | - ]); |
|
| 72 | + ] ); |
|
| 73 | 73 | } |
| 74 | 74 | |
| 75 | 75 | /** |
@@ -80,12 +80,12 @@ discard block |
||
| 80 | 80 | public function translate( $original, $domain, array $args ) |
| 81 | 81 | { |
| 82 | 82 | $domains = apply_filters( 'site-reviews/translator/domains', [Application::ID] ); |
| 83 | - if( !in_array( $domain, $domains )) { |
|
| 83 | + if( !in_array( $domain, $domains ) ) { |
|
| 84 | 84 | return $original; |
| 85 | 85 | } |
| 86 | 86 | $args = $this->normalizeTranslationArgs( $args ); |
| 87 | 87 | $strings = $this->getTranslationStrings( $args['single'], $args['plural'] ); |
| 88 | - if( empty( $strings )) { |
|
| 88 | + if( empty($strings) ) { |
|
| 89 | 89 | return $original; |
| 90 | 90 | } |
| 91 | 91 | $string = current( $strings ); |
@@ -101,7 +101,7 @@ discard block |
||
| 101 | 101 | */ |
| 102 | 102 | protected function getTranslationStrings( $single, $plural ) |
| 103 | 103 | { |
| 104 | - return array_filter( glsr( Translation::class )->translations(), function( $string ) use( $single, $plural ) { |
|
| 104 | + return array_filter( glsr( Translation::class )->translations(), function( $string ) use($single, $plural) { |
|
| 105 | 105 | return $string['s1'] == html_entity_decode( $single, ENT_COMPAT, 'UTF-8' ) |
| 106 | 106 | && $string['p1'] == html_entity_decode( $plural, ENT_COMPAT, 'UTF-8' ); |
| 107 | 107 | }); |
@@ -127,10 +127,10 @@ discard block |
||
| 127 | 127 | */ |
| 128 | 128 | protected function translatePlural( $domain, array $string, array $args ) |
| 129 | 129 | { |
| 130 | - if( !empty( $string['s2'] )) { |
|
| 130 | + if( !empty($string['s2']) ) { |
|
| 131 | 131 | $args['single'] = $string['s2']; |
| 132 | 132 | } |
| 133 | - if( !empty( $string['p2'] )) { |
|
| 133 | + if( !empty($string['p2']) ) { |
|
| 134 | 134 | $args['plural'] = $string['p2']; |
| 135 | 135 | } |
| 136 | 136 | return get_translations_for_domain( $domain )->translate_plural( |
@@ -147,7 +147,7 @@ discard block |
||
| 147 | 147 | */ |
| 148 | 148 | protected function translateSingle( $domain, array $string, array $args ) |
| 149 | 149 | { |
| 150 | - if( !empty( $string['s2'] )) { |
|
| 150 | + if( !empty($string['s2']) ) { |
|
| 151 | 151 | $args['single'] = $string['s2']; |
| 152 | 152 | } |
| 153 | 153 | return get_translations_for_domain( $domain )->translate( |