Handle url rediredcts with changed trailing slashes

1 files changed, 21 insertions, 8 deletions

diff --git a/src/gui/accountstate.cpp b/src/gui/accountstate.cpp
index ef3286704..86cc4bdc0 100644
--- a/src/gui/accountstate.cpp
+++ b/src/gui/accountstate.cpp
@@ -33,20 +33,33 @@ Q_LOGGING_CATEGORY(lcAccountState, "gui.account.state", QtInfoMsg)
 
 void AccountState::updateUrlDialog(AccountPtr account, const QUrl &newUrl, std::function<void()> callback)
 {
-    auto diag = new QMessageBox(QMessageBox::Warning, tr("Url update requested for %1").arg(account->displayName()),
-        tr("The url for %1 changed from %2 to %3, do you want to accept the changed url").arg(account->displayName(), account->url().toString(), newUrl.toString()),
-        QMessageBox::NoButton, ocApp()->gui()->settingsDialog());
-    diag->setAttribute(Qt::WA_DeleteOnClose);
-    auto yes = diag->addButton(tr("Change url permanently to %1").arg(newUrl.toString()), QMessageBox::AcceptRole);
-    diag->addButton(tr("Reject"), QMessageBox::RejectRole);
-    QObject::connect(diag, &QMessageBox::finished, account.data(), [diag, yes, newUrl, account, callback] {
-        if (diag->clickedButton() == yes) {
+    auto accept = [=](bool accepted) {
+        if (accepted) {
             account->setUrl(newUrl);
             Q_EMIT account->wantsAccountSaved(account.data());
         }
         if (callback) {
             callback();
         }
+    };
+    // the urls are identical, previous versions of owncloud cropped the /
+    if (newUrl.path() == QLatin1Char('/')) {
+        auto tmp = newUrl;
+        tmp.setPath(QString());
+        if (tmp == account->url()) {
+            // silently accept the /
+            accept(true);
+            return;
+        }
+    }
+    auto diag = new QMessageBox(QMessageBox::Warning, tr("Url update requested for %1").arg(account->displayName()),
+        tr("The url for %1 changed from %2 to %3, do you want to accept the changed url?").arg(account->displayName(), account->url().toString(), newUrl.toString()),
+        QMessageBox::NoButton, ocApp()->gui()->settingsDialog());
+    diag->setAttribute(Qt::WA_DeleteOnClose);
+    auto yes = diag->addButton(tr("Change url permanently to %1").arg(newUrl.toString()), QMessageBox::AcceptRole);
+    diag->addButton(tr("Reject"), QMessageBox::RejectRole);
+    QObject::connect(diag, &QMessageBox::finished, account.data(), [diag, yes, accept] {
+        accept(diag->clickedButton() == yes);
     });
     diag->show();
 }