# HG changeset patch
# User Chris Cannam
# Date 1300366270 0
# Node ID 93feb59187f5be16230ffdbd5d095f801eda6e0b
# Parent  383223b1dd345754f676586fb27ed8e1b7352a11
Add a dedicated dialog for "authorization failed" messages. Resolves #104

diff -r 383223b1dd34 -r 93feb59187f5 mainwindow.cpp
--- a/mainwindow.cpp	Wed Mar 16 19:44:51 2011 +0000
+++ b/mainwindow.cpp	Thu Mar 17 12:51:10 2011 +0000
@@ -1722,6 +1722,16 @@
     }
 }
 
+void MainWindow::reportAuthFailed(QString output)
+{
+    MoreInformationDialog::warning
+        (this,
+         tr("Authorization failed"),
+         tr("Authorization failed"),
+         tr("You may have entered an incorrect user name or password, or the remote URL may be wrong.<br><br>Or you may lack the necessary permissions on the remote repository.<br><br>Check with the administrator of your remote repository if necessary."),
+         output);
+}
+
 void MainWindow::commandStarting(HgAction action)
 {
     // Annoyingly, hg stat actually modifies the working directory --
@@ -1747,7 +1757,14 @@
     setstr = tr("Settings");
 #endif
 
-    // Some commands we just have to ignore bad return values from:
+    // Some commands we just have to ignore bad return values from,
+    // and some output gets special treatment.
+
+    // Note our fallback case should always be to report a
+    // non-specific error and show the text -- in case output scraping
+    // fails (as it surely will).  Note also that we must force the
+    // locale in order to ensure the output is scrapable; this happens
+    // in HgRunner and may break some system encodings.
 
     switch(action.action) {
     case ACT_NONE:
@@ -1777,9 +1794,16 @@
         enableDisableActions();
         break; // go on to default report
     case ACT_INCOMING:
-        // returns non-zero code and no output if the check was
-        // successful but there are no changes pending
-        {
+        if (output.contains("authorization failed")) {
+            reportAuthFailed(output);
+            return;
+        } else {
+            // Incoming returns non-zero code and no output if the
+            // check was successful but there are no changes
+            // pending. This is the only case where we need to remove
+            // warning messages, because it's the only case where a
+            // non-zero code can be returned even though the command
+            // has for our purposes succeeded
             QString replaced = output;
             while (1) {
                 QString r1 = replaced;
@@ -1787,14 +1811,27 @@
                 if (r1 == replaced) break;
                 replaced = r1.trimmed();
             }
-//            DEBUG << "incoming: text is " << output << endl;
-//            DEBUG << "replaced is " << replaced << endl;
             if (replaced == "") {
                 showIncoming("");
                 return;
             }
         }
         break; // go on to default report
+    case ACT_PULL:
+        if (output.contains("authorization failed")) {
+            reportAuthFailed(output);
+            return;
+        }
+        break; // go on to default report
+    case ACT_PUSH:
+        if (output.contains("creates new remote heads")) {
+            reportNewRemoteHeads(output);
+            return;
+        } else if (output.contains("authorization failed")) {
+            reportAuthFailed(output);
+            return;
+        }
+        break; // go on to default report
     case ACT_QUERY_HEADS:
         // fails if repo is empty; we don't care (if there's a genuine
         // problem, something else will fail too).  Pretend it
@@ -1808,12 +1845,6 @@
         // and some return with failure codes when something as basic
         // as the user closing the window via the wm happens
         return;
-    case ACT_PUSH:
-        if (output.contains("creates new remote heads")) {
-            reportNewRemoteHeads(output);
-            return;
-        }
-        break; // go on to default report
     case ACT_MERGE:
     case ACT_RETRY_MERGE:
         MoreInformationDialog::information
diff -r 383223b1dd34 -r 93feb59187f5 mainwindow.h
--- a/mainwindow.h	Wed Mar 16 19:44:51 2011 +0000
+++ b/mainwindow.h	Thu Mar 17 12:51:10 2011 +0000
@@ -125,6 +125,7 @@
     void readSettings();
     void splitChangeSets(QStringList *list, QString hgLogOutput);
     void reportNewRemoteHeads(QString);
+    void reportAuthFailed(QString);
     void writeSettings();
 
     QStringList listAllUpIpV4Addresses();