[korganizer] [Bug 391769] KOrganizer 5.9.0: Incident times of tasks are not updated when edited by keyboard only

[Denis Kurz]

https://bugs.kde.org/show_bug.cgi?id=391769

Denis Kurz <kdenis at posteo.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.9.3
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/inc
                   |                            |idenceeditor/10056772fd32c8
                   |                            |9dd5e881b495f597393b92f2ee

--- Comment #3 from Denis Kurz <kdenis at posteo.de> ---
Git commit 10056772fd32c89dd5e881b495f597393b92f2ee by Denis Kurz.
Committed on 11/10/2018 at 21:26.
Pushed by dkurz into branch 'Applications/18.08'.

Fix keyboard editting of start/end times of VTODOs

Summary:
Editting start/end times of VTODOs using only the keyboard did not
trigger a check whether the VTODO is dirty. If other fields were updated
along with start/end times, everything was fine, but only editting those
times was simply ignored. The patch causes a dirty check for keyboard
edits of the corresponding KTimeComboBoxes.
FIXED-IN: 5.9.3

Test Plan:
I changed start and end times of a test VTODO. Without the patch,
the change was not saved when hitting OK, nor did I hit my dirty-check
breakpoint. With the patch, everything works as expected: Keyboard editting
results in an updated VTODO, as verified via the Month View tooltip and by
opening the IncidenceEditor again for that VTODO. VEVENTs do not have this
problem even without the patch.

Reviewers: #kde_pim, winterz

Reviewed By: #kde_pim, winterz

Subscribers: kde-pim

Tags: #kde_pim

Differential Revision: https://phabricator.kde.org/D16114

M  +4    -0    src/incidencedatetime.cpp

https://commits.kde.org/incidenceeditor/10056772fd32c89dd5e881b495f597393b92f2ee

-- 
You are receiving this mail because:
You are the assignee for the bug.