Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the fiscal start day of the month for relative quarter reports #2065

Open
wants to merge 2 commits into
base: stable
Choose a base branch
from

Conversation

agwekixj
Copy link
Contributor

@agwekixj agwekixj commented Mar 3, 2025

Only the quarter's start month in reports is adjusted when Absolute is selected in the Start Date on the Accounting Period preferences. This branch adjusts both the quarter's start day and month when Absolute is selected or when Relative and Today, Start of this month, or Start of previous month are selected.

Only the quarter's start month in reports is adjusted when Absolute is selected in the Start Date on the Accounting Period preferences. This branch adjusts both the quarter's start day and month when Absolute is selected or when Relative and Today, Start of this month, or Start of previous month are selected.
@agwekixj
Copy link
Contributor Author

agwekixj commented Mar 3, 2025

The test errors are expected as we're changing the prior relative quarter start behavior.

Commented out the relative quarter tests that are no longer going to reliably succeed.
@jralls
Copy link
Member

jralls commented Mar 3, 2025

The test errors are expected as we're changing the prior relative quarter start behavior.

In only one place, so that the reports' understanding of the relative quarter differs from the rest of the program's. This is particularly amusing because it answers my "what am I missing" from #2064: In the absence of preferences, which is the case with unit tests, the default accounting period is TODAY.

That aside it appears from grepping around that the only other place that would be affected by this policy change is in the price database, where leaving the last quote of each quarter is an option for deleting old quotes. That's not exactly high-risk.

However while I don't advocate as extensive a testing range as Graham Menhemnitt it does deserve more testing, not less.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants