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

Deprecate/remove unused checkVersion() functions #18134

Merged
merged 1 commit into from
Aug 12, 2020

Conversation

colemanw
Copy link
Member

Overview

Deprecate 1 unused function and remove another unused function.

Before

2 unused functions.

After

CRM_Core_DAO::checkVersion() marked as deprecated.
CRM_Upgrade_Form::checkVersion() removed.

Comments

Both functions are unused according to my grepping. The one in CRM_Core_DAO could theoretically be used somewhere in the extension universe (though I doubt it), so I marked it as deprecated to provide a transition period.

The other function in CRM_Upgrade_Form is very "internal" and IMO very unlikely to be in use anywhere, so I deleted it.

@civibot
Copy link

civibot bot commented Aug 12, 2020

(Standard links)

@civibot civibot bot added the master label Aug 12, 2020
@demeritcowboy
Copy link
Contributor

I don't see where they are used in core either. And the upgrade one looks like it was used in earlier versions of incremental upgrades: civicrm/civicrm-svn@dc511ab#diff-49f0a52691f7a2889a9f20651f317956

@colemanw colemanw merged commit a248952 into civicrm:master Aug 12, 2020
@colemanw colemanw deleted the deprecateCheckVersion branch August 12, 2020 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants