Fix gis functions

CHANGELOG.md

@@ -5,7 +5,6 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 ## [Unreleased]
 
 ### Added
-
 - `IComparableType` implementation to `WebFile` class.
 - Feature GeoDistance is implemented.
 
@@ -16,10 +15,11 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 ### Removed
 
 ### Fixed
-
 - Resolve hierarchy references for `Created` master only (removed excess `UPDATE`).
 - Update data objects with static properties inherited from DataObject.
-
+- Remove an external context from PutIdentifierIntoBrackets() method implementation in PostgresDataService.
+- Fix the return value type of GisExtensions GeoDistance, GeomDistance LINQ gis-functions prototypes.
+
 ### Security
 
 ## [6.0.0] - 2021-05-25

ICSSoft.STORMNET.Business.LINQProvider/Extensions/GisExtensions.cs

@@ -1,5 +1,6 @@
 namespace ICSSoft.STORMNET.Business.LINQProvider.Extensions
 {
+    using System;
     using System.IO;
 
     using Microsoft.Spatial;
@@ -11,50 +12,54 @@ public static class GisExtensions
     {
         /// <summary>
         /// Вычисляет расстояние между двумя объектами Geography.
-        /// Метод может использоваться только в запросе к LINQProvider. В остальных случаях всегда будет возвращать 0.0 .
+        /// Метод может использоваться только в запросе к LINQProvider,
+        /// в остальных случаях метод вызовет исключение NotImplementedException.
         /// </summary>
         /// <param name="geo1">Объект 1 Geography.</param>
         /// <param name="geo2">Объект 2 Geography.</param>
         /// <returns>Возвращает минимальное расстояние между двумя объектами.</returns>
-        public static double GeoDistance(this Geography geo1, Geography geo2)
+        public static double? GeoDistance(this Geography geo1, Geography geo2)
         {
-            return 0.0;
+            throw new NotImplementedException();
         }
 
         /// <summary>
         /// Вычисляет расстояние между двумя объектами Geometry.
-        /// Метод может использоваться только в запросе к LINQProvider. В остальных случаях всегда будет возвращать 0.0 .
+        /// Метод может использоваться только в запросе к LINQProvider,
+        /// в остальных случаях метод вызовет исключение NotImplementedException.
         /// </summary>
         /// <param name="geo1">Объект 1 Geometry.</param>
         /// <param name="geo2">Объект 2 Geometry.</param>
         /// <returns>Возвращает минимальное расстояние между двумя объектами.</returns>
-        public static double GeomDistance(this Geometry geo1, Geometry geo2)
+        public static double? GeomDistance(this Geometry geo1, Geometry geo2)
         {
-            return 0.0;
+            throw new NotImplementedException();
         }
 
         /// <summary>
         /// Тестирует пересечение двух объектов Geography.
-        /// Метод может использоваться только в запросе к LINQProvider. В остальных случаях всегда будет возвращать false.
+        /// Метод может использоваться только в запросе к LINQProvider,
+        /// в остальных случаях метод вызовет исключение NotImplementedException.
         /// </summary>
         /// <param name="geo1">Объект 1 Geography.</param>
         /// <param name="geo2">Объект 2 Geography.</param>
         /// <returns>Если существует пересечение двух объектов, то возвращает true, иначе false.</returns>
         public static bool GeoIntersects(this Geography geo1, Geography geo2)
         {
-            return false;
+            throw new NotImplementedException();
         }
 
         /// <summary>
         /// Тестирует пересечение двух объектов Geometry.
-        /// Метод может использоваться только в запросе к LINQProvider. В остальных случаях всегда будет возвращать false.
+        /// Метод может использоваться только в запросе к LINQProvider,
+        /// в остальных случаях метод вызовет исключение NotImplementedException.
         /// </summary>
         /// <param name="geo1">Объект 1 Geometry.</param>
         /// <param name="geo2">Объект 2 Geometry.</param>
         /// <returns>Если существует пересечение двух объектов, то возвращает true, иначе false.</returns>
         public static bool GeomIntersects(this Geometry geo1, Geometry geo2)
         {
-            return false;
+            throw new NotImplementedException();
         }
 
         /// <summary>

ICSSoft.STORMNET.Business.LINQProvider/LcsQueryProvider.cs

@@ -60,7 +60,7 @@ public override object Execute(Expression expression)
                 case LcsReturnType.Count:
                     return _ds.GetObjectsCount(lcs);
                 default:
-                    throw new Exception("Linq провайдер не поддрживает тип: " + lcs.ReturnType);
+                    throw new Exception("Linq провайдер не поддерживает тип: " + lcs.ReturnType);
             }
         }
 

ICSSoft.STORMNET.Business.PostgresDataService/PostgresDataService.cs

@@ -545,34 +545,7 @@ public override System.Data.IDbConnection GetConnection()
         /// <returns>Identifier with brackets.</returns>
         public override string PutIdentifierIntoBrackets(string identifier)
         {
-            string postgresIdentifier = PrepareIdentifier(identifier);
-
-            // Multithreading app with single DS may be failed.
-            if (!isGenerateSqlSelect)
-            {
-                return postgresIdentifier;
-            }
-
-            if (!dictionaryShortNames.ContainsKey(postgresIdentifier))
-            {
-                dictionaryShortNames[postgresIdentifier] = null;
-            }
-
-            if (postgresIdentifier.IndexOf(".", StringComparison.Ordinal) != -1)
-            {
-                postgresIdentifier = "\"" + GenerateShortName(postgresIdentifier) + "\"";
-            }
-            else
-            {
-                if (dictionaryShortNames[postgresIdentifier] == null)
-                {
-                    dictionaryShortNames[postgresIdentifier] = GenerateShortName(postgresIdentifier);
-                }
-
-                postgresIdentifier = dictionaryShortNames[postgresIdentifier];
-            }
-
-            return postgresIdentifier;
+            return PutIdentifierIntoBrackets(identifier, isGenerateSqlSelect);
         }
 
         /// <summary>
@@ -907,6 +880,7 @@ public override void GenerateSQLRowNumber(LoadingCustomizationStruct customizati
                 else if (string.IsNullOrEmpty(orderByExprForPaging))
                 {
                     orderByExprForPaging = $"{nl}ORDER BY {SQLWhereLanguageDef.StormMainObjectKey}";
+                    query += orderByExprForPaging;
                 }
 
                 resQuery =
@@ -951,6 +925,44 @@ public override IDictionary<int, string> GetObjectIndexesWithPks(
             return ret;
         }
 
+        /// <summary>
+        /// Put identifier into brackets.
+        /// </summary>
+        /// <param name="identifier">Identifier in query.</param>
+        /// <param name="generateSqlSelectFlag">A flag, indicating that a call is made from GenerateSQLSelect() method.</param>
+        /// <returns>Identifier with brackets.</returns>
+        protected string PutIdentifierIntoBrackets(string identifier, bool generateSqlSelectFlag)
+        {
+            string postgresIdentifier = PrepareIdentifier(identifier);
+
+            // Multithreading app with single DS may be failed.
+            if (!generateSqlSelectFlag)
+            {
+                return postgresIdentifier;
+            }
+
+            if (!dictionaryShortNames.ContainsKey(postgresIdentifier))
+            {
+                dictionaryShortNames[postgresIdentifier] = null;
+            }
+
+            if (postgresIdentifier.IndexOf(".", StringComparison.Ordinal) != -1)
+            {
+                postgresIdentifier = "\"" + GenerateShortName(postgresIdentifier) + "\"";
+            }
+            else
+            {
+                if (dictionaryShortNames[postgresIdentifier] == null)
+                {
+                    dictionaryShortNames[postgresIdentifier] = GenerateShortName(postgresIdentifier);
+                }
+
+                postgresIdentifier = dictionaryShortNames[postgresIdentifier];
+            }
+
+            return postgresIdentifier;
+        }
+
         private IDictionary<int, string> GetObjectIndexesWithPksImplementation(
             LoadingCustomizationStruct lcs,
             FunctionalLanguage.Function limitFunction,

NewPlatform.Flexberry.ORM.PostgresDataService.nuspec

@@ -2,7 +2,7 @@
 <package xmlns="http://schemas.microsoft.com/packaging/2012/06/nuspec.xsd">
   <metadata>
     <id>NewPlatform.Flexberry.ORM.PostgresDataService</id>
-    <version>6.1.0-beta01</version>
+    <version>6.1.0-beta02</version>
     <title>Flexberry ORM</title>
     <authors>New Platform Ltd</authors>
     <owners>New Platform Ltd</owners>
@@ -14,6 +14,8 @@
     <releaseNotes>
       Performance
       1. Remove `order by stormmainobjectkey` for inner query.
+      Fixed
+      1. Remove an external context from PutIdentifierIntoBrackets() method implementation.
     </releaseNotes>
     <copyright>Copyright New Platform Ltd 2021</copyright>
     <tags>Flexberry ORM</tags>

NewPlatform.Flexberry.ORM.nuspec

@@ -2,7 +2,7 @@
 <package xmlns="http://schemas.microsoft.com/packaging/2012/06/nuspec.xsd">
   <metadata>
     <id>NewPlatform.Flexberry.ORM</id>
-    <version>6.0.1-beta02</version>
+    <version>6.0.1-beta03</version>
     <title>Flexberry ORM</title>
     <authors>New Platform Ltd</authors>
     <owners>New Platform Ltd</owners>
@@ -18,7 +18,7 @@
       Fixed
       1. Resolve hierarchy references for Created master only (removed excess UPDATE).
       2. Update data objects with static properties inherited from DataObject.
-
+      3. Fix the return value type of GisExtensions GeoDistance, GeomDistance LINQ gis-functions prototypes.
     </releaseNotes>
     <copyright>Copyright New Platform Ltd 2021</copyright>
     <tags>Flexberry ORM</tags>