ProjectExplorer: Simplify node creation

Simplifies node creation by not trying to manually split / concatenate
paths.

Change-Id: I5cb4e9391eb8aef6e6942b8b424ac6ef451d1aff
Reviewed-by: hjk <hjk@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
This commit is contained in:
Marcus Tillmanns
2022-10-18 08:16:02 +02:00
parent 5b7f5fc093
commit d1284570d6

View File

@@ -39,37 +39,24 @@ static FolderNode *recursiveFindOrCreateFolderNode(FolderNode *folder,
const FilePath &overrideBaseDir,
const FolderNode::FolderNodeFactory &factory)
{
Utils::FilePath path = overrideBaseDir.isEmpty() ? folder->filePath() : overrideBaseDir;
QList<FilePath> paths;
const Utils::FilePath basePath = overrideBaseDir.isEmpty() ? folder->filePath()
: overrideBaseDir;
Utils::FilePath path = basePath.isEmpty() ? directory : basePath.relativeChildPath(directory);
Utils::FilePath directoryWithoutPrefix;
bool isRelative = false;
if (path.isEmpty() || path.isRootPath()) {
directoryWithoutPrefix = directory;
isRelative = false;
} else {
if (directory.isChildOf(path) || directory == path) {
isRelative = true;
directoryWithoutPrefix = directory.relativeChildPath(path);
} else {
isRelative = false;
path.clear();
directoryWithoutPrefix = directory;
}
while (!path.isEmpty()) {
paths.append(path);
path = path.parentDir();
}
QStringList parts = directoryWithoutPrefix.path().split('/', Qt::SkipEmptyParts);
if (directory.osType() != OsTypeWindows && !isRelative && !parts.isEmpty())
parts[0].prepend('/');
std::reverse(std::begin(paths), std::end(paths));
ProjectExplorer::FolderNode *parent = folder;
for (const QString &part : std::as_const(parts)) {
path = path.pathAppended(part);
// Find folder in subFolders
FolderNode *next = parent->folderNode(path);
FolderNode *parent = folder;
for (const auto &currentPath : paths) {
FolderNode *next = parent->folderNode(currentPath);
if (!next) {
// No FolderNode yet, so create it
auto tmp = factory(path);
tmp->setDisplayName(part);
auto tmp = factory(currentPath);
tmp->setDisplayName(currentPath.fileName());
next = tmp.get();
parent->addNode(std::move(tmp));
}