Bug 798946 - start/end of current/last quarter have off-by-one error

jralls · jralls · commit a475172d8c21 · 2025-01-20T11:40:44.000-08:00
diff --git a/libgnucash/engine/gnc-option-date.cpp b/libgnucash/engine/gnc-option-date.cpp
@@ -527,7 +527,7 @@ gnc_relative_date_to_time64(RelativeDatePeriod period)
         {
             auto delta = (now.tm_mon >= acct_per.tm_mon ?
                           ( now.tm_mon - acct_per.tm_mon) % 3 :
-                          3 - ((acct_per.tm_mon - now.tm_mon) % 3));
+                          (3 - acct_per.tm_mon - now.tm_mon) % 3);
             now.tm_mon = now.tm_mon - delta;
         }
             [[fallthrough]];

Original file line number	Diff line number	Diff line change
`@@ -527,7 +527,7 @@ gnc_relative_date_to_time64(RelativeDatePeriod period)`
`527`	`527`	`{`
`528`	`528`	`auto delta = (now.tm_mon >= acct_per.tm_mon ?`
`529`	`529`	`( now.tm_mon - acct_per.tm_mon) % 3 :`
`530`		`- 3 - ((acct_per.tm_mon - now.tm_mon) % 3));`
	`530`	`+ (3 - acct_per.tm_mon - now.tm_mon) % 3);`
`531`	`531`	`now.tm_mon = now.tm_mon - delta;`
`532`	`532`	`}`
`533`	`533`	`[[fallthrough]];`