Correctly display message when unable to get weather info & add qwhatsthis to configuration widget.

2 files changed, 10 insertions, 4 deletions

diff --git a/noncore/todayplugins/weather/weatherconfig.cpp b/noncore/todayplugins/weather/weatherconfig.cpp
index fe10883..99ee2a0 100644
--- a/noncore/todayplugins/weather/weatherconfig.cpp
+++ b/noncore/todayplugins/weather/weatherconfig.cpp
@@ -53,54 +53,59 @@ WeatherPluginConfig::WeatherPluginConfig( QWidget *parent,  const char* name)
         config.setGroup( "Config" );
 
         QFontMetrics fm = fontMetrics();
         int fh = fm.height();
 
         QGridLayout *layout = new QGridLayout( this );
         layout->setSpacing( 4 );
         layout->setMargin( 4 );
 
         QLabel *label = new QLabel( tr( "Enter ICAO location identifier:" ), this );
         label->setMaximumHeight( fh + 3 );
         layout->addMultiCellWidget( label, 0, 0, 0, 1 );
+    QWhatsThis::add( label, tr( "Enter the 4 letter code for the desired location here.  See http://www.nws.noaa.gov/tg/siteloc.shtml to find a location near you." ) );
 
         locationEdit = new QLineEdit( config.readEntry( "Location", "" ), this );
         locationEdit->setMaximumHeight( fh + 5 );
         locationEdit->setFocus();
     layout->addMultiCellWidget( locationEdit, 1, 1, 0, 1 );
+    QWhatsThis::add( locationEdit, tr( "Enter the 4 letter code for the desired location here.  See http://www.nws.noaa.gov/tg/siteloc.shtml to find a location near you." ) );
 
         label = new QLabel( tr( "Visit http://www.nws.noaa.gov/tg/siteloc.shtml to find the nearest location." ), this );
         label->setAlignment( AlignHCenter | WordBreak );
         label->setMaximumHeight( label->height() );
         layout->addMultiCellWidget( label, 2, 2, 0, 1 );
         //layout->addWidget( label, 2, 0 );
 
         //QPushButton *pb = new QPushButton( Resource::loadPixmap( "copy" ),
                 //tr( "Copy link" ), this );
         //connect( pb, SIGNAL( clicked() ), this, SLOT( slotCopyLink() ) );
         //layout->addWidget( pb, 2, 1 );
 
         metricCB = new QCheckBox( tr( "Use metric units" ), this );
         metricCB->setMaximumHeight( fh + 5 );
         metricCB->setChecked( config.readBoolEntry( "Metric", TRUE ) );
         layout->addMultiCellWidget( metricCB, 3, 3, 0, 1 );
+    QWhatsThis::add( metricCB, tr( "Click here to select type of units displayed." ) );
 
         label = new QLabel( tr( "Update frequency (in minutes):" ), this );
         label->setMaximumHeight( fh + 3 );
         layout->addWidget( label, 4, 0 );
+    QWhatsThis::add( label, tr( "Select how often (in minutes) you want the weather to be updated." ) );
 
         timerDelaySB = new QSpinBox( 1, 60, 1, this );
         timerDelaySB->setMaximumHeight( fh + 5 );
         timerDelaySB->setValue( config.readNumEntry( "Frequency", 5 ) );
         layout->addWidget( timerDelaySB, 4, 1 );
+    QWhatsThis::add( timerDelaySB, tr( "Select how often (in minutes) you want the weather to be updated." ) );
 }
 
 
 void WeatherPluginConfig::writeConfig()
 {
         Config config( "todayweatherplugin");
         config.setGroup( "Config" );
 
         config.writeEntry( "Location", locationEdit->text().upper().stripWhiteSpace() );
         config.writeEntry( "Metric", metricCB->isChecked() );
         config.writeEntry( "Frequency", timerDelaySB->value() );
 
@@ -115,13 +120,13 @@ void WeatherPluginConfig::doLookup()
 {
         system( "weather" );
 }
 
 /*
 
 Doesn't seem to like QPEApplication::clipboard()...
 
 void WeatherPluginConfig::slotCopyLink()
 {
         QPEApplication::clipboard()->setText( "http://www.nws.noaa.gov/tg/siteloc.shtml" );
 }
-*/
\ No newline at end of file
+*/
diff --git a/noncore/todayplugins/weather/weatherpluginwidget.cpp b/noncore/todayplugins/weather/weatherpluginwidget.cpp
index f4ea0f2..800fdcb 100644
--- a/noncore/todayplugins/weather/weatherpluginwidget.cpp
+++ b/noncore/todayplugins/weather/weatherpluginwidget.cpp
@@ -105,25 +105,26 @@ void WeatherPluginWidget::retreiveData()
         OProcess *proc = new OProcess;
 
         *proc << "wget" << "-q" << remoteFile << "-O" << localFile;
         connect( proc, SIGNAL( processExited( OProcess * ) ), this, SLOT( dataRetrieved( OProcess * ) ) );
         proc->start();
 }
 
 void WeatherPluginWidget::displayWeather()
 {
         weatherData = QString::null;
 
         QFile file( localFile );
-        if ( file.open( IO_ReadOnly ) )
+        
+        if ( file.size() > 0 && file.open( IO_ReadOnly ) )
         {
                 QTextStream data( &file );
                 while ( !data.eof() )
                 {
                         weatherData.append( data.readLine() );
                 }
                 file.close();
                 weatherData = weatherData.simplifyWhiteSpace();
 
                 QString tmpstr;
 
                 tmpstr.append( tr( "Temp: " ) );
@@ -141,25 +142,25 @@ void WeatherPluginWidget::displayWeather()
                 weatherLabel->setText( tmpstr );
 
                 tmpstr = "todayweatherplugin/";
                 getIcon( weatherData );
                 tmpstr.append( dataStr );
                 QImage logo1 = Resource::loadImage( tmpstr );
                 QPixmap pic;
                 pic.convertFromImage( logo1 );
                 weatherIcon->setPixmap( pic );
         }
         else
         {
-                weatherLabel->setText( tr( "Current weather data not available.\nTry looking out the window." ) );
+                weatherLabel->setText( tr( "Current weather data not available." ) );
         }
 }
 
 void WeatherPluginWidget::getTemp( const QString &data )
 {
         int value;
         bool ok;
 
         int pos = data.find( QRegExp( "M?[0-9]+/M?[0-9]+" ), 20 );
         if ( pos > -1 )
         {
                 if ( data.at( pos ) == 'M' )
@@ -310,15 +311,15 @@ void WeatherPluginWidget::getIcon(const QString &data )
                 dataStr = "sleet";
         }
 }
 
 void WeatherPluginWidget::dataRetrieved( OProcess *process )
 {
         if ( process->normalExit() )
         {
                 displayWeather();
         }
         else
         {
-                weatherLabel->setText( tr( "Current weather data not available.\nTry looking out the window." ) );
+                weatherLabel->setText( tr( "Current weather data not available." ) );
         }
 }