Bug 267944 - Album properties - Date manually entered not saved when clicking Ok
Summary: Album properties - Date manually entered not saved when clicking Ok
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Albums-MainView (show other bugs)
Version: 1.9.0
Platform: Compiled Sources Linux
: NOR minor
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-08 09:07 UTC by Ludovic Pageaud
Modified: 2017-07-15 05:43 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 2.0.0
Sentry Crash Report:


Attachments
Proposition to solve the issue (2.25 KB, patch)
2011-03-08 09:07 UTC, Ludovic Pageaud
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ludovic Pageaud 2011-03-08 09:07:14 UTC
Created attachment 57765 [details]
Proposition to solve the issue

Version:           1.9.0 (using KDE 4.5.1) 
OS:                Linux

In album properties,
When manually writing date in album date box (no click on calendar), then clicking Ok, the date is not updated in album properties.


Reproducible: Didn't try

Steps to Reproduce:
right click album -> properties
Write a valid date (2011-02-23) in the album date box (no click in calendar and do not press return)
Click Ok

Actual Results:  
If I check again the album date in album properties, the date is back to the previous one.


Expected Results:  
I expect the entered date to be saved in album properties.

Only way to get it working is to press return just after writing the date. But in this case the album properties window is closed. This is not convenient if one wanted to update other properties.

I suggest the attached patch. It is a dirty one but at least it gives an idea on how to solve this issue.
Comment 1 caulier.gilles 2011-03-08 13:34:37 UTC
Git commit 32568845cac47f99b7dfa42ef206bd79162ef568 by Gilles Caulier.
Committed on 08/03/2011 at 13:32.
Pushed by cgilles into branch 'master'.

if date is entered by hand in text field, take a care about it if date is valid.
BUG: 267944

M  +38   -11   digikam/album/albumpropsedit.cpp     

http://commits.kde.org/digikam/32568845cac47f99b7dfa42ef206bd79162ef568