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

Incompatible with 5.61+? #335

Open
jofranz opened this issue Aug 24, 2023 · 5 comments
Open

Incompatible with 5.61+? #335

jofranz opened this issue Aug 24, 2023 · 5 comments

Comments

@jofranz
Copy link

jofranz commented Aug 24, 2023

➜  uk.co.vedaconsulting.gdpr (master) check-extensions-for-removed-core-code.sh -d .                                                                                                                             ✭
./CRM/Gdpr/Form/Search/GroupcontactDetails.php:
CRM_Core_Form_Date::buildDateRange($form, 'subscription_date', 1, '_low', '_high', E::ts('From'), FALSE);

@eileenmcnaughton
Touches #330

@eileenmcnaughton
Copy link
Contributor

Yeah - looks like that deprecation was in 2020 so it would definitely be good to stop calling if from this extension - it seems this extension calls a few core furnction that are not supported for non-core use

@eileenmcnaughton
Copy link
Contributor

Note I don't think the presence of deprecation warnings makes it incompatible - live sites should not be displaying deprecation warnings (but staging & dev should so they they do get fixed)

@jofranz
Copy link
Author

jofranz commented Aug 30, 2023

@eileenmcnaughton Correct me if I'm wrong but CRM_Core_Form_Date::buildDateRange has been kicked out of CiviCRM core which breaks this extension if this method is called.

@eileenmcnaughton
Copy link
Contributor

@jofranz ah right - I was looking at the deprecated one - that one must have been through it's deprecation & been removed

Easiest is for the extension to take a copy of the function into the extension, although a better approach would be to replace the custom search (which we are winding down support for) with a search kit display.

Presumably the extension works other than that search & just the search could be removed/ disabled?

@VangelisP
Copy link

@jofranz while I don't want to overtake this discussion, just wanted to say that I was hitting the same issue and I've spotted on your initial post that you've been using a bash script that you're calling to detect removed code from extension(s): check-extensions-for-removed-core-code.sh , what is this and where can I find it 😄 ?

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

No branches or pull requests

3 participants