Deprecate Node::getTitle() & other entity-type-specific calls in favour of Entity::label()

getTitle() was added in the original Node/EntityNG conversion on the request of @webchick because she thought that label() was confusing. That might have changed since then, it was a long time ago and generic entity API was new thing back then. You likely still want to involve here in this.

If/when this is greenlighted we might as well deprecate Term::getName(), since it is a similar wrapper around $this->label()

Nay sayers might argue the entity label should match the input label, so node's 'Title' matches the code Node::getTitle(). This is currently true for the content entity types in core: node, taxonomy term, media. Actually MediaInterface was recently expanded to explicitly use getName / setName in #2897009: MediaInterface is missing setName() and getName()

Assigning to webchick

Sure. I'm not gonna argue with people who actually use the subsystem every day. :)

WimLeers created #2897009: MediaInterface is missing setName() and getName() while working on #2835767-46: Media + REST: comprehensive test coverage for Media + MediaType entity types. I can't find why it was a bug, but maybe he can provide some insight?

Should we also deprecate wrapper methods around bundle(), like NodeInterface::getType()?

This needs separate issues per entity type to clean-up usage as well but first it needs approval

I am +1 on doing the change across all entities. No need for multiple methods doing the same thing, which is actually what is confusing.

+1 I think people are used to the generic entity methods, also sometimes I see something like getType() and have to remind myself it's an alias for bundle().

If this doesn't make it into 8.8.x, it could still go into 8.9.x but the deprecations will need to be for 10.0.x

+++ b/core/modules/node/src/NodeInterface.php
@@ -56,6 +56,11 @@ public function getType();
    * @return string
    *   Title of the node.
+   *
+   * @deprecated in drupal:8.8.0, will be removed from drupal:9.0.0. Use
+   *   \Drupal\Core\Entity\Entity::label() instead
+   *
+   * @see https://www.drupal.org/node/3085319
    */
   public function getTitle();

what about getType()/bundle()?

Not sure if we split the issue/issues on entity type or type of method. Lets see how big it will be with all the replacements done in tests and so on.

In addition to getType()/bundle(), what about setName($name)/set($name, $value, $notify = TRUE) ? Should it be deprecated as well? Whether do it in a separate issue?

no, setName() and set() are not the same thing.

Re #27, let me reword it.

setName() is just an variation of setTitle().

Node::getTitle() is getting deprecated which makes me wondering should Node::setTitle() be deprecated.

Further, I am thinking of introducing a common method setLabel() to the base class ContentEntityBase.

Then setTitle(), setName() calls can be deprecated in favour of using setLabel().

Or deprecate them directly, as setTitle() is just a wrapper of the set() method. (see below). Personally, I prefer the former way -- introduce setLabel(), and deprecate setTitle(), setName()

/**
   * {@inheritdoc}
   */
  public function setTitle($title) {
    $this->set('title', $title);
    return $this;
  }

1. +++ b/core/modules/node/src/Entity/Node.php
   @@ -197,7 +197,8 @@ public function access($operation = 'view', AccountInterface $account = NULL, $r
      public function getTitle() {
   -    return $this->get('title')->value;
   +    @trigger_error(__NAMESPACE__ . '\Node::getTitle is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Use \Drupal\Core\Entity\Entity::label() instead. See https://www.drupal.org/node/3085319', E_USER_DEPRECATED);
   +    return $this->label();
      }
   

   And here, should Use \Drupal\Core\Entity\Entity::label() instead be Use \Drupal\node\Entity\Node::label() instead?