| @@ -2,22 +2,18 @@ | ||
| 2 | 2 | |
| 3 | 3 | namespace FluentSupport\App\Http\Controllers; |
| 4 | 4 | |
| 5 | 5 | use FluentSupport\App\Modules\IntegrationSettingsModule; |
| 6 | -use FluentSupport\Framework\Request\Request; | |
| 6 | +use FluentSupport\Framework\Http\Request\Request; | |
| 7 | 7 | |
| 8 | 8 | /** |
| 9 | - * SlackController class is responsible for getting and save Slack settings | |
| 10 | - * @package FluentSupport\App\Http\Controllers | |
| 11 | - * | |
| 12 | - * @version 1.0.0 | |
| 9 | + * Slack integration settings. | |
| 13 | 10 | */ |
| 14 | 11 | class SlackController extends Controller |
| 15 | 12 | { |
| 16 | 13 | /** |
| 17 | - * getSettings method will generate data for Slack settings | |
| 18 | 14 | * @param Request $request |
| 19 | - * @return false | |
| 15 | + * @return array|false | |
| 20 | 16 | */ |
| 21 | 17 | public function getSettings(Request $request) |
| 22 | 18 | { |
| 23 | 19 | $settingsKey = $request->getSafe('integration_key', 'sanitize_text_field'); |
| @@ -25,9 +21,8 @@ | ||
| 25 | 21 | return IntegrationSettingsModule::getSettings($settingsKey, true); |
| 26 | 22 | } |
| 27 | 23 | |
| 28 | 24 | /** |
| 29 | - * saveSettings method will save settings data for Slack | |
| 30 | 25 | * @param Request $request |
| 31 | 26 | * @return array |
| 32 | 27 | */ |
| 33 | 28 | public function saveSettings(Request $request) |
| @@ -32,9 +27,10 @@ | ||
| 32 | 27 | */ |
| 33 | 28 | public function saveSettings(Request $request) |
| 34 | 29 | { |
| 35 | 30 | $settingsKey = $request->getSafe('integration_key' , 'sanitize_text_field'); |
| 36 | - $settings = wp_unslash($request->getSafe('settings')); | |
| 31 | + $settings = wp_unslash($request->get('settings', null)); | |
| 32 | + $settings = is_array($settings) ? map_deep($settings, 'sanitize_text_field') : []; | |
| 37 | 33 | $settings = IntegrationSettingsModule::saveSettings($settingsKey, $settings); |
| 38 | 34 | |
| 39 | 35 | if(!$settings || is_wp_error($settings)) { |
| 40 | 36 | $errorMessage = (is_wp_error($settings)) ? $settings->get_error_message() : __('Settings failed to save', 'fluent-support'); |