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

[Views integration] How is caching supposed to work? #48

Open
jensschuppe opened this issue May 26, 2021 · 4 comments
Open

[Views integration] How is caching supposed to work? #48

jensschuppe opened this issue May 26, 2021 · 4 comments
Labels

Comments

@jensschuppe
Copy link
Collaborator

$options['cache'] = empty($view->query->options['cache']) ? NULL : $view->query->options['cache'];

This seems to have been ported from 7.x, but I could neither get it to work nor do I understand where this caching option should be coming from.

Can anyone involved shed some light on whether Views caching (at least time-based) in the View's configuration is supposed to work with caching CiviMRF calls at all?

@fabioneves @jaapjansma @kainuk

@jaapjansma
Copy link
Contributor

In Drupal 7 you can set a caching timeout in the views ui. This is per view.
This caching setting is then passed on to civi mc restface calls.

@jensschuppe
Copy link
Collaborator Author

Thanks, @jaapjansma, so this will need to be adapted for D8 since caching configuration in D8 is per Views display and can only be used if time-based caching is enabled, if I got that correctly.

Has anyone taken on that for D8 yet?

@jaapjansma
Copy link
Contributor

Yes indeed this need to be adapted to d8. I also believe that the caching setting in d7 is per display of a view.

@jensschuppe
Copy link
Collaborator Author

Linking #62

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

No branches or pull requests

2 participants