Bug 424074

Summary:	Events are always saved as all-day in day and week views
Product:	[Applications] korganizer	Reporter:	odzinic
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	gjditchfield
Priority:	NOR
Version:	5.14.1
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description odzinic 2020-07-10 17:37:13 UTC

SUMMARY
When I create an event in either the day or week view, even if I don't select "All Day" as the event time period the event will get saved as all day. 

STEPS TO REPRODUCE
1. Create an event for a certain time slot
2. Ensure All Day is not selected
3. Save the event

OBSERVED RESULT
If the event was created in the day view then no visual of the event will appear. Switching to the week view will display the event you created lasting the entire day even if you specified a certain time period. Creating the event in the week view will display the event lasting all day instantly.

EXPECTED RESULT
The event gets created for the specified time period instead of all day

SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.18.5
KDE Frameworks Version: 5.70.0
Qt Version: 5.15.0

ADDITIONAL INFORMATION
This issue has been tested on Kubuntu and Manjaro KDE and has occurred in both.

Comment 1 gjditchfield 2020-07-12 18:22:29 UTC

Which locale do you use?

Does the problem go away if you switch to the American English locale (for example, by executing "LANG=en_US korganizer" from the command line)?  If it does, this is probably a duplicate of bug 420468.  a fix for that bug is arriving shortly in KWidgetsAddons 5.72.0

Comment 2 odzinic 2020-07-12 18:31:44 UTC

(In reply to gjditchfield from comment #1)
> Which locale do you use?
> 
> Does the problem go away if you switch to the American English locale (for
> example, by executing "LANG=en_US korganizer" from the command line)?  If it
> does, this is probably a duplicate of bug 420468.  a fix for that bug is
> arriving shortly in KWidgetsAddons 5.72.0

I am using the en_CA locale which seems to match that bug. But when I launch korganizer with "LANG=en_US korganizer", I still get the same issue. Also I noticed that every time I double click a time period, I get the following output: 

org.kde.pim.incidenceeditor: free slot calculation: invalid range. range(  0 ) / mSlotResolutionSeconds( 900 ) =  0

This appears when creating a task in both day and week views.

Comment 3 gjditchfield 2020-07-12 19:46:27 UTC

(In reply to odzinic from comment #2)
> I am using the en_CA locale which seems to match that bug. But when I launch
> korganizer with "LANG=en_US korganizer", I still get the same issue.

Hm.  I use en_CA in KDE Neon, and the LANG work-around worked last week, but not today!

Today, I have to use "LC_TIME=en_US.UTF-8 korganizer". Does that work better for you?

> Also I noticed that every time I double click a time period, I get the 
> following output: 
> 
> org.kde.pim.incidenceeditor: free slot calculation: invalid range. range(  0
> ) / mSlotResolutionSeconds( 900 ) =  0

I see that a lot.  It seems to be harmless.

Comment 4 odzinic 2020-07-12 20:43:41 UTC

> Hm.  I use en_CA in KDE Neon, and the LANG work-around worked last week, but
> not today!
> 
> Today, I have to use "LC_TIME=en_US.UTF-8 korganizer". Does that work better
> for you?

That one worked! So looking at the commit the issue is caused by the extra punctuation used in the time format. Interesting. I'll use the workaround until that fix gets upstreamed. Appreciate it!

*** This bug has been marked as a duplicate of bug 420468 ***