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
avpadernoComment #3
avpadernoComment #4
mohit1604 commentedComment #5
mohit1604 commented@kiamlaluno Thanks for the issue. Please review the patch :)
Comment #6
mohit1604 commentedComment #7
chiranjeeb2410 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!