Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0050922Openbravo ERPA. Platformpublic2022-11-16 16:272022-11-18 09:23
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PriorityimmediateSeveritycriticalReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionPR22Q4.1Fixed in VersionPR22Q4.1 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2022-10-31
Regression introduced in releasePR22Q3.2
Regression introduced by commit
Triggers an Emergency PackNo
Summary0050922: Filter in valid tree structure results in message related to cycles in the hierarchy
DescriptionIn this issue [1] we tried to handle properly the filter behaviour when there are cycles in the hierarchy of the data being fetched.

That fix has a bug that results in cycles being detected if the filtered text is present in a child node and one of its ancestors

[1] https://issues.openbravo.com/view.php?id=50549 [^]
Steps To Reproduce- Log in in the openbravo backoffice
- Change to a System Administrator role
- Open the Menu window
- Click the Tree button to show the tree grid view
- In the Name column enter the text "User" (without the quotes). An error related to data cycles is displayed, but there are no cycles in the hierarchy
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0050921 closed AugustoMauch Filter in valid tree structure results in message related to cycles in the hierarchy 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2022-11-16 16:32AugustoMauchTypedefect => backport
2022-11-16 16:32AugustoMauchTarget Version => PR22Q4.1
2022-11-17 20:13hgbotNote Added: 0143805
2022-11-18 09:23hgbotResolutionopen => fixed
2022-11-18 09:23hgbotStatusscheduled => closed
2022-11-18 09:23hgbotNote Added: 0143811
2022-11-18 09:23hgbotFixed in Version => PR22Q4.1
2022-11-18 09:23hgbotNote Added: 0143812

Notes
(0143805)
hgbot   
2022-11-17 20:13   
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/770 [^]
(0143811)
hgbot   
2022-11-18 09:23   
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/770 [^]
(0143812)
hgbot   
2022-11-18 09:23   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: d05adc75f38ccbba39aeb343bf6f0df3527ffdc2
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 17-11-2022 20:13:12
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/d05adc75f38ccbba39aeb343bf6f0df3527ffdc2 [^]

Fixes ISSUE-50922: Data cycles are now properly detected in TreeDatasourceService

The problem was that we were not properly checking the presence of data cycles. When filtering a node, we want to include also all its
ancestors. When looking for cycles, instead of looking for repeated nodes in the ancestor list, we were looking for repeated nodes in the
list of nodes that were going to be returned, that contains all the nodes that match the filter plus its ancestors.

---
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/TreeDatasourceService.java
---