Tue, 19 May 2015 22:35:00 +0200Fixed issue 29894: Merged [gsi-removal] project
David Baz Fayos <david.baz@openbravo.com> [Tue, 19 May 2015 22:35:00 +0200] rev 26665
Fixed issue 29894: Merged [gsi-removal] project

http://wiki.openbravo.com/w/index.php?title=Projects:OpenID_Removal

Mon, 18 May 2015 13:54:14 +0200[gsi-removal] Change 'Iterator' to 'for' loop in Login.java
David Baz Fayos <david.baz@openbravo.com> [Mon, 18 May 2015 13:54:14 +0200] rev 26664
[gsi-removal] Change 'Iterator' to 'for' loop in Login.java

Mon, 18 May 2015 13:53:39 +0200[gsi-removal] Update classpath template with the removed sources
David Baz Fayos <david.baz@openbravo.com> [Mon, 18 May 2015 13:53:39 +0200] rev 26663
[gsi-removal] Update classpath template with the removed sources

Mon, 18 May 2015 13:53:00 +0200[gsi-removal] Update license with the removed libraries
David Baz Fayos <david.baz@openbravo.com> [Mon, 18 May 2015 13:53:00 +0200] rev 26662
[gsi-removal] Update license with the removed libraries

Tue, 12 May 2015 16:40:13 +0200[gsi-removal] Restored AD_TABLE and AD_COLUMN of the removed modules
David Baz Fayos <david.baz@openbravo.com> [Tue, 12 May 2015 16:40:13 +0200] rev 26661
[gsi-removal] Restored AD_TABLE and AD_COLUMN of the removed modules
to avoid warnings during the export.database

Mon, 11 May 2015 16:17:23 +0200[gsi-removal] Added 'service.integration.google' and 'service.integration.openid' modules:
David Baz Fayos <david.baz@openbravo.com> [Mon, 11 May 2015 16:17:23 +0200] rev 26660
[gsi-removal] Added 'service.integration.google' and 'service.integration.openid' modules:

In order to avoid problems with localization modules having dependencies to these ones,
a basic version of them have been restored

Wed, 06 May 2015 15:34:31 +0200[gsi-removal] Removed 'service.integration.google' and 'service.integration.openid' modules:
David Baz Fayos <david.baz@openbravo.com> [Wed, 06 May 2015 15:34:31 +0200] rev 26659
[gsi-removal] Removed 'service.integration.google' and 'service.integration.openid' modules:

- Removed source files
- Added a 'Module Merge' in 'v3 Framework' module. It also includes the merge that
was contained in 'service.integration.openid' to avoid problems while upgrading
from a very old Openbravo 3 version
- Removed the dependencies of these modules from 'v3' module
- Removed 'accesses' registered in sampledata to windows/views provided by the removed modules

Wed, 06 May 2015 15:28:17 +0200[gsi-removal] Modified Login to use the new stuff:
David Baz Fayos <david.baz@openbravo.com> [Wed, 06 May 2015 15:28:17 +0200] rev 26658
[gsi-removal] Modified Login to use the new stuff:

Now, 'out of the box', the Google Sign In icon is just informative.
In order to make it runs, the 'Google Sign In' module (Professional) should be insalled

Wed, 06 May 2015 15:24:57 +0200[gsi-removal] Added new Google Sign In icon
David Baz Fayos <david.baz@openbravo.com> [Wed, 06 May 2015 15:24:57 +0200] rev 26657
[gsi-removal] Added new Google Sign In icon

Wed, 06 May 2015 15:23:26 +0200[gsi-removal] Added new messages
David Baz Fayos <david.baz@openbravo.com> [Wed, 06 May 2015 15:23:26 +0200] rev 26656
[gsi-removal] Added new messages

Wed, 06 May 2015 15:19:36 +0200[gsi-removal] Moved 'OBSEIG_ShowGIcon' preference to 'client.application'
David Baz Fayos <david.baz@openbravo.com> [Wed, 06 May 2015 15:19:36 +0200] rev 26655
[gsi-removal] Moved 'OBSEIG_ShowGIcon' preference to 'client.application'

Wed, 06 May 2015 15:16:58 +0200[gsi-removal] Moved 'guice', 'httpclient', 'httpcore' and 'httpmime' libraries to 'client.application'
David Baz Fayos <david.baz@openbravo.com> [Wed, 06 May 2015 15:16:58 +0200] rev 26654
[gsi-removal] Moved 'guice', 'httpclient', 'httpcore' and 'httpmime' libraries to 'client.application'

Tue, 19 May 2015 16:37:18 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Tue, 19 May 2015 16:37:18 +0000] rev 26653
CI: merge back from main

Tue, 19 May 2015 16:21:12 +0000CI: update AD_MODULE to version 26651
RM packaging bot <staff.rm@openbravo.com> [Tue, 19 May 2015 16:21:12 +0000] rev 26652
CI: update AD_MODULE to version 26651

Tue, 19 May 2015 10:47:47 +0200related to issue 29751: fixed success status when executing bg processes
Asier Lostalé <asier.lostale@openbravo.com> [Tue, 19 May 2015 10:47:47 +0200] rev 26651
related to issue 29751: fixed success status when executing bg processes

When a background process finsished successfully its status was incorrectly
set to error instead of success

Mon, 18 May 2015 23:31:48 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Mon, 18 May 2015 23:31:48 +0000] rev 26650
CI: merge back from main

Mon, 18 May 2015 23:16:35 +0000CI: update AD_MODULE to version 26647
RM packaging bot <staff.rm@openbravo.com> [Mon, 18 May 2015 23:16:35 +0000] rev 26649
CI: update AD_MODULE to version 26647

Mon, 18 May 2015 18:20:40 +0200Fixes issue 29896: Prevents unneded datasource request when filtering
Augusto Mauch <augusto.mauch@openbravo.com> [Mon, 18 May 2015 18:20:40 +0200] rev 26648
Fixes issue 29896: Prevents unneded datasource request when filtering

When the grid was filtered in a scrolled down grid, two datasource requests to load the grid were being done instead of one. This happened due to two reasons:
- The function scrollTo of the grid's body was being invoked while the grid was being filtered. This function can be skipped in this case because when a datasource request is triggered due to a filter event, the grid is always automatically scrolled up to the top
- If the OBViewGrid.getRecord function was invoked in the middle of a datasource request, an additional request was done. To prevent this, in this case the function returns Array.LOADING

To detect if the grid is in the process of doing a datasource request, the isFilteringExternally flag is used. This flag is set in the handleFilterEditorSubmit function, and only if the change in the filter editor is going to trigger a datasource request. The flag is reset in the callback of that function.

This previous fix did not work properly, as the criteria passed to the willFetchData function was not proper, it needed to the processed in the convertCriteria function.

Mon, 18 May 2015 16:50:47 +0200Related to bug 29256: Code Review
Unai Martirena <unai.martirena@openbravo.com> [Mon, 18 May 2015 16:50:47 +0200] rev 26647
Related to bug 29256: Code Review

Add coalesce in case there is no batch associated to set GL Journal Org, in order to avoid issues in the if condition done right after the query if null values are compared

Thu, 14 May 2015 14:20:33 +0200Fixes issue 29256: Error while completing a Simple G/L Journal in Oracle
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Thu, 14 May 2015 14:20:33 +0200] rev 26646
Fixes issue 29256: Error while completing a Simple G/L Journal in Oracle

A query in gl_journal_post has been changed to avoid errors in Oracle when retrieving a null id

Mon, 18 May 2015 11:06:08 +0200Fixes issue 27170: Performance problems in CharacteristicValueEventHandler
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Mon, 18 May 2015 11:06:08 +0200] rev 26645
Fixes issue 27170: Performance problems in CharacteristicValueEventHandler

onUpdate method has been changed to execute an update query instead of two OBCriteria queries, which improves the performance of the event handler.

Sat, 16 May 2015 03:19:14 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Sat, 16 May 2015 03:19:14 +0000] rev 26644
CI: merge back from main

Sat, 16 May 2015 03:04:03 +0000CI: update AD_MODULE to version 26642
RM packaging bot <staff.rm@openbravo.com> [Sat, 16 May 2015 03:04:03 +0000] rev 26643
CI: update AD_MODULE to version 26642

Fri, 15 May 2015 18:27:18 +0200Related with issue 29896: Fix backed out because it is breaks int-initial tests
Augusto Mauch <augusto.mauch@openbravo.com> [Fri, 15 May 2015 18:27:18 +0200] rev 26642
Related with issue 29896: Fix backed out because it is breaks int-initial tests

A side effect of the changeset is making Selenium unable to know when a grid has been loaded under certain circumstances.

Fri, 15 May 2015 12:01:31 +0200Fixes issue 29897: False positives in GLJournalAccountingCheck validation
Eduardo Argal Guibert <eduardo.argal@openbravo.com> [Fri, 15 May 2015 12:01:31 +0200] rev 26641
Fixes issue 29897: False positives in GLJournalAccountingCheck validation
Missing ad_table_id constraint ends up in wrong validation when there are old records using numeric values for ids.

Fri, 15 May 2015 12:23:20 +0200Related to issue 29766: Retail Operations Buffer: store all transactions in operations table before processing
Martin Taal <martin.taal@openbravo.com> [Fri, 15 May 2015 12:23:20 +0200] rev 26640
Related to issue 29766: Retail Operations Buffer: store all transactions in operations table before processing
Solve javadoc errors

Fri, 15 May 2015 09:58:34 +0200Fixes issue 29896: Prevents unneded datasource request when filtering
Augusto Mauch <augusto.mauch@openbravo.com> [Fri, 15 May 2015 09:58:34 +0200] rev 26639
Fixes issue 29896: Prevents unneded datasource request when filtering

When the grid was filtered in a scrolled down grid, two datasource requests to load the grid were bein
g done instead of one. This happened due to two reasons:
- The function scrollTo of the grid's body was being invoked while the grid was being filtered. This f
unction can be skipped in this case because when a datasource request is triggered due to a filter eve
nt, the grid is always automatically scrolled up to the top
- If the OBViewGrid.getRecord function was invoked in the middle of a datasource request, an additiona
l request was done. To prevent this, in this case the function returns Array.LOADING

To detect if the grid is in the process of doing a datasource request, the isFilteringExternally flag
is used. This flag is set in the handleFilterEditorSubmit function, and only if the change in the filt
er editor is going to trigger a datasource request. The flag is reset in the callback of that function
.

Thu, 14 May 2015 15:46:56 +0000CI: merge back from main
RM packaging bot <staff.rm@openbravo.com> [Thu, 14 May 2015 15:46:56 +0000] rev 26638
CI: merge back from main

Thu, 14 May 2015 15:29:39 +0000CI: update AD_MODULE to version 26633
RM packaging bot <staff.rm@openbravo.com> [Thu, 14 May 2015 15:29:39 +0000] rev 26637
CI: update AD_MODULE to version 26633

Thu, 14 May 2015 14:59:55 +0200ted to issue 29751: Killable processes
Asier Lostalé <asier.lostale@openbravo.com> [Thu, 14 May 2015 14:59:55 +0200] rev 26636
ted to issue 29751: Killable processes

Few minor enhacements:
- When a process is killed, its status in process monitor is shown as "Killed
by User"
- When user clicks on "Kill Process" button change the message from "Process
killed successfully" to "Kill signal sent to process" as sending the kill
signal doesn't imply necessaryly the process to be successfully killed
- Changed UI message levels when "Kill Process" button is clicked:
- If signal is sent: from success to info
- If process does not implement KillableProcess: from info to warn
- Added ProcessBundle to KillableProcess.kill method so process instance's
information can be accessed (ie. allows to add extra log when killing the
process)