Bug 175999 - saving scrollback history overwrites existing file
Summary: saving scrollback history overwrites existing file
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: 2.1
Platform: Ubuntu Unspecified
: NOR normal
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-24 17:56 UTC by Dovydas
Modified: 2008-12-31 19:52 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dovydas 2008-11-24 17:56:09 UTC
Version:           2.1 (using KDE 4.1.2)
Installed from:    Ubuntu Packages

I am trying to save scrollback output. 
I click on "Scrollback" - > "Save Output..." and click on any existing file and - OUCH! - the original file gets immediately overwritten!

I expect the clicked file name to appear in the filename field so I could edit the filename. I don't expect it to be immediately overwritten.

I have lost an important file! It happens regardless of which global mouse mode is enabled: single-click or double-click.
Comment 1 Dario Andres 2008-12-27 22:29:23 UTC
Here using:

Qt: 4.4.3 + qt-copy-patches-889120
KDE: 4.1.86 (KDE 4.1.86 (KDE 4.2 >= 20081221))
kdelibs svn rev. 902031 / kdebase svn rev. 902032
on ArchLinux x86_64 - Kernel 2.6.27.10

I can confirm this bug. The file dialog doesn't ask for overwriting the file.

The file gets overwriting when clicking it in the file list (I have "single click" setting), or when selecting it (right click to only select the file) and then pressing "OK" 

(that should be a bug in kfile file dialog, but I haven't seen anything like this before..). Adding kdelibs to CC
Comment 2 Robert Knight 2008-12-28 01:01:34 UTC
Hi Dario,

I'm marking this as normal rather than grave because it will only affect a relatively small number of users in exceptional cases - I usually reserve 'grave' and 'critical' for things which affect a large number of users on a regular basis.
Comment 3 Dario Andres 2008-12-31 19:52:56 UTC
Here using:

Qt: 4.4.3 + qt-copy-patches-889120
KDE: 4.1.86 (KDE 4.1.86 (KDE 4.2 >= 20081221))
kdelibs svn rev. 903705 / kdebase svn rev. 903706
on ArchLinux x86_64 - Kernel 2.6.27.10

This seems to be fixed (since r902755 commit)