int-upgrade-pgsql - Build # 2472 - Still Failing:

Check console output at https://ci.openbravo.com/job/int-upgrade-pgsql/2472/ to 
view the results.


Committers since last success:

Changes for Build #2468

    David Miguelez <david.migue...@openbravo.com> null
    Related to Issue 38288. Code review changes

* Moved validation of Payment Plan before performing any logic
* Changed logic to retrieve messages, it now usses API from OBMessageUtils
        
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java

    Armaignac <collazoan...@gmail.com> null
    Fixes issue 38288: Sales Invoice Payment Plan Due date can be modified when
related payment is in Remitted status

Remittances only accept documents with due date prior to their expected date,
the due date of a payment plan can be increased because the process doesn't
properly check the that condition

A new hook is created to check the edited payment schedule lines and avoid 
validation
issues
        
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ModifyPaymentPlanHook.java
        
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanHookCaller.java
        
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ModifyPaymentPlanActionHandler.java

Changes for Build #2469

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: updated modulescript version limit
        
src-util/modulescript/build/classes/org/openbravo/modulescript/UpdatePersistOrgInfo.class
        
src-util/modulescript/src/org/openbravo/modulescript/UpdatePersistOrgInfo.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: code review improvements
Improvements in JUnit tests:
 Avoid duplication of code
 Remove useless javadoc or comments
 Reorganize methods
 Remove useless log
 Improve asserts
 Renamed some classes
 Reduced classes and methods visibility where possible
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoCornerCaseOrgTest.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoTestSuite.java
        src-test/src/org/openbravo/test/AllAntTaskTests.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoConstants.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoSetReadyTest.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoUtility.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistOrgInfoComplexOrgTreeTest.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoCornerCaseOrg.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: code review improvements
Updated sampledata for QA Testing client
        referencedata/sampledata/QA_Testing/AD_ORG.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: code review improvements
Properly format modulescript
Removed info message in module script
Increased limits
Ensure we don't work with * org and any organization where the persist org is 
already available
        
src-util/modulescript/build/classes/org/openbravo/modulescript/UpdatePersistOrgInfo.class
        
src-util/modulescript/build/classes/org/openbravo/modulescript/UpdatePersistOrgInfoData.class
        
src-util/modulescript/src/org/openbravo/modulescript/UpdatePersistOrgInfo.java
        
src-util/modulescript/src/org/openbravo/modulescript/UpdatePersistOrgInfo_data.xsql

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: code review improvements
Renamed field group to Inherited Information
Reordered fields
        src-db/database/sourcedata/AD_FIELD.xml
        src-db/database/sourcedata/AD_FIELDGROUP.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: code review improvements
Improved functions documentation and comments
        src-db/database/model/functions/AD_GET_ORG_LE_BU.xml
        src-db/database/model/functions/AD_ORG_GETCALENDAROWNER.xml
        src-db/database/model/functions/AD_ORG_GETPERIODCONTROLALLOW.xml
        src-db/database/model/functions/AD_ORG_READY.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Related to issue 33650: code review improvements
Removed new reference Organization Selector for Persist Information and use 
instead existing Organization Simple Selector
        src-db/database/sourcedata/AD_COLUMN.xml
        src-db/database/sourcedata/AD_REFERENCE.xml
        src-db/database/sourcedata/OBUISEL_SELECTOR.xml

    Atul Gaware <atul.gaw...@openbravo.com> null
    Related to issue 33650:Make use of persist org info column

Use ad_inheritedcalendar_id
        src-db/database/model/functions/C_GET_DEFAULT_PERIOD.xml
        src-db/database/model/functions/C_INVOICE_POST.xml
        src-db/database/model/functions/C_TAXPAYMENT_POST.xml
        src-db/database/model/triggers/C_PERIOD_TRG.xml
        src-db/database/sourcedata/AD_VAL_RULE.xml
        
src/org/openbravo/erpCommon/ad_reports/ReportTrialBalanceDetail_data.xsql

    Atul Gaware <atul.gaw...@openbravo.com> null
    Related to issue 33650:Sample data updated with persist org info

Existing organization are updated with persist org info as part
of sample data.
        referencedata/sampledata/F_B_International_Group/AD_ORG.xml

    Atul Gaware <atul.gaw...@openbravo.com> null
    Related to issue 33650:Added junit test cases for persist org info task

** Added junit tests to verify persist info data in organization
for corner case orgs (*, null, org that does not exists)
** Added junit tests which creates a complex org tree, sets org as ready
with cascade Yes and No option and verifies persist org info.
** Added junit tests which create single org, two orgs one under another,
sets org as ready with cascade = Yes, Cascade = No option.
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoConstants.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoCornerCaseOrg.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoSetReadyTest.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoTest.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistInfoUtility.java
        
src-test/src/org/openbravo/test/generalsetup/enterprise/organization/ADOrgPersistOrgInfoComplexOrgTreeTest.java
        src-test/src/org/openbravo/test/AllAntTaskTests.java

    Atul Gaware <atul.gaw...@openbravo.com> null
    Related to issue 33650:Modulescript to set persist org info

** Added Modulescript to populate AD_Org persist info columns
** Variables are used to get persist org info and then sent
to update method to avoid trigger mutating error in orcl
instance
        
src-util/modulescript/build/classes/org/openbravo/modulescript/UpdatePersistOrgInfo.class
        
src-util/modulescript/build/classes/org/openbravo/modulescript/UpdatePersistOrgInfoData.class
        
src-util/modulescript/src/org/openbravo/modulescript/UpdatePersistOrgInfo.java
        
src-util/modulescript/src/org/openbravo/modulescript/UpdatePersistOrgInfo_data.xsql

    Atul Gaware <atul.gaw...@openbravo.com> null
    Fixes Issue 33650:Add columns in AD_Org table to persist data in order
to improve performance

** Add columns to store calendar owner org, period control allowed org,
legal entity org, business unit org and calendar of the period
control allowed org of the organization, Add field group and ready only
fields in organization window tab to show persist org information.
Field group is collapse by default.
** Added new function AD_ORG_GETCALENDAROWNERTN and
AD_ORG_GETPERIODCONTROLALLOWTN, AD_GET_ORG_LE_BU_TREENODE
from the original one with performance issues notes in it to only use
when organization are not set ready else use persist info from AD_Org
directly.
** Refactor AD_ORG_GETCALENDAROWNER, AD_ORG_GETPERIODCONTROLALLOW and
AD_GET_ORG_LE_BU to make use of respective new columns to use
AD_CALENDAROWNER_ORG_ID, AD_PERIODCONTROLALLOWED_ORG_ID,
AD_LEGALENTITY_ORG_ID or AD_BUSINESSUNIT_ORG_ID from AD_ORG directly.
** Modified AD_ORG_READY to set persist info columns in AD_ORG.
** To avoid triggers mutating error in oracle instance, instead of
calling directly functions to get persist org info in
update query, variables are used to get persist org info from ad_org
and then ad_org is updated.
** Modified AD_GET_ORG_LE_BU_TREENODE, remove condition
v_org_header_id<>'0' from while clause in p_type='BU' block.
        src-db/database/model/functions/AD_GET_ORG_LE_BU_TREENODE.xml
        src-db/database/model/functions/AD_ORG_GETCALENDAROWNERTN.xml
        src-db/database/model/functions/AD_ORG_GETPERIODCONTROLALLOWTN.xml
        src-db/database/model/functions/AD_GET_ORG_LE_BU.xml
        src-db/database/model/functions/AD_ORG_GETCALENDAROWNER.xml
        src-db/database/model/functions/AD_ORG_GETPERIODCONTROLALLOW.xml
        src-db/database/model/functions/AD_ORG_READY.xml
        src-db/database/model/tables/AD_ORG.xml
        src-db/database/sourcedata/AD_COLUMN.xml
        src-db/database/sourcedata/AD_ELEMENT.xml
        src-db/database/sourcedata/AD_FIELD.xml
        src-db/database/sourcedata/AD_FIELDGROUP.xml
        src-db/database/sourcedata/AD_REFERENCE.xml
        src-db/database/sourcedata/OBUISEL_SELECTOR.xml

Changes for Build #2470

    Armaignac <collazoan...@gmail.com> null
    Fixes issue 38302:Tax tab info is not correctly recalculated when negative 
child
tax defined as "cascade" is modified

When an orderlinetax is removed and the line order it's not take into account
the triggers calculations can be incorrect.

Now the orderlinetax are removed in the proper order (lineno asc) to avoid
wrong calculations in the triggers.
        src-db/database/model/triggers/C_ORDERLINE_TRG2.xml

Changes for Build #2471

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    Fixed issue 38350: Unmark in development modules in production instances
      
    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ndmp] Minor code improvements
        
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelApplicationInitializer.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Code review fixes.

- Fixed java file formatting.
        
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelApplicationInitializer.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Code review fixes.

- Improved comment in trigger
- Added a failsafe action in ActiveInstanceProcess when ADCS is not available
 (ant activate.instance)
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/window/ApplicationDictionaryCachedStructures.java
        
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelApplicationInitializer.java
        src-db/database/model/triggers/AD_MODULE_TRG.xml
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Code review fixes.

- Changed error message
- Java format fixes
- Change public API method names
- Checking if there are modules as "In Development" in the initializer
- Moving initializer logic to KernelApplicationInitializer
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/window/ApplicationDictionaryCachedStructures.java
        
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelApplicationInitializer.java
        src-db/database/sourcedata/AD_MESSAGE.xml
        src/org/openbravo/erpCommon/ad_forms/InstancePurpose.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ProductionModuleInitializer.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Merge with pi
      
    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Simplified ADCS logic
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ProductionModuleInitializer.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/window/ApplicationDictionaryCachedStructures.java
        src/org/openbravo/erpCommon/ad_forms/InstancePurpose.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Fixed trigger for Oracle
        src-db/database/model/triggers/AD_MODULE_TRG.xml

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Moved logic to ADCS and merged trigger with an existing one
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ProductionModuleInitializer.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/window/ApplicationDictionaryCachedStructures.java
        src-db/database/model/triggers/AD_MODULE_TRG.xml
        src/org/openbravo/erpCommon/ad_forms/InstancePurpose.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ModuleDevelopmentStatusHelper.java
        src-db/database/model/triggers/AD_MODULE_NODEV_IN_PROD_TRG.xml

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] InstancePurpose process requires to receive the purpose as parameter.
        src/org/openbravo/erpCommon/ad_forms/InstancePurpose.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Refactored repeated code
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ModuleDevelopmentStatusHelper.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ProductionModuleInitializer.java
        src/org/openbravo/erpCommon/ad_forms/InstancePurpose.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Created an initializer that checks purpose and modules at 
initialization
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ProductionModuleInitializer.java
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Created a trigger that ensures modules cannot be set as in 
development.
        src-db/database/model/triggers/AD_MODULE_NODEV_IN_PROD_TRG.xml
        src-db/database/sourcedata/AD_MESSAGE.xml

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] merge with pi
      
    Javier Armendáriz <javier.armenda...@openbravo.com> null
    [ndmp] Removing development flag in modules when registering for production.
        src/org/openbravo/erpCommon/obps/ActiveInstanceProcess.java

Changes for Build #2472

    Armaignac <collazoan...@gmail.com> null
    Fixes issue 38316:Warehouse in context (session) is different from the 
warehouse
on the login in some circumstances, could affect tax selection

Under some circumstances the warehouse in the session is different from the 
warehouse
on the login that could affect tax selection

Now in case there does not exist any warehouse defined for the specific 
organization
one is selected from the organization natural tree
        src/org/openbravo/base/secureApp/LoginUtils.java




Last 20 lines of the console output:

[...truncated 2537 lines...]
+ (( i=0 ))
+ (( i<12 ))
++ jps
++ grep Bootstrap
++ true
+ '[' '' = '' ']'
+ break
++ jps
++ grep Bootstrap
++ true
+ '[' '' '!=' '' ']'
+ true
+ '[' 0 '!=' 0 ']'
+ echo

POST BUILD TASK : SUCCESS
END OF POST BUILD TASK : 0
Email was triggered for: Failure - Any
Sending email for trigger: Failure - Any
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openbravo-builds mailing list
Openbravo-builds@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-builds

Reply via email to