Bug 257692 - Tried to change a time of a task. Time selection dialog shows 0 0 and I couldn't change the time.
Summary: Tried to change a time of a task. Time selection dialog shows 0 0 and I could...
Status: VERIFIED FIXED
Alias: None
Product: KOrganizer Mobile
Classification: Applications
Component: tasks (show other bugs)
Version: unspecified
Platform: Maemo 5 Linux
: VHI major
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-23 13:10 UTC by Ludwig Reiter
Modified: 2011-01-19 11:07 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ludwig Reiter 2010-11-23 13:10:39 UTC
Version:           unspecified (using Devel) 
OS:                Linux

version: 20101122.1199717-1maemo1.1199691 (enterprise5)

Tried to change the time of a task with the Time Selection dialog, but it displayed 0 0 and it wasn't possible to change anything.

Reproducible: Always

Steps to Reproduce:
1. Start to create a task.
2. Click on the start time.
3. Try to change the time.

Actual Results:  
Dialog shows 0 0 and is not changable.

Expected Results:  
Should show the preselected time and should be changable.
Comment 1 Artur Souza (MoRpHeUz) 2010-11-26 10:47:04 UTC
SVN commit 1200922 by asouza:

Make date and time selectors set proper values

A possible bug in QML was not properly propagating the changes
to the widgets. One workaround for that is calling a function
instead to set the value instead of relying on the signals.

Now dates and times are shown correctly (instead of 0's) but
we still need to setup to the right values (right now they show
the 'current/last setup' date and time.

CCBUG:257236
CCBUG:257692


 M  +10 -2     ClockDialog.qml  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1200922
Comment 2 Artur Souza (MoRpHeUz) 2010-11-26 15:46:00 UTC
SVN commit 1201019 by asouza:

Properly setup the initial values of widgets

This patch (still needs some polishing, but already works) fix the
issue of the initial value of the variables not being properly
propagated through the tree until it reached the parent.

Actually the parent was receiving the changes but wasn't acting
on those changes, what led to the bugs below. It's fixed now
and just needs some polishing.

BUG:257236
BUG:257692


 M  +16 -1     CalendarDialog.qml  
 M  +12 -0     ClockDialog.qml  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1201019
Comment 3 Anselmo L. S. Melo (anselmolsm) 2010-11-26 16:36:29 UTC
SVN commit 1201037 by amelo:

Select time using the clock hands

Besides the selectors, now it's possible again to set the
time using the clock. Tests on device probably will show
the need of adjustments.

CCBUG:252526
CCBUG:257692



 M  +28 -15    Clock.qml  
 M  +2 -0      calendar/ClockDialog.qml  
 M  +69 -96    calendar/clockhelper.cpp  
 M  +25 -16    calendar/clockhelper.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1201037