From b58f695cf8ed19fca6c0f4dc64cb27f37cc59b2b Mon Sep 17 00:00:00 2001
From: hjk <hjk@qt.io>
Date: Wed, 27 Sep 2017 12:06:11 +0200
Subject: [PATCH] ProjectExplorer: Sort dependency names alphabetically

Task-number: QTCREATORBUG-18843
Change-Id: Iba25e9ea292b34a7479e0456d005d02b678477f5
Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
---
 src/plugins/projectexplorer/dependenciespanel.cpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/plugins/projectexplorer/dependenciespanel.cpp b/src/plugins/projectexplorer/dependenciespanel.cpp
index fed8a6a653d..595b5ca5a7b 100644
--- a/src/plugins/projectexplorer/dependenciespanel.cpp
+++ b/src/plugins/projectexplorer/dependenciespanel.cpp
@@ -29,7 +29,9 @@
 
 #include <coreplugin/fileiconprovider.h>
 #include <coreplugin/icore.h>
+
 #include <utils/detailswidget.h>
+#include <utils/algorithm.h>
 
 #include <QDebug>
 #include <QSize>
@@ -47,10 +49,8 @@ namespace Internal {
 DependenciesModel::DependenciesModel(Project *project, QObject *parent)
     : QAbstractListModel(parent)
     , m_project(project)
-    , m_projects(SessionManager::projects())
 {
-    // We can't select ourselves as a dependency
-    m_projects.removeAll(m_project);
+    resetModel();
 
     SessionManager *sessionManager = SessionManager::instance();
     connect(sessionManager, &SessionManager::projectRemoved,
@@ -59,7 +59,6 @@ DependenciesModel::DependenciesModel(Project *project, QObject *parent)
             this, &DependenciesModel::resetModel);
     connect(sessionManager, &SessionManager::sessionLoaded,
             this, &DependenciesModel::resetModel);
-//    qDebug()<<"Dependencies Model"<<this<<"for project"<<project<<"("<<project->file()->fileName()<<")";
 }
 
 void DependenciesModel::resetModel()
@@ -67,6 +66,9 @@ void DependenciesModel::resetModel()
     beginResetModel();
     m_projects = SessionManager::projects();
     m_projects.removeAll(m_project);
+    Utils::sort(m_projects, [](Project *a, Project *b) {
+        return a->displayName() < b->displayName();
+    });
     endResetModel();
 }