D15309: [Calendar] Wrap day name index around

[Kai Uwe Broulik]

broulik created this revision.
broulik added a reviewer: Plasma.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
broulik requested review of this revision.

REVISION SUMMARY
  Otherwise when the week starts on a day other than Sunday or Monday we access invalid day names.
  
  BUG: 390330

TEST PLAN
  TIL there's locales where weeks start on a Saturday :)
  Ran `LC_TIME=ar_EG plasmawindowed org.kde.plasma.digitalclock` and hat all weekdays shown properly. Previously it would only show Saturday and Sunday headings

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D15309

AFFECTED FILES
  src/declarativeimports/calendar/qml/DaysCalendar.qml

To: broulik, #plasma
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180906/2d0b9636/attachment.html>