#7469 - Fixed Ambiguous extension method issue by renaming our extens…

…ion method from Value to GetValue
umbraco · Jan 17, 2020 · baef282 · baef282
1 parent 6637bf5
commit baef282
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Umbraco.ModelsBuilder.Embedded/PublishedElementExtensions.cs b/src/Umbraco.ModelsBuilder.Embedded/PublishedElementExtensions.cs
@@ -17,13 +17,14 @@ public static class PublishedElementExtensions
         /// <summary>
         /// Gets the value of a property.
         /// </summary>
-        public static TValue GetValue<TModel, TValue>(this TModel model, Expression<Func<TModel, TValue>> property, string culture = null, string segment = null, Fallback fallback = default, TValue defaultValue = default)
+        public static TValue ValueByExpression<TModel, TValue>(this TModel model, Expression<Func<TModel, TValue>> property, string culture = null, string segment = null, Fallback fallback = default, TValue defaultValue = default)
             where TModel : IPublishedElement
         {
             var alias = GetAlias(model, property);
             return model.Value<TValue>(alias, culture, segment, fallback, defaultValue);
         }
 
+        //This cannot be public due to ambiguous issue with external ModelsBuilder if we do not rename.
         private static string GetAlias<TModel, TValue>(TModel model, Expression<Func<TModel, TValue>> property)
         {
             if (property.NodeType != ExpressionType.Lambda)