diff options
| author | Ilia Kats <[email protected]> | 2020-12-15 22:20:51 +0100 |
|---|---|---|
| committer | Ilia Kats <[email protected]> | 2020-12-17 13:04:01 +0100 |
| commit | acc42f918925983e2e83501cd2bacc7952a55718 (patch) | |
| tree | 343af939dc0b54b07070396d99bab4bc5f5b48fc /src/kitemviews/kfileitemlistwidget.cpp | |
| parent | d74853ef850fdabf2dfad8fcbe634af8185fce36 (diff) | |
fix display of folder sizes for empty folders
also fixes 1 byte error in size calculation for all folders
Diffstat (limited to 'src/kitemviews/kfileitemlistwidget.cpp')
| -rw-r--r-- | src/kitemviews/kfileitemlistwidget.cpp | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/src/kitemviews/kfileitemlistwidget.cpp b/src/kitemviews/kfileitemlistwidget.cpp index 69a40ddf3..66fcafaf6 100644 --- a/src/kitemviews/kfileitemlistwidget.cpp +++ b/src/kitemviews/kfileitemlistwidget.cpp @@ -53,20 +53,16 @@ QString KFileItemListWidgetInformant::roleText(const QByteArray& role, if (role == "size") { if (values.value("isDir").toBool()) { - // The item represents a directory. - if (!roleValue.isNull()) { - const int count = values.value("count").toInt(); - if (count > 0) { - if (DetailsModeSettings::directorySizeCount()) { - // Show the number of sub directories instead of the file size of the directory. - text = i18ncp("@item:intable", "%1 item", "%1 items", count); - } else { - // if we have directory size available - if (roleValue != -1) { - const KIO::filesize_t size = roleValue.value<KIO::filesize_t>(); - text = KFormat().formatByteSize(size); - } - } + if (!roleValue.isNull() && roleValue != -1) { + // The item represents a directory. + if (DetailsModeSettings::directorySizeCount()) { + // Show the number of sub directories instead of the file size of the directory. + const int count = values.value("count").toInt(); + text = i18ncp("@item:intable", "%1 item", "%1 items", count); + } else { + // if we have directory size available + const KIO::filesize_t size = roleValue.value<KIO::filesize_t>(); + text = KFormat().formatByteSize(size); } } } else { |
