Thu, 27 Feb 2014 16:14:53 +0100[Tree Extension] Adds CheckTreeOperationManager for the Menu Tree
Augusto Mauch <augusto.mauch@openbravo.com> [Thu, 27 Feb 2014 16:14:53 +0100] rev 22577
[Tree Extension] Adds CheckTreeOperationManager for the Menu Tree

The MenuTreeOperationManager checks that a node cannot be moved if its module is not in development

Thu, 27 Feb 2014 15:19:55 +0100code formatting
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 27 Feb 2014 15:19:55 +0100] rev 22576
code formatting

Thu, 27 Feb 2014 13:25:38 +0100[Tree Extension] Refreshing the tree grid did not work
Augusto Mauch <augusto.mauch@openbravo.com> [Thu, 27 Feb 2014 13:25:38 +0100] rev 22575
[Tree Extension] Refreshing the tree grid did not work

Thu, 27 Feb 2014 12:27:32 +0100[Tree Extension] Adds TreeDatasourceService to work with upgraded smartclient
Augusto Mauch <augusto.mauch@openbravo.com> [Thu, 27 Feb 2014 12:27:32 +0100] rev 22574
[Tree Extension] Adds TreeDatasourceService to work with upgraded smartclient

The upgraded version of smartclient sends the parentId property to the datasource in two different ways:
- If grid is filtered, the parentId is included as a criteria
- If not, it is included as a parameter

Wed, 26 Feb 2014 18:12:08 +0100[Tree Extension] Merge with smartclient upgrade
Augusto Mauch <augusto.mauch@openbravo.com> [Wed, 26 Feb 2014 18:12:08 +0100] rev 22573
[Tree Extension] Merge with smartclient upgrade

Wed, 26 Feb 2014 16:12:29 +0100[Tree Extension] Removes unused methods from ADTreeDatasourceService
Augusto Mauch <augusto.mauch@openbravo.com> [Wed, 26 Feb 2014 16:12:29 +0100] rev 22572
[Tree Extension] Removes unused methods from ADTreeDatasourceService

Wed, 26 Feb 2014 16:09:57 +0100[Tree Extension] Code review changes in LinkToParentTreeDatasourceService
Augusto Mauch <augusto.mauch@openbravo.com> [Wed, 26 Feb 2014 16:09:57 +0100] rev 22571
[Tree Extension] Code review changes in LinkToParentTreeDatasourceService

-Fixed some SQL Injections
-Removed unused variable

Wed, 26 Feb 2014 15:02:36 +0100[Tree Extension] Avoids API change in InitialSetupUtility
Augusto Mauch <augusto.mauch@openbravo.com> [Wed, 26 Feb 2014 15:02:36 +0100] rev 22570
[Tree Extension] Avoids API change in InitialSetupUtility

These two functions have been restored and flagged as deprecated:

public static List<org.openbravo.model.ad.domain.List> treeRelation() throws Exception

public static List<TableTree> treeRelation() throws Exception

Tue, 25 Feb 2014 18:54:39 +0100[Tree Extension] Changes modulescript to make it work in Oracle
Augusto Mauch <augusto.mauch@openbravo.com> [Tue, 25 Feb 2014 18:54:39 +0100] rev 22569
[Tree Extension] Changes modulescript to make it work in Oracle

Tue, 25 Feb 2014 18:12:30 +0100[Tree Extension] PLs use ad_table_id instead of treetype to fetch trees
Augusto Mauch <augusto.mauch@openbravo.com> [Tue, 25 Feb 2014 18:12:30 +0100] rev 22568
[Tree Extension] PLs use ad_table_id instead of treetype to fetch trees

Tue, 25 Feb 2014 16:21:31 +0100[Tree Extension] Merge with pi@22129
Augusto Mauch <augusto.mauch@openbravo.com> [Tue, 25 Feb 2014 16:21:31 +0100] rev 22567
[Tree Extension] Merge with pi@22129

Tue, 25 Feb 2014 12:11:20 +0100[Tree Extension] Code review changes in TreeDomainType
Augusto Mauch <augusto.mauch@openbravo.com> [Tue, 25 Feb 2014 12:11:20 +0100] rev 22566
[Tree Extension] Code review changes in TreeDomainType

-Fixed wrong JavaDoc
-Improved the way the key columns are obtained

Tue, 25 Feb 2014 10:04:08 +0100[Tree Extension] Adds eventhandler to control only one ADTree per table
Augusto Mauch <augusto.mauch@openbravo.com> [Tue, 25 Feb 2014 10:04:08 +0100] rev 22565
[Tree Extension] Adds eventhandler to control only one ADTree per table

Tue, 25 Feb 2014 09:04:47 +0100[Tree Extension] Adds JavaDoc to TreeTablesEventHandler
Augusto Mauch <augusto.mauch@openbravo.com> [Tue, 25 Feb 2014 09:04:47 +0100] rev 22564
[Tree Extension] Adds JavaDoc to TreeTablesEventHandler

Mon, 24 Feb 2014 18:39:24 +0100[Tree Extension] Use xsql to reparent the children of a deleted node
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 18:39:24 +0100] rev 22563
[Tree Extension] Use xsql to reparent the children of a deleted node

Mon, 24 Feb 2014 17:46:09 +0100[Tree Extension] Use selectTreeIDWithTableId instead of selectTreeID
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 17:46:09 +0100] rev 22562
[Tree Extension] Use selectTreeIDWithTableId instead of selectTreeID

WindowTreeData.selectTreeID obtains adtrees based on its treetype. The treetype is no longer to be maintained, so from now on the selectTreeIDWithTableId should be used. This function uses the ad_table_id instead of the treetype to obtain the adtrees.

Mon, 24 Feb 2014 16:22:39 +0100[Tree Extension] Remove unused element to fix db inconsistency
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 16:22:39 +0100] rev 22561
[Tree Extension] Remove unused element to fix db inconsistency

Mon, 24 Feb 2014 15:39:55 +0100[Tree Extension] Adds missing UpdateADTreesData.class
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 15:39:55 +0100] rev 22560
[Tree Extension] Adds missing UpdateADTreesData.class

Mon, 24 Feb 2014 14:04:08 +0100[Tree Extension] Changes in InitialClientSetup
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 14:04:08 +0100] rev 22559
[Tree Extension] Changes in InitialClientSetup

One step of the InitialClientSetup process creates the ADTrees for the new client. The list of ADTrees to be created used to be obtained using the ADTreeType Type list. This list is no longer going to be maintained. Instead, now the list of ADTrees to be created is obtained using the list of tables that have a tree defined with the ADTree tree structure.

Some more changes have been done to take into account that now ad_trees are associated to tables by means of the ad_table_id column, instead of the treetype column.

Mon, 24 Feb 2014 13:57:43 +0100[Tree Extension] Adds UpdateADTrees modulescript
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 13:57:43 +0100] rev 22558
[Tree Extension] Adds UpdateADTrees modulescript

This modulescript fills in the ad_table_id field of the ad_trees that do not have it defined. There is a 1:1 relation between the treetype column of ad_tree and the treetype column of ad_table, so this relation is used to set the ad_table_id column of the ad_tree table.

Mon, 24 Feb 2014 13:10:53 +0100[Tree Extension] Takes into account ROOT_NODE_DB in the moveNode method
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 13:10:53 +0100] rev 22557
[Tree Extension] Takes into account ROOT_NODE_DB in the moveNode method

Now that the ROOT_NODE_CLIENT property is equals to -1, if a node is moved to the root, its parentId will be -1 in the moveNode method. Depending of whether the tree is the org tree or not, this parentId will have to be converted to ROOT_NODE_DB or to null.

Mon, 24 Feb 2014 13:03:26 +0100[Tree Extension] Menu tree is ordered
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 13:03:26 +0100] rev 22556
[Tree Extension] Menu tree is ordered

Mon, 24 Feb 2014 12:35:31 +0100[Tree Extension] Improves Java Doc in ADTreeDatasourceService
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 12:35:31 +0100] rev 22555
[Tree Extension] Improves Java Doc in ADTreeDatasourceService

Mon, 24 Feb 2014 12:33:42 +0100[Tree Extension] Defines ROOT_NODE_DB and ROOT_NODE_CLIENT
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 12:33:42 +0100] rev 22554
[Tree Extension] Defines ROOT_NODE_DB and ROOT_NODE_CLIENT

The ROOT_NODE constant, defined in TreeDatasourceService, was used to set the parent_id of the root nodes (being root nodes, they actually don't have a parent). It was not possible to use '0' as ROOT_NODE in the client, because the AD_ORG table has a root node (* organization) with nodeId = '0'. The parent_id of this node would have to be '0', which results in a loop.

Therefore, '-1' is going to be used to represent the parent of root nodes in the client (ROOT_NODE_CLIENT), while '0' is still going to be used to represent the parent_id of root nodes in the database (ROOT_NODE_DB).

Mon, 24 Feb 2014 11:20:12 +0100[Tree Extension] Removes reference to SelectorConstants in FKTreeUIDefinition
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 11:20:12 +0100] rev 22553
[Tree Extension] Removes reference to SelectorConstants in FKTreeUIDefinition

Mon, 24 Feb 2014 11:09:59 +0100[Tree Extension] Trees with treetype NEW are not handled in WindowTreeUtility
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 11:09:59 +0100] rev 22552
[Tree Extension] Trees with treetype NEW are not handled in WindowTreeUtility

Mon, 24 Feb 2014 11:01:43 +0100[Tree Extension] Is it safe to assume that a table has at most one ADTree
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 11:01:43 +0100] rev 22551
[Tree Extension] Is it safe to assume that a table has at most one ADTree

Tables can define several trees, but only one of them can use the ADTree tree structure. This has been applied to WindowTree. Also, a restriction to use the current client has been added to the criteria used to obtain an ADTree

Mon, 24 Feb 2014 10:32:11 +0100[Tree Extension] Removes not needed oncreatedefault from ad_ref_tree_field
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 10:32:11 +0100] rev 22550
[Tree Extension] Removes not needed oncreatedefault from ad_ref_tree_field

Mon, 24 Feb 2014 09:52:55 +0100[Tree Extension] Adds on delete cascade to AD_REF_TREE and AD_TABLE_TREE
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 09:52:55 +0100] rev 22549
[Tree Extension] Adds on delete cascade to AD_REF_TREE and AD_TABLE_TREE

Mon, 24 Feb 2014 09:32:37 +0100[Tree Extension] HQL Where clause is not included in the tree selector ftl
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 24 Feb 2014 09:32:37 +0100] rev 22548
[Tree Extension] HQL Where clause is not included in the tree selector ftl