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

FIX Accountancy generic suggest account not proposed #12319

Closed
wants to merge 1 commit into from

Conversation

aspangaro
Copy link
Member

Before :

image

After (With constant ACCOUNTANCY_AUTOFILL_ACCOUNT_WITH_GENERIC)

image

@eldy
Copy link
Member

eldy commented Nov 4, 2019

The rule was to check the box when Dolibarr is sure "automatically".
Here, there is no account for the product, so dolibarr can suggest the default value, but he is not sure it is good so i think we must have the value "autoselected" but no the checkbox.
So default behavior should be an intermediate of actual and the one suggested with option ACCOUNTANCY_AUTOFILL_ACCOUNT_WITH_GENERIC, shouldn't it ?

@aspangaro
Copy link
Member Author

Thanks @eldy for your reflexion.

No, until now (branch v9 and previous), the checkbox is check at the "green" first level (With product account defined and default_account defined) and also at the "orange" second level (Only with default account_defined) to win a lot of time especially when there are a lot of lines waiting when you start an accounting for example or for companies where there are extremely fast lines and they use half the accountancy module.
For the moment, it's a regression. I validate the constant because I think it's a good idea but even compared to my patch, it would be necessary to reverse the operation. Enable by default the checkbox with a level 2 and disable the checkbox with the constant. What do you think about that ?

Available tomorrow to talk about it in person if you want

@eldy eldy closed this in de77e39 Nov 7, 2019
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