Notes |
|
(0069410)
|
hgbot
|
2014-08-17 00:25
|
|
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 64b43ec6b05c4ac2f611446513d888f110c9b2d7
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sun Aug 17 00:25:34 2014 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/64b43ec6b05c4ac2f611446513d888f110c9b2d7 [^]
Fixes issue 27351: When calling the 'getGlobalComponentResources' method of the 'MobileCoreComponentProvider' class, the array 'globalResources' will not be recreated if it is not necessary
---
M src/org/openbravo/mobile/core/MobileCoreComponentProvider.java
---
|
|
|
(0069422)
|
hgbot
|
2014-08-17 15:09
|
|
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 6a6753151bd4f219ed85b64c7b590c84e1955f1f
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sun Aug 17 15:09:18 2014 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/6a6753151bd4f219ed85b64c7b590c84e1955f1f [^]
Fixes issue 27351: 'getGlobalComponentResources' have to be set as synchronized because multiple calls were corrupting the globalResources array
Mark variables as final is not necessary but a nice plus
---
M src/org/openbravo/mobile/core/MobileCoreComponentProvider.java
---
|
|
|
(0069423)
|
hgbot
|
2014-08-17 15:14
|
|
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: ac81e375391e199954d7316baecd60a6948fc927
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sun Aug 17 15:14:47 2014 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/ac81e375391e199954d7316baecd60a6948fc927 [^]
Fixes issue 27351: When calling the 'getGlobalComponentResources' method of the 'OBPOSComponentProvider' class, the array 'globalResources' will not be recreated if it is not necessary
- finals is a bonus
---
M src/org/openbravo/retail/posterminal/OBPOSComponentProvider.java
---
|
|
|
(0069429)
|
hgbot
|
2014-08-17 17:09
|
|
Repository: erp/pmods/org.openbravo.retail.returns
Changeset: fcc0100ea23be8e37f663bd23cfdf1c10968e69f
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sun Aug 17 17:09:48 2014 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/fcc0100ea23be8e37f663bd23cfdf1c10968e69f [^]
Fixes issue 27351: 'getGlobalComponentResources' have to be set as synchronized because multiple calls were corrupting the globalResources array
Marking variables as final was not necessary but a nice plus
---
M src/org/openbravo/retail/returns/ReturnsComponentProvider.java
---
|
|
|
(0069430)
|
hgbot
|
2014-08-17 17:10
|
|
Repository: erp/pmods/org.openbravo.retail.discounts
Changeset: 3dcc63c26236df1ff330fdcef3fe670058e796ba
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sun Aug 17 17:10:25 2014 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/3dcc63c26236df1ff330fdcef3fe670058e796ba [^]
Fixes issue 27351: 'getGlobalComponentResources' have to be set as synchronized because multiple calls were corrupting the globalResources array
Marking variables as final was not necessary but a nice plus
---
M src/org/openbravo/retail/discounts/DiscountsComponentProvider.java
---
|
|
|
(0069433)
|
hgbot
|
2014-08-17 17:35
|
|
|
|
(0069434)
|
hgbot
|
2014-08-17 17:37
|
|
|
|
(0069435)
|
hgbot
|
2014-08-17 17:37
|
|
|
|
(0069437)
|
hgbot
|
2014-08-17 18:01
|
|
Repository: erp/devel/pi
Changeset: 87b13b8ceac8a153efaaecae2cc6eb08a4203593
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sun Aug 17 18:01:11 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/87b13b8ceac8a153efaaecae2cc6eb08a4203593 [^]
Fixes issue 27351: 'getGlobalComponentResources' have to be set as synchronized because multiple calls were corrupting the globalResources array
Marking variables as final was not necessary but a nice plus
(client.kernel)
(service.datasource)
---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
---
|
|
|
|
|
|
(0069602)
|
mtaal
|
2014-08-22 10:28
|
|
I think the java code should be different:
- make the globalresources member private and place the declaration in the top of the class after the static members/methods
- initialize the globalresources with null, don't make it final
then make the getGlobalComponentResources synchronized and structure it like this:
public List<ComponentResource> getGlobalComponentResources() {
if (globalResources != null) {
return globalResources;
}
globalResources = new ArrayList<ComponentResource>();
gr. Martin |
|
|
(0069631)
|
hgbot
|
2014-08-23 01:34
|
|
Repository: erp/devel/pi
Changeset: daa69c127184d785356a2268620511c2ae0a0e56
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sat Aug 23 01:31:40 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/daa69c127184d785356a2268620511c2ae0a0e56 [^]
Fixes issue 27351: refactor (kernel, datasource)
---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
---
|
|
|
(0069632)
|
hgbot
|
2014-08-23 01:34
|
|
|
|
(0069633)
|
hgbot
|
2014-08-23 01:34
|
|
|
|
(0069634)
|
hgbot
|
2014-08-23 01:34
|
|
|
|
(0069635)
|
hgbot
|
2014-08-23 01:35
|
|
|
|
(0069636)
|
hgbot
|
2014-08-23 09:08
|
|
Repository: erp/devel/pi
Changeset: 077fca7b3c709f3447fe0fab9d0ad1a9d37b7c6b
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sat Aug 23 09:05:26 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/077fca7b3c709f3447fe0fab9d0ad1a9d37b7c6b [^]
Issue 27351. Backed out rev. 24307, changeset: daa69c127184
refactor (kernel, datasource)
may the changeset not pass the tests
---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
---
|
|
|
(0069638)
|
mtaal
|
2014-08-23 10:50
|
|
|
|
(0069639)
|
hgbot
|
2014-08-23 10:52
|
|
Repository: erp/devel/pi
Changeset: a2f9c55b5ca95422e3580ec97b08b1d10ff744ac
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Sat Aug 23 01:31:40 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/a2f9c55b5ca95422e3580ec97b08b1d10ff744ac [^]
Fixes issue 27351: refactor (kernel, datasource) (II)
---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
---
|
|
|
|
|
|
|
|
|
(0069655)
|
mtaal
|
2014-08-23 23:47
|
|
|
|
|
|