While working on translation integration for commerce rfay and I came across the method TranslationNodeHandler::getHumanReadableId(). Since we couldn't find any call to this method at all and the interface doesn't define it, it looks like this method is superfluous. True?

Attached patch removes the method.

CommentFileSizeAuthor
translation-superfluous-method-getHumanReadableId.patch600 bytesdas-peter

Comments

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Needs review » Reviewed & tested by the community

Yes, it has been obsoleted by TranslationHandlerInterface::getLabel().

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Reviewed & tested by the community » Fixed

Thanks for reporting, reviewing, and testing! Committed to HEAD.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)
Issue tags: -dcsprint5

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

Issue tags: +dcsprint5

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