Review Request 121174: Fix checking for file overwriting on file creation

Jarosław Staniek staniek at kde.org
Thu Nov 20 14:39:44 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121174/
-----------------------------------------------------------

(Updated Nov. 20, 2014, 2:39 p.m.)


Status
------

This change has been marked as submitted.


Review request for Calligra, Andrius da Costa Ribas, Friedrich W. H. Kossebau, Adam Pigg, Roman Shtemberko, Radosław Wicik, and Wojciech Kosowicz.


Repository: calligra


Description
-------

Fix checking for file overwriting on file creation

Startup file handler: update filename after adding extension,
otherwise checking for overwriting isn't accurate.

Example of inaccurate behaviour:
User enters /tmp/a and /tmp/a does not exist.
Kexi changes it to /tmp/a.kexi but still checks if /tmp/a exists.
If /tmp/a.kexi exists, user won't be notified and asked.

In addition the handler now checks if user entered existing
directory path without filename; if so, it warns about that.
This avoids creating nonsense ".kexi" files (that look like hidden).

The fix works for CSV creation too.


Diffs
-----

  kexi/main/startup/KexiStartupFileHandler.cpp ff483e7 
  kexi/main/startup/KexiStartupFileHandler.h d203bb4 
  kexi/main/startup/KexiNewProjectAssistant.cpp 633d919 

Diff: https://git.reviewboard.kde.org/r/121174/diff/


Testing
-------

Creation works as before


Thanks,

Jarosław Staniek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141120/3b1e02c8/attachment.htm>


More information about the calligra-devel mailing list