Review Request 129709: Fix checking for valid date entered

David Jarvie djarvie at kde.org
Sat Dec 31 00:13:33 UTC 2016


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

(Updated Dec. 31, 2016, 12:13 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and John Layt.


Changes
-------

Submitted with commit 1550b2c83f4842eb491b906ff7029fc11fd07d8a by David Jarvie to branch master.


Repository: kwidgetsaddons


Description
-------

Checks to determine whether an entered date is valid are wrong or missing in various places in the code. This is due to:
- not testing if a minimum or maximum date is set before comparing a date to the minimum/maximum;
- always rejecting new minimum/maximum dates if the other maximum/minimum date is not set;
- not testing dates for validity when set by the date picker or menu.

This results in the following bugs currently:
- When an up/down arrow or page up/down key is pressed to change the date, and the minimum and maximum dates are not set, it is always considered invalid and the date is not changed.
- When the DateKeywords option is set, and the minimum and maximum dates are not set, the only date which is displayed in the menu is "No Date".
- setMinimumDate() and resetMinimumDate() do nothing if no maximum date is currently set.
- setMaximumDate() and resetMaximumDate() do nothing if no minimum date is currently set.
- resetDateRange() does nothing.

This patch fixes the above bugs.

Documentation comments in the header file are also improved.


Diffs
-----

  autotests/kdatecomboboxtest.cpp c15525a 
  src/kdatecombobox.h d9a20ca 
  src/kdatecombobox.cpp ad1d085 

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


Testing
-------

Updated autotests pass. GUI changes tested with KAlarm, including setting DateKeywords. The bugs described above are now fixed.


Thanks,

David Jarvie

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161231/dc7e02cd/attachment.html>


More information about the Kde-frameworks-devel mailing list