diff options
author | jacobkeeler <jacob.keeler@livioradio.com> | 2020-08-07 14:23:47 -0400 |
---|---|---|
committer | jacobkeeler <jacob.keeler@livioradio.com> | 2020-08-07 14:23:47 -0400 |
commit | 40040d31593db2eb3ec1fb615a466ac3ec5bde19 (patch) | |
tree | 004b7ef6df317ddd31ba3ae00dfdbe735b2589b6 | |
parent | e7dcbdb73b612202b3aac7ee32d3e026c41c45e0 (diff) | |
download | sdl_core-40040d31593db2eb3ec1fb615a466ac3ec5bde19.tar.gz |
Address comments
13 files changed, 44 insertions, 31 deletions
diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/on_ui_subtle_alert_pressed_notification.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/on_ui_subtle_alert_pressed_notification.h index fd6aa2e837..c08746ba3d 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/on_ui_subtle_alert_pressed_notification.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/on_ui_subtle_alert_pressed_notification.h @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_request.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_request.h index 1b286f068a..b6b232516d 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_request.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_request.h @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_response.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_response.h index efd07b1de5..c363677ab0 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_response.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/hmi/ui_subtle_alert_response.h @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/on_subtle_alert_pressed_notification.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/on_subtle_alert_pressed_notification.h index 7836e48054..68789438b8 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/on_subtle_alert_pressed_notification.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/on_subtle_alert_pressed_notification.h @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_request.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_request.h index 313106e659..4789d35e60 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_request.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_request.h @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_response.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_response.h index 133ed7b045..c9be00dd37 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_response.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/subtle_alert_response.h @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/on_ui_subtle_alert_pressed_notification.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/on_ui_subtle_alert_pressed_notification.cc index e720a4b4a8..b2f5f80055 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/on_ui_subtle_alert_pressed_notification.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/on_ui_subtle_alert_pressed_notification.cc @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without @@ -34,6 +34,7 @@ namespace sdl_rpc_plugin { using namespace application_manager; + namespace commands { OnUISubtleAlertPressedNotification::OnUISubtleAlertPressedNotification( diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_request.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_request.cc index ddb9a8831b..b2a5c7b4a8 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_request.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_request.cc @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without @@ -34,6 +34,7 @@ namespace sdl_rpc_plugin { using namespace application_manager; + namespace commands { UISubtleAlertRequest::UISubtleAlertRequest( @@ -46,7 +47,15 @@ UISubtleAlertRequest::UISubtleAlertRequest( application_manager, rpc_service, hmi_capabilities, - policy_handler) {} + policy_handler) { + const auto& msg_params = (*message_)[strings::msg_params]; + if (msg_params[strings::duration]) { + uint32_t request_timeout = msg_params[strings::duration].asUInt(); + default_timeout_ += request_timeout; + } else { + default_timeout_ = 0; + } +} UISubtleAlertRequest::~UISubtleAlertRequest() {} diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_response.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_response.cc index 093bf6f5e4..51e9802db7 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_response.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/hmi/ui_subtle_alert_response.cc @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without @@ -34,6 +34,7 @@ namespace sdl_rpc_plugin { using namespace application_manager; + namespace commands { UISubtleAlertResponse::UISubtleAlertResponse( diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_subtle_alert_pressed_notification.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_subtle_alert_pressed_notification.cc index 279de2f1c6..ce1d1b16d1 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_subtle_alert_pressed_notification.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_subtle_alert_pressed_notification.cc @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without @@ -32,10 +32,10 @@ #include "sdl_rpc_plugin/commands/mobile/on_subtle_alert_pressed_notification.h" #include "application_manager/application_impl.h" -#include "interfaces/MOBILE_API.h" namespace sdl_rpc_plugin { using namespace application_manager; + namespace commands { OnSubtleAlertPressedNotification::OnSubtleAlertPressedNotification( diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_request.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_request.cc index 44eab8090b..23604a6b93 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_request.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_request.cc @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without @@ -34,17 +34,17 @@ #include "application_manager/application_impl.h" #include "application_manager/message_helper.h" -#include "interfaces/MOBILE_API.h" namespace sdl_rpc_plugin { using namespace application_manager; + namespace commands { SubtleAlertRequest::SubtleAlertRequest( const application_manager::commands::MessageSharedPtr& message, ApplicationManager& application_manager, - app_mngr::rpc_service::RPCService& rpc_service, - app_mngr::HMICapabilities& hmi_capabilities, + rpc_service::RPCService& rpc_service, + HMICapabilities& hmi_capabilities, policy::PolicyHandlerInterface& policy_handler) : CommandRequestImpl(message, application_manager, @@ -57,7 +57,10 @@ SubtleAlertRequest::SubtleAlertRequest( , is_ui_subtle_alert_sent_(false) , is_tts_stop_speaking_sent_(false) , subtle_alert_result_(hmi_apis::Common_Result::INVALID_ENUM) - , tts_speak_result_(hmi_apis::Common_Result::INVALID_ENUM) {} + , tts_speak_result_(hmi_apis::Common_Result::INVALID_ENUM) { + subscribe_on_event(hmi_apis::FunctionID::UI_OnResetTimeout); + subscribe_on_event(hmi_apis::FunctionID::TTS_OnResetTimeout); +} SubtleAlertRequest::~SubtleAlertRequest() {} @@ -202,7 +205,7 @@ bool SubtleAlertRequest::PrepareResponseParameters( PrepareResultForMobileResponse(ui_subtle_alert_info, tts_alert_info); /* result=false if UI interface is ok and TTS interface = UNSUPPORTED_RESOURCE - * and sdl receive TTS.IsReady=true or SDL doesn't receive responce for + * and sdl receive TTS.IsReady=true or SDL doesn't receive response for * TTS.IsReady. */ if (result && ui_subtle_alert_info.is_ok && @@ -221,16 +224,6 @@ bool SubtleAlertRequest::PrepareResponseParameters( tts_response_info_); return result; } - if ((ui_subtle_alert_info.is_ok || ui_subtle_alert_info.is_not_used) && - tts_alert_info.is_unsupported_resource && - HmiInterfaces::STATE_AVAILABLE == tts_alert_info.interface_state) { - tts_response_info_ = "Unsupported phoneme type sent in a prompt"; - info = app_mngr::commands::MergeInfos(ui_subtle_alert_info, - ui_response_info_, - tts_alert_info, - tts_response_info_); - return result; - } // Ignore TTS.Speak result if ABORTED due to TTS.StopSpeaking request if (is_tts_stop_speaking_sent_ && hmi_apis::Common_Result::ABORTED == tts_alert_info.result_code) { @@ -299,8 +292,7 @@ bool SubtleAlertRequest::Validate(uint32_t app_id) { // check if mandatory params(alertText1 and TTSChunk) specified if ((!(*message_)[strings::msg_params].keyExists(strings::alert_text1)) && (!(*message_)[strings::msg_params].keyExists(strings::alert_text2)) && - (!(*message_)[strings::msg_params].keyExists(strings::tts_chunks) && - (1 > (*message_)[strings::msg_params][strings::tts_chunks].length()))) { + (!(*message_)[strings::msg_params].keyExists(strings::tts_chunks))) { LOG4CXX_ERROR(logger_, "Mandatory parameters are missing"); SendResponse(false, mobile_apis::Result::INVALID_DATA, diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_response.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_response.cc index cb5e842385..ad9252997b 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_response.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subtle_alert_response.cc @@ -1,5 +1,5 @@ /* - Copyright (c) 2019, Ford Motor Company, Livio + Copyright (c) 2020, Ford Motor Company, Livio All rights reserved. Redistribution and use in source and binary forms, with or without @@ -37,6 +37,7 @@ namespace sdl_rpc_plugin { using namespace application_manager; + namespace commands { SubtleAlertResponse::SubtleAlertResponse( diff --git a/src/components/application_manager/src/hmi_capabilities_impl.cc b/src/components/application_manager/src/hmi_capabilities_impl.cc index 401fa52341..439530444f 100644 --- a/src/components/application_manager/src/hmi_capabilities_impl.cc +++ b/src/components/application_manager/src/hmi_capabilities_impl.cc @@ -278,6 +278,15 @@ void InitCapabilities() { hmi_apis::Common_TextFieldName::addressLines)); text_fields_enum_name.insert(std::make_pair( std::string("phoneNumber"), hmi_apis::Common_TextFieldName::phoneNumber)); + text_fields_enum_name.insert( + std::make_pair(std::string("subtleAlertText1"), + hmi_apis::Common_TextFieldName::subtleAlertText1)); + text_fields_enum_name.insert( + std::make_pair(std::string("subtleAlertText2"), + hmi_apis::Common_TextFieldName::subtleAlertText2)); + text_fields_enum_name.insert(std::make_pair( + std::string("subtleAlertSoftButtonText"), + hmi_apis::Common_TextFieldName::subtleAlertSoftButtonText)); text_fields_enum_name.insert(std::make_pair( std::string("turnText"), hmi_apis::Common_TextFieldName::turnText)); text_fields_enum_name.insert(std::make_pair( |