summaryrefslogtreecommitdiffstats
path: root/voice
AgeCommit message (Collapse)AuthorFilesLines
2020-03-23rework message hierarchyRaquel Medina3-54/+70
Rework message hierarchy with the final objective of splitting libqtappfw into several libraries. This commit carries the following changes: - Simplify message hierarchy, keeping abstract Message class, adding specialization for call and event messages, keeping ResponseMessage, and removing all module specific specializations. - Add MessageFactory class to create message objects. - Change messages life cycle: using smart pointers and removing QObject from message hierarchy (a Message is not a QObject anymore and thus 'deleteLater()' is not available). - Adapt all modules to use new message hierarchy. - Keep ResponseMessage original constructor to avoid breaking TaskManager. - Message constructors have been kept public, but will go private on a follow-up patch (once TaskManager class has been modified to use new MessageFactory). Bug-AGL: SPEC-3112 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: I3a7a6325209ddeca2293f1ac745371861a947bfb
2020-03-09eliminate unneeded dependencies on public headersRaquel Medina2-11/+13
- Remove unneeded dependencies from libqtappfw public headers, specifically MessageEngine and Message hierarchy header files, in preparation for refactoring work on these areas. This change helps shield client apps from the upcoming changes in the library, and highlight the separation between the library's core and the various modules providing public interfaces to those apps. - Make MessageType an scoped enum to fwd declare it. - Update Copyright statement on files updated by this patch. Bug-AGL: SPEC-3112 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: I12fd9cdc034aa1166f6448c07befadd9d2042b05
2020-01-03voice: remove workarounds for broken vshl-core event JSONScott Murray3-102/+7
Now that the broken event argument JSON from the vshl-core binding has a fix, the workaround of selectively reparsing the voice event arguments needs to be removed, as it breaks with the corrected events. Bug-AGL: SPEC-3084 Signed-off-by: Scott Murray <scott.murray@konsulko.com> Change-Id: Ie158ae326a3dc9c7cf4b08dbdbc729f5587e3758 (cherry picked from commit 77392aeda562e3bbb4f311a99469156cb1a66d75)
2019-12-29voice: add vendor name to voiceagent modelScott Murray4-1/+7
Add the vendor name to the voiceagent model to allow using it applications, and fixed a typo and a couple of possible state name omissions. Bug-AGL: SPEC-2981 Signed-off-by: Scott Murray <scott.murray@konsulko.com> Change-Id: I65a59ec17d0e003bc114fa85d120ee773fee0f1a
2019-12-17fix parsing for all events, overloading for specific needsicefish_8.99.4icefish/8.99.48.99.4Raquel Medina2-0/+88
commit 21bba7d, introduced extra parsing ops to process voice_xxxx_event payloads, which carry extra whitespaces and eol chars; however that change introduced a regression for other events. To solve the parsing issue for all events processed by libqtappfw, overloading is applied to 'fromJdoc' method. Bug-AGL: SPEC-3060 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: I175848cba7f9f9e2e8b4e5984e7abc2a2fc12464
2019-12-13fix parsing event payload in presence of '\n' chars,icefish_8.99.3icefish/8.99.38.99.3Raquel Medina5-89/+105
and include signal emit for voice status and cbl changes. - fix parsing event payload in presence of whitespaces; - fix parsing voice events payload in presence of '\n' chars; - add missing model updates when status or cbl data changes; - fix authorization status enum value typo; Bug-AGL: SPEC-2981 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: I718bdfea1221211ed4c62ce1b695eabe83ab9590
2019-12-12voice: register for all vshl events on start upRaquel Medina2-14/+9
libqtappfw should register for login events from start up (along with other vshl events), so that the information is available even if an user didnt kick start the authorization process via settings app. - register to all vshl events on start up; - rename 'subscribeToLoginEvents' to 'triggerCBLProcess', which is indeed the operation implemented behind this method. Bug-AGL: SPEC-2981 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: I8aea5cac0b6d17b8ca3838d0958d2128cc745c0f
2019-12-12voice: remove vshl-capabilities dependenciesRaquel Medina6-138/+85
Various fixes for voice component: - fix incorrect check on vap (voice agent profile) pointers; - remove dependencies on vshl-capabilities: this module will communicate via vshl-core api; - remove dependencis on alexa-voiceagent; - use a more generic name for cbl related operations. CBL in this environment is understood as code based login (for voice agent authorization via user login to a cloud service), however, in the case of alexa it is known as code based linking; - add check for voice_xxx_event eventdata which in ocasions is an empty object, i.e., the payload is missing. Bug-AGL: SPEC-2981 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: Icaab7a21bedb3e2696d541567162ec46b047a305
2019-12-04voice: add status events to VshlCoreVoiceMessages.Raquel Medina1-5/+7
- Add connection, auth and dialog status events to VshlCoreVoiceMessage private events list. - Fix logic error in AlexaVoiceMessage's auth events processing. Error introduced by commit 'da091649e775b23a35eb88df1c2cf9ce3fb350f7'. Bug-AGL: SPEC-2880 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: Ia402c8307b16595be5a8f9a1ed79d5eeb0e08036
2019-11-29voice: add initial support for voice agents configurationRaquel Medina11-0/+1137
Bug-AGL: SPEC-2981 Signed-off-by: Raquel Medina <raquel.medina@konsulko.com> Change-Id: I0195d914dc10f4fcdea1cb6df0e6a5859ad8269d