Add missing Content Translation dependancy [#3174757]

Comment	File	Size	Author
#3	3174757-2_opigno_course-add_missing_content_translation_dependency.patch	647 bytes	spheresh
	opigno_course-add_missing_content_translation_dependency-D8.patch	387 bytes	somersoft

Comment #1

4 October 2020 at 14:01

somersoft created an issue. See original summary.

Log in or register to post comments

Comment #2

claudiu.cristea

Romanian

Arad 🇷🇴

commented 26 January 2021 at 08:56

Status:

Needs review

» Needs work

But why should field.field.media.image_png.field_media_image depend on content_translation? I guess the site should work w/o content_translation enabled. Enabling the module should be optional. Wouldn't be better if we remove the dependency from field.field.media.image_png.field_media_image?

Log in or register to post comments

Comment #3

spheresh commented 2 February 2021 at 13:22

Status	File	Size
new	3174757-2_opigno_course-add_missing_content_translation_dependency.patch	647 bytes

Because that this is the only place where the dependency on the content translation and there are no other configs of content translation, I think we should add a patch that removes this dependency.