FreeEMS Issues - Loader
View Issue Details
0000728LoaderGeneral Featurespublic2012-10-06 11:142012-10-06 23:12
Fred 
Fred 
normalminoralways
closedfixed 
LinuxDebianSid/Unstable
0.1.0-SNAPSHOT 
0.1.00.1.0 
Bug
low
0000728: Chosen file name for rip does not automatically get extension
If the typed name has .s19 then it should be used as is, if it doesn't it should be appended.
Calling it a bug because the save dialogue lists .s19 as an option. If it didn't the behaviour would be OK.
No tags attached.

Notes
(0002268)
sean94z   
2012-10-06 11:45   
Thx for taking the time to file the issue. Let me test my fix and if it checks out I'll push it up. Who should I assign verification to ?

thx
(0002269)
sean94z   
2012-10-06 12:09   
Please verify at your earliest convenience.

commit ce1889bdd204837f6328c6fa3da9d0af15b848e6
Author: Sean Keys <skeys@ipdatasys.com>
Date: Sat Oct 6 05:00:35 2012 -0700

    [0000728] Fixed and verified.
(0002270)
Fred   
2012-10-06 12:39   
(14:12:05) Fred: tells you you're going to overwrite the extensionless version, you hit ok, and it writes to the extensioned version
(14:12:31) Fred: bad on 2 counts < confirmed
(14:12:39) Fred: 1 pointless warning < confirmed
(14:12:41) Fred: 2 no warning < confirmed!

(14:36:00) Fred: it likely won't warn on subsequent over writes now
(14:37:06) Fred: yeah, just silently kills your existing file...
(0002272)
sean94z   
2012-10-06 19:36   
There are quite a few complaints about this from qt users. It seems the trick is to just be explicit when calling the fileDialoge. Please try this, when you have a min.

commit b2f8a4312af174670167526886cd7f63b0854de3
Author: Sean Keys <skeys@ipdatasys.com>
Date: Sat Oct 6 12:29:18 2012 -0700

    [0000728] Manually invoking the qtfiledialog and being explicit.
(0002273)
Fred   
2012-10-06 19:59   
Tested as fixed!
(0002275)
sean94z   
2012-10-06 20:56   
thx for testing :)
(0002277)
sean94z   
2012-10-06 23:12   
Took me so long to fix you would think I had paint in my eyes.

https://www.youtube.com/watch?v=vfxVaD-vVpY [^]