Add rename local project option

app/CMakeLists.txt

@@ -241,6 +241,7 @@ if (ENABLE_TESTS)
       test/testimageutils.cpp
       test/testlayertree.cpp
       test/testlinks.cpp
+      test/testlocalprojectsmanager.cpp
       test/testmaptools.cpp
       test/testmerginapi.cpp
       test/testmodels.cpp
@@ -268,6 +269,7 @@ if (ENABLE_TESTS)
       test/testimageutils.h
       test/testlayertree.h
       test/testlinks.h
+      test/testlocalprojectsmanager.h
       test/testmaptools.h
       test/testmerginapi.h
       test/testmodels.h

app/projectsmodel.cpp

@@ -53,6 +53,7 @@ void ProjectsModel::initializeProjectsModel()
   QObject::connect( mLocalProjectsManager, &LocalProjectsManager::localProjectAdded, this, &ProjectsModel::onProjectAdded );
   QObject::connect( mLocalProjectsManager, &LocalProjectsManager::aboutToRemoveLocalProject, this, &ProjectsModel::onAboutToRemoveProject );
   QObject::connect( mLocalProjectsManager, &LocalProjectsManager::localProjectDataChanged, this, &ProjectsModel::onProjectDataChanged );
+  QObject::connect( mLocalProjectsManager, &LocalProjectsManager::localProjectRenamed, this, &ProjectsModel::onProjectRenamed );
   QObject::connect( mLocalProjectsManager, &LocalProjectsManager::dataDirReloaded, this, &ProjectsModel::loadLocalProjects );
 
   emit modelInitialized();
@@ -380,6 +381,32 @@ void ProjectsModel::removeLocalProject( const QString &projectId )
   mLocalProjectsManager->removeLocalProject( projectId );
 }
 
+QString ProjectsModel::renameLocalProject( const QString &projectId, const QString &newName )
+{
+  return mLocalProjectsManager->renameLocalProject( projectId, newName );
+}
+
+void ProjectsModel::onProjectRenamed( const QString &oldProjectId, const LocalProject &localProject )
+{
+  int ix = projectIndexFromId( oldProjectId );
+
+  if ( ix < 0 )
+    return;
+
+  Project &project = mProjects[ix];
+
+  project.local = localProject;
+
+  QModelIndex editIndex = index( ix );
+  emit dataChanged( editIndex, editIndex );
+
+  if ( mActiveProjectId == oldProjectId )
+  {
+    mActiveProjectId = localProject.id();
+    emit activeProjectIdChanged( mActiveProjectId );
+  }
+}
+
 void ProjectsModel::migrateProject( const QString &projectId )
 {
   int ix = projectIndexFromId( projectId );

app/projectsmodel.h

@@ -127,6 +127,9 @@ class ProjectsModel : public QAbstractListModel
     //! Forwards call to LocalProjectsManager to remove local project
     Q_INVOKABLE void removeLocalProject( const QString &projectId );
 
+    //! Forwards call to LocalProjectsManager to rename local project
+    Q_INVOKABLE QString renameLocalProject( const QString &projectId, const QString &newName );
+
     //! Migrates local project to mergin
     Q_INVOKABLE void migrateProject( const QString &projectId );
 
@@ -170,6 +173,7 @@ class ProjectsModel : public QAbstractListModel
     void onProjectAdded( const LocalProject &project );
     void onAboutToRemoveProject( const LocalProject &project );
     void onProjectDataChanged( const LocalProject &project );
+    void onProjectRenamed( const QString &oldProjectId, const LocalProject &project );
 
     void onAuthChanged();
 

app/qml/CMakeLists.txt

@@ -184,6 +184,7 @@ set(MM_QML
     project/components/MMProjectStatusItem.qml
     project/components/MMProjectWizardDelegate.qml
     project/components/MMProjectDelegate.qml
+    project/components/MMRenameProjectDialog.qml
     settings/MMAboutPage.qml
     settings/MMChangelogPage.qml
     settings/MMLogPage.qml

app/qml/project/MMProjectList.qml

@@ -139,7 +139,7 @@ Item {
           return ["changes", "remove"]
         }
         else if ( !model.ProjectIsMergin && model.ProjectIsLocal ) {
-          return ["upload", "remove"]
+          return ["upload", "remove", "rename"]
         }
         return ["download"]
       }
@@ -176,6 +176,10 @@ Item {
       }
       onStopSyncRequested: controllerModel.stopProjectSync( projectId )
       onShowChangesRequested: root.showLocalChangesRequested( projectId )
+      onRenameRequested: () => {
+        internal.projectIdToRename = projectId
+        renameDialog.open()
+      }
     }
   }
 
@@ -306,6 +310,25 @@ Item {
     }
   }
 
+  MMProjectComponents.MMRenameProjectDialog {
+    id: renameDialog
+
+    onRenameClicked: function( newName ) {
+      if ( !internal.projectIdToRename ) {
+        return
+      }
+
+      const renameResult = controllerModel.renameLocalProject( internal.projectIdToRename, newName )
+
+      if ( !renameResult ) {
+        renameDialog.close()
+      }
+      else {
+        renameDialog.errorText = renameResult
+      }
+    }
+  }
+
   MMDownloadProjectDialog {
     id: downloadProjectDialog
 
@@ -314,4 +337,10 @@ Item {
       downloadProjectDialog.relatedProjectId = ""
     }
   }
+
+  QtObject {
+    id: internal
+
+    property string projectIdToRename: ""
+  }
 }

app/qml/project/components/MMProjectDelegate.qml

@@ -25,7 +25,7 @@ Control {
   property bool projectIsInSync: false
   property real projectSyncProgress: 0.0
 
-  property var projectActionButtons: [] // possible values: upload, changes, sync, download, remove
+  property var projectActionButtons: [] // possible values: upload, changes, sync, download, remove, rename
 
   property bool projectIsOpened: false
 
@@ -35,6 +35,7 @@ Control {
   signal removeRequested()
   signal stopSyncRequested()
   signal showChangesRequested()
+  signal renameRequested()
 
   height: implicitHeight
 
@@ -307,6 +308,11 @@ Control {
         "name": qsTr("Upload"),
         "iconSource": __style.uploadIcon,
         "callback": () => root.migrateRequested()
+      },
+      "rename": {
+        "name": qsTr("Rename the local project"),
+        "iconSource": __style.editIcon,
+        "callback": () => root.renameRequested()
       }
     }
   }

app/qml/project/components/MMRenameProjectDialog.qml

@@ -0,0 +1,57 @@
+/***************************************************************************
+ *                                                                         *
+ *   This program is free software; you can redistribute it and/or modify  *
+ *   it under the terms of the GNU General Public License as published by  *
+ *   the Free Software Foundation; either version 2 of the License, or     *
+ *   (at your option) any later version.                                   *
+ *                                                                         *
+ ***************************************************************************/
+
+import QtQuick
+
+import "../../components"
+import "../../inputs"
+
+MMDrawer {
+  id: root
+
+  property alias errorText: newNameField.errorMsg
+
+  signal renameClicked( string newName )
+
+  drawerHeader.title: qsTr( "Rename project" )
+  drawerHeader.titleFont: __style.t2
+
+  onAboutToShow: () => {
+    newNameField.errorMsg = ""
+    newNameField.text = ""
+  }
+
+  drawerContent: Column {
+    id: contentColumn
+
+    width: parent.width
+    spacing: newNameField.errorMsg ? __style.margin12 : __style.spacing40
+
+    MMTextInput {
+      id: newNameField
+
+      width: contentColumn.width
+      textFieldBackground.color: __style.lightGreenColor
+
+      placeholderText: qsTr( "Enter the new name" )
+
+      onTextEdited: () => newNameField.errorMsg = ""
+    }
+
+    MMButton {
+      width: contentColumn.width
+
+      text: qsTr( "Confirm" )
+
+      onClicked: {
+        root.renameClicked( newNameField.text )
+      }
+    }
+  }
+}

app/test/inputtests.cpp

@@ -27,6 +27,7 @@
 #include "testimageutils.h"
 #include "testmaptools.h"
 #include "testlayertree.h"
+#include "testlocalprojectsmanager.h"
 #include "testactiveproject.h"
 #include "testprojectchecksumcache.h"
 #include "testmultieditmanager.h"
@@ -172,6 +173,11 @@ int InputTests::runTest() const
     TestLayerTree layerTreeTest;
     nFailed = QTest::qExec( &layerTreeTest, mTestArgs );
   }
+  else if ( mTestRequested == "--testLocalProjectsManager" )
+  {
+    TestLocalProjectsManager localProjectsManagerTest;
+    nFailed = QTest::qExec( &localProjectsManagerTest, mTestArgs );
+  }
   else if ( mTestRequested == "--testActiveProject" )
   {
     TestActiveProject activeProjectTest( mApi );