Review Request 111413: Plan - Set First Day of Week to Monday if using ISO Week for week numbering

[Friedrich W. H. Kossebau]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111413/#review44297
-----------------------------------------------------------


Just pushed the current version (extended by linking change for libkdgantt CMakeLists.txt) to 2.7 branch, so the fix is at least in 2.7.5, to be tagged today :)
So you just have to commit the improved version to master, no backport needed.

- Friedrich W. H. Kossebau


On July 6, 2013, 12:51 a.m., Alvaro Soliverez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111413/
> -----------------------------------------------------------
> 
> (Updated July 6, 2013, 12:51 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Bugs: 321290
>     http://bugs.kde.org/show_bug.cgi?id=321290
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> Plan uses week numbering extensively. The default week numbering in KDE is ISO Week, which uses Monday as first day of the week, regardless of other locale settings.
> 
> To make it consistent through the application, if week numbering format is ISO Week, set the first day of the week to Monday, and display it accordingly in charts and graphs.
> 
> Without this patch, the week could be set to start on Sunday, and week would be shown on graphs as ranging from Sunday to Saturday, but the week number of working days would be off by one.
> 
> Example: 
> 
> - Week numbering: ISO Week
> - First day of week: Sunday
> 
> Without patch:
> - Week displayed: 2013-07-07 to 2013-07-13
> - Week number shown on charts: 27
> 
> - Expected week number: 28 (Week 27 ends on Sunday 2013-07-07)
> 
> With patch:
> - Week displayed: 2013-07-08 to 2013-07-14
> - Week number shown on charts: 28
> 
> 
> Additionally, there were a few calls to QDate::week() to get the week number, instead of using KGlobalCalendar.
> 
> 
> Diffs
> -----
> 
>   plan/libs/ui/kptganttview.cpp 81073a3 
>   plan/libs/models/kptaccountsmodel.cpp 71c3100 
>   plan/libs/models/kcalendar/kdatetable.cpp b389345 
>   3rdparty/kdgantt/kdganttdatetimegrid.cpp 79c405c 
> 
> Diff: http://git.reviewboard.kde.org/r/111413/diff/
> 
> 
> Testing
> -------
> 
> Tested on a live file. It displays calendars and gantt charts correctly.
> 
> 
> Thanks,
> 
> Alvaro Soliverez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20131124/b58c6297/attachment.htm>