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

problem with counter_cache #149

Open
NazarK opened this issue Sep 27, 2023 · 1 comment
Open

problem with counter_cache #149

NazarK opened this issue Sep 27, 2023 · 1 comment

Comments

@NazarK
Copy link

NazarK commented Sep 27, 2023

if association is in "include" option, counter_cache field should be in "except" option, otherwise counter_cache field is multiplied by 2 and doesnt represent actual number of associated records (maybe should be mentioned in docs or managed automatically if possible)

@moiristo
Copy link
Owner

Hi, thanks for the feedback! As it is part of the configuration of associations, I think I agree it should be managed automatically. Will have a look when I have the time. PRs are welcome too ;)

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

No branches or pull requests

2 participants