From 8d6a540dfa3d24989ed7870dc3ead7fec44dfb8e Mon Sep 17 00:00:00 2001
From: Sergey Krivchenko <45521776+sergey-krivchenko@users.noreply.github.com>
Date: Sat, 2 Nov 2019 00:01:55 +0100
Subject: [PATCH] Updated Template References section for static classes

a fix for the latest comment in #26 .
Using a static class does not indicate that it is used only for holding constants. Usually a class name qualifier is used for this (e.g. `static class Constants`). I think the sentence should just say that it is a recommended way to define constants, following the conventions for defining constants in the [Microsoft C# Programming Guide](https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/how-to-define-constants). An explanation for this can be that static classes can only hold static members, including constants, and it cannot be mistakenly instantiated or derived from.
---
 helix/principles/templates/references.rst | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/helix/principles/templates/references.rst b/helix/principles/templates/references.rst
index d0d821d..1306c9f 100644
--- a/helix/principles/templates/references.rst
+++ b/helix/principles/templates/references.rst
@@ -16,9 +16,9 @@ Define constants for a module’s templates in a single static class named
 ``Templates``. This static class is located in the root namespace for the module.
 This makes it easy to explicitly reference a template in the business
 logic or views of the module and makes it easier to discover references to a template
-or field. The conventions define this as a static class to clearly signal the
-``Templates`` type's unique function as a constants holder only, following the
-`conventions for Constants in the Microsoft C# Programming Guide <https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/how-to-define-constants>`__.
+or field. The conventions define this as a static class to follow the
+`conventions for Constants in the Microsoft C# Programming Guide <https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/how-to-define-constants>`__, as it indicates that the type
+can only contain static members, including constants, and cannot be mistakenly instantiated or derived from.
 
 The ``Templates`` static class should have a nested static class for each template
 which each contains an ID member and a nested static class, Fields, which
@@ -139,4 +139,4 @@ rather an *is* operator.
             .Select(x => new {
                 Uri = x.UniqueId,
                 Database = Factory.GetDatabase(x.UniqueId.DatabaseName)
-            }).ToList();
\ No newline at end of file
+            }).ToList();