From d9dfea8ef55de0d07d0159a4752a34b5cda557e8 Mon Sep 17 00:00:00 2001
From: Ronald Barendse <ronald@panoramastudios.nl>
Date: Fri, 17 Jan 2020 22:43:11 +0100
Subject: [PATCH] Add remarks and better summary on SetVariesBy extension
 methods

---
 src/Umbraco.Core/ContentVariationExtensions.cs | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/Umbraco.Core/ContentVariationExtensions.cs b/src/Umbraco.Core/ContentVariationExtensions.cs
index 4a918ca70d..7cec5df27f 100644
--- a/src/Umbraco.Core/ContentVariationExtensions.cs
+++ b/src/Umbraco.Core/ContentVariationExtensions.cs
@@ -305,6 +305,9 @@ namespace Umbraco.Core
         /// <param name="contentType">The content type.</param>
         /// <param name="variation">The variation to set or remove.</param>
         /// <param name="value">If set to <c>true</c> sets the variation; otherwise, removes the variation.</param>
+        /// <remarks>
+        /// This method does not support setting the variation to nothing.
+        /// </remarks>
         public static void SetVariesBy(this IContentTypeBase contentType, ContentVariation variation, bool value = true) => contentType.Variations = contentType.Variations.SetVariesBy(variation, value);
 
         /// <summary>
@@ -313,17 +316,23 @@ namespace Umbraco.Core
         /// <param name="propertyType">The property type.</param>
         /// <param name="variation">The variation to set or remove.</param>
         /// <param name="value">If set to <c>true</c> sets the variation; otherwise, removes the variation.</param>
+        /// <remarks>
+        /// This method does not support setting the variation to nothing.
+        /// </remarks>
         public static void SetVariesBy(this PropertyType propertyType, ContentVariation variation, bool value = true) => propertyType.Variations = propertyType.Variations.SetVariesBy(variation, value);
 
         /// <summary>
-        /// Sets or removes the variation depending on the specified value.
+        /// Returns the variations with the variation set or removed depending on the specified value.
         /// </summary>
         /// <param name="variations">The existing variations.</param>
         /// <param name="variation">The variation to set or remove.</param>
         /// <param name="value">If set to <c>true</c> sets the variation; otherwise, removes the variation.</param>
         /// <returns>
-        /// The variations with the specified variation set or removed.
+        /// The variations with the variation set or removed.
         /// </returns>
+        /// <remarks>
+        /// This method does not support setting the variation to nothing.
+        /// </remarks>
         public static ContentVariation SetVariesBy(this ContentVariation variations, ContentVariation variation, bool value = true)
         {
             return value