Fix IReferenceNullDelete with hierarchy #201

Anisimova2020 · 2021-10-07T09:37:14Z

No description provided.

…erencesNullDelete.

bratchikov · 2021-10-08T08:44:06Z

...atform.Flexberry.ORM.IntegratedTests/ICSSoft.STORMNET.Business/IReferencesNullDeleteTests.cs

+ /// Конструктор.
+ /// </summary>
+ public IReferencesNullDeleteTests()
+ : base("IRefNullDel")


Имя будет длинноватое. Надо не более 5 символов.

Поправлено.

bratchikov · 2021-10-08T08:45:16Z

NewPlatform.Flexberry.ORM.IntegratedTests/XUnitTestRunnerInitializer.cs

+ System.Reflection.Assembly.Load("NewPlatform.Flexberry.ORM.Tests.Objects"),
+ };
+
+ ICSSoft.STORMNET.Business.OrmConfigurator.SetAssembliesForIReferencesNullDeleteSearch(assemblies);


Давай лучше это где-то в конструкторе интеграционных тестов сделаем?

Мне кажется, в интеграционных тестах это наиболее подходящая сущность.
Есть ещё базовый класс для интеграционных тестов, но там такая инициализация вероятно будет не к месту.

bratchikov · 2021-10-08T08:45:45Z

NewPlatform.Flexberry.ORM.IntegratedTests/XUnitTestRunnerInitializer.cs

@@ -29,6 +29,12 @@ public XUnitTestRunnerInitializer(IMessageSink messageSink)
 string outputConfigFile = ConfigurationManager.OpenExeConfiguration(ConfigurationUserLevel.None).FilePath;
 File.Copy(configFile, outputConfigFile, true);
 #endif
+ var assemblies = new[]
+ {
+ System.Reflection.Assembly.Load("NewPlatform.Flexberry.ORM.Tests.Objects"),


Лучше взять какой-нибудь тип из объектов и взять у него сборку. Через строку - не вариант.

Хорошо. Делала по аналогии, как у нас в ODataService.

Поправлено.

bratchikov · 2021-10-08T09:11:21Z

ICSSoft.STORMNET.Business/Interfaces/InterfaceBusinessServer.cs

 /// <returns>Набор объектов, для которых переданный является мастером.</returns>
- public static List<ReferencePropertyInfo> GetReferencedDataObjectsInfo(DataObject masterObject)
+ public static List<ReferencePropertyInfo> GetReferencedDataObjectsInfo(


Раз это необязательный вызов, а лишь добавление доп. сборок, предлагаю переименовать на SetupAdditionalAssemblies.

Поправлено.

bratchikov · 2021-10-08T09:12:32Z

ICSSoft.STORMNET.Business/OrmConfigurator.cs

+ /// <summary>
+ /// Класс с настройками, используемыми в ORM.
+ /// </summary>
+ public static class OrmConfigurator


Предлагаю эту прослойку не вводить. Есть же уже InterfaceBusinesServer со статическими методами.

…esNullDelete with self-reference

bratchikov

Ещё бы проверить как скажется это изменение на производительности сервиса данных. Можно погонять тесты производительности, которые запускаются вручную.

bratchikov · 2021-10-15T13:08:34Z

ICSSoft.STORMNET.Business/SQLDataService/SQLDataService.cs

@@ -4508,7 +4508,13 @@ private void AddDependencies(Type to, Type @from, Dictionary<Type, List<Type>> d
 /// <param name="currentType">Текущий обрабатываемый тип.</param>
 /// <param name="dependencies">Дополняемые зависимости.</param>
 /// <param name="extraUpdateList">Дополнительные объекты на обновление.</param>
- private void GetDependencies(DataObject currentObject, Type currentType, Dictionary<Type, List<Type>> dependencies, List<DataObject> extraUpdateList)
+ /// <param name="selfReferences">Структура для хранения информации об иерархических мастеровых ссылках объекта самогона себя.</param>


самогона?

…exberry.ORM into fix-200-i-ref-null-delete

sonarcloud · 2022-12-06T08:36:28Z

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
21 Code Smells

0.0% Coverage
0.0% Duplication

s-andrey · 2022-12-08T06:15:56Z

ICSSoft.STORMNET.Business/Interfaces/InterfaceBusinessServer.cs

+ public static List<ReferencePropertyInfo> GetReferencedDataObjectsInfo(
+ DataObject masterObject, IEnumerable<Assembly> searchAssemblies = null)
+ {
+ var realMasterObjectType = masterObject.GetType();


var заменить на тип

s-andrey · 2022-12-08T06:16:10Z

ICSSoft.STORMNET.Business/Interfaces/InterfaceBusinessServer.cs

+ return cache[realMasterObjectType];
+ }
+
+ var referencePropertyInfos = new List<ReferencePropertyInfo>();


s-andrey · 2022-12-08T06:16:20Z

ICSSoft.STORMNET.Business/Interfaces/InterfaceBusinessServer.cs

@@ -187,7 +249,7 @@ public DataObject[] OnUpdateIReferencesCascadeDelete(IReferencesCascadeDelete Up
 }

 List<ReferencePropertyInfo> referencePropertyInfos;
- var referenceObjectList = GetReferencedDataObjects(updatedDataObject, out referencePropertyInfos);
+ var referenceObjectList = GetReferencedDataObjects(updatedDataObject, out referencePropertyInfos, AssembliesForIReferencesCascadeDeleteSearch);


s-andrey · 2022-12-08T06:16:33Z

ICSSoft.STORMNET.Business/Interfaces/InterfaceBusinessServer.cs

@@ -216,7 +278,8 @@ public DataObject[] OnUpdateIReferencesNullDelete(IReferencesNullDelete UpdatedO
 }

 List<ReferencePropertyInfo> referencePropertyInfos;
- var referenceObjectList = GetReferencedDataObjects(updatedDataObject, out referencePropertyInfos);
+ var referenceObjectList = GetReferencedDataObjects(


s-andrey · 2022-12-08T06:17:00Z

ICSSoft.STORMNET.Business/SQLDataService/SQLDataService.cs

@@ -4887,8 +4925,9 @@ private void GetOrderFromDependenciesHelper(Type currentType, Dictionary<Type, L
 var alteredLastList = new Dictionary<DataObject, Collections.CaseSensivityStringDictionary>();
 var updateList = new Dictionary<DataObject, UpdaterObject>();

- List<DataObject> extraUpdateList = new List<DataObject>();
- Dictionary<Type, List<Type>> dependencies = new Dictionary<Type, List<Type>>();
+ var extraUpdateList = new List<DataObject>();


s-andrey · 2022-12-08T06:17:12Z

ICSSoft.STORMNET.Business/SQLDataService/SQLDataService.cs

+ /* Ещё возможна редкая ситуация, когда у объекта есть ссылка на самого себя. Один объект удаляется, а ссылающийся на него обнуляет ссылку.
+ В этом случае "стандартный" механизм, что над таблицей операции выполняются в порядке "Delete, Insert, Update" не подходит,
+ потому что сначала нужен "Update" с занулением ссылки, а потом "Delete" с удалением сущности.*/
+ var alteredFirstSpecialList = new Dictionary<DataObject, Collections.CaseSensivityStringDictionary>();


и нижу все var

hvostya added 3 commits October 7, 2021 14:20

Added simple test for IReferenceNullDelete.

45a33f8

Try to fix server tests

40a268c

Added new class to configurate list of assemblies for processing IRef…

b58178b

…erencesNullDelete.

bratchikov requested review from NicholasNoise and mao29 October 8, 2021 08:46

bratchikov reviewed Oct 8, 2021

View reviewed changes

hvostya added 6 commits October 8, 2021 21:11

Changes by code review.

7486279

Added simple unit tests on IReferencesNullDelete.

2a69662

Added simple tests fir IReferencesCascadeDelete.

4ad205b

Added tests on BS interfaces with hierarchy.

bc6809e

Added fixes to sql-order at DataService in order to support IReferenc…

6b87018

…esNullDelete with self-reference

Added integration test on SQLDataService

f9c1969

Anisimova2020 marked this pull request as ready for review October 13, 2021 13:57

Anisimova2020 requested a review from bratchikov October 13, 2021 13:57

bratchikov reviewed Oct 15, 2021

View reviewed changes

bratchikov linked an issue Oct 20, 2021 that may be closed by this pull request

Реализовать тесты для IReferencesNullDelete #200

Open

bratchikov changed the title ~~Added simple test for IReferenceNullDelete.~~ Fix IReferenceNullDelete with hierarchy Oct 20, 2021

bratchikov added the bug label Oct 20, 2021

hvostya added 2 commits November 11, 2021 19:11

Added performance test and some cache for BS interfaces.

538b7d0

Removed mistype.

9a23ef2

Anisimova2020 requested a review from bratchikov November 15, 2021 11:41

Merge branch 'develop' of https://github.com/Flexberry/NewPlatform.Fl…

9ac409a

…exberry.ORM into fix-200-i-ref-null-delete

s-andrey reviewed Dec 8, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix IReferenceNullDelete with hierarchy #201

Fix IReferenceNullDelete with hierarchy #201

Anisimova2020 commented Oct 7, 2021

bratchikov Oct 8, 2021

Anisimova2020 Dec 6, 2022

bratchikov Oct 8, 2021

Anisimova2020 Oct 8, 2021

bratchikov Oct 8, 2021

Anisimova2020 Oct 8, 2021

Anisimova2020 Dec 6, 2022

bratchikov Oct 8, 2021

Anisimova2020 Dec 6, 2022

bratchikov Oct 8, 2021

Anisimova2020 Dec 6, 2022

bratchikov left a comment

bratchikov Oct 15, 2021

sonarcloud bot commented Dec 6, 2022

s-andrey Dec 8, 2022

s-andrey Dec 8, 2022

s-andrey Dec 8, 2022

s-andrey Dec 8, 2022

s-andrey Dec 8, 2022

s-andrey Dec 8, 2022

s-andrey Dec 8, 2022

Fix IReferenceNullDelete with hierarchy #201

Are you sure you want to change the base?

Fix IReferenceNullDelete with hierarchy #201

Conversation

Anisimova2020 commented Oct 7, 2021

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

bratchikov left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sonarcloud bot commented Dec 6, 2022

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment