Bug 371865

Summary: Exporting laps to CSV or JSON doesn’t work
Product: [Applications] kronometer Reporter: Karl Ove Hufthammer <karl>
Component: generalAssignee: Elvis Angelaccio <elvis.angelaccio>
Status: RESOLVED FIXED    
Severity: normal CC: cfeck
Priority: NOR    
Version First Reported In: 2.1.1   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://commits.kde.org/kronometer/6cc662646a3165f94a466bc086fe3c21ef4d704a Version Fixed/Implemented In: 2.2.0
Sentry Crash Report:

Description Karl Ove Hufthammer 2016-10-30 13:59:00 UTC 
User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0
Build Identifier: 

Exporting laps to CSV or JSON files silently fails. There is no error message, but no file is actually stored on disk.

Reproducible: Always

Steps to Reproduce:
1. Record a few laps.
2. Select ‘File → Export Laps as’.
3. Enter a file name and click ‘Save’.
Actual Results:  
No file is stored. No error message is shown.

Expected Results:  
A CSV or JSON files with the lap times should be stored in the selected folder.

This happens both if I use a file name with a .csv extension, a .json extension or no extension. I observe this behaviour with the latest Git version (as of 2016-10-31).
Comment 1 Elvis Angelaccio 2016-10-30 16:52:34 UTC 
Hi, thanks for the report. This is caused by a bug in the Plasma's file dialog and I'm trying to fix it there. Unfortunately it also requires new api in Qt, so it's taking quite some time to get this fixed.
Comment 2 Elvis Angelaccio 2017-06-11 10:31:27 UTC 
Git commit 6cc662646a3165f94a466bc086fe3c21ef4d704a by Elvis Angelaccio.
Committed on 11/06/2017 at 10:24.
Pushed by elvisangelaccio into branch 'master'.

Fix laps export

QFileDialog::selectedNameFilter() is broken in the Plasma file dialog.
Port to Qt 5.9's selectedMimeTypeFilter() which will work with
plasma-integration >= 5.10.
FIXED-IN: 2.2.0

M  +1    -1    CMakeLists.txt
M  +4    -4    src/gui/mainwindow.cpp
M  +2    -2    src/gui/mainwindow.h

https://commits.kde.org/kronometer/6cc662646a3165f94a466bc086fe3c21ef4d704a