housecleaning

Author: skttl

src/Our.Umbraco.DocTypeGridEditor/Models/DetachedPublishedProperty.cs

@@ -25,7 +25,6 @@ public DetachedPublishedProperty(PublishedPropertyType propertyType, object valu
 
             _rawValue = value;
 
-            // TODO: FIXME!
             _sourceValue = new Lazy<object>(() => _propertyType.ConvertSourceToInter(null, _rawValue, _isPreview));
             _objectValue = new Lazy<object>(() => _propertyType.ConvertInterToObject(null, PropertyEditors.PropertyCacheLevel.None, _sourceValue.Value, _isPreview));
             _xpathValue = new Lazy<object>(() => _propertyType.ConvertInterToXPath(null, PropertyEditors.PropertyCacheLevel.None, _sourceValue.Value, _isPreview));

src/Our.Umbraco.DocTypeGridEditor/Models/UnpublishedContent.cs

@@ -138,7 +138,6 @@ private Dictionary<string, IPublishedProperty> MapProperties(ServiceContext serv
                     Current.PropertyEditors.TryGet(propertyType.DataType.EditorAlias, out var editor);
                     if (editor != null)
                     {
-                        // TODO: FIXME!
                         value = editor.GetValueEditor().ConvertDbToString(property.PropertyType, value, services.DataTypeService);
                     }
                 }

src/Our.Umbraco.DocTypeGridEditor/Web/Controllers/DocTypeGridEditorApiController.cs

@@ -126,11 +126,6 @@ public object GetDataTypePreValues(string dtdId)
                 return null;
 
             // Convert to editor config
-            //var preValue = Services.DataTypeService.GetPreValuesCollectionByDataTypeId(dtd.Id);
-            //var propEditor = PropertyEditorResolver.Current.GetByAlias(dtd.PropertyEditorAlias);
-            //return propEditor.PreValueEditor.ConvertDbToEditor(propEditor.DefaultPreValues, preValue);
-            
-            // TODO: FIXME! This is not the data we were expecting, not sure what we were actually expecting and why :)
             var dataType = Current.Services.DataTypeService.GetDataType(dtd.Id);
             var propEditor = dataType.Editor;
             var content = propEditor.GetValueEditor().ConvertDbToString(new PropertyType(dataType), dataType.Configuration, Current.Services.DataTypeService);
@@ -154,23 +149,6 @@ public HttpResponseMessage GetPreviewMarkup([FromBody] PreviewData data, [FromUr
                 }
             }
 
-            // NOTE: The previous previewer had a content node associated with the request,
-            // meaning that an implementation may have used this to traverse the content-tree.
-            // In order to maintain backward-compatibility, we must ensure the PublishedContentRequest context.
-
-            // TODO: Do we still need this hack?
-            //if (UmbracoContext.PublishedContentRequest == null)
-            //{
-            //    UmbracoContext.PublishedContentRequest = new PublishedContentRequest(
-            //        Request.RequestUri,
-            //        UmbracoContext.RoutingContext,
-            //        UmbracoConfig.For.UmbracoSettings().WebRouting,
-            //        null)
-            //    {
-            //        PublishedContent = page
-            //    };
-            //}
-
             // Set the culture for the preview
             if (page != null)
             {