Added version handling

1 files changed, 59 insertions, 22 deletions

diff --git a/noncore/settings/aqpkg/networkpkgmgr.cpp b/noncore/settings/aqpkg/networkpkgmgr.cpp
index 3971aea..02e4e73 100644
--- a/noncore/settings/aqpkg/networkpkgmgr.cpp
+++ b/noncore/settings/aqpkg/networkpkgmgr.cpp
@@ -45,2 +45,4 @@ using namespace std;
 
+extern int compareVersions( const char *v1, const char *v2 );
+
 NetworkPackageManager::NetworkPackageManager( DataManager *dataManager, QWidget *parent, const char *name)
@@ -234,3 +236,6 @@ void NetworkPackageManager :: serverSelected( int )
             if ( it->getVersion() != it->getInstalledVersion() )
-                    text += "*";
+            {
+                if ( compareVersions( it->getInstalledVersion(), it->getVersion() ) == 1 )
+                    text += "*";
+            }
         }
@@ -535,18 +540,12 @@ InstallData NetworkPackageManager :: dealWithItem( QCheckListItem *item )
 
-        // Sticky option not implemented yet, but will eventually allow
-        // the user to say something like 'remove all'
-        if ( stickyOption == "" )
+        // Now see if version is newer or not
+        int val = compareVersions( p->getInstalledVersion(), p->getVersion() );
+        if ( val == -2 )
         {
-            QString msgtext;
-            msgtext.sprintf( "Do you wish to remove or reinstall\n%s?", (const char *)name );
-            switch( QMessageBox::information( this, "Remove or ReInstall",
-                                msgtext, "Remove", "ReInstall" ) )
-            {
-                case 0: // Try again or Enter
-                    item.option = "D";
-                    break;
-                case 1: // Quit or Escape
-                    item.option = "U";
-                    break;
-            }
+            // Error - should handle
+        }
+        else if ( val == -1 )
+        {
+            // Version available is older - remove only
+            item.option = "R";
         }
@@ -554,14 +553,52 @@ InstallData NetworkPackageManager :: dealWithItem( QCheckListItem *item )
         {
-//            item.option = stickyOption;
+            QString caption;
+            QString text;
+            QString secondButton;
+            QString secondOption;
+            if ( val == 0 )
+            {
+                // Version available is the same - option to remove or reinstall
+                caption = "Do you wish to remove or reinstall\n%s?";
+                text = "Remove or ReInstall";
+                secondButton = "ReInstall";
+                secondOption = "R";
+            }
+            else if ( val == 1 )
+            {
+                // Version available is newer - option to remove or upgrade
+                caption = "Do you wish to remove or upgrade\n%s?";
+                text = "Remove or Upgrade";
+                secondButton = "Upgrade";
+                secondOption = "U";
+            }
+
+            // Sticky option not implemented yet, but will eventually allow
+            // the user to say something like 'remove all'
+            if ( stickyOption == "" )
+            {
+                QString msgtext;
+                msgtext.sprintf( caption, (const char *)name );
+                switch( QMessageBox::information( this, text,
+                                    msgtext, "Remove", secondButton ) )
+                {
+                    case 0: // Try again or Enter
+                        item.option = "D";
+                        break;
+                    case 1: // Quit or Escape
+                        item.option = secondOption;
+                        break;
+                }
+            }
+            else
+            {
+//                item.option = stickyOption;
+            }
         }
+
         
         // Check if we are reinstalling the same version
-        if ( p->getVersion() != p->getInstalledVersion() )
+        if ( item.option != "R" )
            item.recreateLinks = true;
         else
-        {
-            if ( item.option == "U" )
-                item.option = "R";
            item.recreateLinks = false;
-        }