Problem/Motivation

When rendering the video field class, the code is trying to retrieve a setting called "class" that might not always be set. When not set, a warning is thrown.

Proposed resolution

Check that the setting does exist.

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None

Original report

Notice: Undefined index: class in video_embed_field_handle_vimeo() (Line 519 of video_embed_field/video_embed_field.handlers.inc).

CommentFileSizeAuthor
#4 2589857-4.patch877 bytesmichaellenahan
#2 2589857-2.patch560 bytesmichaellenahan
Command icon Show commands

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

michaellenahan created an issue. See original summary.

michaellenahan’s picture

FileSize
560 bytes

Here's the patch.

michaellenahan’s picture

Status: Active » Needs review
michaellenahan’s picture

FileSize
877 bytes

Oh, it's needed for video_embed_field_handle_youtube() as well.

Parker Sydnor Historic Site’s picture

Can you please let me know if this can be fixed with administrative controls in the Drupal website?

david_garcia’s picture

Status: Needs review » Reviewed & tested by the community
david_garcia’s picture

Issue summary: View changes
Chris Matthews’s picture

kunal_sahu made their first commit to this issue’s fork.

kunal_sahu’s picture

Hi @all Thanks you for all your patches. The patch 2589857-4.patch

</code> applied successfully.

<code>PS C:\xampp\htdocs\drupal\web\themes\custom\video_embed_field-2589857> git apply -v .\2589857-4.patch
Checking patch video_embed_field.handlers.inc...
Applied patch video_embed_field.handlers.inc cleanly.

Creating an MR for helping out maintainer. Thanks michaellenahan for the patch . Thanks