int-initial-pgsql - Build # 948 - Unstable:
Check console output at https://ci.openbravo.com/job/int-initial-pgsql/948/ to
view the results.
Committers since last success:
Changes for Build #948
RM packaging bot <[email protected]> null
Merge back from main
RM packaging bot <[email protected]> null
Merge temporary head for 3.0PR15Q2.1
RM packaging bot <[email protected]> null
Added signature for changeset fdb1e0cd936d
.hgsigs
RM packaging bot <[email protected]> null
Added tag 3.0PR15Q2.1 for changeset e861122b0b1d
.hgtags
RM packaging bot <[email protected]> null
Update AD_MODULE version to 3.0PR15Q2.1
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.base.weld/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.base.weld/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.application/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.application/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.htmlwidget/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.htmlwidget/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.kernel/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.kernel/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.myob/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.myob/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.querylist/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.querylist/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.widgets/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.widgets/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.financial.paymentreport/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.financial.paymentreport/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.reports.ordersawaitingdelivery/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.reports.ordersawaitingdelivery/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.datasource/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.datasource/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.integration.google/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.integration.google/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.integration.openid/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.integration.openid/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.json/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.json/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.userinterface.selector/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.userinterface.selector/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.userinterface.skin.250to300Comp/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.userinterface.skin.250to300Comp/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.userinterface.smartclient/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.userinterface.smartclient/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.utility.cleanup.log/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.utility.cleanup.log/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.v3.datasets/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.v3.datasets/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.v3.framework/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.v3.framework/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.v3/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.v3/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
src-db/database/sourcedata/AD_MODULE.xml
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29808: Impossible to create several calendars for the same
organization
Two pieces of code were affected by this bug:
PeriodEventHandler.java: EntityPersistenceEventObserver in charge of checking
overlap in manual inserts/updates (or any java process) in c_period table
C_YEARPERIODS: db function associated to the create periods process inside the
Fiscal Calendar | Year tab. It also verifies the periods don't overlap other
periods.
The fix consists in checking that there is no date overlap per calendar. Before
this fix the calendar wasn't taken into account, so it was not possible to
define several calendars for the same organization with the same periods.
src-db/database/model/functions/C_YEARPERIODS.xml
src/org/openbravo/event/PeriodEventHandler.java
Víctor Martínez Romanos <[email protected]> null
Related to issue 29808: Backout wrong changeset
The changeset pushed was not the good one, and failed in Oracle
src-db/database/model/functions/C_YEARPERIODS.xml
src/org/openbravo/event/PeriodEventHandler.java
Unai Martirena <[email protected]> null
Related to bug 29884: 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
src-db/database/model/functions/GL_JOURNAL_POST.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29884: 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
src-db/database/model/functions/GL_JOURNAL_POST.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29889 & Fixes issue 29887: Error in Price Correction Background
IsCostCalculated will not be considered to set CheckPriceDifference flag, when
completing an invoice.
Instead, when running Price Correction Background, transactions will be
filtered by IsCostCalculated to avoid calculate price differences in
transactions where cost has not been calculated.
src-db/database/model/functions/C_INVOICE_POST.xml
src/org/openbravo/costing/PriceDifferenceProcess.java
Eduardo Argal Guibert <[email protected]> null
Fixes issue 29899: 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.
src-util/buildvalidation/src/org/openbravo/buildvalidation/GLJournalAccountingCheck_data.xsql
Unai Martirena <[email protected]> null
Related to bu 29891: Adjust costing tests to the new behavior
src-test/src/org/openbravo/test/costing/TestCosting.java
Unai Martirena <[email protected]> null
Fixes bug 29891: Total Movement qty fixed in costing tab with backdated trx
While working with cost adjustments, on certain cases the existing Costing
records changes. This can happen because the cost has been recalculated due to
backdated transactions, price adjustments, manual cost corrections, etc. In all
this cases the 'Total Movement Quantity' field was not being correctly updated.
This field has to store the current stock of the product on that moment. So,
each time the costing record is updated it is being checked if this value
changes, and if it has changed the current stock is set.
src/org/openbravo/costing/AverageCostAdjustment.java
Augusto Mauch <[email protected]> null
Fixes bug 29838: Prevents unlimited datasource requests when filtering the
grid
The problem was that the logic to check that if a datasource request was
triggered by scrolling up in a grid that did not have its initial rows loaded
(see [1]) did not work well when the user filtered the grid after having
scrolled down the grid till loading its second page. This caused the totalRows
property of the grid to be miscalculated, and this triggered the undefinite
amount of datasource requests.
The logic to check if the request was triggered by scrolling up has been
changed. Now, instead of checking low-level smartclient properties like
lastScrollTop, we check if there are rows loaded in the positions just after
the page that was just received. Only in that case we want to prevent resetting
the totalRows property of the ResultSet with the value returned by the
datasource.
[1]
https://code.openbravo.com/erp/devel/pi/rev/c51dce7e9fd3c47915464ab4f565a9d1cee60e3b
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29827: Open/Close Period Control shows calendars associated to
organizations
The Open/Close Period Control window was showing all the periods available at
the C_Period table.
The target of this window is to open/close periods in fiscal calendars, so it
should only show periods belonging to a fiscal calendar linked to an
organization set as ready.
The fix has 2 parts:
1. Added a hql where clause to include only periods belonging to a fiscal
calendar linked to an organization set as ready.
2. The hql filter clause was wrong because it was filtering by the
c_period.ad_org_id. Instead it should be filtering by the organization linked
to the calendar's periods
src-db/database/sourcedata/AD_TAB.xml
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29808: Impossible to create several calendars for the same
organization
Two pieces of code were affected by this bug:
PeriodEventHandler.java: EntityPersistenceEventObserver in charge of checking
overlap in manual inserts/updates (or any java process) in c_period table
C_YEARPERIODS: db function associated to the create periods process inside the
Fiscal Calendar | Year tab. It also verifies the periods don't overlap other
periods.
The fix consists in checking that there is no date overlap per calendar. Before
this fix the calendar wasn't taken into account, so it was not possible to
define several calendars for the same organization with the same periods.
src-db/database/model/functions/C_YEARPERIODS.xml
src/org/openbravo/event/PeriodEventHandler.java
Unai Martirena <[email protected]> null
Fixes bug 29836: Avoid creating unnecessary backdated adjustments.
The problem was happening when costing precision was different from standard
precision. When calculating the expected cost of a transaction to know if an
adjustment is necessary to be created, it has to be rounded to standard
precision because the amounts always have to be created in standard precision.
In this case, while comparing expected cost with actual cost from database, as
in database values are rounded to standard precision, and expected cost was
being rounded to cost precission, an adjustment was being created with the
difference, that corresponds just to the precision lost, and finally an
adjustment of Zero amount was being created.
src/org/openbravo/costing/AverageCostAdjustment.java
Sandra Huguet <[email protected]> null
Fixed bug 29780 The done button appears disabled when it should not
null parameter when it should be the view in recalcDisplayLogicOrReadOnlyLogic
call in updateDifference function
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addPayment.js
RM packaging bot <[email protected]> null
Merge temporary head for 3.0PR15Q1.4
RM packaging bot <[email protected]> null
Added signature for changeset ac3537eef819
.hgsigs
RM packaging bot <[email protected]> null
Added tag 3.0PR15Q1.4 for changeset d5ec99ff8e8e
.hgtags
RM packaging bot <[email protected]> null
Update AD_MODULE version to 3.0PR15Q1.4
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.base.weld/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.base.weld/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.application/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.application/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.htmlwidget/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.htmlwidget/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.kernel/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.kernel/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.myob/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.myob/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.querylist/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.querylist/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.client.widgets/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.client.widgets/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.financial.paymentreport/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.financial.paymentreport/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.reports.ordersawaitingdelivery/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.reports.ordersawaitingdelivery/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.datasource/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.datasource/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.integration.google/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.integration.google/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.integration.openid/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.integration.openid/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.service.json/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.service.json/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.userinterface.selector/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.userinterface.selector/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.userinterface.skin.250to300Comp/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.userinterface.skin.250to300Comp/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.userinterface.smartclient/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.userinterface.smartclient/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.v3.datasets/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.v3.datasets/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.v3.framework/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.v3.framework/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
modules/org.openbravo.v3/src-db/database/sourcedata/AD_MODULE.xml
modules/org.openbravo.v3/src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
src-db/database/sourcedata/AD_MODULE.xml
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29809: Impossible to create several calendars for the same
organization
Two pieces of code were affected by this bug:
PeriodEventHandler.java: EntityPersistenceEventObserver in charge of checking
overlap in manual inserts/updates (or any java process) in c_period table
C_YEARPERIODS: db function associated to the create periods process inside the
Fiscal Calendar | Year tab. It also verifies the periods don't overlap other
periods.
The fix consists in checking that there is no date overlap per calendar. Before
this fix the calendar wasn't taken into account, so it was not possible to
define several calendars for the same organization with the same periods.
src-db/database/model/functions/C_YEARPERIODS.xml
src/org/openbravo/event/PeriodEventHandler.java
Víctor Martínez Romanos <[email protected]> null
Related to issue 29809: Backout wrong changeset
The changeset pushed was not the good one, and failed in Oracle
src-db/database/model/functions/C_YEARPERIODS.xml
src/org/openbravo/event/PeriodEventHandler.java
Unai Martirena <[email protected]> null
Related to bug 29885: 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
src-db/database/model/functions/GL_JOURNAL_POST.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29885: 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
src-db/database/model/functions/GL_JOURNAL_POST.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29890 & Fixes issue 29888: Error in Price Correction Background
IsCostCalculated will not be considered to set CheckPriceDifference flag, when
completing an invoice.
Instead, when running Price Correction Background, transactions will be
filtered by IsCostCalculated to avoid calculate price differences in
transactions where cost has not been calculated.
src-db/database/model/functions/C_INVOICE_POST.xml
src/org/openbravo/costing/PriceDifferenceProcess.java
Eduardo Argal Guibert <[email protected]> null
Fixes issue 29900: 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.
src-util/buildvalidation/src/org/openbravo/buildvalidation/GLJournalAccountingCheck_data.xsql
Jorge Garcia <[email protected]> null
Fixed issue 29776: Header of warehouse picking module do not refresh
Header of warehouse picking module do not refresh in some circunstances.
Issue of platform.
The problem was that the toolbar didn't update when the Manage Incidences
process finished.
The solution is to update the toolbar when the ActionHandler return the
responded JSON.
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-parameter-window-view.js
Unai Martirena <[email protected]> null
Fixes bug 29892: Total Movement qty fixed in costing tab with backdated trx
While working with cost adjustments, on certain cases the existing Costing
records changes. This can happen because the cost has been recalculated due to
backdated transactions, price adjustments, manual cost corrections, etc. In all
this cases the 'Total Movement Quantity' field was not being correctly updated.
This field has to store the current stock of the product on that moment. So,
each time the costing record is updated it is being checked if this value
changes, and if it has changed the current stock is set.
src/org/openbravo/costing/AverageCostAdjustment.java
Alvaro Ferraz <[email protected]> null
Fixes issue 29873: Wrong accounts when posting multi-general ledger G/L
Journal
When getting accounts from GL Item (while posting a Simple G/L Journal setted
as multi-general ledger), they were setted wrongly to accounting lines
src/org/openbravo/erpCommon/ad_forms/DocLine_GLJournal.java
Alvaro Ferraz <[email protected]> null
Fixes issue 29876: Avoid error when posting a multi-general ledger G/L
Journal
General Ledger will be taken from G/L Item instead from Simple G/L Journal when
it is marked as multi-general ledger, because in this case General Ledger from
Simple G/L Journal will be empty
src/org/openbravo/erpCommon/ad_forms/DocGLJournal_data.xsql
Augusto Mauch <[email protected]> null
Fixes bug 29839: Prevents unlimited datasource requests when filtering the
grid
The problem was that the logic to check that if a datasource request was
triggered by scrolling up in a grid that did not have its initial rows loaded
(see [1]) did not work well when the user filtered the grid after having
scrolled down the grid till loading its second page. This caused the totalRows
property of the grid to be miscalculated, and this triggered the undefinite
amount of datasource requests.
The logic to check if the request was triggered by scrolling up has been
changed. Now, instead of checking low-level smartclient properties like
lastScrollTop, we check if there are rows loaded in the positions just after
the page that was just received. Only in that case we want to prevent resetting
the totalRows property of the ResultSet with the value returned by the
datasource.
[1]
https://code.openbravo.com/erp/devel/pi/rev/c51dce7e9fd3c47915464ab4f565a9d1cee60e3b
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29823: Open/Close Period Control shows calendars associated to
organizations
The Open/Close Period Control window was showing all the periods available at
the C_Period table.
The target of this window is to open/close periods in fiscal calendars, so it
should only show periods belonging to a fiscal calendar linked to an
organization set as ready.
The fix has 2 parts:
1. Added a hql where clause to include only periods belonging to a fiscal
calendar linked to an organization set as ready.
2. The hql filter clause was wrong because it was filtering by the
c_period.ad_org_id. Instead it should be filtering by the organization linked
to the calendar's periods
src-db/database/sourcedata/AD_TAB.xml
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29809: Impossible to create several calendars for the same
organization
Two pieces of code were affected by this bug:
PeriodEventHandler.java: EntityPersistenceEventObserver in charge of checking
overlap in manual inserts/updates (or any java process) in c_period table
C_YEARPERIODS: db function associated to the create periods process inside the
Fiscal Calendar | Year tab. It also verifies the periods don't overlap other
periods.
The fix consists in checking that there is no date overlap per calendar. Before
this fix the calendar wasn't taken into account, so it was not possible to
define several calendars for the same organization with the same periods.
src-db/database/model/functions/C_YEARPERIODS.xml
src/org/openbravo/event/PeriodEventHandler.java
Unai Martirena <[email protected]> null
Fixes bug 29837: Avoid creating unnecessary backdated adjustments.
The problem was happening when costing precision was different from standard
precision. When calculating the expected cost of a transaction to know if an
adjustment is necessary to be created, it has to be rounded to standard
precision because the amounts always have to be created in standard precision.
In this case, while comparing expected cost with actual cost from database, as
in database values are rounded to standard precision, and expected cost was
being rounded to cost precission, an adjustment was being created with the
difference, that corresponds just to the precision lost, and finally an
adjustment of Zero amount was being created.
src/org/openbravo/costing/AverageCostAdjustment.java
Sandra Huguet <[email protected]> null
related to issue 28971, related to issue 29781
handleButtonsStatus function does not exist in q1
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addPayment.js
Sandra Huguet <[email protected]> null
related to issue 29781 apply js beautifier
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addPayment.js
Sandra Huguet <[email protected]> null
Fixed bug 29781 The done button appears disabled when it should not
null parameter when it should be the view in recalcDisplayLogicOrReadOnlyLogic
call in updateDifference function
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addPayment.js
Inigo Sanchez <[email protected]> null
Fixed bug 29760:Problems with inherited permissions in process definition
The problem was that when a process containing parameters defined as "window" is
launched , this manual role has not access to windows contained in that process
definition.
The cause of this issue is that before 14Q3, no security check was done on P&E
grids, so data always was retrieved.From 15Q1, security is checked requiring
explicit access to P&E grid.
The issue is fixed by inheriting access from the process, this is if the process
is accessible the grid within the P&E doesn't require to have explicit access
but inherits from the process itself.
modules/org.openbravo.client.application/src/org/openbravo/client/application/process/BaseProcessActionHandler.java
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceServlet.java
Unai Martirena <[email protected]> null
Fixes bug 29761: getCallableResult broken when using null parameters.
Removed TEST value for default
src-core/src/org/openbravo/database/RDBMSIndependent.java
Alvaro Ferraz <[email protected]> null
Fixes issue 29736: Wrong accounting in Cost Adjusment
Wrong accounting of the cost adjusment lines in case of Goods Movement From.
In this case, accounting should consider negative amounts, in order to have a
net effect of the goods movement adjustment accounting as 0.
src/org/openbravo/erpCommon/ad_forms/DocCostAdjustment.java
Eduardo Argal Guibert <[email protected]> null
Fixes issue 29727: Pressing Cancel in Match Statement or in Process
Reconciliation does not trigger a refresh.
Cancel button has been removed as it was considered confusing. Now there are
two buttons, one for closing the window called 'OK', as it was before, and
another one called 'Reconcile'.
Both triggers refresh. Messages have been added to improve user understanding
on how the window works. New message to inform the user any change will be
persisted. This message can be disabled once clear and will not popup again.
Message to run algorithm as well improved.
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/MatchStatementOnLoadGetPreferenceActionHandler.java
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/MatchStatementOnLoadPreferenceActionHandler.java
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_ELEMENT.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MESSAGE.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_REFERENCE.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_REF_LIST.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/MatchStatementActionHandler.java
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-matchStatement.js
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-parameter-window-view.js
Alvaro Ferraz <[email protected]> null
Fixes issue 29722: Payment navigation from Transaction tab is working wrong
Issotrx auxiliary input query in Transaction tab has been changed to get into
account related payment isreceipt field, when navigating from it to Payment
In/Out window
Also, SE_Payment_Transaction and SE_Trxtype_Transaction callouts have been
changed to overwrite issotrx session value when updating payment or
transactiontype fields
src-db/database/sourcedata/AD_AUXILIARINPUT.xml
src/org/openbravo/erpCommon/ad_callouts/SE_Payment_Transaction.java
src/org/openbravo/erpCommon/ad_callouts/SE_Trxtype_Transaction.java
David Baz Fayos <[email protected]> null
Fixed issue 29692: DateTime now works ok with 12:XX:XX times
The root of the problem was that there was a 'typo' and the
'minus' sign was missing in the 'if' statement in charge
of evaluate if it was in 24h mode or not. This was used later
to substract 12h to the entered date if this was in the
12:XX:XX form, change that obviously doesn't apply in the
24h mode
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/utilities/ob-utilities-date.js
Jorge Garcia <[email protected]> null
Fixed issue 29705: Description field is set with "null" in the FA
Transaction
Description field is filled in with "null" in the FA Transaction when matched
to GL Item.
The problem was in the Match Statement process, more specifically, in the
Add new transaction button. The AddTransanctionActionHandler didn't
take in consideration that the received description could be null.
The fix for this issue is to check if the received description is null
or blank.
Now, the description field fill in the correct description.
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddTransactionActionHandler.java
Unai Martirena <[email protected]> null
Fixes bug 29678: Fix NPE in Costing Background with Standard Costing Rule
In Cost Adjustment project new method has been added to calculate an opening
inventory cost. This method uses 'CostingUtils.getStandardCostDefinition'
method, that tries to get an standard cost for that opening inventory. If it
does not found any standard or legacy cost, it returns null. So, in this moment
a NPE error was happening. To fix this problem, at this point a controlled
error is raised to notify the user that no cost has been found.
src/org/openbravo/costing/StandardAlgorithm.java
Víctor Martínez Romanos <[email protected]> null
Related to issue 29645: changed column in selector field
to get the right name, description and help for the selector's column
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/OBUISEL_SELECTOR_FIELD.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29645: BP selector in AddTransaction shows all bp
"Business Partner selector" selector of "Business Partner" paramenter in
"AddTransaction" process definition, has been changed into "Business Partner
selector without default expressions" selector; in order to show all business
partners in the dropdown of the selector.
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_REFERENCE.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/OBUISEL_SELECTOR.xml
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/OBUISEL_SELECTOR_FIELD.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29660: GLItem lost in Add new Transaction window
Neither payment nor glitem will be deleted when changing to Bank fee in Add new
Transaction window. Instead, when clicking on Done and creating the
transaction, AddTransactionActionHandler will check if the transaction is of
type Bank fee and in this case it will set as null both fields.
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddTransactionActionHandler.java
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addTransaction.js
Víctor Martínez Romanos <[email protected]> null
Fixed bug 29655: Payment selector now filters by child organizations
The payment selector was incorrectly filtering by child organizations because
the order of the parameters in the call to the AD_ISORGINCLUDED function was
wrong.
It now gets all the payments belonging to the organization and any child
organization of the tab's organization.
src-db/database/sourcedata/OBUISEL_SELECTOR.xml
Alvaro Ferraz <[email protected]> null
Fixes issue 29634: Foreign currency not setted in transaction
When creating a transaction manually from a payment, foreign currency, foreign
conversion rate and foreign amount fields were not setted to transaction. Now,
they will be setted as in TransactionDao.java, when the transaction is
automatically created from the payment.
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_TransactionProcess.java
Unai Martirena <[email protected]> null
Fixes bug 29620: Correctly used credit when paying an invoice and refunding
There were two issues while managing this scenario:
[1] In 'addCredit' method inside AddPaymentActionHandler.java class, a loop of
Credit to Use grid is done, creating Payment Credit Used records assigned to
the new created payment of the invoice. This method was not taking into account
that could be credit amount that exceeds the invoice amount (amount to be
refunded) and it was assigning all the credit amount as used to the payment of
the invoice. So the refund payment that is created right after was been left
without any used credit payment.
[2] After fixing the previous method, the second issue appears. The
'updateUsedCredit' method inside FIN_PaymentProcess.java class retrieves all
the payments of the business partner that has generated credit, and it assigns
as used to the refund payment. The problem with this function is that before
the Add Payment refactor it was not possible to select which credit payments we
wanted to consume, it was only a total credit amount to be consumed and the
system behind was being consuming any of them. But after the refactor it is
possible to select the ones that want to be consumed.
In order to fix this last issue the selected credit line ids are added as a
parameter to 'updateUsedCredit' method.
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddPaymentActionHandler.java
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
Inigo Sanchez <[email protected]> null
Fixed bug 29629: setup application is not taking into account the context
name
The setup application is not taking into account the context name that users
are configuring. The values of "deploy-name" and "context-root" properties
of org.eclipse.wst.common.component are not updating.
Now it has been solved and if user set "context-name" in setup application.
this value is updating in org.eclipse.wst.common.component. For example if
a user set the context name as "test" the URL to go to Openbravo is
"http://localhost:8080/test" instead of default one
"http://localhost:8080/openbravo".
src/org/openbravo/configuration/ConfigurationApp.java
Asier Lostalé <[email protected]> null
related to bug 29902: added test case
src-test/src/org/openbravo/test/scheduling/ProcessSchedulingTest.java
src-test/src/org/openbravo/test/AllAntTaskTests.java
Asier Lostalé <[email protected]> null
fixed bug 29902: ProcessBundle.setCloseConnection(false) closes connection
When a process is invoked through ProcessRunner and its bundle is set as
setCloseConnection(false) there were 2 problems:
- connection was close after execution so it could not be reused by another
process invoking it
- ProcessRunner tried to update the process run status on a closed
connection
causing an error
The problem is caused by the fix for issue #27878 which correctly closes the
connection in DalConnectionProvider when invoking releaseCommitConnection. But
when setCloseConnection it relied in that bug not closing the connection to
leave it open.
The solution consists in doing directly the commit or rollback in
DalBaseProcess
without closing the connection.
src/org/openbravo/service/db/DalBaseProcess.java
Last 20 lines of the console output:
[...truncated 7355 lines...]
DEBUG: Tomcat stop called with parameters: ENABLED=ENABLE
* Stopping Tomcat servlet engine for Openbravo tomcat
...done.
executing script 'Check log'
[int-initial-pgsql] $ /bin/bash /tmp/build_step_template2036718558167492808.sh
ENABLE
DEBUG: Check log called with parameters: ENABLED=
Errors in openbravo.log:
b3e0edee 2015-05-21 16:11:55,757 [DefaultQuartzScheduler_Worker-2] ERROR
org.openbravo.erpCommon.ad_forms.AcctServer - Exception in AcctServer.run
Recording test results
Archiving artifacts
Checking \] ERROR|\] WARN
/srv/ci/workspace/int-initial-pgsql/SANDBOX/openbravo.log:
b3e0edee 2015-05-21 16:11:55,757 [DefaultQuartzScheduler_Worker-2] ERROR
org.openbravo.erpCommon.ad_forms.AcctServer - Exception in AcctServer.run
Build step 'Jenkins Text Finder' changed build result to UNSTABLE
Email was triggered for: Unstable
Sending email for trigger: Unstable
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
Openbravo-builds mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-builds