Opened 9 years ago

Closed 9 years ago

#8956 closed defect (fixed)

The Weather Plugin Fails to Run

Reported by: Bill Meek <keemllib@…> Owned by: beirdo
Priority: minor Milestone: unknown
Component: Plugin - MythWeather Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The Weather plugin fails to run, using the Terra theme. The following appears in mythfrontend.log.

2010-09-17 19:43:09.010 Error: container 'mythweather' is missing child 'update_text'
2010-09-17 19:43:09.010 Window weatherbase is missing required elements.

I believe this is related to [26095].

Adding the following allows the Weather plugin to run:

Index: mythweather/mythweather/weather.cpp
===================================================================
--- mythweather/mythweather/weather.cpp	(revision 26362)
+++ mythweather/mythweather/weather.cpp	(working copy)
@@ -84,7 +84,7 @@
 
     UIUtilE::Assign(this, m_pauseText, "pause_text", &err);
     UIUtilE::Assign(this, m_headerText, "header", &err);
-    UIUtilE::Assign(this, m_updatedText, "update_text", &err);
+    UIUtilE::Assign(this, m_updatedText, "updatetime", &err);
 
     if (err)
     {

This is not a fix, because the following 4 errors now appear in the log when the Weather plugin is started (not tested with other themes either):

2010-09-17 20:33:57.667 Widget not found updatetime
2010-09-17 20:33:57.675 Widget not found updatetime
2010-09-17 20:33:58.005 Widget not found updatetime
2010-09-17 20:33:58.815 Widget not found updatetime
Please attach all output as a file in bug reports.
MythTV Version   : 26362
MythTV Branch    : trunk
Network Protocol : 62
Library API      : 0.23.20100913-1
QT Version       : 4.6.2
Options compiled in:
 linux debug using_alsa using_jack using_oss using_pulse using_pulseoutput using_backend using_bindings_perl using_bindings_python using_dvb using_frontend using_iptv using_ivtv using_mheg using_opengl_video using_opengl_vsync using_qtdbus using_qtwebkit using_v4l using_x11 using_xrandr using_xv using_xvmc using_xvmc_vld using_xvmcw using_bindings_perl using_bindings_python using_mythtranscode using_opengl using_vdpau using_ffmpeg_threads using_live using_mheg

Change History (3)

comment:1 Changed 9 years ago by beirdo

This is not at all the correct fix.

It seems that the Terra theme is broken by [26095] which is incorrect. The code itself, however, is not broken.

comment:2 Changed 9 years ago by beirdo

Status: newaccepted

comment:3 Changed 9 years ago by beirdo

Resolution: fixed
Status: acceptedclosed

(In [26369]) Reverting [26095]. The "update_text" field is not for the update time, and is in fact a required field in weatherbase.

Fixes #8956

Note: See TracTickets for help on using tickets.