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

Convert Custom Data cache group to be using standard cache backend #14582

Merged

Conversation

seamuslee001
Copy link
Contributor

Overview

This is a partial of #14487 which converts the custom data cache group to standard cache definition

Before

Uses deprecated caching functions

After

Uses standard caching functions and cache definition

Technical Details

If the PR introduces noteworthy technical changes, please describe them here. Provide code snippets if necessary

Comments

When r-running this the reviewer should preform an advanced search and filter on a custom field, and confirm that when the advanced search pane re-loads that the custom field value that was set is retained

@civibot
Copy link

civibot bot commented Jun 18, 2019

(Standard links)

@civibot civibot bot added the master label Jun 18, 2019
@seamuslee001 seamuslee001 force-pushed the convert_custom_data_cache_group branch 2 times, most recently from 603cd77 to 883d7dc Compare June 22, 2019 23:34
@seamuslee001
Copy link
Contributor Author

Jenkins re test this please

@seamuslee001 seamuslee001 force-pushed the convert_custom_data_cache_group branch from 883d7dc to cd5cd44 Compare June 25, 2019 21:53
@seamuslee001 seamuslee001 force-pushed the convert_custom_data_cache_group branch 2 times, most recently from 8154c60 to 341914b Compare July 12, 2019 13:52
@seamuslee001
Copy link
Contributor Author

@eileenmcnaughton this should be fine for review now, it seems to only get set when editing cache data

@eileenmcnaughton
Copy link
Contributor

@seamuslee001 looks like the one I merged made this stale

@seamuslee001 seamuslee001 force-pushed the convert_custom_data_cache_group branch from 341914b to 5d27ade Compare July 14, 2019 04:40
@seamuslee001
Copy link
Contributor Author

@eileenmcnaughton rebased now

@eileenmcnaughton
Copy link
Contributor

ok I think this is in keeping with other related PRs

@seamuslee001 seamuslee001 merged commit de8fc5b into civicrm:master Jul 14, 2019
@seamuslee001 seamuslee001 deleted the convert_custom_data_cache_group branch July 14, 2019 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants