search

cYbercOsmOnauT / wysiwygsceditorphpbb

8 stars 11 forks source link

YouTube videos not showing in post #24

Closed AutoJunkie79 closed 7 years ago

AutoJunkie79 commented 7 years ago

when attempting to embed a YouTube video, the clip shows up fine in the editor but not in the post. I'm not much of a developer, but it does appear that the video is being embedded in the final post even though it is not visible.

phpBB 3.2.0 PHP 7.1.1 ComBoot Theme (I switched back to prosilver and result is the same)

Enabled Extensions:

Installed but not Enabled:

[ Working in the editor ]

editor

[ Actual post ]

post

[ Inpsector Code Snippet ]

inspect

AutoJunkie79 commented 7 years ago

Ok, so I did some more research, it appears this should have been fixed in SCEditor v1.5.0: https://github.com/samclarke/SCEditor/issues/489

However I see the same error in my inspector console:

console

and I appear to have SCEditor 1.5.2, I only recently installed your extension:

sceditor_ver_verify

Not sure to go from here, thank you for looking into this! Let me know how I can help.

AutoJunkie79 commented 7 years ago

Also, the URL I actually put into the editor was https://youtu.be/25QyCxVkXwQ

cYbercOsmOnauT commented 7 years ago

The problem occurs because you have a secure domain and the YT videos gets loaded over unsecure http. I will look into this.

AutoJunkie79 commented 7 years ago

The URL we see in the inspector/console is not the one I put into the editor. I copied and pasted the https://youtu.be/25QyCxVkXwQ URL straight from the YouTube page after clicking on "Share". It seems like the editor or something else might be parsing the URL I entered, taking the unique ID of the video (25QyCxVkXwQ) and re-inserting it as a variable into the non-secure version. This probably explains why the video does not show in the preview either.

cYbercOsmOnauT commented 7 years ago

Go to your ACP - POSTING - BBCodes, scroll down to youtube and click on Edit inside the HTML Replacement change http to https and save it.

I will change this too in my project.

AutoJunkie79 commented 7 years ago

Good find, I didn't even think to look there! Still very new to this. Thanks again for your help!

cYbercOsmOnauT commented 7 years ago

Don't close it until I commit the solution for the new users ;)

AutoJunkie79 commented 7 years ago

Sorry very new to GitHub too. ^_^