Part of #2545972: Remove all code usages SafeMarkup::checkPlain() and rely more on Twig autoescaping

Problem/Motivation

SafeMarkup::checkPlain() is unnecessary.

Proposed resolution

EntityListBuilder does not need to SafeMarkup::checkPlain() the entity label.

Remaining tasks

Do it
Review
Commit

User interface changes

None

API changes

EntityListBuilder::getLabel() is deprecated.

Data model changes

None

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott created an issue. See original summary.

alexpott’s picture

Status: Active » Needs review
FileSize
16.49 KB
Xano’s picture

Cool stuff! (Comment as requested, review coming later if needed)

alexpott’s picture

alexpott’s picture

Fixing docs...

lauriii’s picture

Status: Needs review » Reviewed & tested by the community

If this passes its RTBC I think

alexpott’s picture

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed 42c9be9 on 8.0.x
    Issue #2560851 by alexpott, Xano: Deprecate EntityListBuilder::getLabel...

Status: Fixed » Closed (fixed)

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