Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0019464Openbravo ERP01. General setuppublic2012-01-11 16:542012-01-25 10:29
Reportermalsasua 
Assigned Tomarvintm 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionCommunity Appliance
Product Version3.0MP6.1 
Target Version3.0MP8Fixed in Version3.0MP8 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browserGoogle Chrome
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0019464: menu entries are displayed, but the parent node is not active
Descriptionif there is a menu entry that it is "is summary", his father is not active and his father in ad_treenode is not active, the menu entry is displayed in tree, and his father is not displayed

If the environment started in 2.50 version, in this version, when you deactivate a menu entry, the element ad_treenode is deactivated too.
Steps To Reproduce
. deactivate the menu entry "sales management"
. in database, deactivate the ad_treenode corresponding to this record
Proposed Solution
Additional Information
TagsRegression
Relationships
related to defect 00186473.0MP5 closed marvintm A menu item hidden on 2.50 can not be shown after upgrade to Openbravo 3. 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-01-11 16:54malsasuaNew Issue
2012-01-11 16:54malsasuaAssigned To => jonalegriaesarte
2012-01-11 16:54malsasuaWeb browser => Google Chrome
2012-01-11 16:54malsasuaModules => Core
2012-01-11 16:54malsasuaResolution time => 1329260400
2012-01-11 17:02malsasuaIssue Monitored: networkb
2012-01-11 17:02malsasuaRelationship addedrelated to 0018647
2012-01-11 17:03marvintmAssigned Tojonalegriaesarte => marvintm
2012-01-12 17:36marvintmTag Attached: Regression
2012-01-12 17:39marvintmNote Added: 0044308
2012-01-12 17:40hgbotCheckin
2012-01-12 17:40hgbotNote Added: 0044310
2012-01-12 17:40hgbotStatusnew => resolved
2012-01-12 17:40hgbotResolutionopen => fixed
2012-01-12 17:40hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/c263f4e3ec2253e2bf7734bb7b991156bc4cb43f [^]
2012-01-14 21:21hudsonbotCheckin
2012-01-14 21:21hudsonbotNote Added: 0044383
2012-01-23 17:21malsasuaNote Added: 0044488
2012-01-25 10:29alostaleNote Added: 0044547
2012-01-25 10:29alostaleStatusresolved => closed
2012-01-25 10:29alostaleFixed in Version => 3.0MP8

Notes
(0044308)
marvintm   
2012-01-12 17:39   
This is a regression introduced in the related issue 18647. It is important to note that:
- The problem only happens with summary menus _at a second level_, that is, summary menus which have a summary menu parent.

- The problem only happens with summary menus which have both its treenode and ad_menu set as disabled. Disabled menus with enabled treenodes are not affected.

After the fix, the ad_menu configuration for the isactive column will prevail over the treenode (as intended by the original fix, which was unfortunately not completely correct).
(0044310)
hgbot   
2012-01-12 17:40   
Repository: erp/devel/pi
Changeset: c263f4e3ec2253e2bf7734bb7b991156bc4cb43f
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Thu Jan 12 17:36:12 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c263f4e3ec2253e2bf7734bb7b991156bc4cb43f [^]

Fixed issue 19464. Summary menu entries will be hidden correctly.

---
M modules/org.openbravo.client.application/src/org/openbravo/client/application/MenuManager.java
---
(0044383)
hudsonbot   
2012-01-14 21:21   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/0f91795528e4 [^]

Maturity status: Test
(0044488)
malsasua   
2012-01-23 17:21   
checked:
functionality: the behavior is right:
 I have deactivated a issummary node and I have deactivated the node in ad_treenode table, and his sons are not displayed
 The behaviour of the other nodes have not been changed
(0044547)
alostale   
2012-01-25 10:29   
Code reviewed and tested on pi@85a366534d3c