Skip to content

[Ref] Mark unused date functions as deprecated #17487

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

Merged
merged 1 commit into from
Jun 12, 2020

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Flags unused functions for removal

Before

Not clear functions are on their way out

After

Functions are clearly deprecated

Technical Details

Comments

@civibot
Copy link

civibot bot commented Jun 4, 2020

(Standard links)

@civibot civibot bot added the master label Jun 4, 2020
@eileenmcnaughton eileenmcnaughton changed the title Mark unused functions as deprecated [Ref] Mark unused functions as deprecated Jun 8, 2020
@mlutfy mlutfy changed the title [Ref] Mark unused functions as deprecated [Ref] Mark unused date functions as deprecated Jun 11, 2020
@mlutfy
Copy link
Member

mlutfy commented Jun 11, 2020

Seems fine by me, but will let someone else more qualified to merge :)

@eileenmcnaughton
Copy link
Contributor Author

@mlutfy this is really a case of doing a grep & seeing if you agree they aren't used

@mlutfy
Copy link
Member

mlutfy commented Jun 11, 2020

I'm still seeing this:

$ grep -r returnDateRangeSelector
CRM/Core/Form/Date.php:      = CRM_Core_Form_Date::returnDateRangeSelector(
CRM/Core/Form/Date.php:  public static function returnDateRangeSelector(
bgm@erdenet:~/repositories/civicrm-core$ grep -r addDateRangeToForm
CRM/Core/Form/Date.php:    CRM_Core_Form_Date::addDateRangeToForm(
CRM/Core/Form/Date.php:  public static function addDateRangeToForm(

Other than that, I grepped in a set of extensions and they were not used there.

@eileenmcnaughton
Copy link
Contributor Author

@mlutfy yep - aren't those calls from the deprecated functions?

@mlutfy mlutfy merged commit ac50a85 into civicrm:master Jun 12, 2020
@mlutfy
Copy link
Member

mlutfy commented Jun 12, 2020

Long day ;-)

@eileenmcnaughton eileenmcnaughton deleted the date branch June 12, 2020 22:12
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