From fdf53b85501d43923733845ec503e7114fc4955b Mon Sep 17 00:00:00 2001 From: Shannon Date: Thu, 25 Jun 2015 17:10:45 +0200 Subject: [PATCH] puts the Flickr provider back for backwards compat, adds a todo and updates the release config with all providers --- .../config/EmbeddedMedia.Release.config | 72 +++++++++++++++---- .../EmbedProviders/AbstractOEmbedProvider.cs | 2 + .../Media/EmbedProviders/Flickr.cs | 25 +++++++ src/Umbraco.Web/Umbraco.Web.csproj | 1 + 4 files changed, 88 insertions(+), 12 deletions(-) create mode 100644 src/Umbraco.Web/Media/EmbedProviders/Flickr.cs diff --git a/src/Umbraco.Web.UI/config/EmbeddedMedia.Release.config b/src/Umbraco.Web.UI/config/EmbeddedMedia.Release.config index fe58a8fa84..6472968e44 100644 --- a/src/Umbraco.Web.UI/config/EmbeddedMedia.Release.config +++ b/src/Umbraco.Web.UI/config/EmbeddedMedia.Release.config @@ -1,17 +1,11 @@ - + - - - - - - @@ -20,16 +14,54 @@ xml + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - @@ -67,5 +99,21 @@ - + + + + + + + + + + + + + + + + + diff --git a/src/Umbraco.Web/Media/EmbedProviders/AbstractOEmbedProvider.cs b/src/Umbraco.Web/Media/EmbedProviders/AbstractOEmbedProvider.cs index 2bf01d40b2..51622ed504 100644 --- a/src/Umbraco.Web/Media/EmbedProviders/AbstractOEmbedProvider.cs +++ b/src/Umbraco.Web/Media/EmbedProviders/AbstractOEmbedProvider.cs @@ -8,6 +8,8 @@ using Umbraco.Core.Media; namespace Umbraco.Web.Media.EmbedProviders { + //TODO: Make all Http calls async + public abstract class AbstractOEmbedProvider: IEmbedProvider { public virtual bool SupportsDimensions diff --git a/src/Umbraco.Web/Media/EmbedProviders/Flickr.cs b/src/Umbraco.Web/Media/EmbedProviders/Flickr.cs new file mode 100644 index 0000000000..944513ee4f --- /dev/null +++ b/src/Umbraco.Web/Media/EmbedProviders/Flickr.cs @@ -0,0 +1,25 @@ +using System; +using System.ComponentModel; +using System.Web; + +namespace Umbraco.Web.Media.EmbedProviders +{ + [EditorBrowsable(EditorBrowsableState.Never)] + [Obsolete("This is no longer used and will be removed from the codebase in the future, for Flickr, use the Umbraco.Web.Media.EmbedProviders.OEmbedPhoto provider")] + public class Flickr : AbstractOEmbedProvider + { + public override string GetMarkup(string url, int maxWidth, int maxHeight) + { + var flickrUrl = BuildFullUrl(url, maxWidth, maxHeight); + var doc = GetXmlResponse(flickrUrl); + + string imageUrl = doc.SelectSingleNode("/oembed/url").InnerText; + string imageWidth = doc.SelectSingleNode("/oembed/width").InnerText; + string imageHeight = doc.SelectSingleNode("/oembed/height").InnerText; + string imageTitle = doc.SelectSingleNode("/oembed/title").InnerText; + + return string.Format("\"{3}\"", + imageUrl, imageWidth, imageHeight, HttpUtility.HtmlEncode(imageTitle)); + } + } +} \ No newline at end of file diff --git a/src/Umbraco.Web/Umbraco.Web.csproj b/src/Umbraco.Web/Umbraco.Web.csproj index 17322a218f..ea3f838076 100644 --- a/src/Umbraco.Web/Umbraco.Web.csproj +++ b/src/Umbraco.Web/Umbraco.Web.csproj @@ -302,6 +302,7 @@ +