[Openbravo-builds] int-api - Build # 3951 - Failure!

3 views
Skip to first unread message

jen...@ci.openbravo.com

unread,
May 14, 2019, 10:52:15 AM5/14/19
to atul....@openbravo.com, asier....@openbravo.com, carlos...@openbravo.com, openbrav...@lists.sourceforge.net, staf...@openbravo.com, staf...@openbravo.com
int-api - Build # 3951 - Failure:

Check console output at https://ci.openbravo.com/job/int-api/3951/ to view the results.


Committers since last success:

Changes for Build #3951

Carlos Aristu <carlos...@openbravo.com> null
fixes issue 39034: Remove xstream-1.3.1 library

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Avoid potential NPE
src/org/openbravo/scheduling/ProcessContext.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Fix test, JSONObject type is now supported
src-test/src/org/openbravo/test/system/JSONSerialization.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Support parameters of JSONObject type
src/org/openbravo/scheduling/ParameterSerializer.java
src/org/openbravo/scheduling/ProcessBundle.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Do not catch exception, throwing it instead
src-test/src/org/openbravo/test/system/JSONSerialization.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Remove this
src/org/openbravo/scheduling/ProcessContext.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Log exception
src/org/openbravo/scheduling/ProcessContext.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Do not need to declare method as synchronized
src/org/openbravo/scheduling/ProcessContext.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Log improvements
src/org/openbravo/scheduling/OBScheduler.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Do not need ProcessBundleParameter
src/org/openbravo/scheduling/ParameterSerializer.java
src/org/openbravo/scheduling/ProcessBundleParameter.java

Asier Lostalé <asier....@openbravo.com> null
[remove-xstream] no need to create temporary ProcessBundleParam instances
src/org/openbravo/scheduling/ParameterSerializer.java
src/org/openbravo/scheduling/ProcessBundleParameter.java

Asier Lostalé <asier....@openbravo.com> null
[remove-xstream] static instance initialization
src/org/openbravo/scheduling/ParameterSerializer.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Minor code improvements

- Use diamond operator
- Use Integer.toString(int)
- Remove unused code
src/org/openbravo/erpCommon/ad_process/ApplyModulesCallServlet.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Code improvements in ApplyModulesResponse

- Reduced the visibility of the class
- Removed unused methods
src/org/openbravo/erpCommon/ad_process/ApplyModulesCallServlet.java
src/org/openbravo/erpCommon/ad_process/ApplyModulesResponse.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Correct serialization of errors and warnings arrays
src/org/openbravo/erpCommon/ad_process/ApplyModules.html
src/org/openbravo/erpCommon/ad_process/ApplyModulesResponse.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Merge with pi

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Remove xstream library
legal/Licensing.txt
lib/runtime/xstream-1.3.1.jar

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] ApplyModulesResponse.toJSON() method

Use ApplyModulesResponse.toJSON() and OBError.toJSON() in ApplyModulesCallServlet to replace the usage of xstream
src/org/openbravo/erpCommon/ad_process/ApplyModulesCallServlet.java
src/org/openbravo/erpCommon/ad_process/ApplyModulesResponse.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Minor code improvements

- Improve setters code
- Use diamond operator
src/org/openbravo/erpCommon/utility/OBError.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] OBError.toJSON() method
src-test/src/org/openbravo/test/system/JSONSerialization.java
src/org/openbravo/erpCommon/utility/OBError.java
src/org/openbravo/erpCommon/utility/WindowTree.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Minor code improvements

- Do not need to invoke toString() of "channel" String
- Extracted code into new "scheduleProcess" private method
- Remove unneed assignment for "data" variable
- Use faster indexOf(char) method
- Use diamond operator
src/org/openbravo/scheduling/OBScheduler.java
src/org/openbravo/scheduling/ProcessBundle.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Rename test class
src-test/src/org/openbravo/test/system/JSONSerialization.java
src-test/src/org/openbravo/test/AllAntTaskTests.java
src-test/src/org/openbravo/test/system/JsonSerialization.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] New logic to (de)serialize ProcessBundle parameters
src/org/openbravo/scheduling/ParameterSerializationException.java
src/org/openbravo/scheduling/ParameterSerializer.java
src/org/openbravo/scheduling/ProcessBundleParameter.java
src-test/src/org/openbravo/test/system/JsonSerialization.java
src/org/openbravo/scheduling/OBScheduler.java
src/org/openbravo/scheduling/ProcessBundle.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Fix test
src-test/src/org/openbravo/test/system/JsonSerialization.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Handle properties not defined
src/org/openbravo/scheduling/ProcessContext.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Add test cases into a suite
src-test/src/org/openbravo/test/AllAntTaskTests.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Fix class name
src-test/src/org/openbravo/test/system/JsonSerialization.java

Carlos Aristu <carlos...@openbravo.com> null
[remove-xstream] Use JSONObject to (de)serialize ProcessContext

- Changes not not make use of xstream library in ProcessContext
- Added test cases to cover Process context (de)serialization
src-test/src/org/openbravo/test/system/JsonSerialization.java
src/org/openbravo/scheduling/ProcessContext.java

Atul Gaware <atul....@openbravo.com> null
Fixes Issue 40865:Set VOLATILITY LEVEL for Database functions

** Set volatility level as IMMUTABLE for ad_isjavascriptword, ad_isjavaword,
ad_mapping_format, ad_module_checked, c_add_months2, c_datedayinmonth,
c_debt_payment_status, c_divide, c_get_isoweek_day, c_ignore_accent,
c_invoicetodate, c_trim, last_day, m_get_offer_price, m_get_offer_std_price,
m_get_offers_price, m_get_offers_std_price, m_warehouse_default_rule,
m_warehouse_default_rule_param

** Rest are set with volatility level as STABLE
modules/org.openbravo.advpaymentmngt/src-db/database/model/functions/APRM_ISPAYMENTCONFIRMED.xml
modules/org.openbravo.advpaymentmngt/src-db/database/model/functions/APRM_SEQNOINVPAIDSTATUS.xml
modules/org.openbravo.advpaymentmngt/src-db/database/model/functions/APRM_SEQNUMBERPAYMENTSTATUS.xml
src-db/database/model/functions/AD_COLUMN_IDENTIFIER.xml
src-db/database/model/functions/AD_COLUMN_IDENTIFIER_REF_SQL.xml
src-db/database/model/functions/AD_COLUMN_IDENTIFIER_SQL.xml
src-db/database/model/functions/AD_COLUMN_IDENTIFIER_STD.xml
src-db/database/model/functions/AD_DBA_USER_CONST_SEARCHCOND.xml
src-db/database/model/functions/AD_GET_DOCTYPE.xml
src-db/database/model/functions/AD_GET_DOCTYPE_EXPENSES.xml
src-db/database/model/functions/AD_GET_DOC_LE_BU.xml
src-db/database/model/functions/AD_GET_LEVEL_NODE_TREE.xml
src-db/database/model/functions/AD_GET_NODE_TREE_BYLEVEL.xml
src-db/database/model/functions/AD_GET_ORG_LE_BU.xml
src-db/database/model/functions/AD_GET_ORG_LE_BU_TREENODE.xml
src-db/database/model/functions/AD_GET_PREFERENCE_VALUE.xml
src-db/database/model/functions/AD_ISJAVASCRIPTWORD.xml
src-db/database/model/functions/AD_ISJAVAWORD.xml
src-db/database/model/functions/AD_ISMEMBERINCLUDED.xml
src-db/database/model/functions/AD_ISORGINCLUDED.xml
src-db/database/model/functions/AD_ISORGINCLUDED_TREENODE.xml
src-db/database/model/functions/AD_MAPPING_FORMAT.xml
src-db/database/model/functions/AD_MESSAGE_GET2.xml
src-db/database/model/functions/AD_MODULE_CHECKED.xml
src-db/database/model/functions/AD_MODULE_DEPENDENT.xml
src-db/database/model/functions/AD_MODULE_IDENTIFIER.xml
src-db/database/model/functions/AD_ORGTYPE_ISLE_ISBU.xml
src-db/database/model/functions/AD_ORGTYPE_ISTRANS_ALLOWED.xml
src-db/database/model/functions/AD_ORG_CHK_CALENDAR.xml
src-db/database/model/functions/AD_ORG_CHK_READY.xml
src-db/database/model/functions/AD_ORG_CHK_SCHEMAS.xml
src-db/database/model/functions/AD_ORG_GETCALENDAROWNER.xml
src-db/database/model/functions/AD_ORG_GETCALENDAROWNERTN.xml
src-db/database/model/functions/AD_ORG_GETPERIODCONTROLALLOW.xml
src-db/database/model/functions/AD_ORG_GETPERIODCONTROLALLOWTN.xml
src-db/database/model/functions/AD_ORG_ISINNATURALTREE.xml
src-db/database/model/functions/AD_PARENT_MENU_ELEMENT.xml
src-db/database/model/functions/AD_PARENT_TREE.xml
src-db/database/model/functions/C_ADD_MONTHS2.xml
src-db/database/model/functions/C_AGING_GET_SCOPE.xml
src-db/database/model/functions/C_BASE_CONVERT.xml
src-db/database/model/functions/C_CALCULATECOST_CATSALARY.xml
src-db/database/model/functions/C_CALCULATE_CATSALARY.xml
src-db/database/model/functions/C_CHK_OPEN_PERIOD.xml
src-db/database/model/functions/C_CHK_OPEN_PERIOD_GLJ.xml
src-db/database/model/functions/C_CONVERT_DOC_CURRENCY.xml
src-db/database/model/functions/C_CURRENCY_CONVERT.xml
src-db/database/model/functions/C_CURRENCY_CONVERT0.xml
src-db/database/model/functions/C_CURRENCY_CONVERT_PRECISION.xml
src-db/database/model/functions/C_CURRENCY_CONVERT_RATE.xml
src-db/database/model/functions/C_CURRENCY_ISOSYM.xml
src-db/database/model/functions/C_CURRENCY_RATE.xml
src-db/database/model/functions/C_CURRENCY_ROUND.xml
src-db/database/model/functions/C_CURRENCY_SYMBOL.xml
src-db/database/model/functions/C_DATEDAYINMONTH.xml
src-db/database/model/functions/C_DB_PERCENTAGE_PAID.xml
src-db/database/model/functions/C_DEBT_PAYMENT_PAID.xml
src-db/database/model/functions/C_DEBT_PAYMENT_STATUS.xml
src-db/database/model/functions/C_DEBT_PAYMENT_STATUS0.xml
src-db/database/model/functions/C_DIVIDE.xml
src-db/database/model/functions/C_GETBPLOCATIONID.xml
src-db/database/model/functions/C_GETINVOICELINE_TAXAMOUNT.xml
src-db/database/model/functions/C_GETINVOICESTATUSFROMSHIPMENT.xml
src-db/database/model/functions/C_GETTAX.xml
src-db/database/model/functions/C_GETWITHHOLDING.xml
src-db/database/model/functions/C_GET_CURRENCY_PRECISION.xml
src-db/database/model/functions/C_GET_DEFAULT_PERIOD.xml
src-db/database/model/functions/C_GET_IBAN_DISPLAYED_ACCOUNT.xml
src-db/database/model/functions/C_GET_ISOWEEK_DAY.xml
src-db/database/model/functions/C_GET_NET_AMOUNT_FROM_GROSS.xml
src-db/database/model/functions/C_GET_NET_PRICE_FROM_GROSS.xml
src-db/database/model/functions/C_GET_PRODUCT_TAXCATEGORY.xml
src-db/database/model/functions/C_GET_SERVICE_AMOUNT.xml
src-db/database/model/functions/C_GET_TAX_AMT_FROM_NET.xml
src-db/database/model/functions/C_IGNORE_ACCENT.xml
src-db/database/model/functions/C_INVOICELINE_TAX.xml
src-db/database/model/functions/C_INVOICETODATE.xml
src-db/database/model/functions/C_INVOICE_PERCENTAGE_PAID.xml
src-db/database/model/functions/C_LOCATION_DESCRIPTION.xml
src-db/database/model/functions/C_LOCATION_IDENTIFIER.xml
src-db/database/model/functions/C_LOCATION_NAME.xml
src-db/database/model/functions/C_ORDER_STATUS.xml
src-db/database/model/functions/C_PAYMENTDUEDATE.xml
src-db/database/model/functions/C_TAX_GET_LEVEL.xml
src-db/database/model/functions/C_TAX_GET_ROOT.xml
src-db/database/model/functions/C_TAX_ISMEMBER.xml
src-db/database/model/functions/C_TRIM.xml
src-db/database/model/functions/C_UOM_CONVERT.xml
src-db/database/model/functions/FIN_AGING_GETDOCNO.xml
src-db/database/model/functions/FIN_AGING_INVOICECURRENCY_RATE.xml
src-db/database/model/functions/FIN_AGING_ISDOUBTFULTDEBT.xml
src-db/database/model/functions/FIN_AGING_PAYMENTCURRENCY_RATE.xml
src-db/database/model/functions/MRP_CHECK_PLANNINGMETHOD.xml
src-db/database/model/functions/M_BOM_PRICELIMIT.xml
src-db/database/model/functions/M_BOM_PRICELIST.xml
src-db/database/model/functions/M_BOM_PRICESTD.xml
src-db/database/model/functions/M_BOM_QTY_AVAILABLE.xml
src-db/database/model/functions/M_BOM_QTY_ONHAND.xml
src-db/database/model/functions/M_BOM_QTY_ORDERED.xml
src-db/database/model/functions/M_BOM_QTY_RESERVED.xml
src-db/database/model/functions/M_CHECK_STOCK.xml
src-db/database/model/functions/M_GETJSONDESCRIPTION.xml
src-db/database/model/functions/M_GET_CONVERTED_AUMQTY.xml
src-db/database/model/functions/M_GET_CONVERTED_QTY.xml
src-db/database/model/functions/M_GET_COST.xml
src-db/database/model/functions/M_GET_NO_TRX_PRODUCT_COST.xml
src-db/database/model/functions/M_GET_OFFER.xml
src-db/database/model/functions/M_GET_OFFERS_NAME.xml
src-db/database/model/functions/M_GET_OFFERS_PRICE.xml
src-db/database/model/functions/M_GET_OFFERS_STD_PRICE.xml
src-db/database/model/functions/M_GET_OFFER_PRICE.xml
src-db/database/model/functions/M_GET_OFFER_STD_PRICE.xml
src-db/database/model/functions/M_GET_PARETO_ABC.xml
src-db/database/model/functions/M_GET_PARETO_ABC_LEGACY.xml
src-db/database/model/functions/M_GET_PRICELIST_VERSION.xml
src-db/database/model/functions/M_GET_PRODUCT_COST.xml
src-db/database/model/functions/M_GET_PRODUCT_PRICE.xml
src-db/database/model/functions/M_GET_PRODUCT_PRICE_DATE.xml
src-db/database/model/functions/M_GET_TRANSACTION_COST.xml
src-db/database/model/functions/M_GET_TRX_AND_PRODUCT_COST.xml
src-db/database/model/functions/M_ISPARENT_CH_VALUE.xml
src-db/database/model/functions/M_WAREHOUSE_DEFAULT_RULE.xml
src-db/database/model/functions/M_WAREHOUSE_DEFAULT_RULE_PARAM.xml
src-db/database/model/postscript-PostgreSql.sql
src-db/database/model/prescript-PostgreSql.sql




Last 20 lines of the console output:

[...truncated 516 lines...]

org.openbravo.erpCommon.ad_process:
Missing
class org.openbravo.erpCommon.ad_process.ApplyModulesResponse: missing in /srv/ci/workspace/int-full-pgsql/SANDBOX/api-checks/output/java
+ failure=1
+ exit 1
Build step 'Execute shell' marked build as failure
Performing Post build task...
Match found for : : True
Logical operation result is TRUE
Running script : cp /srv/ci/workspace/int-full-pgsql/SANDBOX/api-checks/output/java.japi.gz .
[int-api] $ /bin/bash -xe /tmp/hudson6949180339118771441.sh
+ cp /srv/ci/workspace/int-full-pgsql/SANDBOX/api-checks/output/java.japi.gz .
POST BUILD TASK : SUCCESS
END OF POST BUILD TASK : 0
Archiving artifacts
Checking console output
Email was triggered for: Failure - Any
Sending email for trigger: Failure - Any

jen...@ci.openbravo.com

unread,
May 16, 2019, 6:47:18 AM5/16/19
to asier....@openbravo.com, openbrav...@lists.sourceforge.net, staf...@openbravo.com, staf...@openbravo.com
int-api - Build # 3952 - Fixed:

Check console output at https://ci.openbravo.com/job/int-api/3952/ to view the results.
Changes for Build #3952

Asier Lostalé <asier....@openbravo.com> null
fixed bug 40886: incorrect audit trail for DAL WebServices

DAL WebServices were incorrectly audited because ad_context_info was not set as
it was disabled for the thread during login and they are executed in the same
thread.

This fix consists in:
1. Now after login completion, thread is reset to allow auditing of actions
occurred afterward.
2. Correct user id and new WebService process type is set to be audited. Note
this didn't occur even before the regression was introduced, reulting in
records in ad_audit_trail without user or process information.
src-db/database/sourcedata/AD_REF_LIST.xml
src/org/openbravo/erpCommon/security/SessionLogin.java
src/org/openbravo/service/web/BaseWebServiceServlet.java




Last 20 lines of the console output:

[...truncated 517 lines...]
Total: 100% good

Error Summary:

Errors:
+ exit 0
Performing Post build task...
Match found for : : True
Logical operation result is TRUE
Running script : cp /srv/ci/workspace/int-full-pgsql/SANDBOX/api-checks/output/java.japi.gz .
[int-api] $ /bin/bash -xe /tmp/hudson3169831880658957452.sh
+ cp /srv/ci/workspace/int-full-pgsql/SANDBOX/api-checks/output/java.japi.gz .
POST BUILD TASK : SUCCESS
END OF POST BUILD TASK : 0
Archiving artifacts
Checking console output
Email was triggered for: Fixed
Trigger Success was overridden by another trigger and will not send an email.
Sending email for trigger: Fixed
Reply all
Reply to author
Forward
0 new messages