kde/dolphin
kde/dolphin
1
0
Fork 0
mirror of https://invent.kde.org/system/dolphin synced 2024-08-22 01:40:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

KFileItemModel: DetailsModeSettings::directorySizeCount forces m_sortDirsFirst

Browse source 
Before this patch when !m_sortDirsFirst and DetailsModeSettings::directorySizeCount() == true, the ordering of folders before files would be affected by the sortOrder. I.e descending order would put the folders after the files.
This commit is contained in:
Méven Car 2021-03-11 08:09:18 +01:00
parent c0e3a5afb2
commit 0429e5330e
1 changed files with 16 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
src/kitemviews/kfileitemmodel.cpp
View file

@ -1719,7 +1719,7 @@ bool KFileItemModel::lessThan(const ItemData* a, const ItemData* b, const QColla
}
}
if (m_sortDirsFirst) {
if (m_sortDirsFirst || (DetailsModeSettings::directorySizeCount() && m_sortRole == SizeRole)) {
const bool isDirA = a->item.isDir();
const bool isDirB = b->item.isDir();
if (isDirA && !isDirB) {
@ -1768,30 +1768,25 @@ int KFileItemModel::sortRoleCompare(const ItemData* a, const ItemData* b, const
break;
case SizeRole: {
if (DetailsModeSettings::directorySizeCount() && (itemA.isDir() || itemB.isDir())) {
if (DetailsModeSettings::directorySizeCount() && itemA.isDir()) {
// folders first then
if (itemA.isDir() && itemB.isDir()) {
auto valueA = a->values.value("count");
auto valueB = b->values.value("count");
if (valueA.isNull()) {
if (valueB.isNull()) {
return 0;
} else {
return -1;
}
} else if (valueB.isNull()) {
return +1;
// items A and B are folders thanks to lessThan checks
auto valueA = a->values.value("count");
auto valueB = b->values.value("count");
if (valueA.isNull()) {
if (valueB.isNull()) {
return 0;
} else {
if (valueA.toLongLong() < valueB.toLongLong()) {
return -1;
} else {
return +1;
}
return -1;
}
} else if (itemA.isDir()) {
return 1;
} else if (valueB.isNull()) {
return +1;
} else {
return -1;
if (valueA.toLongLong() < valueB.toLongLong()) {
return -1;
} else {
return +1;
}
}
}
KIO::filesize_t sizeA = 0;