Even though api.drupal.org doesn't support classes yet, the help for common.inc/entity_load():

 * @see DrupalEntityController

should be corrected that way:

 * @see DrupalEntityControllerInterface

as defined in entity.inc.

The text above that line had to be modified also.

CommentFileSizeAuthor
#3 core-common-entity_load-DrupalEntityControllerInterface.931352.patch1.41 KBLaurentAjdnik
#1 core-common-entity_load-DrupalEntityControllerInterface.931352.patch1.44 KBLaurentAjdnik

Comments

LaurentAjdnik’s picture

LaurentAjdnik commented
Status: Active » Needs review
Issue tags: +Entity system
StatusFileSize
new core-common-entity_load-DrupalEntityControllerInterface.931352.patch1.44 KB

Status: Needs review » Needs work

The last submitted patch, core-common-entity_load-DrupalEntityControllerInterface.931352.patch, failed testing.

LaurentAjdnik’s picture

LaurentAjdnik commented
Status: Needs work » Needs review
StatusFileSize
new core-common-entity_load-DrupalEntityControllerInterface.931352.patch1.41 KB
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs review » Reviewed & tested by the community

jep, that makes more sense.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Issue tags: +Quick fix
dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Entity system

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

Issue tags: +Quick fix, +Entity system

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.