Problem/Motivation

Fields on configuration form are also trimmed.
But it is not always required.
E.g. Suffix field on Field config form.

Patch will be attached.

CommentFileSizeAuthor
#4 trim-restrict-to-content-entities-3238209-04.patch726 bytesdrpldrp

Issue fork trim-3238209

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

super_romeo created an issue. See original summary.

super_romeo opened merge request !1

mlncn’s picture

mlncn commented
Status: Active » Needs review

Thanks, committed to the dev branch!

For anyone interested— should this be as option?

And is there a better way to consistently catch 'admin' forms.

drpldrp’s picture

drpldrp commented
StatusFileSize
new trim-restrict-to-content-entities-3238209-04.patch726 bytes

Would limiting the trimming to content entities be better?

  • mlncn committed dc6e33e0 on 8.x-1.x authored by drpldrp 
    Issue #3238209 by super_romeo, drpldrp, mlncn: Skip the configuration...
mlncn’s picture

mlncn commented
Version: 8.x-1.x-dev » 2.0.x-dev
Status: Needs review » Fixed

Doing it drpldrp's way in the new 2.0.x branch!

(Also with a switch from get_class to instanceof which i think is more robust.)

Status: Fixed » Closed (fixed)

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