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

New:Algerian Chart of accounts #9391

Merged
merged 2 commits into from
Sep 3, 2018
Merged

New:Algerian Chart of accounts #9391

merged 2 commits into from
Sep 3, 2018

Conversation

bafbes
Copy link
Contributor

@bafbes bafbes commented Sep 2, 2018

New:Algerian Chart of accounts - First version

@bafbes bafbes changed the title New:Algerian Chart of accounts - First version New:Algerian Chart of accounts Sep 3, 2018
@eldy eldy merged commit b9066fc into Dolibarr:develop Sep 3, 2018
@eldy
Copy link
Member

eldy commented Sep 3, 2018

I revert the merge because there is 2 errors into file.

  • You must first remove column rowid
  • The field account_parent must be a key on account_number not on rowid (so value into account_parent must be existing values found into account_number).

@hregis
Copy link
Contributor

hregis commented Sep 3, 2018

@bafbes @eldy
ah mince j'étais en train de corriger ! ;-)

@bafbes
Copy link
Contributor Author

bafbes commented Sep 3, 2018

I used the benin chart of accounts as a model in the file llx_accounting_account_bj.sql. The french chart of accounts ( in the file llx_accounting_account_fr.sql) does not either show the parent account number in in the account_parent field. Look at the first non root record in it : " INSERT INTO llx_accounting_account (rowid, fk_pcg_version, pcg_type, pcg_subtype, account_number, account_parent, label, active) VALUES ( 1,'PCG99-ABREGE','CAPIT', 'CAPITAL', '101', '1401', 'Capital', 1);"
Count '1401' is not the parent account numer of '101' either but its parent rowid. Right ?
So which file must be taken as a model ?

@eldy
Copy link
Member

eldy commented Sep 3, 2018

Problem is more complex than that.
Finally the PR of abbes is correct and i will add a line of comment into it to be able to fix file on the fly on v9.

@bafbes bafbes deleted the abb80016 branch October 4, 2018 19:06
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.

3 participants