Review Request 123383: Saving-as Kexi object should close window of object that will be overwritten

Jarosław Staniek staniek at kde.org
Thu Apr 16 18:14:53 BST 2015


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


Works! But... wouldn't it be simple to just use KexiProject::item(KexiPart::Info *i, const QString &name) in getNewObjectInfo() where name is originalName?


kexi/main/KexiMainWindow.cpp (line 2709)
<https://git.reviewboard.kde.org/r/123383/#comment54041>

    overwrited -> overwritten (everwhere)



kexi/main/KexiMainWindow.cpp (line 2719)
<https://git.reviewboard.kde.org/r/123383/#comment54042>

    Will this succeed in all cases?


- Jarosław Staniek


On April 16, 2015, 4:07 p.m., Roman Shtemberko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123383/
> -----------------------------------------------------------
> 
> (Updated April 16, 2015, 4:07 p.m.)
> 
> 
> Review request for Calligra, Adam Pigg, Jarosław Staniek, Radosław Wicik, and Wojciech Kosowicz.
> 
> 
> Bugs: 344656
>     http://bugs.kde.org/show_bug.cgi?id=344656
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> See [Bug 344656](https://bugs.kde.org/show_bug.cgi?id=344656)
> 
> 
> Diffs
> -----
> 
>   kexi/main/KexiMainWindow.cpp 532d3fb 
>   kexi/widget/KexiNameDialog.h 143f917 
>   kexi/widget/KexiNameDialog.cpp 09355d0 
> 
> Diff: https://git.reviewboard.kde.org/r/123383/diff/
> 
> 
> Testing
> -------
> 
> Window to be overwrited closes before actual overwriting.
> 
> 
> Thanks,
> 
> Roman Shtemberko
> 
>

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


More information about the calligra-devel mailing list