Closed (won't fix)
Project:
Varbase Media
Version:
9.0.24
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Mar 2024 at 16:16 UTC
Updated:
5 Mar 2024 at 04:35 UTC
Jump to comment: Most recent
The embedded video shows a 404 error when clicked on.
This occurs when the user is logged in.
This issue occurs because the JavaScript code is executing multiple times.
Prevent multiple executions.
| Comment | File | Size | Author |
|---|---|---|---|
| prevent-multiple-executions.patch | 3.64 KB | mutasim al-shoura |
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
rajab natshahComment #3
rishabjasrotia commentedComment #4
rishabjasrotia commentedHello @rajab-natshah,
The patch you have added does not belong to this module source code.
I am not able to apply this patch!
Comment #5
rishabjasrotia commentedComment #6
rajab natshahNo further support for the Varbase Media
9.0.xbranch.If this patch is for the
9.1.xbranch, It will be reviwed.If not it will be closed as want fix.
Comment #7
mutasim al-shoura commentedSorry, the code is related to this issue. I updated the patch there.
https://www.drupal.org/project/varbase_media/issues/3260536
Comment #8
rajab natshahNoted; Mutasim
Thanks, for reporting and patching.
Thank you, Rishab, for testing the patch.
You may close this issue as won't fix or as doublicated
Let us move the converstion to #3260536: Add an experimental Lazy Load OEmbed field formatter for remote videos in Varbase Media
We may add it to the
9.1.xbranch and maybe to10.0.xAs we drpped support for Drupal 9 ( only trying to save time for Varbase
9.1.xand Varbase10.0.x)Comment #9
rishabjasrotia commentedClosing this issue since no relevant code changes needed.