Review Request: Avoid processing parameters if the number of arguments is not met in Text Labels

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102725/#review7187
-----------------------------------------------------------


This review has been submitted with commit ead886e476ed8be103bbdb3cdc763c0f526f9206 by David E. Narváez to branch KDE/4.7.

- Commit Hook


On Sept. 29, 2011, 4 a.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102725/
> -----------------------------------------------------------
> 
> (Updated Sept. 29, 2011, 4 a.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Description
> -------
> 
> Changed the finished variable to finish, as I think is the way the code was intended to be. Also fixes the second validation that took place (calling canFinish in the finishPressed method) since first validation happens in the validatePage method of the Arguments page.
> 
> 
> This addresses bug 282913.
>     http://bugs.kde.org/show_bug.cgi?id=282913
> 
> 
> Diffs
> -----
> 
>   modes/label.cc d0c855d 
>   modes/textlabelwizard.cc 0721227 
> 
> Diff: http://git.reviewboard.kde.org/r/102725/diff/diff
> 
> 
> Testing
> -------
> 
> 1) Created a new label
> 2) Introduced a parameter %1
> 3) Clicked Next
> 4) Clicked Finish without specifying a parameter
> 5) Clicked Ok in the error popup
> 
> No crash, no second validation.
> 
> 
> Thanks,
> 
> David Narváez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20111009/228f337d/attachment.html>