Problem/Motivation

I tried to extend the EntityUrlGeneratorBase for a custom media url generator and noticed that throughout the class the class ContentEntityBase instead of the interface ContentEntityInterface is used.

Proposed resolution

ContentEntityInterface is a drop-in replacement for ContentEntityBase in the type definitions and docblocks.

CommentFileSizeAuthor
#2 simple_sitemap-replace_class_with_interface-3181879-2.patch2.55 KBtgauges
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tgauges created an issue. See original summary.

tgauges’s picture

tgauges CreditAttribution: tgauges commented
Assigned: tgauges » Unassigned
Status: Active » Needs review
FileSize
simple_sitemap-replace_class_with_interface-3181879-2.patch2.55 KB
gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and at gbyte commented
Category: Bug report » Task

  • gbyte committed 411db77 on 8.x-3.x 
    Issue #3181879 by tgauges, gbyte: Replace ContentEntityBase with...
gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and at gbyte commented
Status: Needs review » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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