Bug 443751 - When pushing "Rescan" the number of GB already scanned doesn't reset to 0.
Summary: When pushing "Rescan" the number of GB already scanned doesn't reset to 0.
Status: RESOLVED FIXED
Alias: None
Product: filelight
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Debian stable Linux
: NOR normal
Target Milestone: ---
Assignee: Martin Sandsmark
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-14 23:18 UTC by Tyler
Modified: 2022-04-26 19:44 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Video shows me rescanning the drive and the vyte count not reseting. (2.25 MB, video/mp4)
2021-10-14 23:18 UTC, Tyler
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tyler 2021-10-14 23:18:21 UTC
Created attachment 142449 [details]
Video shows me rescanning the drive and the vyte count not reseting.

SUMMARY
When you scan a drive with filelight, you have the option to rescan at the top. If you push rescan, it shows the progress in the middle. The progress should reset to 0 everytime you rescan, but instead it keeps building up more and more. I am extremely aware of how poorly I'm describing this, so I've attached a video.

STEPS TO REPRODUCE
1. Open filelight
2. Scan your drive and wait for it to finish
3. Click rescan at the top

OBSERVED RESULT
The total number of bytes scanned keeps increasing with each rescan

EXPECTED RESULT
The total number of bytes scanned should reset to 0 each time you rescan

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 5.20.5
KDE Plasma Version: 5.20.5
KDE Frameworks Version: 5.78.0
Qt Version: 5.15.2
Comment 1 Harald Sitter 2022-04-26 19:44:32 UTC
Git commit e873c3bfa2e6f971e1699a5377a0e46c7e363442 by Harald Sitter.
Committed on 26/04/2022 at 19:21.
Pushed by sitter into branch 'master'.

grand qml rewrite

this moves the entire main ui (excluding settings) to qtquick from
qwidgets.

this loses some of the more esoteric kxmlgui features but should be a
good path forward on the balance

the actual map is still being qpainted, so this isn't super ideal but
the plan there is to hopefully use kquickcharts eventually instead of a
custom implementation.

settings dialog still needs porting but is kept qwidget for now to at
least reduce the scope of this port slightly
Related: bug 445018, bug 445026, bug 445027, bug 449688

M  +2    -1    .kde-ci.yml
M  +2    -1    CMakeLists.txt
M  +6    -9    src/CMakeLists.txt
M  +6    -5    src/Config.h
M  +1    -1    src/app/CMakeLists.txt
M  +5    -2    src/fileTree.h
M  +9    -23   src/main.cpp
A  +264  -0    src/mainContext.cpp     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
A  +81   -0    src/mainContext.h     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
D  +0    -587  src/mainWindow.cpp
D  +0    -118  src/mainWindow.h
D  +0    -71   src/overviewWidget.cpp
D  +0    -23   src/overviewWidget.h
D  +0    -121  src/progressBox.cpp
D  +0    -48   src/progressBox.h
A  +14   -0    src/qml/AboutPage.qml     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
A  +9    -0    src/qml/Action.qml     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
A  +165  -0    src/qml/MapPage.qml     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
A  +72   -0    src/qml/OverviewPage.qml     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
A  +183  -0    src/qml/main.qml     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
A  +13   -0    src/qml/qml.qrc
A  +906  -0    src/radialMap/item.cpp     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
R  +36   -43   src/radialMap/item.h [from: src/radialMap/widget.h - 051% similarity]
D  +0    -308  src/radialMap/labels.cpp
A  +42   -0    src/radialMap/labels.h     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
M  +20   -8    src/radialMap/map.cpp
M  +4    -1    src/radialMap/map.h
A  +17   -0    src/radialMap/radialMap.cpp     [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)]
D  +0    -212  src/radialMap/widget.cpp
D  +0    -395  src/radialMap/widgetEvents.cpp
M  +10   -1    src/scan.cpp
M  +8    -3    src/scan.h

https://invent.kde.org/utilities/filelight/commit/e873c3bfa2e6f971e1699a5377a0e46c7e363442