Fri, 05 Feb 2016 14:38:57 +0100[pi-security-hqlinjection] Refactor of the code
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 05 Feb 2016 14:38:57 +0100] rev 28947
[pi-security-hqlinjection] Refactor of the code

The code has been refactored and now the preference values is going to be taken into account
to get the where clause from the _where parameter or not.

Mon, 01 Feb 2016 15:09:09 +0100[pi-security-hqlinjection] Create a new preference
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 01 Feb 2016 15:09:09 +0100] rev 28946
[pi-security-hqlinjection] Create a new preference

A new preference has been created "Allow Where Parameter". This preference
allows the where parameter in the manual datasources.

Mon, 01 Feb 2016 12:54:13 +0100[pi-security-hqlinjection] Merge with pi
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 01 Feb 2016 12:54:13 +0100] rev 28945
[pi-security-hqlinjection] Merge with pi

Fri, 29 Jan 2016 14:01:24 +0100[pi-security-hqlinjection] Code review fix for Alert Management window
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 29 Jan 2016 14:01:24 +0100] rev 28944
[pi-security-hqlinjection] Code review fix for Alert Management window

Before, the where clause was being overwritten for every manual dataosurces.
This has been fixed by creating a new method which tells if a datasource is manual
or not and if it is manual, now, the where parametes is added the same way as it is
for the selector items.

Fri, 29 Jan 2016 09:43:44 +0100[pi-security-hqlinjection] Merge with pi
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 29 Jan 2016 09:43:44 +0100] rev 28943
[pi-security-hqlinjection] Merge with pi

Fri, 29 Jan 2016 09:42:35 +0100[pi-security-hqlinjection] Fixed the selector Datasource
Naroa Iriarte <naroa.iriarte@openbravo.com> [Fri, 29 Jan 2016 09:42:35 +0100] rev 28942
[pi-security-hqlinjection] Fixed the selector Datasource

Mon, 25 Jan 2016 16:33:52 +0100[pi-security-hqlinjection] Merge with pi
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 25 Jan 2016 16:33:52 +0100] rev 28941
[pi-security-hqlinjection] Merge with pi

Mon, 25 Jan 2016 16:19:49 +0100[pi-security-hqlinjection] Refactor of ob-view-grid and ob-pick-and-execute
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 25 Jan 2016 16:19:49 +0100] rev 28940
[pi-security-hqlinjection] Refactor of ob-view-grid and ob-pick-and-execute

The new function isFilterClauseApplied has been moved to the parent class OBGrid.

Mon, 25 Jan 2016 13:18:03 +0100[pi-security-hqlinjection] Create the proper code in the backend
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 25 Jan 2016 13:18:03 +0100] rev 28939
[pi-security-hqlinjection] Create the proper code in the backend

The way of handling the where and filter clauses now it is different. In the DefaultJsonDataService.java, a
protected method has been created, to be able to invoke it in the same package.
This method is the one which creates the query that, before, was sent in the _where parameter of the datasouce.

Thu, 21 Jan 2016 13:14:29 +0100[pi-security-hqlinjection] Remove the where parameter form selectors
Naroa Iriarte <naroa.iriarte@openbravo.com> [Thu, 21 Jan 2016 13:14:29 +0100] rev 28938
[pi-security-hqlinjection] Remove the where parameter form selectors

The selectos have only where clause, they do not have a filter clause,
so, the isFilterApplied parameter has no sense in this cases.

Wed, 20 Jan 2016 10:42:37 +0100[pi-security-hqlinjection] Merge with pi
Naroa Iriarte <naroa.iriarte@openbravo.com> [Wed, 20 Jan 2016 10:42:37 +0100] rev 28937
[pi-security-hqlinjection] Merge with pi

Wed, 20 Jan 2016 10:37:33 +0100[pi-security-hqlinjection] Improvements in grid and pick and execute funtions
Naroa Iriarte <naroa.iriarte@openbravo.com> [Wed, 20 Jan 2016 10:37:33 +0100] rev 28936
[pi-security-hqlinjection] Improvements in grid and pick and execute funtions

Functions have been modified to be more accurate with the result and also the
pick and execute ftl has been modified, the where clause is not anymore shown in
the view.

Wed, 20 Jan 2016 10:34:35 +0100[pi-security-hqlinjection] A new method addded to handle selector ftl
Naroa Iriarte <naroa.iriarte@openbravo.com> [Wed, 20 Jan 2016 10:34:35 +0100] rev 28935
[pi-security-hqlinjection] A new method addded to handle selector ftl

A new method has been added to handle the ftl of the selectors, this method
checks if there exists a whereClause and returns a boolean.

Tue, 19 Jan 2016 17:33:28 +0100[pi-security-hqlinjection] Don't send _where in selectors
Naroa Iriarte <naroa.iriarte@openbravo.com> [Tue, 19 Jan 2016 17:33:28 +0100] rev 28934
[pi-security-hqlinjection] Don't send _where in selectors

The _where parameter is no longer send in the selectors datasource.
Now a boolean parameter is sent, "whereClause" this is true when there
exists a where clause in the selector and false when not.

Tue, 19 Jan 2016 15:52:21 +0100[pi-security-hqlinjection] Don't send _where param in Pick and Execute windows
Naroa Iriarte <naroa.iriarte@openbravo.com> [Tue, 19 Jan 2016 15:52:21 +0100] rev 28933
[pi-security-hqlinjection] Don't send _where param in Pick and Execute windows

The _where parameter is not sent in the pick and execute windows' datasource.
Now a boolean parameter is sent, "isFilterApplied", this parameter is true when
there is a filter applied in a pick and execute window and it is false when there
is not a filter applied or it has been cleaned up.

Tue, 19 Jan 2016 10:55:51 +0100[pi-security-hqlinjection] Send a boolean
Naroa Iriarte <naroa.iriarte@openbravo.com> [Tue, 19 Jan 2016 10:55:51 +0100] rev 28932
[pi-security-hqlinjection] Send a boolean

Only send a boolean parameter that is true when a filter is applied in
the grid. And do not show the query in the view of the window.

Mon, 18 Jan 2016 18:03:20 +0100[pi-security-hqlinjection] new name for filter clause
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 18 Jan 2016 18:03:20 +0100] rev 28931
[pi-security-hqlinjection] new name for filter clause

existsFilterClause has been created in the ftl to get that name
in the view. The filterClause has been put again to get the query
in the view generation.

Mon, 18 Jan 2016 17:08:11 +0100[pi-security-hqlinjection] The isFilterApplied parameter added
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 18 Jan 2016 17:08:11 +0100] rev 28930
[pi-security-hqlinjection] The isFilterApplied parameter added

If a filter is applied, then this parameter will be true. If a filter is
deleted, thenm this parameter will be false.

Mon, 18 Jan 2016 12:55:02 +0100[pi-security-hqlinjection] The hasFilterClause parameter added
Naroa Iriarte <naroa.iriarte@openbravo.com> [Mon, 18 Jan 2016 12:55:02 +0100] rev 28929
[pi-security-hqlinjection] The hasFilterClause parameter added

The _where parameter which was showing a query now is not send.
A new parameter called hasFilterClause has been added.
This parameter is a boolean that is true when there exists a whereclause or
a filterClause.

Wed, 13 Apr 2016 11:24:18 +0200Fixed issue 32600: Export csv was not working fine
Naroa Iriarte <naroa.iriarte@openbravo.com> [Wed, 13 Apr 2016 11:24:18 +0200] rev 28928
Fixed issue 32600: Export csv was not working fine

The csv files where not correctly exported from the client if the client was in a different
timezone.
The problem was that in the AdvancedQueryBuilder class, the way of setting the date time was
not correct.
The Calendar.HOUR constant was being used instead of Calendar.HOUR_OF_DAY. The first, is to
set or get the hour indicating the morning or the afternoon, for example the 11:00:00 PM, the
hour is 11. The second one, the one that is used now, is used for the 24 hours clock, so, if
the time isn 11:00:00 the HOUR_OF_DAY is 23.

Wed, 13 Apr 2016 14:57:26 +0200fixed issue 32570: allow to define cross-organization references
Asier Lostalé <asier.lostale@openbravo.com> [Wed, 13 Apr 2016 14:57:26 +0200] rev 28927
fixed issue 32570: allow to define cross-organization references

Wed, 13 Apr 2016 14:51:17 +0200[cross-org] adding allow cross org column in sourcedata
Asier Lostalé <asier.lostale@openbravo.com> [Wed, 13 Apr 2016 14:51:17 +0200] rev 28926
[cross-org] adding allow cross org column in sourcedata

Tue, 12 Apr 2016 09:22:47 +0200[cross-org] pi merge
Asier Lostalé <asier.lostale@openbravo.com> [Tue, 12 Apr 2016 09:22:47 +0200] rev 28925
[cross-org] pi merge

Tue, 12 Apr 2016 07:46:05 +0200[cross-org] Expect exception, GenericJDBCException is thrown in PG not ORA
Asier Lostalé <asier.lostale@openbravo.com> [Tue, 12 Apr 2016 07:46:05 +0200] rev 28924
[cross-org] Expect exception, GenericJDBCException is thrown in PG not ORA

Tue, 12 Apr 2016 07:45:29 +0200[cross-org] reuse existing ExpectedException rule in parent
Asier Lostalé <asier.lostale@openbravo.com> [Tue, 12 Apr 2016 07:45:29 +0200] rev 28923
[cross-org] reuse existing ExpectedException rule in parent

Mon, 11 Apr 2016 13:33:17 +0200[cross-org] test cases: reset column with modules in dev
Asier Lostalé <asier.lostale@openbravo.com> [Mon, 11 Apr 2016 13:33:17 +0200] rev 28922
[cross-org] test cases: reset column with modules in dev

Fri, 08 Apr 2016 15:11:43 +0200[cross-org] remove some clutter
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 15:11:43 +0200] rev 28921
[cross-org] remove some clutter

Fri, 08 Apr 2016 15:08:25 +0200[cross-org] recovered unintentionally deleted code
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 15:08:25 +0200] rev 28920
[cross-org] recovered unintentionally deleted code

to manage the case admin mode is locally set before having a context

Fri, 08 Apr 2016 14:31:44 +0200[cross-org] format
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 14:31:44 +0200] rev 28919
[cross-org] format

Fri, 08 Apr 2016 13:20:28 +0200[cross-org] prevent Allowed_Cross_Org_Link modification if mod is not in dev
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 13:20:28 +0200] rev 28918
[cross-org] prevent Allowed_Cross_Org_Link modification if mod is not in dev

Fri, 08 Apr 2016 11:16:49 +0200[cross-org] removing code duplicities
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 11:16:49 +0200] rev 28917
[cross-org] removing code duplicities

* Added stakcs to AdminType enum to remove "magic numbers"
* Making use of this enum, generic methods can be implemented instead of
duplicating them for each case

Fri, 08 Apr 2016 10:44:36 +0200[cross-org] handle properly stacks for set/restore cross org admin mode
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 10:44:36 +0200] rev 28916
[cross-org] handle properly stacks for set/restore cross org admin mode

Fri, 08 Apr 2016 10:11:50 +0200[cross-org] typo
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Apr 2016 10:11:50 +0200] rev 28915
[cross-org] typo

Fri, 01 Apr 2016 14:31:56 +0200[cross-org] updated (c) year
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 01 Apr 2016 14:31:56 +0200] rev 28914
[cross-org] updated (c) year

Fri, 01 Apr 2016 14:29:43 +0200[cross-org] javadoc
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 01 Apr 2016 14:29:43 +0200] rev 28913
[cross-org] javadoc

Fri, 01 Apr 2016 14:29:28 +0200[cross-org] reformat using homogeneously spaces instead of tabs
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 01 Apr 2016 14:29:28 +0200] rev 28912
[cross-org] reformat using homogeneously spaces instead of tabs

Fri, 01 Apr 2016 11:44:15 +0200[cross-org] pi merge
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 01 Apr 2016 11:44:15 +0200] rev 28911
[cross-org] pi merge

Thu, 18 Feb 2016 17:28:39 +0100[cross-org] fixed test cleanup
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 17:28:39 +0100] rev 28910
[cross-org] fixed test cleanup

Thu, 18 Feb 2016 17:05:24 +0100[cross-org] added test cases for datasource fetch
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 17:05:24 +0100] rev 28909
[cross-org] added test cases for datasource fetch

Thu, 18 Feb 2016 16:32:04 +0100[cross-org] comments + constants
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:32:04 +0100] rev 28908
[cross-org] comments + constants

Thu, 18 Feb 2016 16:29:10 +0100[cross-org] renamed field to be consistent with the rest
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:29:10 +0100] rev 28907
[cross-org] renamed field to be consistent with the rest

Thu, 18 Feb 2016 16:28:38 +0100[cross-org] added boolean DB constraint
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:28:38 +0100] rev 28906
[cross-org] added boolean DB constraint

Thu, 18 Feb 2016 16:23:21 +0100[cross-org] removed unneeded code
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:23:21 +0100] rev 28905
[cross-org] removed unneeded code

Thu, 18 Feb 2016 16:10:01 +0100[cross-org] ADMIN_TRACE_SIZE is not final anymore
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:10:01 +0100] rev 28904
[cross-org] ADMIN_TRACE_SIZE is not final anymore

* This prevents warn messages for dead code
* A setter has also been created to allow changing it

Thu, 18 Feb 2016 16:05:27 +0100[cross-org] test case for cross org admin mode working as stack
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:05:27 +0100] rev 28903
[cross-org] test case for cross org admin mode working as stack

Thu, 18 Feb 2016 16:05:05 +0100[cross-org] javadoc
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 16:05:05 +0100] rev 28902
[cross-org] javadoc

Thu, 18 Feb 2016 15:57:52 +0100[cross-org] rename for consistency
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:57:52 +0100] rev 28901
[cross-org] rename for consistency

Thu, 18 Feb 2016 15:56:18 +0100[cross-org] removed duplicated test case
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:56:18 +0100] rev 28900
[cross-org] removed duplicated test case

Thu, 18 Feb 2016 15:47:27 +0100[cross-org] proper implementation for cross org admin stack
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:47:27 +0100] rev 28899
[cross-org] proper implementation for cross org admin stack

Thu, 18 Feb 2016 15:46:47 +0100[cross-org] test cases for cross org admin mode
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:46:47 +0100] rev 28898
[cross-org] test cases for cross org admin mode

Thu, 18 Feb 2016 15:45:13 +0100[cross-org] ALLOWED_CROSS_ORG_LINK onCreateDefault
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:45:13 +0100] rev 28897
[cross-org] ALLOWED_CROSS_ORG_LINK onCreateDefault

Thu, 18 Feb 2016 15:23:29 +0100[cross-org] pi merge
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:23:29 +0100] rev 28896
[cross-org] pi merge

Thu, 18 Feb 2016 15:22:50 +0100[cross-org] partial backout of changeset ae290995c081
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 15:22:50 +0100] rev 28895
[cross-org] partial backout of changeset ae290995c081

Some column postions were unintentionally changed. This causes conflicts
merging with pi.

Thu, 18 Feb 2016 11:23:05 +0100[cross-org] testing with a more limited role (access only to Spain, no *)
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 11:23:05 +0100] rev 28894
[cross-org] testing with a more limited role (access only to Spain, no *)

Thu, 18 Feb 2016 10:38:51 +0100[cross-org] test cases covering DAL one-to-many bags fetch
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 10:38:51 +0100] rev 28893
[cross-org] test cases covering DAL one-to-many bags fetch

Thu, 18 Feb 2016 09:44:34 +0100[cross-org] added test cases for one to many relationships
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 09:44:34 +0100] rev 28892
[cross-org] added test cases for one to many relationships

Thu, 18 Feb 2016 08:55:23 +0100[cross-org] clean up after test execution
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 08:55:23 +0100] rev 28891
[cross-org] clean up after test execution

Thu, 18 Feb 2016 08:31:31 +0100[cross-org] basic implementation for cross org references:
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 08:31:31 +0100] rev 28890
[cross-org] basic implementation for cross org references:

- Allowed cross org is read in base Column model
- This value is kept in run-time in memory model
- New crossOrgReferenceAdminMode can be set in OBContext
- This admin mode is taken into account to check org of FKs

Thu, 18 Feb 2016 08:26:20 +0100[cross-org] test cases for column allowing cross org, but not in org admin mode
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 18 Feb 2016 08:26:20 +0100] rev 28889
[cross-org] test cases for column allowing cross org, but not in org admin mode

Wed, 17 Feb 2016 16:55:49 +0100[cross-org] DB infra to define allowed cross org link
Asier Lostalé <asier.lostale@openbravo.com> [Wed, 17 Feb 2016 16:55:49 +0100] rev 28888
[cross-org] DB infra to define allowed cross org link