Skip to content

Enable drupal-check in workflow for 4.x #1130

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

Open
kedarkhaire opened this issue Feb 12, 2025 · 2 comments
Open

Enable drupal-check in workflow for 4.x #1130

kedarkhaire opened this issue Feb 12, 2025 · 2 comments
Assignees

Comments

@kedarkhaire
Copy link
Collaborator

Description

Here in Drupal v11, drupal-check is removed as it throws errors for deprecated classes and functions.
These classes & functions are deprecated in Drupal v10.3 and will be removed in Drupal v12 but they are no issue for Drupal v11.
The deprecated functions and classes are suppressed in v3.x-latest version for D10.3 compatibility.

Issues related for these are as follows

  1. getName() is deprecated - https://www.drupal.org/node/3310017
  2. class RenderElement is deprecated - https://www.drupal.org/node/3436275
  3. renderPlain() is deprecated - https://www.drupal.org/node/3407994
  4. class constant EXISTS_REPLACE is deprecated - https://www.drupal.org/node/3426517

Partial fixes for these functions and classes are done in these PRs - #1115 , #1116 , #1117 , #1118
But still these changes will need through testing and dependent updates to other pages as well.

Apigee Info

Apigee Edge 4.0.0 & Drupal 11

@kedarkhaire
Copy link
Collaborator Author

getName() is deprecated - TeamPermissionHandler.php ModuleHandlerInterface to be updated with ModuleExtensionList for apigee/apigee-m10n-drupal#529

@kedarkhaire
Copy link
Collaborator Author

renderPlain() is deprecated - AppListBuilder.php Update renderPlain() with renderInIsolation() on 293 for apigee/apigee-m10n-drupal#529

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

1 participant