Skip to content

Commit

Permalink
RDK-45352: Upgrade Xumo TV to use Thunder R4.4.1 (rdkcentral#5397)
Browse files Browse the repository at this point in the history
* RDK-45352: Upgrade Xumo TV to use Thunder R4.4.1

Reason for change: ResidentApp plugin unable to activate due to unable to get the Initialize/activating Notification from Thunder(R4.4.1)
Test Procedure: Verify in Jenkin Build
Risks: High
Signed-off-by: Thamim Razith [email protected]
  • Loading branch information
tabbas651 authored Jun 10, 2024
1 parent 5ab4dc2 commit f34f560
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 7 deletions.
3 changes: 3 additions & 0 deletions RDKShell/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ All notable changes to this RDK Service will be documented in this file.

* For more details, refer to [versioning](https://github.com/rdkcentral/rdkservices#versioning) section under Main README.

## [1.5.3] - 2024-06-10
### Changed
- ResidentApp plugin unable to activate due to unable to get the Initialize/activating Notification from Thunder(R4.4.1)

## [1.5.2] - 2024-06-04
### Changed
Expand Down
8 changes: 4 additions & 4 deletions RDKShell/RDKShell.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1453,7 +1453,7 @@ namespace WPEFramework {
}

#ifdef USE_THUNDER_R4
void RDKShell::MonitorClients::Initialize(VARIABLE_IS_NOT_USED const string& callsign, VARIABLE_IS_NOT_USED PluginHost::IShell* service)
void RDKShell::MonitorClients::Initialize(const string& callsign, PluginHost::IShell* service)
{
handleInitialize(service);
}
Expand All @@ -1470,10 +1470,10 @@ namespace WPEFramework {
//StateChange(service);
handleDeactivated(service);
}
void RDKShell::MonitorClients::Deinitialized(VARIABLE_IS_NOT_USED const string& callsign, VARIABLE_IS_NOT_USED PluginHost::IShell* service)
{
void RDKShell::MonitorClients::Deinitialized(const string& callsign, PluginHost::IShell* service)
{
handleDeinitialized(service);
}
}
void RDKShell::MonitorClients::Unavailable(const string& callsign, PluginHost::IShell* service)
{}
#endif /* USE_THUNDER_R4 */
Expand Down
14 changes: 11 additions & 3 deletions RDKShell/RDKShell.h
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,11 @@ namespace WPEFramework {
RDKShell& mShell;
};

class MonitorClients : public PluginHost::IPlugin::INotification {
class MonitorClients : public PluginHost::IPlugin::INotification
#if ((THUNDER_VERSION >= 4) && (THUNDER_VERSION_MINOR >= 4))
, public PluginHost::IPlugin::ILifeTime
#endif
{
private:
MonitorClients() = delete;
MonitorClients(const MonitorClients&) = delete;
Expand All @@ -414,6 +418,9 @@ namespace WPEFramework {
public:
BEGIN_INTERFACE_MAP(MonitorClients)
INTERFACE_ENTRY(PluginHost::IPlugin::INotification)
#if ((THUNDER_VERSION >= 4) && (THUNDER_VERSION_MINOR >= 4))
INTERFACE_ENTRY(PluginHost::IPlugin::ILifeTime)
#endif
END_INTERFACE_MAP

private:
Expand All @@ -422,13 +429,14 @@ namespace WPEFramework {
void handleActivated(PluginHost::IShell* shell);
void handleDeactivated(PluginHost::IShell* shell);
void handleDeinitialized(PluginHost::IShell* shell);

#ifdef USE_THUNDER_R4
virtual void Initialize(VARIABLE_IS_NOT_USED const string& callsign, VARIABLE_IS_NOT_USED PluginHost::IShell* plugin);
virtual void Initialize(const string& callsign, PluginHost::IShell* plugin);
virtual void Activation(const string& name, PluginHost::IShell* plugin);
virtual void Deactivation(const string& name, PluginHost::IShell* plugin);
virtual void Activated(const string& callSign, PluginHost::IShell* plugin);
virtual void Deactivated(const string& callSign, PluginHost::IShell* plugin);
virtual void Deinitialized(VARIABLE_IS_NOT_USED const string& callsign, VARIABLE_IS_NOT_USED PluginHost::IShell* plugin);
virtual void Deinitialized(const string& callsign, PluginHost::IShell* plugin);
virtual void Unavailable(const string& callSign, PluginHost::IShell* plugin);
#endif /* USE_THUNDER_R4 */
private:
Expand Down

0 comments on commit f34f560

Please sign in to comment.