← All changes
|
app/Http/Controllers/AIActivityLoggerController.php
+13
-15
1.10.2
→
2.4.5
View file →
| @@ -2,16 +2,12 @@ | ||
| 2 | 2 | |
| 3 | 3 | namespace FluentSupport\App\Http\Controllers; |
| 4 | 4 | |
| 5 | 5 | use FluentSupport\App\Services\Helper; |
| 6 | -use FluentSupport\Framework\Request\Request; | |
| 6 | +use FluentSupport\Framework\Http\Request\Request; | |
| 7 | 7 | |
| 8 | 8 | /** |
| 9 | - * ActivityLoggerController class for REST API | |
| 10 | - * This class is responsible for getting data for all request related to activity and activity settings | |
| 11 | - * @package FluentSupport\App\Http\Controllers | |
| 12 | - * | |
| 13 | - * @version 1.0.0 | |
| 9 | + * AI activity log endpoints and settings. | |
| 14 | 10 | */ |
| 15 | 11 | class AIActivityLoggerController extends Controller |
| 16 | 12 | { |
| 17 | 13 | public function getAIActivities(Request $request) |
| @@ -16,44 +12,46 @@ | ||
| 16 | 12 | { |
| 17 | 13 | public function getAIActivities(Request $request) |
| 18 | 14 | { |
| 19 | 15 | try { |
| 16 | + $filters = $request->get('filters', null); | |
| 17 | + $filters = is_array($filters) ? map_deep($filters, 'sanitize_text_field') : []; | |
| 18 | + | |
| 20 | 19 | return Helper::getAIActivities( [ |
| 21 | 20 | 'page' => $request->getSafe('page', 'intval', 1), |
| 22 | 21 | 'per_page' => $request->getSafe('per_page', 'intval', 10), |
| 23 | 22 | 'from' => $request->getSafe('from', 'sanitize_text_field', ''), |
| 24 | 23 | 'to' => $request->getSafe('to', 'sanitize_text_field', ''), |
| 25 | - 'filters' => $request->getSafe('filters', null, []), | |
| 24 | + 'filters' => $filters, | |
| 26 | 25 | ] ); |
| 27 | 26 | } catch (\Exception $e) { |
| 28 | 27 | return $this->sendError([ |
| 29 | - 'message' => $e->getMessage() | |
| 28 | + 'message' => Helper::getSafeErrorMessage($e) | |
| 30 | 29 | ]); |
| 31 | 30 | } |
| 32 | 31 | } |
| 33 | 32 | |
| 34 | 33 | /** |
| 35 | - * updateSettings method will update existing activity settings | |
| 36 | 34 | * @return \WP_REST_Response | array |
| 37 | 35 | */ |
| 38 | 36 | public function updateSettings (Request $request) |
| 39 | 37 | { |
| 40 | - $settings = $request->get('ai_activity_settings'); | |
| 38 | + $settings = $request->get('ai_activity_settings', null); | |
| 39 | + $settings = is_array($settings) ? $settings : []; | |
| 41 | 40 | $settings = [ |
| 42 | - 'delete_days' => intval($settings['delete_days']), | |
| 43 | - 'disable_logs' => sanitize_text_field($settings['disable_logs']) | |
| 41 | + 'delete_days' => intval($settings['delete_days'] ?? 0), | |
| 42 | + 'disable_logs' => sanitize_text_field($settings['disable_logs'] ?? '') | |
| 44 | 43 | ]; |
| 45 | 44 | try { |
| 46 | 45 | return Helper::updateAISettings($settings); |
| 47 | 46 | } catch (\Exception $e) { |
| 48 | 47 | return $this->sendError([ |
| 49 | - 'message' => $e->getMessage() | |
| 48 | + 'message' => Helper::getSafeErrorMessage($e) | |
| 50 | 49 | ]); |
| 51 | 50 | } |
| 52 | 51 | } |
| 53 | 52 | |
| 54 | 53 | /** |
| 55 | - * getSettings method will get the list of activity settings and return | |
| 56 | 54 | * @return \WP_REST_Response | array |
| 57 | 55 | */ |
| 58 | 56 | public function getSettings() |
| 59 | 57 | { |
| @@ -60,9 +58,9 @@ | ||
| 60 | 58 | try { |
| 61 | 59 | return Helper::getSettings(); |
| 62 | 60 | } catch (\Exception $e) { |
| 63 | 61 | return $this->sendError([ |
| 64 | - 'message' => $e->getMessage() | |
| 62 | + 'message' => Helper::getSafeErrorMessage($e) | |
| 65 | 63 | ]); |
| 66 | 64 | } |
| 67 | 65 | } |
| 68 | 66 | |