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

Purge deprecated options #768

Open
vincent3569 opened this issue Apr 3, 2014 · 2 comments
Open

Purge deprecated options #768

vincent3569 opened this issue Apr 3, 2014 · 2 comments

Comments

@vincent3569
Copy link
Collaborator

hi

I add a revue on options values in my dev database.
I saw some combinews-* options.
as combinews is deprecated, these options could be purged during setup process, don't you think ?

same way for other options for deprecated plugin or themes.

@sbillard
Copy link
Contributor

sbillard commented Apr 3, 2014

They will get purged if you have enabled the GalleryArticles plugin and used its option to "import" previous combi-news articles. (This option is available only if the zenpage_combinews option was enabled as otherwise presumably you were not using combi-news.)

As to the other options, They can be purged, but since the deprecated function is still available they probably should not be purged.

In general the issue of cleaning out obsolete options is problematic. It really takes human intelligence to make the decisions. There are clues stored along with the option that can help. For instance for theme options the theme name is stored. But we really do not have a indicator that the theme has been removed, so the user who does that would need to purge its options.

There is not much harm in leaving obsolete options. They take up little space and do not impact any performance.

@acrylian acrylian added this to the 2.0 milestone Apr 15, 2014
@sbillard sbillard removed this from the 2.0 milestone Apr 15, 2014
@sbillard
Copy link
Contributor

sbillard commented Jun 4, 2014

This feature is available in ZenPhoto20 with the "purgeObjects" plugin. There is then a new purge subtab under the options tab.

@acrylian acrylian added this to the 2.0 milestone Jul 21, 2014
@acrylian acrylian changed the title 1.4.6 purge deprecated options Purge deprecated options Jul 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants