Bug 257692

Summary:	Tried to change a time of a task. Time selection dialog shows 0 0 and I couldn't change the time.
Product:	[Applications] KOrganizer Mobile	Reporter:	Ludwig Reiter <ludwig.reiter>
Component:	tasks	Assignee:	kdepim bugs <kdepim-bugs>
Status:	VERIFIED FIXED
Severity:	major	CC:	bjoern.ricks
Priority:	VHI
Version:	unspecified
Target Milestone:	---
Platform:	Maemo 5
OS:	Linux
Latest Commit:		Version Fixed In:

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