| @@ -126,9 +126,8 @@ | ||
| 126 | 126 | * @return WP_Error|boolean |
| 127 | 127 | */ |
| 128 | 128 | public function _permission_check( WP_REST_Request $request ) { |
| 129 | 129 | $this->request = $request; |
| 130 | - $this->api_key = $this->utils('options')->get( 'api_key' ); | |
| 131 | 130 | if(!current_user_can('delete_posts')){ |
| 132 | 131 | return false; |
| 133 | 132 | } |
| 134 | 133 | |
| @@ -162,9 +161,9 @@ | ||
| 162 | 161 | * @return WP_Error |
| 163 | 162 | */ |
| 164 | 163 | protected function permission_error( $message, $endpoint = '') { |
| 165 | 164 | if( empty( $message ) ) { |
| 166 | - $message = __( 'Your session has expired. Please log in again.', 'templately' ); | |
| 165 | + $message = __( 'Sorry, you need to login/re-login again.', 'templately' ); | |
| 167 | 166 | } |
| 168 | 167 | |
| 169 | 168 | $_additional_data = [ |
| 170 | 169 | 'status' => rest_authorization_required_code(), |
| @@ -173,27 +172,18 @@ | ||
| 173 | 172 | if( ! empty( $endpoint ) ) { |
| 174 | 173 | $_additional_data['endpoint'] = $endpoint; |
| 175 | 174 | } |
| 176 | 175 | |
| 177 | - // Delete user logged meta data — pinned to the acting user, otherwise | |
| 178 | - // Options::user_id() resolves a linked user to the global-login | |
| 179 | - // administrator and clears the administrator's session instead. | |
| 180 | - $options = $this->utils('options'); | |
| 181 | - $options->use_current_user( true ); | |
| 176 | + // Delete user logged meta data | |
| 177 | + $this->utils('options') | |
| 178 | + ->remove('user') | |
| 179 | + ->remove('favourites') | |
| 180 | + ->remove('reviews') | |
| 181 | + ->remove('cloud_activity') | |
| 182 | + ->remove('api_key'); | |
| 182 | 183 | |
| 183 | - try { | |
| 184 | - $options | |
| 185 | - ->remove('user') | |
| 186 | - ->remove('favourites') | |
| 187 | - ->remove('reviews') | |
| 188 | - ->remove('cloud_activity') | |
| 189 | - ->remove('api_key'); | |
| 190 | - | |
| 191 | - if( $options->who_am_i() === 'global' ) { | |
| 192 | - $options->remove_global_login(); | |
| 193 | - } | |
| 194 | - } finally { | |
| 195 | - $options->use_current_user( false ); | |
| 184 | + if( $this->utils('options')->who_am_i() === 'global' ) { | |
| 185 | + $this->utils('options')->remove_global_login(); | |
| 196 | 186 | } |
| 197 | 187 | |
| 198 | 188 | return new WP_Error( 'invalid_api_key', $message, $_additional_data ); |
| 199 | 189 | } |
| @@ -238,24 +228,8 @@ | ||
| 238 | 228 | * @return WP_REST_Response |
| 239 | 229 | */ |
| 240 | 230 | public function success( $data ) { |
| 241 | 231 | return new WP_REST_Response( $data, 200 ); |
| 242 | - } | |
| 243 | - | |
| 244 | - /** | |
| 245 | - * Enhanced success response wrapper that automatically adds success flag and wraps data | |
| 246 | - * | |
| 247 | - * @param $data mixed The data to be wrapped in the response | |
| 248 | - * | |
| 249 | - * @return WP_REST_Response | |
| 250 | - */ | |
| 251 | - public function successWithData( $data ) { | |
| 252 | - $enhanced_response = [ | |
| 253 | - 'success' => true, | |
| 254 | - 'data' => $data | |
| 255 | - ]; | |
| 256 | - | |
| 257 | - return $this->success( $enhanced_response ); | |
| 258 | 232 | } |
| 259 | 233 | /** |
| 260 | 234 | * @param $error_code string |
| 261 | 235 | * @param $error_message string|array |
