First - incorrect use checkbox/value for "Convert on save" field. "value" type field should have only "value" instead "default_value" for checkbox. Also, I've noted, that sometimes even if "default_value" set to TRUE or 1, it still renders unchecked, so I've added "attributes" workaround.

Second issue - "convert_on_save" saved in different place when "value" field type used in video_convert_process() function.

Patch attached. Review and comments are welcome!

CommentFileSizeAuthor
video.module.checkbox.patch3.79 KBpavel.karoukin

Comments

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Needs review » Reviewed & tested by the community
hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Reviewed & tested by the community » Patch (to be ported)
hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Patch (to be ported) » Reviewed & tested by the community
hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Reviewed & tested by the community » Closed (fixed)