← All changes
|
src/Controllers/Rest/class-log-rest-controller.php
+16
-10
3.0.5
→
4.1.0
View file →
| @@ -7,9 +7,10 @@ | ||
| 7 | 7 | */ |
| 8 | 8 | |
| 9 | 9 | namespace SeQura\WC\Controllers\Rest; |
| 10 | 10 | |
| 11 | -use SeQura\WC\Services\Interface_Logger_Service; | |
| 11 | +use SeQura\WC\Services\Log\Interface_Logger_Service; | |
| 12 | +use SeQura\Core\Infrastructure\Utility\RegexProvider; | |
| 12 | 13 | use WP_Error; |
| 13 | 14 | use WP_REST_Request; |
| 14 | 15 | use WP_REST_Response; |
| 15 | 16 | |
| @@ -17,10 +18,10 @@ | ||
| 17 | 18 | * REST Log Controller |
| 18 | 19 | */ |
| 19 | 20 | class Log_REST_Controller extends REST_Controller { |
| 20 | 21 | |
| 21 | - const PARAM_IS_ENABLED = 'isEnabled'; | |
| 22 | - const PARAM_LOG_LEVEL = 'level'; | |
| 22 | + private const PARAM_IS_ENABLED = 'isEnabled'; | |
| 23 | + private const PARAM_LOG_LEVEL = 'level'; | |
| 23 | 24 | |
| 24 | 25 | /** |
| 25 | 26 | * Constructor. |
| 26 | 27 | * |
| @@ -25,11 +26,16 @@ | ||
| 25 | 26 | * Constructor. |
| 26 | 27 | * |
| 27 | 28 | * @param string $rest_namespace The namespace. |
| 28 | 29 | * @param Interface_Logger_Service $logger The logger service. |
| 30 | + * @param RegexProvider $regex The regex provider. | |
| 29 | 31 | */ |
| 30 | - public function __construct( $rest_namespace, Interface_Logger_Service $logger ) { | |
| 31 | - parent::__construct( $logger ); | |
| 32 | + public function __construct( | |
| 33 | + $rest_namespace, | |
| 34 | + Interface_Logger_Service $logger, | |
| 35 | + RegexProvider $regex | |
| 36 | + ) { | |
| 37 | + parent::__construct( $logger, $regex ); | |
| 32 | 38 | $this->namespace = $rest_namespace; |
| 33 | 39 | $this->rest_base = '/log'; |
| 34 | 40 | } |
| 35 | 41 | |
| @@ -38,9 +44,9 @@ | ||
| 38 | 44 | * |
| 39 | 45 | * @return void |
| 40 | 46 | */ |
| 41 | 47 | public function register_routes() { |
| 42 | - $this->logger->log_info( 'Hook executed', __FUNCTION__, __CLASS__ ); | |
| 48 | + $this->logger->log_debug( 'Hook executed', __FUNCTION__, __CLASS__ ); | |
| 43 | 49 | $store_id_args = array( self::PARAM_STORE_ID => $this->get_arg_string() ); |
| 44 | 50 | $conf_args = array_merge( |
| 45 | 51 | $store_id_args, |
| 46 | 52 | array( |
| @@ -72,9 +78,9 @@ | ||
| 72 | 78 | ); |
| 73 | 79 | } catch ( \Throwable $e ) { |
| 74 | 80 | $response = new WP_Error( 'error', $e->getMessage() ); |
| 75 | 81 | } |
| 76 | - return rest_ensure_response( $response ); | |
| 82 | + return \rest_ensure_response( $response ); | |
| 77 | 83 | } |
| 78 | 84 | |
| 79 | 85 | /** |
| 80 | 86 | * DELETE logs. |
| @@ -92,9 +98,9 @@ | ||
| 92 | 98 | $response = array(); |
| 93 | 99 | } catch ( \Throwable $e ) { |
| 94 | 100 | $response = new WP_Error( 'error', $e->getMessage() ); |
| 95 | 101 | } |
| 96 | - return rest_ensure_response( $response ); | |
| 102 | + return \rest_ensure_response( $response ); | |
| 97 | 103 | } |
| 98 | 104 | |
| 99 | 105 | /** |
| 100 | 106 | * GET logs configuration. |
| @@ -107,9 +113,9 @@ | ||
| 107 | 113 | $response = $this->get_config_response(); |
| 108 | 114 | } catch ( \Throwable $e ) { |
| 109 | 115 | $response = new WP_Error( 'error', $e->getMessage() ); |
| 110 | 116 | } |
| 111 | - return rest_ensure_response( $response ); | |
| 117 | + return \rest_ensure_response( $response ); | |
| 112 | 118 | } |
| 113 | 119 | |
| 114 | 120 | /** |
| 115 | 121 | * POST logs configuration. |
| @@ -126,9 +132,9 @@ | ||
| 126 | 132 | $response = $this->get_config_response(); |
| 127 | 133 | } catch ( \Throwable $e ) { |
| 128 | 134 | $response = new WP_Error( 'error', $e->getMessage() ); |
| 129 | 135 | } |
| 130 | - return rest_ensure_response( $response ); | |
| 136 | + return \rest_ensure_response( $response ); | |
| 131 | 137 | } |
| 132 | 138 | |
| 133 | 139 | /** |
| 134 | 140 | * Make the logger configuration response. |