From 47d70e858b7ddd1b8f7979a3bc465fb233aaa3ec Mon Sep 17 00:00:00 2001
From: Shannon <unknown@kiln.example.com>
Date: Mon, 10 Jun 2013 10:36:59 -0200
Subject: [PATCH] Getting media picker working

---
 src/Umbraco.Web.UI/Umbraco.Web.UI.csproj      |   3 +
 .../umbraco/Views/Default.cshtml              |   4 +-
 .../Views/common/dialogs/mediapicker.html     |  44 ++++++++
 .../umbraco/js/umbraco.controllers.js         |  10 +-
 .../umbraco/js/umbraco.resources.js           |  73 ++++++++++---
 .../Editors/BackOfficeController.cs           |   3 +-
 ...rApiController.cs => ContentController.cs} |  15 +--
 src/Umbraco.Web/Editors/MediaController.cs    |  55 ++++++++++
 .../Models/ContentEditing/ContentItemBasic.cs |   6 +-
 .../ContentEditing/ContentItemDisplay.cs      |   2 +
 .../Models/ContentEditing/ContentItemSave.cs  |   2 +-
 ...ropertyBase.cs => ContentPropertyBasic.cs} |   6 +-
 .../ContentEditing/ContentPropertyDisplay.cs  |   2 +-
 .../ContentEditing/ContentPropertyDto.cs      |   2 +-
 .../Models/Mapping/ContentModelMapper.cs      |  24 +++-
 .../Models/Mapping/MediaModelMapper.cs        | 103 ++++++++++++++++++
 .../Mvc/AreaRegistrationExtensions.cs         |   2 +-
 src/Umbraco.Web/Mvc/PluginControllerArea.cs   |   4 +-
 src/Umbraco.Web/Umbraco.Web.csproj            |   6 +-
 src/Umbraco.Web/WebBootManager.cs             |   9 +-
 20 files changed, 321 insertions(+), 54 deletions(-)
 create mode 100644 src/Umbraco.Web.UI/umbraco/Views/common/dialogs/mediapicker.html
 rename src/Umbraco.Web/Editors/{ContentEditorApiController.cs => ContentController.cs} (90%)
 create mode 100644 src/Umbraco.Web/Editors/MediaController.cs
 rename src/Umbraco.Web/Models/ContentEditing/{ContentPropertyBase.cs => ContentPropertyBasic.cs} (82%)
 create mode 100644 src/Umbraco.Web/Models/Mapping/MediaModelMapper.cs

diff --git a/src/Umbraco.Web.UI/Umbraco.Web.UI.csproj b/src/Umbraco.Web.UI/Umbraco.Web.UI.csproj
index 4b03a9439b..e34c65aead 100644
--- a/src/Umbraco.Web.UI/Umbraco.Web.UI.csproj
+++ b/src/Umbraco.Web.UI/Umbraco.Web.UI.csproj
@@ -292,6 +292,7 @@
     <Compile Include="App_Plugins\MyPackage\PropertyEditors\ServerInfoPropertyEditor.cs" />
     <Compile Include="App_Plugins\MyPackage\System\MyStartupHandler.cs" />
     <Compile Include="App_Plugins\MyPackage\Trees\LegacyTestTree.cs" />
+    <Compile Include="App_Plugins\TestPackage\PropertyEditors\TestEditor.cs" />
     <Compile Include="Install\Default.aspx.cs">
       <DependentUpon>default.aspx</DependentUpon>
       <SubType>ASPXCodeBehind</SubType>
@@ -563,6 +564,8 @@
     <Content Include="App_Plugins\MyPackage\PropertyEditors\Views\FileUploadEditor.html" />
     <Content Include="App_Plugins\MyPackage\PropertyEditors\Views\PostcodeEditor.html" />
     <Content Include="App_Plugins\MyPackage\PropertyEditors\Views\RegexEditor.html" />
+    <Content Include="App_Plugins\TestPackage\PropertyEditors\Js\Test.js" />
+    <Content Include="App_Plugins\TestPackage\PropertyEditors\Views\Test.html" />
     <Content Include="Config\Splashes\booting.aspx" />
     <Content Include="Config\Splashes\noNodes.aspx" />
     <Content Include="Umbraco\lib\Umbraco\Extensions.js" />
diff --git a/src/Umbraco.Web.UI/umbraco/Views/Default.cshtml b/src/Umbraco.Web.UI/umbraco/Views/Default.cshtml
index e909f90a47..8e684b7729 100644
--- a/src/Umbraco.Web.UI/umbraco/Views/Default.cshtml
+++ b/src/Umbraco.Web.UI/umbraco/Views/Default.cshtml
@@ -21,8 +21,6 @@
 
     @*Currently this needs to be loaded before anything*@
     <script src="@Url.Action("ServerVariables", "BackOffice")" type="text/javascript"></script>
-    
-    
 
     <style type="text/css">
         .validation-message
@@ -68,7 +66,7 @@
     <umb-login></umb-login>
     <umb-notifications></umb-notifications>
 
-    <script  src="lib/require/require.min.js" type="text/javascript"></script>
+    <script src="lib/require/require.min.js" type="text/javascript"></script>
     
     @*We cannot use the data-main attribute above because we need to load in the application 
     via a server side JavaScript request, so we just request it after requirejs*@
diff --git a/src/Umbraco.Web.UI/umbraco/Views/common/dialogs/mediapicker.html b/src/Umbraco.Web.UI/umbraco/Views/common/dialogs/mediapicker.html
new file mode 100644
index 0000000000..8ecb90c060
--- /dev/null
+++ b/src/Umbraco.Web.UI/umbraco/Views/common/dialogs/mediapicker.html
@@ -0,0 +1,44 @@
+<div class="umb-panel" ng-controller="Umbraco.Dialogs.MediaPickerController">
+	<div class="umb-panel-header">
+		<div class="umb-el-wrap umb-panel-buttons">
+	        <div class="pull-right umb-btn-toolbar">
+	         	<button class="btn">Create <i class="icon-upload"></i></button> 
+	         	<button type="button" ng-click="submit(dialogData)" class="btn btn-primary">Select (x)</button>
+	        </div>
+		</div>
+	</div>
+
+
+	<div class="umb-panel-body umb-scrollable" auto-scale="1">
+		<div class="umb-panel">
+			<div class="umb-control-group">
+				<ul class="breadcrumb">
+					<li><strong>You are here:</strong></li>
+					<li><a href="#">Media</a></li>
+				</ul>
+				<div id="search-form" ng-animate="'slide'">
+                        <form class="form-search" ng-controller="SearchController">
+                            <i class="icon-search"></i>
+                            <input type="text" 
+                                   ng-model="ui.searchTerm" 
+                                   class="umb-search-field search-query" 
+                                   placeholder="{{localization.app.search.typeToSearch}}"
+                                   on-blur="deActivateSearch()" 
+                                   on-keyup="performSearch(ui.searchTerm)">
+                        </form>
+                </div>
+				</ul>
+
+					<ul class="thumbnails">						
+						<li class="span2" ng-repeat="image in images">
+							<a href="#" class="thumbnail" ng-class="{selected: dialogData.selection.indexOf(image) > -1}" ng-click="select(image)" prevent-default>
+								<img ng-src="{{image.thumbnail}}" />
+							</a>
+						</li>
+					</ul>	
+			</div>
+		</div>
+
+	</div>
+
+</div>
\ No newline at end of file
diff --git a/src/Umbraco.Web.UI/umbraco/js/umbraco.controllers.js b/src/Umbraco.Web.UI/umbraco/js/umbraco.controllers.js
index e193014a20..707549ae74 100644
--- a/src/Umbraco.Web.UI/umbraco/js/umbraco.controllers.js
+++ b/src/Umbraco.Web.UI/umbraco/js/umbraco.controllers.js
@@ -129,7 +129,7 @@ angular.module('umbraco').controller("MainController",
 });
 
 //used for the media picker dialog
-angular.module("umbraco").controller("Umbraco.Dialogs.ContentPickerController", function ($scope, mediaFactory) {	
+angular.module("umbraco").controller("Umbraco.Dialogs.ContentPickerController", function ($scope, mediaResource) {
 	
 	$scope.$on("treeNodeSelect", function(event, args){
 		$(args.event.target.parentElement).find("i").attr("class", "icon umb-tree-icon sprTree icon-check blue");
@@ -147,8 +147,12 @@ angular.module("umbraco").controller("Umbraco.Dialogs.MacroPickerController", fu
 	};
 });
 //used for the media picker dialog
-angular.module("umbraco").controller("Umbraco.Dialogs.MediaPickerController", function ($scope, mediaFactory) {	
-	$scope.images = mediaFactory.rootMedia();
+angular.module("umbraco").controller("Umbraco.Dialogs.MediaPickerController", function ($scope, mediaResource) {
+
+    mediaResource.rootMedia()
+        .then(function (data) {
+            $scope.images = data;
+        });    
 });
 angular.module("umbraco").controller("Umbraco.Common.LegacyController", 
 	function($scope, $routeParams){
diff --git a/src/Umbraco.Web.UI/umbraco/js/umbraco.resources.js b/src/Umbraco.Web.UI/umbraco/js/umbraco.resources.js
index bca4c9892c..f4bd8e77b0 100644
--- a/src/Umbraco.Web.UI/umbraco/js/umbraco.resources.js
+++ b/src/Umbraco.Web.UI/umbraco/js/umbraco.resources.js
@@ -6,6 +6,44 @@
 'use strict';
 define(['app', 'angular'], function (app, angular) {
 
+    /**
+    * @ngdoc factory 
+    * @name umbraco.resources.treeResource     
+    * @description Loads in data for trees
+    **/
+    function mediaResource($q, $http) {
+        
+        /** internal method to get the api url */
+        function getRootMediaUrl(section) {
+            return Umbraco.Sys.ServerVariables.mediaApiBaseUrl + "GetRootMedia";
+        }
+
+        return {
+            rootMedia: function () {
+
+                var deferred = $q.defer();
+
+                //go and get the tree data
+                $http.get(getRootMediaUrl()).
+                    success(function (data, status, headers, config) {
+                        deferred.resolve(data);
+                    }).
+                    error(function (data, status, headers, config) {
+                        deferred.reject('Failed to retreive data for application tree ' + section);
+                    });
+
+                return deferred.promise;
+
+                return [
+                { id: 1234, src: "/Media/boston.jpg", thumbnail: "/Media/boston.jpg" },
+                { src: "/Media/bird.jpg", thumbnail: "/Media/bird.jpg" },
+                { src: "/Media/frog.jpg", thumbnail: "/Media/frog.jpg" }
+                ];
+            }
+        };
+    }
+    angular.module('umbraco.resources.media', []).factory('mediaResource', mediaResource);
+
 
     /**
     * @ngdoc factory 
@@ -125,7 +163,6 @@ define(['app', 'angular'], function (app, angular) {
     }
     angular.module('umbraco.resources.contentType', []).factory('contentTypeResource', contentTypeResource);
 
-
     /**
     * @ngdoc factory 
     * @name umbraco.resources.contentResource
@@ -135,15 +172,15 @@ define(['app', 'angular'], function (app, angular) {
         
         /** internal method to get the api url */
         function getContentUrl(contentId) {
-            return Umbraco.Sys.ServerVariables.contentEditorApiBaseUrl + "GetContent?id=" + contentId;
+            return Umbraco.Sys.ServerVariables.contentApiBaseUrl + "GetContent?id=" + contentId;
         }
         /** internal method to get the api url */
         function getEmptyContentUrl(contentTypeAlias, parentId) {
-            return Umbraco.Sys.ServerVariables.contentEditorApiBaseUrl + "GetEmptyContent?contentTypeAlias=" + contentTypeAlias + "&parentId=" + parentId;
+            return Umbraco.Sys.ServerVariables.contentApiBaseUrl + "GetEmptyContent?contentTypeAlias=" + contentTypeAlias + "&parentId=" + parentId;
         }
         /** internal method to get the api url for publishing */
         function getSaveUrl() {
-            return Umbraco.Sys.ServerVariables.contentEditorApiBaseUrl + "PostSaveContent";
+            return Umbraco.Sys.ServerVariables.contentApiBaseUrl + "PostSave";
         }
         /** internal method process the saving of data and post processing the result */
         function saveContentItem(content, action) {
@@ -603,19 +640,21 @@ angular.module('umbraco.resources.macro', [])
         }
     };
 });
-angular.module('umbraco.resources.media', [])
-.factory('mediaFactory', function () {
-    var mediaArray = [];
-    return {
-        rootMedia: function(){
-          return [
-          {id: 1234, src: "/Media/boston.jpg", thumbnail: "/Media/boston.jpg" },
-          {src: "/Media/bird.jpg", thumbnail: "/Media/bird.jpg" },
-          {src: "/Media/frog.jpg", thumbnail: "/Media/frog.jpg" }
-          ];
-      }
-  };
-});
+
+//angular.module('umbraco.resources.media', [])
+//.factory('mediaFactory', function () {
+//    var mediaArray = [];
+//    return {
+//        rootMedia: function(){
+//          return [
+//          {id: 1234, src: "/Media/boston.jpg", thumbnail: "/Media/boston.jpg" },
+//          {src: "/Media/bird.jpg", thumbnail: "/Media/bird.jpg" },
+//          {src: "/Media/frog.jpg", thumbnail: "/Media/frog.jpg" }
+//          ];
+//      }
+//  };
+//});
+
 angular.module('umbraco.resources.tags', [])
 .factory('tagsFactory', function () {
 	return {
diff --git a/src/Umbraco.Web/Editors/BackOfficeController.cs b/src/Umbraco.Web/Editors/BackOfficeController.cs
index 9874cc9c54..c23a47260f 100644
--- a/src/Umbraco.Web/Editors/BackOfficeController.cs
+++ b/src/Umbraco.Web/Editors/BackOfficeController.cs
@@ -47,7 +47,8 @@ namespace Umbraco.Web.Editors
             var d = new Dictionary<string, object>
                 {
                     {"umbracoPath", GlobalSettings.Path},
-                    {"contentEditorApiBaseUrl", Url.GetUmbracoApiService<ContentEditorApiController>("PostSaveContent").TrimEnd("PostSaveContent")},
+                    {"contentApiBaseUrl", Url.GetUmbracoApiService<ContentController>("PostSave").TrimEnd("PostSave")},
+                    {"mediaApiBaseUrl", Url.GetUmbracoApiService<MediaController>("GetRootMedia").TrimEnd("GetRootMedia")},
                     {"treeApplicationApiBaseUrl", Url.GetUmbracoApiService<ApplicationTreeApiController>("GetTreeData").TrimEnd("GetTreeData")},
                     {"contentTypeApiBaseUrl", Url.GetUmbracoApiService<ContentTypeApiController>("GetAllowedChildrenForContent").TrimEnd("GetAllowedChildrenForContent")}
                 };
diff --git a/src/Umbraco.Web/Editors/ContentEditorApiController.cs b/src/Umbraco.Web/Editors/ContentController.cs
similarity index 90%
rename from src/Umbraco.Web/Editors/ContentEditorApiController.cs
rename to src/Umbraco.Web/Editors/ContentController.cs
index 257f2d70f6..e105eafb03 100644
--- a/src/Umbraco.Web/Editors/ContentEditorApiController.cs
+++ b/src/Umbraco.Web/Editors/ContentController.cs
@@ -18,16 +18,16 @@ namespace Umbraco.Web.Editors
     /// <summary>
     /// The API controller used for editing content
     /// </summary>
-    [PluginController("UmbracoEditors")]
+    [PluginController("UmbracoApi")]
     [ValidationFilter]
-    public class ContentEditorApiController : UmbracoAuthorizedApiController
+    public class ContentController : UmbracoAuthorizedApiController
     {
         private readonly ContentModelMapper _contentModelMapper;
 
         /// <summary>
         /// Constructor
         /// </summary>
-        public ContentEditorApiController()
+        public ContentController()
             : this(UmbracoContext.Current, new ContentModelMapper(UmbracoContext.Current.Application, new ProfileModelMapper()))
         {            
         }
@@ -37,7 +37,7 @@ namespace Umbraco.Web.Editors
         /// </summary>
         /// <param name="umbracoContext"></param>
         /// <param name="contentModelMapper"></param>
-        internal ContentEditorApiController(UmbracoContext umbracoContext, ContentModelMapper contentModelMapper)
+        internal ContentController(UmbracoContext umbracoContext, ContentModelMapper contentModelMapper)
             : base(umbracoContext)
         {
             _contentModelMapper = contentModelMapper;
@@ -85,6 +85,7 @@ namespace Umbraco.Web.Editors
             {
                 throw new HttpResponseException(HttpStatusCode.NotFound);
             }
+
             var emptyContent = new Content("Empty", parentId, contentType);
             return _contentModelMapper.ToContentItemDisplay(emptyContent);
         }
@@ -95,9 +96,9 @@ namespace Umbraco.Web.Editors
         /// <returns></returns>
         [ContentItemValidationFilter]
         [FileUploadCleanupFilter]
-        public ContentItemDisplay PostSaveContent(
+        public ContentItemDisplay PostSave(
             [ModelBinder(typeof(ContentItemBinder))]
-            ContentItemSave contentItem)
+                ContentItemSave contentItem)
         {
             //If we've reached here it means:
             // * Our model has been bound
@@ -135,4 +136,4 @@ namespace Umbraco.Web.Editors
         }
 
     }
-}
+}
\ No newline at end of file
diff --git a/src/Umbraco.Web/Editors/MediaController.cs b/src/Umbraco.Web/Editors/MediaController.cs
new file mode 100644
index 0000000000..62d8eefa96
--- /dev/null
+++ b/src/Umbraco.Web/Editors/MediaController.cs
@@ -0,0 +1,55 @@
+using System.Collections.Generic;
+using Umbraco.Web.Models.ContentEditing;
+using Umbraco.Web.Models.Mapping;
+using Umbraco.Web.Mvc;
+using Umbraco.Web.WebApi;
+using System.Linq;
+
+namespace Umbraco.Web.Editors
+{
+
+    //internal interface IUmbracoApiService<T>
+    //{
+    //    T Get(int id);
+    //    IEnumerable<T> GetChildren(int id);
+    //    HttpResponseMessage Delete(int id);
+    //    //copy
+    //    //move
+    //    //update
+    //    //create
+    //}
+
+    [PluginController("UmbracoApi")]
+    public class MediaController : UmbracoAuthorizedApiController
+    {
+        private readonly MediaModelMapper _mediaModelMapper;
+
+        /// <summary>
+        /// Constructor
+        /// </summary>
+        public MediaController()
+            : this(UmbracoContext.Current, new MediaModelMapper(UmbracoContext.Current.Application, new ProfileModelMapper()))
+        {            
+        }
+
+        /// <summary>
+        /// Constructor
+        /// </summary>
+        /// <param name="umbracoContext"></param>
+        /// <param name="mediaModelMapper"></param>
+        internal MediaController(UmbracoContext umbracoContext, MediaModelMapper mediaModelMapper)
+            : base(umbracoContext)
+        {
+            _mediaModelMapper = mediaModelMapper;
+        }
+
+        /// <summary>
+        /// Returns the root media objects
+        /// </summary>
+        public IEnumerable<ContentItemBasic<ContentPropertyBasic>> GetRootMedia()
+        {
+            return Services.MediaService.GetRootMedia()
+                           .Select(x => _mediaModelMapper.ToMediaItemSimple(x));
+        }
+    }
+}
diff --git a/src/Umbraco.Web/Models/ContentEditing/ContentItemBasic.cs b/src/Umbraco.Web/Models/ContentEditing/ContentItemBasic.cs
index 2cf545da93..eed05955d0 100644
--- a/src/Umbraco.Web/Models/ContentEditing/ContentItemBasic.cs
+++ b/src/Umbraco.Web/Models/ContentEditing/ContentItemBasic.cs
@@ -11,10 +11,10 @@ namespace Umbraco.Web.Models.ContentEditing
     /// A model representing a basic content item
     /// </summary>
     [DataContract(Name = "content", Namespace = "")]
-    public abstract class ContentItemBasic<T>
-        where T: ContentPropertyBase
+    public class ContentItemBasic<T>
+        where T: ContentPropertyBasic
     {
-        protected ContentItemBasic()
+        public ContentItemBasic()
         {
             //ensure its not null
             _properties = new List<T>();
diff --git a/src/Umbraco.Web/Models/ContentEditing/ContentItemDisplay.cs b/src/Umbraco.Web/Models/ContentEditing/ContentItemDisplay.cs
index b778ecd865..11abf5730c 100644
--- a/src/Umbraco.Web/Models/ContentEditing/ContentItemDisplay.cs
+++ b/src/Umbraco.Web/Models/ContentEditing/ContentItemDisplay.cs
@@ -7,6 +7,8 @@ using System.Linq;
 
 namespace Umbraco.Web.Models.ContentEditing
 {
+
+
     /// <summary>
     /// A model representing a content item to be displayed in the back office
     /// </summary>    
diff --git a/src/Umbraco.Web/Models/ContentEditing/ContentItemSave.cs b/src/Umbraco.Web/Models/ContentEditing/ContentItemSave.cs
index 6c86f9d3e6..e4efea60b0 100644
--- a/src/Umbraco.Web/Models/ContentEditing/ContentItemSave.cs
+++ b/src/Umbraco.Web/Models/ContentEditing/ContentItemSave.cs
@@ -8,7 +8,7 @@ namespace Umbraco.Web.Models.ContentEditing
     /// <summary>
     /// A model representing a content item to be saved
     /// </summary>
-    public class ContentItemSave : ContentItemBasic<ContentPropertyBase>
+    public class ContentItemSave : ContentItemBasic<ContentPropertyBasic>
     {
         public ContentItemSave()
         {
diff --git a/src/Umbraco.Web/Models/ContentEditing/ContentPropertyBase.cs b/src/Umbraco.Web/Models/ContentEditing/ContentPropertyBasic.cs
similarity index 82%
rename from src/Umbraco.Web/Models/ContentEditing/ContentPropertyBase.cs
rename to src/Umbraco.Web/Models/ContentEditing/ContentPropertyBasic.cs
index ae1284cf7f..8604bf05b2 100644
--- a/src/Umbraco.Web/Models/ContentEditing/ContentPropertyBase.cs
+++ b/src/Umbraco.Web/Models/ContentEditing/ContentPropertyBasic.cs
@@ -7,7 +7,7 @@ namespace Umbraco.Web.Models.ContentEditing
     /// Represents a content property to be saved
     /// </summary>
     [DataContract(Name = "property", Namespace = "")]
-    public class ContentPropertyBase
+    public class ContentPropertyBasic
     {
         [DataMember(Name = "id", IsRequired = true)]
         [Required]
@@ -16,7 +16,7 @@ namespace Umbraco.Web.Models.ContentEditing
         [DataMember(Name = "value")]
         public string Value { get; set; }
 
-        protected bool Equals(ContentPropertyBase other)
+        protected bool Equals(ContentPropertyBasic other)
         {
             return Id == other.Id;
         }
@@ -25,7 +25,7 @@ namespace Umbraco.Web.Models.ContentEditing
         {
             if (ReferenceEquals(null, obj)) return false;
             if (ReferenceEquals(this, obj)) return true;
-            var other = obj as ContentPropertyBase;
+            var other = obj as ContentPropertyBasic;
             return other != null && Equals(other);
         }
 
diff --git a/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDisplay.cs b/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDisplay.cs
index 86690d75f8..f3d4a765ee 100644
--- a/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDisplay.cs
+++ b/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDisplay.cs
@@ -7,7 +7,7 @@ namespace Umbraco.Web.Models.ContentEditing
     /// <summary>
     /// Represents a content property that is displayed in the UI
     /// </summary>
-    public class ContentPropertyDisplay : ContentPropertyBase
+    public class ContentPropertyDisplay : ContentPropertyBasic
     {
         [DataMember(Name = "label", IsRequired = true)]
         [Required]
diff --git a/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDto.cs b/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDto.cs
index 01b9bc1152..e16cbb3d2f 100644
--- a/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDto.cs
+++ b/src/Umbraco.Web/Models/ContentEditing/ContentPropertyDto.cs
@@ -7,7 +7,7 @@ namespace Umbraco.Web.Models.ContentEditing
     /// <summary>
     /// Represents a content property from the database
     /// </summary>
-    internal class ContentPropertyDto : ContentPropertyBase
+    internal class ContentPropertyDto : ContentPropertyBasic
     {
         public IDataTypeDefinition DataType { get; set; }
         public string Label { get; set; }
diff --git a/src/Umbraco.Web/Models/Mapping/ContentModelMapper.cs b/src/Umbraco.Web/Models/Mapping/ContentModelMapper.cs
index 8dcff66e6c..02833804cd 100644
--- a/src/Umbraco.Web/Models/Mapping/ContentModelMapper.cs
+++ b/src/Umbraco.Web/Models/Mapping/ContentModelMapper.cs
@@ -106,6 +106,11 @@ namespace Umbraco.Web.Models.Mapping
                 });
         }
 
+        internal ContentItemBasic<ContentPropertyBasic> ToContentItemSimple(IContent content)
+        {
+            return CreateContent<ContentItemBasic<ContentPropertyBasic>, ContentPropertyBasic>(content, null, null);
+        } 
+
         /// <summary>
         /// Creates a new content item
         /// </summary>
@@ -121,7 +126,7 @@ namespace Umbraco.Web.Models.Mapping
             Action<TContentProperty, Property, PropertyEditor> propertyCreatedCallback = null, 
             bool createProperties = true)
             where TContent : ContentItemBasic<TContentProperty>, new()
-            where TContentProperty : ContentPropertyBase, new()
+            where TContentProperty : ContentPropertyBasic, new()
         {
             var result = new TContent
                 {
@@ -150,12 +155,25 @@ namespace Umbraco.Web.Models.Mapping
         private static TContentProperty CreateProperty<TContentProperty>(
             Property property, 
             Action<TContentProperty, Property, PropertyEditor> callback = null)
-            where TContentProperty : ContentPropertyBase, new()
+            where TContentProperty : ContentPropertyBasic, new()
         {
             var editor = PropertyEditorResolver.Current.GetById(property.PropertyType.DataTypeId);
             if (editor == null)
             {
-                throw new NullReferenceException("The property editor with id " + property.PropertyType.DataTypeId + " does not exist");
+                //TODO: Remove this check as we shouldn't support this at all!
+                var legacyEditor = DataTypesResolver.Current.GetById(property.PropertyType.DataTypeId);
+                if (legacyEditor == null)
+                {
+                    throw new NullReferenceException("The property editor with id " + property.PropertyType.DataTypeId + " does not exist");
+                }
+
+                var legacyResult = new TContentProperty
+                {
+                    Id = property.Id,
+                    Value = property.Value.ToString()
+                };
+                if (callback != null) callback(legacyResult, property, null);
+                return legacyResult;
             }
             var result = new TContentProperty
                 {
diff --git a/src/Umbraco.Web/Models/Mapping/MediaModelMapper.cs b/src/Umbraco.Web/Models/Mapping/MediaModelMapper.cs
new file mode 100644
index 0000000000..0e52d87551
--- /dev/null
+++ b/src/Umbraco.Web/Models/Mapping/MediaModelMapper.cs
@@ -0,0 +1,103 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Umbraco.Core;
+using Umbraco.Core.Models;
+using Umbraco.Core.PropertyEditors;
+using Umbraco.Web.Models.ContentEditing;
+
+namespace Umbraco.Web.Models.Mapping
+{
+    internal class MediaModelMapper
+    {
+        private readonly ApplicationContext _applicationContext;
+        private readonly ProfileModelMapper _profileMapper;
+
+        public MediaModelMapper(ApplicationContext applicationContext, ProfileModelMapper profileMapper)
+        {
+            _applicationContext = applicationContext;
+            _profileMapper = profileMapper;
+        }
+       
+        internal ContentItemBasic<ContentPropertyBasic> ToMediaItemSimple(IMedia media)
+        {
+            return CreateMedia<ContentItemBasic<ContentPropertyBasic>, ContentPropertyBasic>(media, null, null);
+        } 
+
+        /// <summary>
+        /// Creates a new content item
+        /// </summary>
+        /// <typeparam name="TContent"></typeparam>
+        /// <typeparam name="TContentProperty"></typeparam>
+        /// <param name="media"></param>
+        /// <param name="contentCreatedCallback"></param>
+        /// <param name="propertyCreatedCallback"></param>
+        /// <param name="createProperties"></param>
+        /// <returns></returns>
+        private TContent CreateMedia<TContent, TContentProperty>(IMedia media,
+            Action<TContent, IMedia> contentCreatedCallback = null,
+            Action<TContentProperty, Property, PropertyEditor> propertyCreatedCallback = null, 
+            bool createProperties = true)
+            where TContent : ContentItemBasic<TContentProperty>, new()
+            where TContentProperty : ContentPropertyBasic, new()
+        {
+            var result = new TContent
+                {
+                    Id = media.Id,
+                    Owner = _profileMapper.ToBasicUser(media.GetCreatorProfile()),
+                    Updator = null,
+                    ParentId = media.ParentId,
+                    UpdateDate = media.UpdateDate,
+                    CreateDate = media.CreateDate,
+                    ContentTypeAlias = media.ContentType.Alias
+                };
+            if (createProperties)
+                result.Properties = media.Properties.Select(p => CreateProperty(p, propertyCreatedCallback)).ToArray();
+            if (contentCreatedCallback != null)
+                contentCreatedCallback(result, media);
+            return result;
+        }
+
+        /// <summary>
+        /// Creates the property with the basic property values mapped
+        /// </summary>
+        /// <typeparam name="TContentProperty"></typeparam>
+        /// <param name="property"></param>
+        /// <param name="callback"></param>
+        /// <returns></returns>
+        private static TContentProperty CreateProperty<TContentProperty>(
+            Property property, 
+            Action<TContentProperty, Property, PropertyEditor> callback = null)
+            where TContentProperty : ContentPropertyBasic, new()
+        {
+            var editor = PropertyEditorResolver.Current.GetById(property.PropertyType.DataTypeId);
+            if (editor == null)
+            {
+                //TODO: Remove this check as we shouldn't support this at all!
+                var legacyEditor = DataTypesResolver.Current.GetById(property.PropertyType.DataTypeId);
+                if (legacyEditor == null)
+                {
+                    throw new NullReferenceException("The property editor with id " + property.PropertyType.DataTypeId + " does not exist");   
+                }
+
+                var legacyResult = new TContentProperty
+                {
+                    Id = property.Id,
+                    Value = property.Value.ToString()
+                };
+                if (callback != null) callback(legacyResult, property, null);
+                return legacyResult;
+
+            }
+            var result = new TContentProperty
+                {
+                    Id = property.Id,
+                    Value = editor.ValueEditor.SerializeValue(property.Value)
+                };
+            if (callback != null) callback(result, property, editor);
+            return result;
+        }
+    }
+}
diff --git a/src/Umbraco.Web/Mvc/AreaRegistrationExtensions.cs b/src/Umbraco.Web/Mvc/AreaRegistrationExtensions.cs
index 7a1b91adae..98a845a2f0 100644
--- a/src/Umbraco.Web/Mvc/AreaRegistrationExtensions.cs
+++ b/src/Umbraco.Web/Mvc/AreaRegistrationExtensions.cs
@@ -36,7 +36,7 @@ namespace Umbraco.Web.Mvc
                                                     bool isMvc = true)
         {
             Mandate.ParameterNotNullOrEmpty(controllerName, "controllerName");
-            Mandate.ParameterNotNullOrEmpty(controllerSuffixName, "controllerSuffixName");
+            Mandate.ParameterNotNull(controllerSuffixName, "controllerSuffixName");
             
             Mandate.ParameterNotNull(controllerType, "controllerType");
             Mandate.ParameterNotNull(routes, "routes");
diff --git a/src/Umbraco.Web/Mvc/PluginControllerArea.cs b/src/Umbraco.Web/Mvc/PluginControllerArea.cs
index 0e34e2cdef..d1e8cf4c1d 100644
--- a/src/Umbraco.Web/Mvc/PluginControllerArea.cs
+++ b/src/Umbraco.Web/Mvc/PluginControllerArea.cs
@@ -70,7 +70,7 @@ namespace Umbraco.Web.Mvc
 		{
 			foreach (var s in surfaceControllers)
 			{
-				var route = this.RouteControllerPlugin(s.ControllerName, s.ControllerType, routes, "Surface", "Index", UrlParameter.Optional, "surface");
+				var route = this.RouteControllerPlugin(s.ControllerName, s.ControllerType, routes, "", "Index", UrlParameter.Optional, "surface");
                 //set the route handler to our SurfaceRouteHandler
                 route.RouteHandler = new SurfaceRouteHandler();
 			}
@@ -85,7 +85,7 @@ namespace Umbraco.Web.Mvc
         {
             foreach (var s in apiControllers)
             {
-                this.RouteControllerPlugin(s.ControllerName, s.ControllerType, routes, "Api", "", UrlParameter.Optional, "api", isMvc: false);
+                this.RouteControllerPlugin(s.ControllerName, s.ControllerType, routes, "", "", UrlParameter.Optional, "api", isMvc: false);
             }
         }
 	}
diff --git a/src/Umbraco.Web/Umbraco.Web.csproj b/src/Umbraco.Web/Umbraco.Web.csproj
index 1c371d025c..e11b295be0 100644
--- a/src/Umbraco.Web/Umbraco.Web.csproj
+++ b/src/Umbraco.Web/Umbraco.Web.csproj
@@ -292,6 +292,7 @@
     <Compile Include="Cache\UserCacheRefresher.cs" />
     <Compile Include="Cache\UserTypeCacheRefresher.cs" />
     <Compile Include="Configuration\WebRouting.cs" />
+    <Compile Include="Editors\ContentController.cs" />
     <Compile Include="Editors\ContentTypeApiController.cs" />
     <Compile Include="HttpCookieExtensions.cs" />
     <Compile Include="Models\ContentEditing\ContentSaveAction.cs" />
@@ -301,6 +302,7 @@
     <Compile Include="Models\Mapping\ContentModelMapper.cs" />
     <Compile Include="FormDataCollectionExtensions.cs" />
     <Compile Include="Models\Mapping\ContentTypeModelMapper.cs" />
+    <Compile Include="Models\Mapping\MediaModelMapper.cs" />
     <Compile Include="Models\Mapping\ProfileModelMapper.cs" />
     <Compile Include="Mvc\BackOfficeArea.cs" />
     <Compile Include="Trees\LegacyTreeApiController.cs" />
@@ -329,7 +331,7 @@
     <Compile Include="Models\ContentEditing\ContentItemDto.cs" />
     <Compile Include="Models\ContentEditing\ContentItemFile.cs" />
     <Compile Include="Models\ContentEditing\ContentItemSave.cs" />
-    <Compile Include="Models\ContentEditing\ContentPropertyBase.cs" />
+    <Compile Include="Models\ContentEditing\ContentPropertyBasic.cs" />
     <Compile Include="Models\ContentEditing\ContentPropertyDisplay.cs" />
     <Compile Include="Models\ContentEditing\ContentPropertyDto.cs" />
     <Compile Include="Standalone\ServiceContextManager.cs" />
@@ -428,7 +430,7 @@
       <SubType>ASPXCodeBehind</SubType>
     </Compile>
     <Compile Include="UrlHelperExtensions.cs" />
-    <Compile Include="Editors\ContentEditorApiController.cs" />
+    <Compile Include="Editors\MediaController.cs" />
     <Compile Include="WebApi\Binders\ContentItemBinder.cs" />
     <Compile Include="WebApi\Filters\ContentItemValidationFilterAttribute.cs" />
     <Compile Include="WebApi\Filters\FileUploadCleanupFilterAttribute.cs" />
diff --git a/src/Umbraco.Web/WebBootManager.cs b/src/Umbraco.Web/WebBootManager.cs
index d88abe2f27..6a1e778f67 100644
--- a/src/Umbraco.Web/WebBootManager.cs
+++ b/src/Umbraco.Web/WebBootManager.cs
@@ -178,7 +178,7 @@ namespace Umbraco.Web
                 SurfaceControllerResolver.Current.RegisteredSurfaceControllers.Concat(
                     UmbracoApiControllerResolver.Current.RegisteredUmbracoApiControllers).ToArray();
 
-            //local controllers do not contain the attribute 			
+            //local controllers do not contain the attribute
             var localControllers = pluginControllers.Where(x => PluginController.GetMetadata(x).AreaName.IsNullOrWhiteSpace());
             foreach (var s in localControllers)
             {
@@ -211,8 +211,7 @@ namespace Umbraco.Web
             var route = RouteTable.Routes.MapHttpRoute(
                 string.Format("umbraco-{0}-{1}", "api", meta.ControllerName),
                 umbracoPath + "/Api/" + meta.ControllerName + "/{action}/{id}",//url to match
-                new { controller = meta.ControllerName, id = UrlParameter.Optional },
-                new { controller = @"(\w+)Api" }); //Must be suffixed with "Api" (i.e. MyApiController)
+                new { controller = meta.ControllerName, id = UrlParameter.Optional });
             //web api routes don't set the data tokens object
             if (route.DataTokens == null)
             {                
@@ -229,9 +228,7 @@ namespace Umbraco.Web
             var route = RouteTable.Routes.MapRoute(
                 string.Format("umbraco-{0}-{1}", "surface", meta.ControllerName),
                 umbracoPath + "/Surface/" + meta.ControllerName + "/{action}/{id}",//url to match
-                new { controller = meta.ControllerName, action = "Index", id = UrlParameter.Optional },
-                //NOTE: There SHOULD be a constraint here to only match controllers with a "SurfaceController" suffix but we forgot to include it in the 
-                // 4.10 release so we can't include it now :(
+                new { controller = meta.ControllerName, action = "Index", id = UrlParameter.Optional },                
                 new[] { meta.ControllerNamespace }); //look in this namespace to create the controller
             route.DataTokens.Add("umbraco", "surface"); //ensure the umbraco token is set                
             route.DataTokens.Add("UseNamespaceFallback", false); //Don't look anywhere else except this namespace!