From d4cfa57b2a38b155c373a6a766d92fc68560f2f8 Mon Sep 17 00:00:00 2001
From: Shannon <sdeminick@gmail.com>
Date: Tue, 11 Jun 2013 14:30:37 +0200
Subject: [PATCH] Got media creation working and fixed updating node names.

---
 .../umbraco/googlemaps/editor.html            |  2 +-
 .../src/common/resources/content.resource.js  |  4 +-
 .../common/resources/contenttype.resource.js  |  2 +-
 .../src/common/resources/media.resource.js    | 32 +++++++++++++
 .../common/resources/mediatype.resource.js    | 34 +++++++++++++
 .../views/content/contentedit.controller.js   |  2 +-
 .../src/views/media/mediacreate.controller.js |  4 +-
 .../src/views/media/mediaedit.controller.js   |  2 +-
 .../Views/FileUploadEditor.html               |  4 +-
 .../Editors/BackOfficeController.cs           |  3 +-
 src/Umbraco.Web/Editors/ContentController.cs  |  2 +-
 .../Editors/ContentTypeApiController.cs       | 48 ++++++++++++++++++-
 src/Umbraco.Web/Editors/MediaController.cs    | 24 +++++++++-
 .../Models/Mapping/BaseContentModelMapper.cs  |  2 +-
 .../Models/Mapping/ContentTypeModelMapper.cs  | 22 +++++++++
 .../WebApi/Binders/ContentItemBaseBinder.cs   | 28 +++++++----
 .../WebApi/Binders/ContentItemBinder.cs       |  2 +-
 .../WebApi/Binders/MediaItemBinder.cs         |  2 +-
 .../ContentItemValidationFilterAttribute.cs   |  2 +-
 19 files changed, 195 insertions(+), 26 deletions(-)
 create mode 100644 src/Umbraco.Web.UI.Client/src/common/resources/mediatype.resource.js

diff --git a/src/Umbraco.Web.UI.Client/build/belle/views/propertyeditors/umbraco/googlemaps/editor.html b/src/Umbraco.Web.UI.Client/build/belle/views/propertyeditors/umbraco/googlemaps/editor.html
index 0f576c3124..9effacac9e 100644
--- a/src/Umbraco.Web.UI.Client/build/belle/views/propertyeditors/umbraco/googlemaps/editor.html
+++ b/src/Umbraco.Web.UI.Client/build/belle/views/propertyeditors/umbraco/googlemaps/editor.html
@@ -1,3 +1,3 @@
 <div ng-controller="Umbraco.Editors.GoogleMapsController" class="span7">
-    <div class="" style="height: 200px;" id="{{model.alias}}_map"></div>
+    <div class="" style="height: 400px;" id="{{model.alias}}_map"></div>
 </div>
\ No newline at end of file
diff --git a/src/Umbraco.Web.UI.Client/src/common/resources/content.resource.js b/src/Umbraco.Web.UI.Client/src/common/resources/content.resource.js
index b358c6d490..e7de992f94 100644
--- a/src/Umbraco.Web.UI.Client/src/common/resources/content.resource.js
+++ b/src/Umbraco.Web.UI.Client/src/common/resources/content.resource.js
@@ -11,7 +11,7 @@ function contentResource($q, $http, umbDataFormatter, umbRequestHelper) {
     }
     /** internal method to get the api url */
     function getEmptyContentUrl(contentTypeAlias, parentId) {
-        return Umbraco.Sys.ServerVariables.contentApiBaseUrl + "GetEmptyContent?contentTypeAlias=" + contentTypeAlias + "&parentId=" + parentId;
+        return Umbraco.Sys.ServerVariables.contentApiBaseUrl + "GetEmpty?contentTypeAlias=" + contentTypeAlias + "&parentId=" + parentId;
     }
     /** internal method to get the api url for publishing */
     function getSaveUrl() {
@@ -49,7 +49,7 @@ function contentResource($q, $http, umbDataFormatter, umbRequestHelper) {
 
         /** returns an empty content object which can be persistent on the content service
             requires the parent id and the alias of the content type to base the scaffold on */
-        getContentScaffold: function (parentId, alias) {
+        getScaffold: function (parentId, alias) {
 
             var deferred = $q.defer();
 
diff --git a/src/Umbraco.Web.UI.Client/src/common/resources/contenttype.resource.js b/src/Umbraco.Web.UI.Client/src/common/resources/contenttype.resource.js
index 177602be76..f69e42dc2c 100644
--- a/src/Umbraco.Web.UI.Client/src/common/resources/contenttype.resource.js
+++ b/src/Umbraco.Web.UI.Client/src/common/resources/contenttype.resource.js
@@ -7,7 +7,7 @@ function contentTypeResource($q, $http) {
 
     /** internal method to get the api url */
     function getChildContentTypesUrl(contentId) {
-        return Umbraco.Sys.ServerVariables.contentTypeApiBaseUrl + "GetAllowedChildrenForContent?contentId=" + contentId;
+        return Umbraco.Sys.ServerVariables.contentTypeApiBaseUrl + "GetAllowedChildren?contentId=" + contentId;
     }
 
     return {
diff --git a/src/Umbraco.Web.UI.Client/src/common/resources/media.resource.js b/src/Umbraco.Web.UI.Client/src/common/resources/media.resource.js
index 628fc6ee91..51373415a8 100644
--- a/src/Umbraco.Web.UI.Client/src/common/resources/media.resource.js
+++ b/src/Umbraco.Web.UI.Client/src/common/resources/media.resource.js
@@ -9,6 +9,12 @@ function mediaResource($q, $http, umbDataFormatter, umbRequestHelper) {
     function getMediaUrl(contentId) {
         return Umbraco.Sys.ServerVariables.mediaApiBaseUrl + "GetById?id=" + contentId;
     }
+    
+    /** internal method to get the api url */
+    function getEmptyMediaUrl(contentTypeAlias, parentId) {
+        return Umbraco.Sys.ServerVariables.mediaApiBaseUrl + "GetEmpty?contentTypeAlias=" + contentTypeAlias + "&parentId=" + parentId;
+    }
+
     /** internal method to get the api url */
     function getRootMediaUrl() {
         return Umbraco.Sys.ServerVariables.mediaApiBaseUrl + "GetRootMedia";
@@ -54,6 +60,32 @@ function mediaResource($q, $http, umbDataFormatter, umbRequestHelper) {
             return deferred.promise;
         },
 
+        /** returns an empty content object which can be persistent on the content service
+            requires the parent id and the alias of the content type to base the scaffold on */
+        getScaffold: function (parentId, alias) {
+
+            var deferred = $q.defer();
+
+            //go and get the data
+            $http.get(getEmptyMediaUrl(alias, parentId)).
+                success(function (data, status, headers, config) {
+                    //set the first tab to active
+                    _.each(data.tabs, function (item) {
+                        item.active = false;
+                    });
+                    if (data.tabs.length > 0) {
+                        data.tabs[0].active = true;
+                    }
+
+                    deferred.resolve(data);
+                }).
+                error(function (data, status, headers, config) {
+                    deferred.reject('Failed to retreive data for empty content item type ' + alias);
+                });
+
+            return deferred.promise;
+        },
+
         rootMedia: function () {
 
             var deferred = $q.defer();
diff --git a/src/Umbraco.Web.UI.Client/src/common/resources/mediatype.resource.js b/src/Umbraco.Web.UI.Client/src/common/resources/mediatype.resource.js
new file mode 100644
index 0000000000..c5a00c6760
--- /dev/null
+++ b/src/Umbraco.Web.UI.Client/src/common/resources/mediatype.resource.js
@@ -0,0 +1,34 @@
+/**
+    * @ngdoc factory 
+    * @name umbraco.resources.mediaTypeResource
+    * @description Loads in data for content types
+    **/
+function mediaTypeResource($q, $http) {
+
+    /** internal method to get the api url */
+    function getChildContentTypesUrl(contentId) {
+        return Umbraco.Sys.ServerVariables.mediaTypeApiBaseUrl + "GetAllowedChildren?contentId=" + contentId;
+    }
+
+    return {
+
+        //return all types allowed under given document
+        getAllowedTypes: function (contentId) {
+
+            var deferred = $q.defer();
+
+            //go and get the tree data
+            $http.get(getChildContentTypesUrl(contentId)).
+                success(function (data, status, headers, config) {
+                    deferred.resolve(data);
+                }).
+                error(function (data, status, headers, config) {
+                    deferred.reject('Failed to retreive data for media id ' + contentId);
+                });
+
+            return deferred.promise;
+        }
+
+    };
+}
+angular.module('umbraco.resources').factory('mediaTypeResource', mediaTypeResource);
\ No newline at end of file
diff --git a/src/Umbraco.Web.UI.Client/src/views/content/contentedit.controller.js b/src/Umbraco.Web.UI.Client/src/views/content/contentedit.controller.js
index a8bc16e44d..c2a3570203 100644
--- a/src/Umbraco.Web.UI.Client/src/views/content/contentedit.controller.js
+++ b/src/Umbraco.Web.UI.Client/src/views/content/contentedit.controller.js
@@ -4,7 +4,7 @@ angular.module("umbraco")
 
 	    if ($routeParams.create) {
 
-	        contentResource.getContentScaffold($routeParams.id, $routeParams.doctype)
+	        contentResource.getScaffold($routeParams.id, $routeParams.doctype)
 	            .then(function (data) {
 	                $scope.content = data;
 	            });
diff --git a/src/Umbraco.Web.UI.Client/src/views/media/mediacreate.controller.js b/src/Umbraco.Web.UI.Client/src/views/media/mediacreate.controller.js
index d474d1f620..318935368f 100644
--- a/src/Umbraco.Web.UI.Client/src/views/media/mediacreate.controller.js
+++ b/src/Umbraco.Web.UI.Client/src/views/media/mediacreate.controller.js
@@ -1,5 +1,5 @@
-function mediaCreateController ($scope, $routeParams,contentTypeResource) {
-    $scope.allowedTypes  = contentTypeResource.getAllowedTypes($scope.currentNode.id);
+function mediaCreateController ($scope, $routeParams,mediaTypeResource) {
+    $scope.allowedTypes = mediaTypeResource.getAllowedTypes($scope.currentNode.id);
 }
 
 angular.module('umbraco')
diff --git a/src/Umbraco.Web.UI.Client/src/views/media/mediaedit.controller.js b/src/Umbraco.Web.UI.Client/src/views/media/mediaedit.controller.js
index 1078f58eee..476ecd3adf 100644
--- a/src/Umbraco.Web.UI.Client/src/views/media/mediaedit.controller.js
+++ b/src/Umbraco.Web.UI.Client/src/views/media/mediaedit.controller.js
@@ -2,7 +2,7 @@ function mediaEditController($scope, $routeParams, mediaResource, notificationsS
 
     if ($routeParams.create) {
 
-        mediaResource.getContentScaffold($routeParams.id, $routeParams.doctype)
+        mediaResource.getScaffold($routeParams.id, $routeParams.doctype)
             .then(function (data) {
                 $scope.content = data;
             });
diff --git a/src/Umbraco.Web.UI/App_Plugins/MyPackage/PropertyEditors/Views/FileUploadEditor.html b/src/Umbraco.Web.UI/App_Plugins/MyPackage/PropertyEditors/Views/FileUploadEditor.html
index 419454f433..586304e3ce 100644
--- a/src/Umbraco.Web.UI/App_Plugins/MyPackage/PropertyEditors/Views/FileUploadEditor.html
+++ b/src/Umbraco.Web.UI/App_Plugins/MyPackage/PropertyEditors/Views/FileUploadEditor.html
@@ -11,8 +11,8 @@
 
     <div ng-show="persistedFiles.length > 0">
         <p>Current files</p>
-        <ul>
-            <li ng-repeat="file in persistedFiles">
+        <ul class="thumbnails">
+            <li class="thumbnail" ng-repeat="file in persistedFiles">
                 <div ng-switch on="file.isImage" >
                     <img ng-src="{{getThumbnail(file)}}" ng-switch-when="true" alt="{{file.file}}"/>
                     <span ng-switch-default>{{file.file}}</span>
diff --git a/src/Umbraco.Web/Editors/BackOfficeController.cs b/src/Umbraco.Web/Editors/BackOfficeController.cs
index c23a47260f..f9a6ba49b1 100644
--- a/src/Umbraco.Web/Editors/BackOfficeController.cs
+++ b/src/Umbraco.Web/Editors/BackOfficeController.cs
@@ -50,7 +50,8 @@ namespace Umbraco.Web.Editors
                     {"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")}
+                    {"contentTypeApiBaseUrl", Url.GetUmbracoApiService<ContentTypeApiController>("GetAllowedChildren").TrimEnd("GetAllowedChildren")},
+                    {"mediaTypeApiBaseUrl", Url.GetUmbracoApiService<MediaTypeApiController>("GetAllowedChildren").TrimEnd("GetAllowedChildren")}
                 };
 
             return JavaScript(ServerVariablesParser.Parse(d));
diff --git a/src/Umbraco.Web/Editors/ContentController.cs b/src/Umbraco.Web/Editors/ContentController.cs
index 757a84722b..5eebc7cc75 100644
--- a/src/Umbraco.Web/Editors/ContentController.cs
+++ b/src/Umbraco.Web/Editors/ContentController.cs
@@ -79,7 +79,7 @@ namespace Umbraco.Web.Editors
         /// <param name="contentTypeAlias"></param>
         /// <param name="parentId"></param>
         /// <returns></returns>
-        public ContentItemDisplay GetEmptyContent(string contentTypeAlias, int parentId)
+        public ContentItemDisplay GetEmpty(string contentTypeAlias, int parentId)
         {
             var contentType = Services.ContentTypeService.GetContentType(contentTypeAlias);
             if (contentType == null)
diff --git a/src/Umbraco.Web/Editors/ContentTypeApiController.cs b/src/Umbraco.Web/Editors/ContentTypeApiController.cs
index 6be37be81f..34995019ef 100644
--- a/src/Umbraco.Web/Editors/ContentTypeApiController.cs
+++ b/src/Umbraco.Web/Editors/ContentTypeApiController.cs
@@ -37,7 +37,7 @@ namespace Umbraco.Web.Editors
         /// Returns the allowed child content type objects for the content item id passed in
         /// </summary>
         /// <param name="contentId"></param>
-        public IEnumerable<object> GetAllowedChildrenForContent(int contentId)
+        public IEnumerable<object> GetAllowedChildren(int contentId)
         {
             var contentItem = Services.ContentService.GetById(contentId);
             if (contentItem == null)
@@ -52,4 +52,50 @@ namespace Umbraco.Web.Editors
             
         }
     }
+
+    /// <summary>
+    /// An API controller used for dealing with content types
+    /// </summary>
+    public class MediaTypeApiController : UmbracoAuthorizedApiController
+    {
+        private readonly MediaTypeModelMapper _mediaTypeModelMapper;
+
+        /// <summary>
+        /// Constructor
+        /// </summary>
+        public MediaTypeApiController()
+            : this(UmbracoContext.Current, new MediaTypeModelMapper(UmbracoContext.Current.Application))
+        {
+        }
+
+        /// <summary>
+        /// Constructor
+        /// </summary>
+        /// <param name="umbracoContext"></param>
+        /// <param name="mediaModelMapper"></param>
+        internal MediaTypeApiController(UmbracoContext umbracoContext, MediaTypeModelMapper mediaModelMapper)
+            : base(umbracoContext)
+        {
+            _mediaTypeModelMapper = mediaModelMapper;
+        }
+
+        /// <summary>
+        /// Returns the allowed child content type objects for the content item id passed in
+        /// </summary>
+        /// <param name="contentId"></param>
+        public IEnumerable<object> GetAllowedChildren(int contentId)
+        {
+            var contentItem = Services.MediaService.GetById(contentId);
+            if (contentItem == null)
+            {
+                throw new HttpResponseException(HttpStatusCode.NotFound);
+            }
+
+
+            return contentItem.ContentType.AllowedContentTypes
+                .Select(x => Services.ContentTypeService.GetMediaType(x.Id.Value))
+                .Select(x => _mediaTypeModelMapper.ToMediaTypeBasic(x));
+
+        }
+    }
 }
\ No newline at end of file
diff --git a/src/Umbraco.Web/Editors/MediaController.cs b/src/Umbraco.Web/Editors/MediaController.cs
index edd08753c8..512045f90c 100644
--- a/src/Umbraco.Web/Editors/MediaController.cs
+++ b/src/Umbraco.Web/Editors/MediaController.cs
@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using System.Net;
 using System.Net.Http;
 using System.Web.Http;
@@ -52,6 +53,24 @@ namespace Umbraco.Web.Editors
             _mediaModelMapper = mediaModelMapper;
         }
 
+        /// <summary>
+        /// Gets an empty content item for the 
+        /// </summary>
+        /// <param name="contentTypeAlias"></param>
+        /// <param name="parentId"></param>
+        /// <returns></returns>
+        public MediaItemDisplay GetEmpty(string contentTypeAlias, int parentId)
+        {
+            var contentType = Services.ContentTypeService.GetMediaType(contentTypeAlias);
+            if (contentType == null)
+            {
+                throw new HttpResponseException(HttpStatusCode.NotFound);
+            }
+
+            var emptyContent = new Umbraco.Core.Models.Media("Empty", parentId, contentType);
+            return _mediaModelMapper.ToMediaItemDisplay(emptyContent);
+        }
+
         /// <summary>
         /// Gets the content json for the content id
         /// </summary>
@@ -107,6 +126,9 @@ namespace Umbraco.Web.Editors
 
             //Now, we just need to save the data
 
+            contentItem.PersistedContent.Name = contentItem.Name;
+            //TODO: We'll need to save the new template, publishat, etc... values here
+
             //Save the property values (for properties that have a valid editor ... not legacy)
             foreach (var p in contentItem.ContentDto.Properties.Where(x => x.PropertyEditor != null))
             {
diff --git a/src/Umbraco.Web/Models/Mapping/BaseContentModelMapper.cs b/src/Umbraco.Web/Models/Mapping/BaseContentModelMapper.cs
index ca23f95664..14512eb02e 100644
--- a/src/Umbraco.Web/Models/Mapping/BaseContentModelMapper.cs
+++ b/src/Umbraco.Web/Models/Mapping/BaseContentModelMapper.cs
@@ -149,7 +149,7 @@ namespace Umbraco.Web.Models.Mapping
                 var legacyResult = new TContentProperty
                     {
                         Id = property.Id,
-                        Value = property.Value.ToString(),
+                        Value = property.Value == null ? "" : property.Value.ToString(),
                         Alias = property.Alias
                     };
                 if (callback != null) callback(legacyResult, property, null);
diff --git a/src/Umbraco.Web/Models/Mapping/ContentTypeModelMapper.cs b/src/Umbraco.Web/Models/Mapping/ContentTypeModelMapper.cs
index 168217b473..30fcb50e32 100644
--- a/src/Umbraco.Web/Models/Mapping/ContentTypeModelMapper.cs
+++ b/src/Umbraco.Web/Models/Mapping/ContentTypeModelMapper.cs
@@ -25,4 +25,26 @@ namespace Umbraco.Web.Models.Mapping
                 };
         }
     }
+
+    internal class MediaTypeModelMapper
+    {
+        private readonly ApplicationContext _applicationContext;
+
+        public MediaTypeModelMapper(ApplicationContext applicationContext)
+        {
+            _applicationContext = applicationContext;
+        }
+
+        public ContentTypeBasic ToMediaTypeBasic(IMediaType contentType)
+        {
+            return new ContentTypeBasic
+            {
+                Alias = contentType.Alias,
+                Id = contentType.Id,
+                Description = contentType.Description,
+                Icon = contentType.Icon,
+                Name = contentType.Name
+            };
+        }
+    }
 }
\ No newline at end of file
diff --git a/src/Umbraco.Web/WebApi/Binders/ContentItemBaseBinder.cs b/src/Umbraco.Web/WebApi/Binders/ContentItemBaseBinder.cs
index f003536e72..8ea7e3df66 100644
--- a/src/Umbraco.Web/WebApi/Binders/ContentItemBaseBinder.cs
+++ b/src/Umbraco.Web/WebApi/Binders/ContentItemBaseBinder.cs
@@ -135,18 +135,30 @@ namespace Umbraco.Web.WebApi.Binders
                 model.PersistedContent = CreateNew(model);
             }
 
-            model.ContentDto = Map(model);
-            //we will now assign all of the values in the 'save' model to the DTO object
-            foreach (var p in model.Properties)
-            {
-                model.ContentDto.Properties.Single(x => x.Id == p.Id).Value = p.Value;
-            }
-
+            //create the dto from the persisted model
+            model.ContentDto = MapFromPersisted(model);
+            
+            //now map all of the saved values to the dto
+            MapPropertyValuesFromSaved(model, model.ContentDto);
+            
             return model;
         }
 
+        /// <summary>
+        /// we will now assign all of the values in the 'save' model to the DTO object
+        /// </summary>
+        /// <param name="saveModel"></param>
+        /// <param name="dto"></param>
+        private static void MapPropertyValuesFromSaved(ContentItemSave<TPersisted> saveModel, ContentItemDto<TPersisted> dto)
+        {          
+            foreach (var p in saveModel.Properties)
+            {
+                dto.Properties.Single(x => x.Alias == p.Alias).Value = p.Value;
+            }
+        }
+
         protected abstract TPersisted GetExisting(ContentItemSave<TPersisted> model);
         protected abstract TPersisted CreateNew(ContentItemSave<TPersisted> model);
-        protected abstract ContentItemDto<TPersisted> Map(ContentItemSave<TPersisted> model);
+        protected abstract ContentItemDto<TPersisted> MapFromPersisted(ContentItemSave<TPersisted> model);
     }
 }
\ No newline at end of file
diff --git a/src/Umbraco.Web/WebApi/Binders/ContentItemBinder.cs b/src/Umbraco.Web/WebApi/Binders/ContentItemBinder.cs
index 66ce8ec263..9856c42304 100644
--- a/src/Umbraco.Web/WebApi/Binders/ContentItemBinder.cs
+++ b/src/Umbraco.Web/WebApi/Binders/ContentItemBinder.cs
@@ -39,7 +39,7 @@ namespace Umbraco.Web.WebApi.Binders
             return new Content(model.Name, model.ParentId, contentType);     
         }
 
-        protected override ContentItemDto<IContent> Map(ContentItemSave<IContent> model)
+        protected override ContentItemDto<IContent> MapFromPersisted(ContentItemSave<IContent> model)
         {
             return _contentModelMapper.ToContentItemDto(model.PersistedContent);
         }
diff --git a/src/Umbraco.Web/WebApi/Binders/MediaItemBinder.cs b/src/Umbraco.Web/WebApi/Binders/MediaItemBinder.cs
index a25ef39206..2dfd0ae131 100644
--- a/src/Umbraco.Web/WebApi/Binders/MediaItemBinder.cs
+++ b/src/Umbraco.Web/WebApi/Binders/MediaItemBinder.cs
@@ -39,7 +39,7 @@ namespace Umbraco.Web.WebApi.Binders
             return new Core.Models.Media(model.Name, model.ParentId, contentType);
         }
 
-        protected override ContentItemDto<IMedia> Map(ContentItemSave<IMedia> model)
+        protected override ContentItemDto<IMedia> MapFromPersisted(ContentItemSave<IMedia> model)
         {
             return _mediaModelMapper.ToMediaItemDto(model.PersistedContent);
         }
diff --git a/src/Umbraco.Web/WebApi/Filters/ContentItemValidationFilterAttribute.cs b/src/Umbraco.Web/WebApi/Filters/ContentItemValidationFilterAttribute.cs
index 3069df8139..9f05e58cc0 100644
--- a/src/Umbraco.Web/WebApi/Filters/ContentItemValidationFilterAttribute.cs
+++ b/src/Umbraco.Web/WebApi/Filters/ContentItemValidationFilterAttribute.cs
@@ -104,7 +104,7 @@ namespace Umbraco.Web.WebApi.Filters
                 }
 
                 //get the posted value for this property
-                var postedValue = postedItem.Properties.Single(x => x.Id == p.Id).Value;
+                var postedValue = postedItem.Properties.Single(x => x.Alias == p.Alias).Value;
 
                 //get the pre-values for this property
                 var preValues = _applicationContext.Services.DataTypeService.GetPreValueAsString(p.DataType.Id);