Vcs: Remove unused GetRepositoryRootOperation

Change-Id: I2443aef788c75a96ba19e2499dfb3f17c201d468
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

src/plugins/bazaar/bazaarcontrol.cpp

@@ -86,7 +86,6 @@ bool BazaarControl::supportsOperation(Operation operation) const
     case Core::IVersionControl::MoveOperation:
     case Core::IVersionControl::CreateRepositoryOperation:
     case Core::IVersionControl::AnnotateOperation:
-    case Core::IVersionControl::GetRepositoryRootOperation:
         break;
     case Core::IVersionControl::SnapshotOperations:
         supported = false;
@@ -134,13 +133,6 @@ bool BazaarControl::vcsAnnotate(const QString &file, int line)
     return true;
 }
 
-QString BazaarControl::vcsGetRepositoryURL(const QString &directory)
-{
-    const QString repositoryRoot = m_bazaarClient->findTopLevelForFile(directory);
-    const BranchInfo branchInfo = m_bazaarClient->synchronousBranchQuery(repositoryRoot);
-    return branchInfo.isBoundToBranch ? branchInfo.branchLocation : QString();
-}
-
 void BazaarControl::changed(const QVariant &v)
 {
     switch (v.type()) {

src/plugins/bazaar/bazaarcontrol.h

@@ -62,7 +62,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
     bool vcsAnnotate(const QString &file, int line);
 
 public slots:

src/plugins/clearcase/clearcasecontrol.cpp

@@ -77,7 +77,6 @@ bool ClearCaseControl::supportsOperation(Operation operation) const
     case DeleteOperation:
     case MoveOperation:
     case AnnotateOperation:
-    case GetRepositoryRootOperation:
         break;
     case CreateRepositoryOperation:
     case SnapshotOperations:
@@ -139,11 +138,6 @@ bool ClearCaseControl::vcsMove(const QString &from, const QString &to)
     return m_plugin->vcsMove(ifrom.absolutePath(), ifrom.fileName(), ito.fileName());
 }
 
-QString ClearCaseControl::vcsGetRepositoryURL(const QString &directory)
-{
-    return m_plugin->vcsGetRepositoryURL(directory);
-}
-
 bool ClearCaseControl::managesDirectory(const QString &directory, QString *topLevel) const
 {
     return m_plugin->managesDirectory(directory, topLevel);

src/plugins/clearcase/clearcasecontrol.h

@@ -61,7 +61,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
 
     bool vcsAnnotate(const QString &file, int line);
 

src/plugins/clearcase/clearcaseplugin.cpp

@@ -1851,11 +1851,6 @@ bool ClearCasePlugin::vcsCheckout(const QString & /*directory*/, const QByteArra
     return false;
 }
 
-QString ClearCasePlugin::vcsGetRepositoryURL(const QString & /*directory*/)
-{
-    return currentState().topLevel();
-}
-
 ///
 /// Check if the directory is managed under ClearCase control.
 ///

src/plugins/clearcase/clearcaseplugin.h

@@ -144,7 +144,6 @@ public:
     bool vcsSetActivity(const QString &workingDir, const QString &title, const QString &activity);
     bool managesDirectory(const QString &directory, QString *topLevel = 0) const;
     bool vcsCheckout(const QString &directory, const QByteArray &url);
-    QString vcsGetRepositoryURL(const QString &directory);
 
     static ClearCasePlugin *instance();
 

src/plugins/coreplugin/iversioncontrol.h

@@ -56,8 +56,7 @@ public:
         AddOperation, DeleteOperation, MoveOperation,
         CreateRepositoryOperation,
         SnapshotOperations,
-        AnnotateOperation,
+        AnnotateOperation
-        GetRepositoryRootOperation
     };
 
     enum OpenSupportMode {
@@ -168,11 +167,6 @@ public:
      */
     virtual bool vcsCreateRepository(const QString &directory) = 0;
 
-    /*!
-     * Called to get the version control repository root.
-     */
-    virtual QString vcsGetRepositoryURL(const QString &directory) = 0;
-
     /*!
      * Topic (e.g. name of the current branch)
      */
@@ -245,7 +239,6 @@ public:
     bool vcsDelete(const QString &) { return false; }
     bool vcsMove(const QString &, const QString &) { return false; }
     bool vcsCreateRepository(const QString &) { return false; }
-    QString vcsGetRepositoryURL(const QString &) { return QString(); }
     bool vcsAnnotate(const QString &, int) { return false; }
 
 private:

src/plugins/cvs/cvscontrol.cpp

@@ -74,7 +74,6 @@ bool CvsControl::supportsOperation(Operation operation) const
     case MoveOperation:
     case CreateRepositoryOperation:
     case SnapshotOperations:
-    case GetRepositoryRootOperation:
         rc = false;
         break;
     }
@@ -117,11 +116,6 @@ bool CvsControl::vcsCreateRepository(const QString &)
     return false;
 }
 
-QString CvsControl::vcsGetRepositoryURL(const QString &)
-{
-    return QString();
-}
-
 bool CvsControl::vcsAnnotate(const QString &file, int line)
 {
     const QFileInfo fi(file);

src/plugins/cvs/cvscontrol.h

@@ -59,7 +59,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
     bool vcsAnnotate(const QString &file, int line);
 
     QString vcsOpenText() const;

src/plugins/git/gitclient.cpp

@@ -3472,20 +3472,6 @@ bool GitClient::cloneRepository(const QString &directory,const QByteArray &url)
     }
 }
 
-QString GitClient::vcsGetRepositoryURL(const QString &directory)
-{
-    QStringList arguments(QLatin1String("config"));
-    QByteArray outputText;
-
-    arguments << QLatin1String("remote.origin.url");
-
-    if (fullySynchronousGit(directory, arguments, &outputText, 0,
-                            VcsBasePlugin::SuppressCommandLogging)) {
-        return commandOutputFromLocal8Bit(outputText);
-    }
-    return QString();
-}
-
 GitSettings *GitClient::settings() const
 {
     return m_settings;

src/plugins/git/gitclient.h

@@ -253,7 +253,6 @@ public:
     bool isFastForwardMerge(const QString &workingDirectory, const QString &branch);
 
     bool cloneRepository(const QString &directory, const QByteArray &url);
-    QString vcsGetRepositoryURL(const QString &directory);
     void fetch(const QString &workingDirectory, const QString &remote);
     bool synchronousPull(const QString &workingDirectory, bool rebase);
     void push(const QString &workingDirectory, const QStringList &pushArgs = QStringList());

src/plugins/git/gitversioncontrol.cpp

@@ -98,7 +98,6 @@ bool GitVersionControl::supportsOperation(Operation operation) const
     case CreateRepositoryOperation:
     case SnapshotOperations:
     case AnnotateOperation:
-    case GetRepositoryRootOperation:
         return true;
     }
     return false;
@@ -133,11 +132,6 @@ bool GitVersionControl::vcsCreateRepository(const QString &directory)
     return m_client->synchronousInit(directory);
 }
 
-QString GitVersionControl::vcsGetRepositoryURL(const QString &directory)
-{
-    return m_client->vcsGetRepositoryURL(directory);
-}
-
 QString GitVersionControl::vcsTopic(const QString &directory)
 {
     QString topic = Core::IVersionControl::vcsTopic(directory);

src/plugins/git/gitversioncontrol.h

@@ -58,7 +58,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
 
     bool vcsAnnotate(const QString &file, int line);
     QString vcsTopic(const QString &directory);

src/plugins/mercurial/mercurialclient.cpp

@@ -251,18 +251,6 @@ QString MercurialClient::shortDescriptionSync(const QString &workingDirectory,
     return shortDescriptionSync(workingDirectory, revision, QLatin1String(defaultFormatC));
 }
 
-QString MercurialClient::vcsGetRepositoryURL(const QString &directory)
-{
-    QByteArray output;
-
-    QStringList arguments(QLatin1String("showconfig"));
-    arguments << QLatin1String("paths.default");
-
-    if (vcsFullySynchronousExec(directory, arguments, &output))
-        return QString::fromLocal8Bit(output);
-    return QString();
-}
-
 bool MercurialClient::managesFile(const QString &workingDirectory, const QString &fileName) const
 {
     QStringList args;

src/plugins/mercurial/mercurialclient.h

@@ -63,7 +63,6 @@ public:
     QString shortDescriptionSync(const QString &workingDirectory, const QString &revision);
     void incoming(const QString &repositoryRoot, const QString &repository = QString());
     void outgoing(const QString &repositoryRoot);
-    QString vcsGetRepositoryURL(const QString &directory);
     bool managesFile(const QString &workingDirectory, const QString &fileName) const;
 
     void annotate(const QString &workingDir, const QString &file,

src/plugins/mercurial/mercurialcontrol.cpp

@@ -112,7 +112,6 @@ bool MercurialControl::supportsOperation(Operation operation) const
     case Core::IVersionControl::MoveOperation:
     case Core::IVersionControl::CreateRepositoryOperation:
     case Core::IVersionControl::AnnotateOperation:
-    case Core::IVersionControl::GetRepositoryRootOperation:
         break;
     case Core::IVersionControl::SnapshotOperations:
         supported = false;
@@ -171,11 +170,6 @@ bool MercurialControl::sccManaged(const QString &filename)
     return mercurialClient->manifestSync(topLevel, topLevelDir.relativeFilePath(filename));
 }
 
-QString MercurialControl::vcsGetRepositoryURL(const QString &directory)
-{
-    return mercurialClient->vcsGetRepositoryURL(directory);
-}
-
 void MercurialControl::changed(const QVariant &v)
 {
     switch (v.type()) {

src/plugins/mercurial/mercurialcontrol.h

@@ -62,7 +62,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
     bool sccManaged(const QString &filename);
     bool vcsAnnotate(const QString &file, int line);
 

src/plugins/perforce/perforceversioncontrol.cpp

@@ -76,7 +76,6 @@ bool PerforceVersionControl::supportsOperation(Operation operation) const
         return supported;
     case CreateRepositoryOperation:
     case SnapshotOperations:
-    case GetRepositoryRootOperation:
         break;
     }
     return false;
@@ -133,11 +132,6 @@ bool PerforceVersionControl::vcsAnnotate(const QString &file, int line)
     return true;
 }
 
-QString PerforceVersionControl::vcsGetRepositoryURL(const QString &)
-{
-    return QString();
-}
-
 QString PerforceVersionControl::vcsOpenText() const
 {
     return tr("&Edit");

src/plugins/perforce/perforceversioncontrol.h

@@ -59,7 +59,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
     bool vcsAnnotate(const QString &file, int line);
     QString vcsOpenText() const;
     QString vcsMakeWritableText() const;

src/plugins/subversion/subversioncontrol.cpp

@@ -93,7 +93,6 @@ bool SubversionControl::supportsOperation(Operation operation) const
     case DeleteOperation:
     case MoveOperation:
     case AnnotateOperation:
-    case GetRepositoryRootOperation:
         break;
     case CreateRepositoryOperation:
     case SnapshotOperations:
@@ -128,11 +127,6 @@ bool SubversionControl::vcsMove(const QString &from, const QString &to)
     return m_plugin->vcsMove(fromInfo.absolutePath(), fromInfo.absoluteFilePath(), toInfo.absoluteFilePath());
 }
 
-QString SubversionControl::vcsGetRepositoryURL(const QString &directory)
-{
-    return m_plugin->vcsGetRepositoryURL(directory);
-}
-
 bool SubversionControl::vcsCreateRepository(const QString &)
 {
     return false;

src/plugins/subversion/subversioncontrol.h

@@ -57,7 +57,6 @@ public:
     bool vcsDelete(const QString &filename);
     bool vcsMove(const QString &from, const QString &to);
     bool vcsCreateRepository(const QString &directory);
-    QString vcsGetRepositoryURL(const QString &directory);
 
     bool vcsAnnotate(const QString &file, int line);
 

src/plugins/subversion/subversionplugin.cpp

@@ -1104,45 +1104,6 @@ bool SubversionPlugin::vcsCheckout(const QString &directory, const QByteArray &u
 
 }
 
-QString SubversionPlugin::vcsGetRepositoryURL(const QString &directory)
-{
-    QXmlStreamReader xml;
-    QStringList args = QStringList(QLatin1String("info"));
-    args << SubversionClient::addAuthenticationOptions(settings()) << QLatin1String("--xml");
-
-    const SubversionResponse response = runSvn(directory, args, 10 * m_settings.timeOutMs(), SuppressCommandLogging);
-    xml.addData(response.stdOut);
-
-    bool repo = false;
-    bool root = false;
-
-    while (!xml.atEnd() && !xml.hasError()) {
-        switch (xml.readNext()) {
-        case QXmlStreamReader::StartDocument:
-            break;
-        case QXmlStreamReader::StartElement:
-            if (xml.name() == QLatin1String("repository"))
-                repo = true;
-            else if (repo && xml.name() == QLatin1String("root"))
-                root = true;
-            break;
-        case QXmlStreamReader::EndElement:
-            if (xml.name() == QLatin1String("repository"))
-                repo = false;
-            else if (repo && xml.name() == QLatin1String("root"))
-                root = false;
-            break;
-        case QXmlStreamReader::Characters:
-            if (repo && root)
-                return xml.text().toString();
-            break;
-        default:
-            break;
-        }
-    }
-    return QString();
-}
-
 bool SubversionPlugin::managesDirectory(const QString &directory, QString *topLevel /* = 0 */) const
 {
     const QDir dir(directory);

src/plugins/subversion/subversionplugin.h

@@ -87,7 +87,6 @@ public:
     bool managesDirectory(const QString &directory, QString *topLevel = 0) const;
     bool managesFile(const QString &workingDirectory, const QString &fileName) const;
     bool vcsCheckout(const QString &directory, const QByteArray &url);
-    QString vcsGetRepositoryURL(const QString &directory);
 
     static SubversionPlugin *instance();