Problem/Motivation

As it has been introduced in #1850080: Entity type labels lack plurality, cannot generate UI text based on label if plural is needed now we have a way to specify plural label for any entity type. Let's do it for Tour module.

Proposed resolution

Add plural labels to all entity types defined by Tour module. More information can be found in the change record: https://www.drupal.org/node/2689949

Remaining tasks

Write the patch.

User interface changes

Nope.

API changes

Nope.

Data model changes

Nope.

CommentFileSizeAuthor
#2 2703103-2.patch594 bytesrosinegrean
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

prics created an issue. See original summary.

rosinegrean’s picture

rosinegrean CreditAttribution: rosinegrean at PitechPlus commented
Status: Active » Needs review
FileSize
2703103-2.patch594 bytes
rosinegrean’s picture

rosinegrean CreditAttribution: rosinegrean at PitechPlus commented
Parent issue: » #1850080: Entity type labels lack plurality, cannot generate UI text based on label if plural is needed
therealssj’s picture

therealssj CreditAttribution: therealssj commented

Just one nit, shouldn't it be "Tour" instead of "tour" for the label?
Otherwise the patch applies well.

rosinegrean’s picture

rosinegrean CreditAttribution: rosinegrean at PitechPlus commented

@therealssj

as you can see in here Entity types can specify plural labels now, capital letters are not required.
Thanks,

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Closed (duplicate)
Related issues: +#2702683: Add plural labels to entity types

These issues were not supposed to be separate issues. It is being handled in one big issue on #2702683: Add plural labels to entity types. Please comment there to get credit, and/or merge your patch in.

rosinegrean’s picture

rosinegrean CreditAttribution: rosinegrean at PitechPlus commented
Issue tags: -DCTransylvania