Small changes

renames and summaries, also fixed wrong logic of property and its use

Author: alex-kulakov

Orm/Xtensive.Orm.PostgreSql/Orm.Providers.PostgreSql/DomainHandler.cs

@@ -21,8 +21,8 @@ public class DomainHandler : Providers.DomainHandler
     /// <summary>
     /// <see langword="true"/> if storage can trim insignificant zeros in numeric values
     /// </summary>
-    protected bool HasNativeTrimOfInsignificantDecimalPoints =>
-      Handlers.ProviderInfo.StorageVersion.Major < 13;
+    protected bool HasNativeTrimOfInsignificantZerosInDecimals =>
+      Handlers.ProviderInfo.StorageVersion.Major >= 13;
 
     /// <inheritdoc/>
     protected override ICompiler CreateCompiler(CompilerConfiguration configuration) =>
@@ -31,7 +31,7 @@ protected override ICompiler CreateCompiler(CompilerConfiguration configuration)
     /// <inheritdoc/>
     protected override IPreCompiler CreatePreCompiler(CompilerConfiguration configuration)
     {
-      if (HasNativeTrimOfInsignificantDecimalPoints) {
+      if (!HasNativeTrimOfInsignificantZerosInDecimals) {
         var decimalAggregateCorrector = new AggregateOverDecimalColumnCorrector(Handlers.Domain.Model);
         return new CompositePreCompiler(decimalAggregateCorrector, base.CreatePreCompiler(configuration));
       }

Orm/Xtensive.Orm.PostgreSql/Orm.Providers.PostgreSql/PostgresqlSqlDml.cs

@@ -18,7 +18,7 @@ namespace Xtensive.Orm.Providers.PostgreSql
   public class PostgresqlSqlDml
   {
     /// <summary>
-    /// Creates expression for native "trim_scale" function call. The function is supported starting from PostgreSQL 13
+    /// Creates an expression for native "trim_scale" function call. The function is supported starting from PostgreSQL 13
     /// </summary>
     public static SqlExpression DecimalTrimScale(SqlExpression operand)
     {

Orm/Xtensive.Orm.PostgreSql/Orm.Providers.PostgreSql/SqlCompiler.cs

@@ -21,7 +21,7 @@ internal class SqlCompiler : Providers.SqlCompiler
   {
     private const int MaxDotnetDecimalPrecision = 28;
 
-    private readonly bool canRemoveInsignificantZerosInDecimal;
+    private readonly bool canRemoveInsignificantZerosInDecimals;
 
     protected override SqlProvider VisitFreeText(FreeTextProvider provider)
     {
@@ -63,7 +63,7 @@ protected override SqlExpression ProcessAggregate(SqlProvider source, List<SqlEx
       var aggregateType = aggregateColumn.AggregateType;
       var originCalculateColumn = source.Origin.Header.Columns[aggregateColumn.SourceIndex];
       if (AggregateRequiresDecimalAdjustments(aggregateColumn)) {
-        if (canRemoveInsignificantZerosInDecimal) {
+        if (canRemoveInsignificantZerosInDecimals) {
           return (IsCalculatedColumn(originCalculateColumn))
             ? PostgresqlSqlDml.DecimalTrimScale(SqlDml.Cast(result, Driver.MapValueType(aggregateColumn.Type)))
             : PostgresqlSqlDml.DecimalTrimScale(result);
@@ -145,7 +145,7 @@ AggregateType.Min or
     public SqlCompiler(HandlerAccessor handlers, CompilerConfiguration configuration)
       : base(handlers, configuration)
     {
-      canRemoveInsignificantZerosInDecimal = handlers.ProviderInfo.StorageVersion.Major >= 13;
+      canRemoveInsignificantZerosInDecimals = handlers.ProviderInfo.StorageVersion.Major >= 13;
     }
   }
 }