Closed (fixed)
Project:
Scheduler
Version:
2.x-dev
Component:
User Interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Jun 2022 at 17:42 UTC
Updated:
2 Nov 2022 at 09:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
jonathan1055 commentedHi @demon326,
Thanks for the info. I have investigated this, and I want to make sure I am doing the same as you:
The screen shots you show are slightly different, so I'm not certain that the steps above are exactly what you are doing. I have added two of mine.
The reason the Scheduler fields do not show in my screen shots is because the form id is 'editor_image_dialog' and this is not catered for in the hook_form_alter processing. But your screen shot is not this form. So could you tell me your actual steps to this form?
Comment #4
demon326 commentedThe following video shows the problem. It shows that a external video gets the options, an image uploaded not.
The option do appear on media/add/image , but not when using the media librarie widget.
https://www.youtube.com/watch?v=p0pPED0iVwQ
Screenshots of settings are in the attachment of this issue.
Comment #5
jonathan1055 commentedAh, now I can see you are using the Media Library module, and have customised the ckeditor toolbar to add the 'insert from media library' button and have enabled the 'embed media' filter.
This problem has already been found and discussed on Scheduler issue #2916730: Scheduler fields displayed in Media Library upload form when media is not enabled for scheduling. The problem is that the fields have to be hidden for all media types here because we cannot properly modify the form. There is no
getFormDisplay() method. The fields should not be showing up for remote video either - the reason that they are is because that form id is 'media_library_add_form_oembed' whereas we only catered for 'media_library_add_form_upload' (which covers all the other media types). You will see if you disable scheduling for Remote Video that the fields still show up. This is why we have to remove them for all media types, as we can't accurately determine which fields should be shown, which are required, etc.I have raised two core issues:
#3269103: Add getFormDisplay() method to media_library AddFormBase
#3269471: Expose 'published' status filter in media library views
If these get fixed then Scheduler can properly modify the media upload and oembed forms.
Comment #6
demon326 commentedThanks for taking your time and to get back at me! I tried searching the que before i created the issue, but did not found something. Now i can stop running in rounds looking for why those fields are not there. Lets hope core devs have some time to look at such little issue...
Comment #7
jonathan1055 commentedI am not surprised that you didn't find the existing issue, as the subject was reporting the opposite problem. I think we need to make the remote video embed form behave in the same way - that was always the intension but I did not realise that the form id was different for that one.
If there is another way to get round this, before those core issues are done, then it would be good. I did investigate that last time, and did not find a way.
Comment #9
jonathan1055 commentedComment #11
jonathan1055 commentedMerged and fixed.