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

Link chart of accounts and bookkeping using rowIDs #7023

Closed
ffries opened this issue Jun 16, 2017 · 2 comments
Closed

Link chart of accounts and bookkeping using rowIDs #7023

ffries opened this issue Jun 16, 2017 · 2 comments
Labels
Won't be fixed/implemented No tasks will be done for this issue (amount of work may be too high compared to benefits)

Comments

@ffries
Copy link

ffries commented Jun 16, 2017

This will not be implemented.


Any change in the accounting chart might break bookeeping, because bookkeeping and chart of accounts are linked using account names, not IDs. We propose to link with IDs.

Current behavior

llx_accounting_bookkeeping is linked to llx_accounting_account using account name.
Therefore, it is difficult to run SQL joins and later build a complete journal.

Expected behavior

Replace
llx_accounting_bookkeeping.numero_compte varchar(32) NOT NULL,
with
llx_accounting_bookkeeping.numero_compte_rowID int NOT NULL,

I will comment later, have no time to comment now.
We might need to duplicate chart of accounts on each year.

@ffries ffries changed the title SPEC: link public.llx_accounting_bookkeeping to llx_accounting_account using rowIDs SPEC: link chart of accounts and bookkeping using rowIDs Jun 16, 2017
@ffries ffries changed the title SPEC: link chart of accounts and bookkeping using rowIDs SPEC: Link chart of accounts and bookkeping using rowIDs Jun 16, 2017
@eldy
Copy link
Member

eldy commented Jun 20, 2017

We don't want to have strict link.
When record are wrote into bookeeping, if you change something on the business side, it must not be changed on bookeeping side. Once an account has been affected, it must not be modified. This is important to be compiant with most countries laws. If you need to modifiy it, this must be done using the bookeeping tool, but there is then no more link with business events.

@eldy eldy added the Won't be fixed/implemented No tasks will be done for this issue (amount of work may be too high compared to benefits) label Jun 20, 2017
@ffries
Copy link
Author

ffries commented Jun 21, 2017

Okay, you are right, sounds clear.

@ffries ffries changed the title SPEC: Link chart of accounts and bookkeping using rowIDs Link chart of accounts and bookkeping using rowIDs Jun 21, 2017
@ffries ffries closed this as completed Jun 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Won't be fixed/implemented No tasks will be done for this issue (amount of work may be too high compared to benefits)
Projects
None yet
Development

No branches or pull requests

2 participants