I intend to do a major update containing multiple issue fixes with a working single patch.

This patch contains multiple fixes from myself, config entity fixes by @romainj, and an additional plugin by @matio89 (Drupal credits will be given accordingly).

The provided patch fixes:

#3253253: Make the module compatible with Drupal 9
#3140233: Automated Drupal 9 compatibility fixes
#2680243: Add a custom extra field
#2965638: Invalid argument supplied for foreach() in FieldEmbedView->fieldRowSettingsForm()
#2965616: Undefined index: name in Drupal\display_fields\Plugin\DisplayFieldsField\FieldEmbedView->getFieldBuild()

Partial fix:
#2946437: Change order of field doesn't work : remaining task : disabling fields still doesn't work with the patch.

The patch does not change anything in the configuration files. It fixes the multiple small code and compatibility issues.

It would be best to create an alpha tagged release for the current state of dev, then create a new alpha release with the fixes.

What do you think? I may end up doing this task in a few weeks to be sure to get your input.

CommentFileSizeAuthor
display-fields-major-cleanup-plus-region-partial-fix_3253253-initial.patch84.25 KBdakwamine

Issue fork display_fields-3268106

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Dakwamine created an issue. See original summary.

dakwamine’s picture

dakwamine
Primary language French
commented

Of course, the new alpha release will be D9 ready.

delta’s picture

delta commented

Of course

Well done

romainj’s picture

romainj commented

Works for me! Thanks @Dakwamine

dakwamine’s picture

dakwamine
Primary language French
commented
Issue summary: View changes
dakwamine’s picture

dakwamine
Primary language French
commented
Issue summary: View changes
delta’s picture

delta commented
Status: Active » Fixed

  • delta committed fa6b4ae on 8.x-1.x 
    Major update for display_fields, Issue #3268106 by Dakwamine, matio89,...
delta’s picture

delta commented
Status: Fixed » Needs review

changing to needs review, moved forward a beta release to ease up testing and patch releases
this has been pushed to the 8.x branch as well, switched the default branch to 9.x

pski made their first commit to this issue’s fork.

  • a414b70 committed on 8.x-1.x 
    Major update for display_fields, Issue #3268106 by Dakwamine, matio89,...
delta’s picture

delta commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.