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

[David Narváez]

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

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.h 8434372 
  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/20110928/ed19bf65/attachment.html>