Review Request 121174: Fix checking for file overwriting on file creation
Jarosław Staniek
staniek at kde.org
Tue Nov 18 23:58:32 GMT 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121174/
-----------------------------------------------------------
Review request for Calligra, Andrius da Costa Ribas, Friedrich W. H. Kossebau, Adam Pigg, 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 ff483e75e26cd2891eab1ce2789f7858dec55dcf
kexi/main/startup/KexiNewProjectAssistant.cpp 633d91988bef3331d72fcbd3d9804456990c1404
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/20141118/e0c6eb3d/attachment.htm>
More information about the calligra-devel
mailing list