Summary: | Date of future payments change automatically | ||
---|---|---|---|
Product: | [Applications] kmymoney | Reporter: | Philippe Debrabant <debrabant.philippe> |
Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | onet.cristian, ostroffjh, ralf.habacker |
Priority: | NOR | ||
Version: | 4.7.1 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kmymoney/4d0d3092e77964cb8005621438a4e4d03defbe94 | Version Fixed In: | 4.8.0 |
Sentry Crash Report: | |||
Bug Depends on: | |||
Bug Blocks: | 348166 |
Description
Philippe Debrabant
2015-03-11 18:54:24 UTC
KMyMoney's schedules currently can't handle the "last day of the month" concept. While "last day of the month" should be a wishlist if it is not already, I'm curious about what happens (or doesn't happen) in this example. The payment date is entered as the 31st of the month. If a subsequent month has fewer days, then the payment is moved to the last day of that month, say the 30th, but that change to the 30th is permanent. Should this type of change be considered similar to what happens if they date is on a weekend? In that case, an adjustment is made for that payment only, but the scheduled date is not changed for future payments. I just wonder if that could be a way to handle this instead of explicitly using the last day of the month. Git commit 4d0d3092e77964cb8005621438a4e4d03defbe94 by Thomas Baumgart. Committed on 15/05/2015 at 17:28. Pushed by tbaumgart into branch 'master'. Don't change the original start date of a schedule Entering schedules updates the next due date of the schedule. Before this change, this also used to change the start date member variable which is the reason that the day of the first execution of the schedule is lost over time when it is modified because it is in the range of 29-31 and the month for the transaction does not have that many days. In this case, the day was adjusted and caused future transaction not to happen on this day anymore. I have updated testcases and added a new one but still we need more testing before we cherry pick this change over to the 4.7 branch. M +1 -1 kmymoney/mymoney/mymoneyschedule.cpp M +48 -0 kmymoney/mymoney/mymoneyscheduletest.cpp M +1 -0 kmymoney/mymoney/mymoneyscheduletest.h http://commits.kde.org/kmymoney/4d0d3092e77964cb8005621438a4e4d03defbe94 |