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

Port status: Extrafield Views Integration #837

Open
elisseck opened this issue Nov 5, 2024 · 2 comments
Open

Port status: Extrafield Views Integration #837

elisseck opened this issue Nov 5, 2024 · 2 comments

Comments

@elisseck
Copy link

elisseck commented Nov 5, 2024

Name of the module, theme, or layout you are working on
Extra Field Views Integration

Link to the drupal.org module, theme, or layout
https://www.drupal.org/project/extrafield_views_integration

(Optional) Link to the drupal.org issue where you notified the Drupal maintainer(s) of this port
https://www.drupal.org/project/extrafield_views_integration/issues/3485760

Do you need assistance with this port?
No

@argiepiano
Copy link

argiepiano commented Nov 5, 2024

Technically this shouldn't be needed if you are using Entity Plus. E+ actually creates handlers for all extra fields that have property info (you need to implement hook_entity_property_info_alter() to define them).

[EDIT]: Hmmm, now I'm thinking that maybe this is not the case. I'd need to test this.

@elisseck
Copy link
Author

elisseck commented Nov 7, 2024

@argiepiano Thanks for the heads up! Definitely worth a test... although it may still be useful to have this for a migration instead of changing everything around to use entity_plus... I'll look into whether it is actually needed and mark it appropriately in the readme.

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

2 participants