Changeset 3710

Timestamp:

03/31/09 12:12:29 (12 months ago)

Author:

kris

Message:

Large update to the PBI updater and tray updater, fixed bugs with checking for updates not showing all the updates properly, due to buffering problems when getting the data back from server. Also added feature which will check the PBI's "made date", and not try to have the user install an older PBI on top of a newer copy. This is helpful when the user is running PBI's which are not yet in the online-update service

Location:

pcbsd/trunk

Files:

• SystemUpdaterTray/UpdaterTray.cpp (modified) (9 diffs)
• SystemUpdaterTray/UpdaterTray.h (modified) (1 diff)
• kcmPBMsource/pbm.cpp (modified) (6 diffs)
• kcmPBMsource/pbm.h (modified) (2 diffs)

pcbsd/trunk/SystemUpdaterTray/UpdaterTray.cpp

@@ -72 +72 @@
 QString PBIProgURL3[900];
 QString PBIProgMD5[900];
+QString PBIProgMdate[900];
 QString PBIProgLoc[900];
 int     PBIProgUpdate[900];
 int     PBIProgFailed[900];
+QString PBIBuffer;
+QString updatePBITextList;
 
 void UpdaterTray::programInit()
@@ -480 +483 @@
    if ( pbistatus == PBI_UPDATES_AVAIL ) 
    {
-      tooltipStr += "<br>" + tr("PBI updates available!");
+      tooltipStr += "<br><br>" + tr("PBI updates available!") + "<br><br>";
+      tooltipStr += updatePBITextList;
    }
    
@@ -649 +653 @@
 void UpdaterTray::slotPBICheckUpdate()
 {
+  // Read the buffer from the last PBI check
+  if (currentWorkingPBI != -1 )
+  {
+    slotReadPBIBuffer();
+  }
+
+  // Empty out our PBI buffer
+  PBIBuffer="";
 
     QString tmp, line;
@@ -715 +727 @@
   
   checkPBIJob = KIO::http_post( PBIUrl, postData.utf8(), KIO::HideProgressInfo);
-  checkPBIJob->addMetaData("content-type", "Content-type: application/x-www-form-urlencoded" );
-
-
   connect(checkPBIJob, SIGNAL(result(KJob *)), this, SLOT(slotPBICheckUpdate() ) );
   connect(checkPBIJob, SIGNAL(data(KIO::Job *, const QByteArray&)), this, SLOT(slotRecieveData(KIO::Job *, const QByteArray& ) ) );
+  checkPBIJob->addMetaData("content-type", "Content-type: application/x-www-form-urlencoded" );
+
+
 
 }
@@ -725 +737 @@
 void UpdaterTray::slotRecieveData(KIO::Job*, const QByteArray& data)
 {
-    QString output(data), line;
+    QString output(data);
+    PBIBuffer = PBIBuffer + output;
+}
+
+void UpdaterTray::slotReadPBIBuffer()
+{
+    QString output = PBIBuffer;
+    QString line;
+
     QStringList outputList;
     if ( output.indexOf("Up2Date") == -1  && ! output.isEmpty())
@@ -759 +779 @@
                  PBIProgMD5[currentWorkingPBI] = line.replace("MD5: ", "");
              } 
+             if ( line.find("MDATE:") == 0)
+             {
+                 PBIProgMdate[currentWorkingPBI] = line.replace("MDATE: ", "");
+             } 
         } // End of For loop reading line-by-line our data
 
@@ -777 +801 @@
   int i = 0, foundUpdates = 0;
   QString text, tmp, newMsg, ProgDirName, iconPath;
+  bool updateAvail;
 
     while (! PBIProgName[i].isEmpty() )
@@ -792 +817 @@
           ProgDirName = ProgDirName + tmp;
           iconPath = "/Programs/" + ProgDirName + "/" + PBIProgIcon[i];
-          foundUpdates++;
+          updateAvail=true;
+
+          // Now check if we have a *newer* PBI installed here, than is in the updater
+          QFile mdatefile( "/Programs/" + ProgDirName + "/.mdate");
+          if ( mdatefile.exists() && ! PBIProgMdate[i].isEmpty() )
+          {
+             if ( mdatefile.open( QIODevice::ReadOnly ) ) {
+               QTextStream stream( &mdatefile );
+               QString line;
+               line = stream.readLine(); // line of text excluding '\n'
+               if ( ! line.isEmpty() ) {
+
+                 // Sort the dates from the .mdate file
+                 QString tmp2 = line;
+                 tmp2.truncate( tmp2.indexOf(" ") );
+                 QString date1 = tmp2;
+                 tmp2 = line;
+                 QString date2 = tmp2.remove( 0, tmp2.indexOf(" ") );
+
+                 // Sort the dates from the updater
+                 tmp2 = PBIProgMdate[i];
+                 tmp2.truncate( tmp2.indexOf(" ") );
+                 QString PBIdate1 = tmp2;
+                 tmp2 = PBIProgMdate[i];
+                 QString PBIdate2 = tmp2.remove( 0, tmp2.indexOf(" ") );
+
+                 // Check if the installed PBI is created later than the online update
+                 int ldate, odate;
+                 bool ok;
+                 ldate = date1.toInt(&ok);
+                 if ( ok )
+                 {
+                   odate = PBIdate1.toInt(&ok);
+                   if ( ok )
+                   {
+                     if ( ldate > odate )
+                     {
+                       updateAvail = false;
+                     } else if (ldate == odate) {
+                        // We checked the year / day, now check the hours seconds
+                        ldate = date2.toInt(&ok);
+                        if ( ok )
+                        {
+                           odate = PBIdate2.toInt(&ok);
+                           if ( ok )
+                           {
+                             if ( ldate > odate )
+                             {
+                               updateAvail = false;
+                             }
+                           }
+                        }
+                     }
+                   }
+                 }
+               
+                 /* QMessageBox::critical( 0, tr("Online Update"), "date1: " + date1, QMessageBox::Ok );
+                 QMessageBox::critical( 0, tr("Online Update"), "date2: " + date2, QMessageBox::Ok );
+                 QMessageBox::critical( 0, tr("Online Update"), "PBIdate1: " + PBIdate1, QMessageBox::Ok );
+                 QMessageBox::critical( 0, tr("Online Update"), "PBIdate2: " + PBIdate2, QMessageBox::Ok ); */
+               }
+             }
+
+              
+          }
+
+          // If this is a valid update, go ahead and increment our update counter
+          if ( updateAvail ) {
+            updatePBITextList = updatePBITextList + PBIProgName[i] + " " +  PBIProgNewVer[i] + "<br>";
+            foundUpdates++;
+          }
        }
        i++;
@@ -798 +893 @@
 
   if ( foundUpdates != 0 ) {
-  // Update our dialog that we are looking for PBI updates right now
-    if (foundUpdates == 1 ) {
-       newMsg = "<font color=\"#ff0000\">" + tmp.setNum(foundUpdates) + " " + tr("PBI Update available!") + "</font>";
-       QImage Icon2;
-       Icon2.load("/PCBSD/SystemUpdater/images/pbiupdates.png");
-       QPixmap PixmapIcon2;
-       PixmapIcon2.convertFromImage(Icon2.scaled(22,22));
-
-       KPassivePopup::message(tr("System Update") , tr("A PBI update is available!"),  PixmapIcon2,  this);
-    } else {
-       newMsg = "<font color=\"#ff0000\">" + tmp.setNum(foundUpdates) + " " + tr("PBI Updates available!") + "</font>";
-       QImage Icon2;
-       Icon2.load("/PCBSD/SystemUpdater/images/pbiupdates.png");
-       QPixmap PixmapIcon2;
-       PixmapIcon2.convertFromImage(Icon2.scaled(22,22));
-
-       KPassivePopup::message(tr("System Update") , tr("PBI updates are available!"),  PixmapIcon2,  this);
-    }
-
+    // Set the status that we have new PBIs
     pbistatus = PBI_UPDATES_AVAIL;
   } else {

pcbsd/trunk/SystemUpdaterTray/UpdaterTray.h

@@ -40 +40 @@
    void slotRecieveData(KIO::Job*, const QByteArray& data);
    void slotCheckAllUpdates();
+   void slotReadPBIBuffer();
    
 protected:

pcbsd/trunk/kcmPBMsource/pbm.cpp

@@ -2067 +2067 @@
 {
 
-    QString tmp, line;
-    QString progName, Arch, Version, PBIUrl;
+  // Read the buffer from the last PBI check
+  if (currentWorkingPBI != -1 )
+  {
+    slotReadPBIBuffer();
+  }
+  
+  // Empty out our PBI buffer
+  PBIBuffer="";
+
+  QString tmp, line;
+  QString progName, Arch, Version, PBIUrl;
 
 
@@ -2129 +2138 @@
 
   QString postData = "PBIName=" + progName + "&PBIVer=" + PBIProgVer[currentWorkingPBI] + "&PCBSDVER=" + Version + "&OSARCH=" + Arch;
+
   
+
   checkPBIJob = KIO::http_post( PBIUrl, postData.utf8(), KIO::HideProgressInfo);
+  connect(checkPBIJob, SIGNAL(result(KJob *)), this, SLOT(slotPBICheckUpdate() ) );
+  connect(checkPBIJob, SIGNAL(data(KIO::Job *, const QByteArray&)), this, SLOT(slotRecieveData(KIO::Job *, const QByteArray& ) ) );
+
   checkPBIJob->addMetaData("content-type", "Content-type: application/x-www-form-urlencoded" );
   
   textPBIStatus->setText(tr("<font color=\"#ff0000\">Checking %1 for updates...</font>").arg(progName) );
 
-  connect(checkPBIJob, SIGNAL(result(KJob *)), this, SLOT(slotPBICheckUpdate() ) );
-  connect(checkPBIJob, SIGNAL(data(KIO::Job *, const QByteArray&)), this, SLOT(slotRecieveData(KIO::Job *, const QByteArray& ) ) );
 
 }
@@ -2142 +2154 @@
 void PBM::slotRecieveData(KIO::Job*, const QByteArray& data)
 {
-    QString output(data), line;
+    QString output(data);
+
+    PBIBuffer = PBIBuffer + output;
+}
+
+void PBM::slotReadPBIBuffer()
+{
+    QString output = PBIBuffer;
+    QString line;
     QStringList outputList;
     if ( output.indexOf("Up2Date") == -1  && ! output.isEmpty())
     {
+        //QMessageBox::critical( 0, tr("Online Update"), "line: " + output, QMessageBox::Ok );
+
         outputList = QStringList::split("\n", output);
         for ( int i = 0; i < outputList.size(); i++)
         {
             line = outputList.at(i);
-           
+
              if (line.find("NewVer:" ) ==  0 )
              {
@@ -2176 +2198 @@
                  PBIProgMD5[currentWorkingPBI] = line.replace("MD5: ", "");
              } 
+             if ( line.find("MDATE:") == 0)
+             {
+                 PBIProgMdate[currentWorkingPBI] = line.replace("MDATE: ", "");
+             } 
         } // End of For loop reading line-by-line our data
 
@@ -2194 +2220 @@
   int i = 0, foundUpdates = 0;
   QString text, tmp, newMsg, ProgDirName, iconPath;
+  bool updateAvail;
 
     while (! PBIProgName[i].isEmpty() )
@@ -2209 +2236 @@
           ProgDirName = ProgDirName + tmp;
           iconPath = "/Programs/" + ProgDirName + "/" + PBIProgIcon[i];
-
-          QPixmap defaultIcon;
-          Q3CheckListItem *item = new Q3CheckListItem ( listViewPBIUpdates, text, Q3CheckListItem::CheckBox  ) ;
-          item->setText(1, tmp.setNum(i));
-
-          defaultIcon.convertFromImage(QImage(iconPath).smoothScale(32,32));
-
-          item->setPixmap ( 0, defaultIcon ) ;
-          listViewPBIUpdates->setColumnAlignment ( 0, Qt::AlignVCenter);
-
-          foundUpdates++;
+          updateAvail = true;
+
+          // Now check if we have a *newer* PBI installed here, than is in the updater
+          QFile mdatefile( "/Programs/" + ProgDirName + "/.mdate");
+          if ( mdatefile.exists() && ! PBIProgMdate[i].isEmpty() )
+          {
+             if ( mdatefile.open( QIODevice::ReadOnly ) ) {
+               Q3TextStream stream( &mdatefile );
+               QString line;
+               line = stream.readLine(); // line of text excluding '\n'
+               if ( ! line.isEmpty() ) {
+
+                 // Sort the dates from the .mdate file
+                 QString tmp2 = line;
+                 tmp2.truncate( tmp2.indexOf(" ") );
+                 QString date1 = tmp2;
+                 tmp2 = line;
+                 QString date2 = tmp2.remove( 0, tmp2.indexOf(" ") );
+
+                 // Sort the dates from the updater
+                 tmp2 = PBIProgMdate[i];
+                 tmp2.truncate( tmp2.indexOf(" ") );
+                 QString PBIdate1 = tmp2;
+                 tmp2 = PBIProgMdate[i];
+                 QString PBIdate2 = tmp2.remove( 0, tmp2.indexOf(" ") );
+
+                 // Check if the installed PBI is created later than the online update
+                 int ldate, odate;
+                 bool ok;
+                 ldate = date1.toInt(&ok);
+                 if ( ok )
+                 {
+                   odate = PBIdate1.toInt(&ok);
+                   if ( ok )
+                   {
+                     if ( ldate > odate )
+                     {
+                       updateAvail = false;
+                     } else if (ldate == odate) {
+                        // We checked the year / day, now check the hours seconds
+                        ldate = date2.toInt(&ok);
+                        if ( ok )
+                        {
+                           odate = PBIdate2.toInt(&ok);
+                           if ( ok )
+                           {
+                             if ( ldate > odate )
+                             {
+                               updateAvail = false;
+                             }
+                           }
+                        }
+                     }
+                   }
+                 }
+               
+                 /* QMessageBox::critical( 0, tr("Online Update"), "date1: " + date1, QMessageBox::Ok );
+                 QMessageBox::critical( 0, tr("Online Update"), "date2: " + date2, QMessageBox::Ok );
+                 QMessageBox::critical( 0, tr("Online Update"), "PBIdate1: " + PBIdate1, QMessageBox::Ok );
+                 QMessageBox::critical( 0, tr("Online Update"), "PBIdate2: " + PBIdate2, QMessageBox::Ok ); */
+               }
+             }
+
+              
+          }
+
+          if ( updateAvail ) {
+           QPixmap defaultIcon;
+           Q3CheckListItem *item = new Q3CheckListItem ( listViewPBIUpdates, text, Q3CheckListItem::CheckBox  ) ;
+           item->setText(1, tmp.setNum(i));
+
+           defaultIcon.convertFromImage(QImage(iconPath).smoothScale(32,32));
+
+           item->setPixmap ( 0, defaultIcon ) ;
+           listViewPBIUpdates->setColumnAlignment ( 0, Qt::AlignVCenter);
+
+           foundUpdates++;
+          }
        }
        i++;

pcbsd/trunk/kcmPBMsource/pbm.h

@@ -87 +87 @@
     void slotInstallUpdatesClicked();
     void slotCheckSysUpdatesFlags();
+    void slotReadPBIBuffer();
 private:
     void RemoveFiles( PBI * pbi );
@@ -168 +169 @@
     QString PBIProgURL3[900];
     QString PBIProgMD5[900];
+    QString PBIProgMdate[900];
     QString PBIProgLoc[900];
     int     PBIProgUpdate[900];
     int     PBIProgFailed[900];
+    QString PBIBuffer;
 signals: