Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The comment describing what ContentEntityBase::isTranslatable() does is the following one.
// Check that the bundle is translatable, the entity has a language defined
// and if we have more than one language on the site.
In American English, Oxford comma should be used and that is not necessary. I would also change from we have more than one language to the site has more than one language.
// Check the bundle is translatable, the entity has a language defined, and
// the site has more than one language.
Comment | File | Size | Author |
---|---|---|---|
#5 | 2940051-5-D8.patch | 885 bytes | mohit1604 |
Comments
Comment #2
apadernoComment #3
apadernoComment #4
mohit1604 CreditAttribution: mohit1604 at Google Summer of Code commentedComment #5
mohit1604 CreditAttribution: mohit1604 at Google Summer of Code commented@kiamlaluno Thanks for the issue. Please review the patch :)
Comment #6
mohit1604 CreditAttribution: mohit1604 at Google Summer of Code commentedComment #7
chiranjeeb2410 CreditAttribution: chiranjeeb2410 at Google Summer of Code commentedPatch applies cleanly and improvements as suggested by @kiamluno in description.
RTBC for me.
Comment #9
catchCommitted/pushed to 8.6.x and cherry-picked to 8.5.x. Thanks!