Skip to content
This repository has been archived by the owner on May 17, 2020. It is now read-only.

Existence check for title in oembedData object. #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nathanielford
Copy link

Checks to make sure that oembedData.title even exists within the object; failure to exist results in 'undefined' to be inserted in the link. At least one third party (Flickr) fails to return this object with a title attribute.

Example:
http://www.flickr.com/photos/ahattar/8158065331/lightbox/
Goes to a login page, but an attempt to embed will return a 'success' result without a title object.

Checks to make sure that oembedData.title even exists within the object; failure to exist results in 'undefined' to be inserted in the link. At least one third party (Flickr) fails to return this object with a title attribute.

Example:
http://www.flickr.com/photos/ahattar/8158065331/lightbox/
Goes to a login page, but an attempt to embed will return a 'success' result without a title object.
@LGCambra
Copy link

LGCambra commented Sep 8, 2014

The NFL would like to continue support for this plugin. Please re-submit your pull request with https://github.com/nfl/jquery-oembed-all. Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants