Bug 440551

Summary: "Export current page" does nothing
Product: [Applications] plasma-systemmonitor Reporter: Syfer Polski <syferpl>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, nate, plasma-bugs-null
Priority: HI    
Version First Reported In: 5.22.4   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/plasma-systemmonitor/commit/d13ed9565150bba454036a99382de381a4620ebe Version Fixed/Implemented In: 5.22.5
Sentry Crash Report:

Description Syfer Polski 2021-08-03 10:44:14 UTC 
SUMMARY
System monitor has a button for exporting the current page. Unfortunately, it never saves a file.

STEPS TO REPRODUCE
1. Open system monitor
2. Press the hamburger(a button with three small horizontal lines)
3. Press "Export current page"
4. Choose a directory and file name
5. Press save

OBSERVED RESULT
No file created in the chosen directory

EXPECTED RESULT
A new file with the chosen name created in the chosen directory

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 5.13.4-1-default
(available in About System)
KDE Plasma Version: 5.22.4
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.2

ADDITIONAL INFORMATION
If you can tell me where plasma-systemmonitor stores its logs, I'm happy to add these. `sudo journalctl -b -u plasma-systemmonitor` found nothing
Comment 1 Nate Graham 2021-08-03 19:30:48 UTC 
Can reproduce.
Comment 2 Bug Janitor Service 2021-08-04 08:49:34 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/156
Comment 3 Nate Graham 2021-08-04 14:45:14 UTC 
Git commit 2eb00082c8a07fb3324643f8faf2d492fa8c1f9d by Nate Graham, on behalf of David Redondo.
Committed on 04/08/2021 at 14:44.
Pushed by ngraham into branch 'master'.

Fix page export

'page' is an oversight from when this was moved from EditablePage
during development.

M  +1    -1    src/main.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/2eb00082c8a07fb3324643f8faf2d492fa8c1f9d
Comment 4 Nate Graham 2021-08-04 14:45:42 UTC 
Git commit d13ed9565150bba454036a99382de381a4620ebe by Nate Graham, on behalf of David Redondo.
Committed on 04/08/2021 at 14:45.
Pushed by ngraham into branch 'Plasma/5.22'.

Fix page export

'page' is an oversight from when this was moved from EditablePage
during development.


(cherry picked from commit 2eb00082c8a07fb3324643f8faf2d492fa8c1f9d)

M  +1    -1    src/main.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/d13ed9565150bba454036a99382de381a4620ebe