up-2.50topi-oracle - Build # 228 - Still Failing:

Check console output at https://ci.openbravo.com/job/up-2.50topi-oracle/228/ to 
view the results.


Committers since last success:

Changes for Build #226

    Augusto Mauch <augusto.ma...@openbravo.com> null
    Related with issue 37867: Commented out code meant for debug has been 
restored
        
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/HQLDataSourceService.java

    David Miguelez <david.migue...@openbravo.com> null
    fixed bug 36829: can't display long process execution logs

  Even in database is defined as an arbitary lenght text, logs were truncated
  to 4K characters. This artificial restriction has been removed, so that logs
  can now be longer.
        src-db/database/sourcedata/AD_COLUMN.xml
        src-db/database/sourcedata/AD_FIELD.xml
        src/org/openbravo/scheduling/ProcessMonitor.java

    Augusto Mauch <augusto.ma...@openbravo.com> null
    Fixes issue 37867: Adds missing admin mode
        
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/HQLDataSourceService.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    fixed bug 37648: slow junit ViewTest test cases

  testViews (renamed to viewsCanBeQueried)
    * Queries all the visible records for all views in system loading those 
records
      in memory. Now, record is limited to just one and we stop querying views 
once
      one record is retrieved.

  test14914
    * All invoices in the instance were loaded in memory loading for those all 
their
      payments and making a copy. Now only one invoice is loaded.
        src-test/src/org/openbravo/test/dal/ViewTest.java

    Mark <markm...@gmail.com> null
    Fixes issue 37823: Sales tax selected in Return To Vendor if the returned 
goods
receipt does not have a purchase order related

A wrong sales tax was selected in Return To Vendor if the returned goods receipt
does not have a purchase order related.

The Tax field has the Tax Id value of the receipt line being copied, in case
that the good receipt line was created from an order then it will have the tax 
id
used on the order line. But in case the goods receipt line was created 
manually, as
it lines aren't related to any order lines, the tax received will be NULL, so 
it is
needed to get the default Tax to be used in the new generated returned line.

In the case of a Return to Vendor is needed to return the default Purchase Tax 
of the
product. In the case of Return from Customer is neede to use the default Sales 
tax
of the product. It was fixed passing this parameter taking into account the 
processed
order is sales transaction or not.
        src/org/openbravo/common/actionhandler/SRMOPickEditLines.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    related to bug 37857: minor optimizations

  * FIC uses multiple ids only for attachemnts, we can initialize it only in 
that case
  * In convertJSONArray use more generic variable name
  * In convertJSONArray we know which will be the array size, so it can be 
initialized
    using it
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/window/FormInitializationComponent.java

    Carlos Aristu <carlos.ari...@openbravo.com> null
    fixes bug 37857: Wrong FIC request after selecting hundreds of records

  The MULTIPLE_ROW_IDS parameter contains the ids of the selected records that 
are sent to the FIC. They are used to check if there exists attachments within 
the selection and in that case enable the attachment button of the toolbar or 
display the number of attachments of the current record in case we are in form 
view.

  This information was being sent as part of the request URL in the HTTP header 
which in case of having hundreds of records, the maximum size allowed for the 
request URL was being exceeded.

  To fix this problem, now the MULTIPLE_ROW_IDS parameter is sent within the 
body of the FIC request.
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/window/FormInitializationComponent.java
        
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-toolbar.js

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Fixed issue 37827: [ri] Added message to Application Dictionary
        src-db/database/sourcedata/AD_MESSAGE.xml
        src/org/openbravo/materialmgmt/refinventory/UnboxProcessor.java

    Armaignac <collazoan...@gmail.com> null
    Fixes issue 37763: Trial Balance Report sometimes returns null pointer 
exception
when include zero figures is checked.

When zero figures is checked the matches accounts are skipped from vector list
causing null entries when the vector values are copied with different sizes

Now the dataZeroFigures size is equal to the vector size to avoid null entries
        src/org/openbravo/erpCommon/ad_reports/ReportTrialBalance.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    realted to issue 37849: remove system key hash in log files

  Restored ActivationKey.getOpsLogId method as it is public and used
        src/org/openbravo/erpCommon/obps/ActivationKey.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    fixed issue 37849: remove system key hash in log files

  This hash has been removed as it is not useful and complicates log parsing
        src/org/openbravo/erpCommon/obps/ActivationKey.java

    Mark <markm...@gmail.com> null
    Fixes issue 37724: Error when cancel and replace a sales order of zero 
amount

When an order with grand total of ZERO is created and booked, no Payment Plan is
created for it, so when she is cancelled and replaced the new created orders 
haven't
payment plans neither.

So, when the Cancell and Replace process is executed on orders of this kind, the
it was failing trying to get get the payment schedule detail of the oldOrder and
an exception was thrown.

To fix this issue, only will be created Payments to orders with grand total 
different
than ZERO to avoid the exception and keep the current behaviour for this kind 
of orders
        src/org/openbravo/erpCommon/businessUtility/CancelAndReplaceUtils.java

    Armaignac <collazoan...@gmail.com> null
    Fixes issue 37837:Tax selected is based on the invoicing address instead of 
the
partner address

Wrong tax is selected when selecting a business partner with different invoicing
and shipping address, the invoicing address was used as shipping

Now the correct business partner address is selected to properly get the product
tax
        src/org/openbravo/common/actionhandler/OrderCreatePOLines.java
        src/org/openbravo/common/actionhandler/SRMOPickEditLines.java
        src/org/openbravo/erpCommon/ad_actionButton/RMInsertOrphanLine.java

    Mark <markm...@gmail.com> null
    Fixed issue 37798: Add Characteristic to Product does not update the
Characteristic Description

When adding a characteristic value to an existing product its Characteristic 
Description
field should update with the new value but wasn't doing as expected.

To fix, ii passed True as commitAndClose to commit and notify the import entry 
creation
on the event handler.
        src/org/openbravo/event/ProductCharacteristicValueEventHandler.java

    David Miguelez <david.migue...@openbravo.com> null
    Fixes Issue 37772. Adds several indexes that improve performance
in basic operations like deleting an Invoice or an Order
        src-db/database/model/tables/C_INVOICELINE.xml
        src-db/database/model/tables/C_ORDER.xml
        src-db/database/model/tables/M_COSTING.xml
        src-db/database/model/tables/M_INOUT.xml
        src-db/database/model/tables/M_MATCHPO.xml

    Carlos Aristu <carlos.ari...@openbravo.com> null
    related to issue 37734: use getClusterServices() method in 
ClusterServiceThread

 This allows us to make more extensible the ClusterServiceManager for testing 
purposes.
        src/org/openbravo/cluster/ClusterServiceManager.java

    Augusto Mauch <augusto.ma...@openbravo.com> null
    Related with issue 37451: Adds tests

Tests have been added to check that the datasource returns properly the results 
using the updated criteria.

Selenium tests with the current infrastructure are not possible, since when 
they filter the grid, they do not
select the checkboxes of the foreign key combo boxes, they just enter the text 
in the filter input, and if done
that way the issue is not reproducible.
        src-test/src/org/openbravo/test/datasource/NonIdForeignKeyFilters.java
        src-test/src/org/openbravo/test/AllWebserviceTests.java

    Augusto Mauch <augusto.ma...@openbravo.com> null
    Fixes issue 37451: Fixes filtering of fks that dont use ids as key

The problem was that the foreign keys that did not use and id column as key 
(i.e. ad_country.ad_language) were not being properly
filtered locally in smartclient, because the criteria used the id of the 
referenced record, that was not available in the browser (it
was available in the query done in the back end, that is why only the 
client-side filtering was not working).

For instance, if the ad_language foreign key reference uses as key 
ad_language.ad_language instead of ad_language.ad_language_id, the criteria
will no longer be:

{
  fieldName: 'language',
  operator: 'equals',
  value: '192'
}

Now it will be:

{
  fieldName: 'language',
  operator: 'equals',
  value: 'en_US'
}

The AdvancedQueryBuilder has been adapted to support the new type of criteria.

There are also changes on ob-grid.js and ob-formitem-fk-filter to ensure that 
the criteria is properly displayed in the filter input.
        
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-fk-filter.js
        
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
        
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/ForeignKeyUIDefinition.java
        
modules/org.openbravo.service.json/src/org/openbravo/service/json/AdvancedQueryBuilder.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Fixed issue 37827: [ri] clear session after each individual test execution
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxForcedReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxFullReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxSeveralStorageDetailsTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Fixed issue 37827: [ri] clear session after each test execution
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTest.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    fixed bug 37437: useless session cache of window.isSOTrx

  Now isSOTrx propery is not cached in session for each window in the system,
  instead its value is obtained from application scoped ADCS when required:
   - In Utility.getConext
   - In OBBindings
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/OBBindings.java
        src/org/openbravo/base/secureApp/Attribute_data.xsql
        src/org/openbravo/base/secureApp/LoginUtils.java
        src/org/openbravo/erpCommon/utility/Utility.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    related to bug 37437: useless session cache of window.isSOTrx

  Reapplied backed out changeset.

  Some code minor clean up:
    - Removed some duplicity
    - Fixed format in some comments
    - ...
        src/org/openbravo/erpCommon/utility/Utility.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    related to bug 37437 related to bug 37544: added test case

  covering regression #37544
        src-test/src/org/openbravo/test/datasource/OtherDatasourceRequests.java
        src-test/src/org/openbravo/test/AllWebserviceTests.java

    Stefan Hühner <stefan.hueh...@openbravo.com> null
    Fixed 37825. Fix javadoc syntax to not show warnings with JDK 9.

Javadoc from Java9 is more strict by enabling a subset of -Xdoclint:html and
complains about any unescaped <,>,& in comments.

Apart from fixing all those in pi also undo the addition of -Xdoclint:none
to build.xml to let javadoc check those again.
        build.xml
        
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/utility/FIN_Utility.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ExtraWindowSettingsInjector.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/ParameterUtils.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/event/ReorderCascadingChildDeletionsHandler.java
        
modules/org.openbravo.client.application/src/org/openbravo/client/application/process/BaseProcessActionHandler.java
        
src-test/src/org/openbravo/advpaymentmngt/test/DocumentNumberGeneration.java
        src-test/src/org/openbravo/test/base/BaseTest.java
        src-test/src/org/openbravo/test/base/OBBaseTest.java
        
src-test/src/org/openbravo/test/db/model/functions/Ad_isorgincludedTest.java
        src-test/src/org/openbravo/test/pricelist/data/PriceListTestData.java
        
src-test/src/org/openbravo/test/role/inheritance/RoleInheritanceTestUtils.java
        
src-test/src/org/openbravo/test/role/inheritance/VerticalInheritance.java
        src-test/src/org/openbravo/test/xml/ClientExportImportTest.java
        src/org/openbravo/base/util/Check.java
        
src/org/openbravo/common/actionhandler/copyfromorderprocess/UpdateTax.java
        
src/org/openbravo/common/datasource/ReturnFromCustomerVendorPropertyFieldsTransformer.java
        src/org/openbravo/dal/core/DalRequestFilter.java
        src/org/openbravo/dal/xml/StaxXMLEntityConverter.java
        src/org/openbravo/erpCommon/ad_forms/Fact.java
        src/org/openbravo/erpCommon/businessUtility/CancelAndReplaceUtils.java
        src/org/openbravo/erpCommon/businessUtility/InitialSetupUtility.java
        src/org/openbravo/erpCommon/businessUtility/PriceAdjustment.java
        src/org/openbravo/erpCommon/modules/ImportModule.java
        src/org/openbravo/erpCommon/modules/VersionUtility.java
        src/org/openbravo/erpCommon/obps/ActivationKey.java
        src/org/openbravo/erpCommon/utility/BasicUtility.java
        src/org/openbravo/erpCommon/utility/HttpsUtils.java
        src/org/openbravo/erpCommon/utility/OBMessageUtils.java
        src/org/openbravo/erpCommon/utility/Utility.java
        src/org/openbravo/jmx/MBeanRegistry.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Fixed issue 37827: [ri] updated copyright year
        src/org/openbravo/erpCommon/ad_callouts/SE_RefInventory_RefInvType.java
        src/org/openbravo/event/ReferenceInventoryEventHandler.java
        src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java

    Javier Armendáriz <javier.armenda...@openbravo.com> null
    Fixed bug 37587: Opening tab with display logic using aux input appears 
empty.

ShowTabIf logic applied only if the view does not come from a direct link call. 
This fixed the issue with no wait until the contextInfo is fully loaded.
        
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Fixed issue 37827: [ri] Removed TODO as no need of hooks
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    Fixed issue 37827: [ri] Added Referenced Inventory support
      
    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Fixes exception in Oracle,
Add reservation preference if necessary
        src-db/database/model/functions/M_MOVEMENT_POST.xml
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Shared flag is always readonly
        src-db/database/sourcedata/AD_FIELD.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    pi merge
      
    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Avoid editing Attributes when linked to RI
        src-db/database/sourcedata/AD_MESSAGE.xml
        src/org/openbravo/erpCommon/info/AttributeSetInstance.java
        src/org/openbravo/erpCommon/utility/AttributeSetInstanceValue.java
        src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Improvements in Box/Unbox grid
        src-db/database/sourcedata/AD_FIELD.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] fixes issues found in CI
        src-db/database/model/tables/M_REFINVENTORY.xml
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    pi merge
      
    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Force Bin or Attribute at reservation header
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxForcedReservation.java
        src-db/database/sourcedata/AD_MESSAGE.xml
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestSuite.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java
        src/org/openbravo/materialmgmt/refinventory/ReservationManager.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Added storage details check after unboxing with reservations
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxReservationTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] extended allocated flag check
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] improved unbox reservation tests
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] simplified unbox reservation tests
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] removed useless throws
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] removed useless null checks
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] More improvements in JUnit tests
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Run more tests for all unboxing flows
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] properly test reservation qty in boxing
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] removed useless method in tests and simplified test condition
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] fixed getStorageDetails in tests
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryExceptionTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] minor fixes en tests
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Added missed class
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Support for reservations
JUnit tests in progress
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxFullReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservationTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxSeveralStorageDetailsTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryExceptionTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventorySequenceTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestSuite.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxReservationTest.java
        src/org/openbravo/materialmgmt/refinventory/ReservationManager.java
        src-db/database/model/functions/M_MOVEMENT_POST.xml
        src-db/database/sourcedata/AD_MESSAGE.xml
        src-test/src/org/openbravo/test/AllAntTaskTests.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java
        src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Tests for Box and Unbox referenced inventory
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTest.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java
        
src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java
        src-test/src/org/openbravo/test/AllAntTaskTests.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Removed unused method
        src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Refactor in Processors to avoid code duplication
        
src/org/openbravo/common/actionhandler/ReferencedInventoryUnBoxHandler.java
        src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
        src/org/openbravo/materialmgmt/refinventory/UnboxProcessor.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] getReferencedInventory only makes sense in boxing
        src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
        src/org/openbravo/materialmgmt/refinventory/UnboxProcessor.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Fixed several issues
In Box the new storage bin parameter must be mandatory
Unbox did wrong verification on previously linked to a referenced inventory
        src-db/database/sourcedata/AD_MESSAGE.xml
        src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
        
src/org/openbravo/common/actionhandler/ReferencedInventoryBoxHandler.java
        src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Referenced Inventory window is a Transactional window
        src-db/database/sourcedata/AD_WINDOW.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Added validation to avoid child referenced inventories
        src-db/database/model/tables/M_STORAGE_DETAIL.xml
        src-db/database/sourcedata/AD_MESSAGE.xml
        src-db/database/sourcedata/AD_TAB.xml
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Added validation RI in unique bin
        src-db/database/sourcedata/AD_MESSAGE.xml
        src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    pi merge
      
    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    pi merge
      
    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Represent ReferencedInventoy inside [] in attribute desc
        src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Added ReferencedInventory to attribute set instance description
        src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] New selector in StorageDetail.Locator
        src-db/database/sourcedata/AD_COLUMN.xml

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Box and Unbox processes
        
src/org/openbravo/common/actionhandler/ReferencedInventoryBoxHandler.java
        
src/org/openbravo/common/actionhandler/ReferencedInventoryUnBoxHandler.java
        src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
        
src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
        src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
        src/org/openbravo/materialmgmt/refinventory/UnboxProcessor.java
        
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/utility/FIN_Utility.java
        src-db/database/model/functions/M_MOVEMENT_POST.xml
        src-db/database/model/functions/M_UPDATE_INVENTORY.xml
        src-db/database/model/tables/M_REFINVENTORY.xml
        src-db/database/model/triggers/M_MOVEMENTLINE_TRG.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_MESSAGE.xml
        src-db/database/sourcedata/AD_REFERENCE.xml
        src-db/database/sourcedata/AD_TAB.xml
        src-db/database/sourcedata/AD_WINDOW.xml
        src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
        src-db/database/sourcedata/OBUIAPP_PROCESS.xml
        src-db/database/sourcedata/OBUIAPP_REF_WINDOW.xml
        src/org/openbravo/erpCommon/ad_callouts/SE_RefInventory_RefInvType.java
        src/org/openbravo/event/ReferenceInventoryEventHandler.java
        src/org/openbravo/erpCommon/utility/ReferencedInventoryUtil.java

    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    pi merge
      
    Víctor Martínez Romanos <victor.marti...@openbravo.com> null
    [ri] Added data model, windows and sequence management code
        src-db/database/model/tables/M_REFINVENTORY.xml
        src-db/database/model/tables/M_REFINVENTORY_TYPE.xml
        src/org/openbravo/erpCommon/ad_callouts/SE_RefInventory_RefInvType.java
        src/org/openbravo/erpCommon/utility/ReferencedInventoryUtil.java
        src/org/openbravo/event/ReferenceInventoryEventHandler.java
        src-db/database/model/tables/M_ATTRIBUTESETINSTANCE.xml
        src-db/database/model/tables/M_MOVEMENTLINE.xml
        src-db/database/model/tables/M_STORAGE_DETAIL.xml
        src-db/database/sourcedata/AD_CALLOUT.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_MENU.xml
        src-db/database/sourcedata/AD_MODEL_OBJECT.xml
        src-db/database/sourcedata/AD_MODEL_OBJECT_MAPPING.xml
        src-db/database/sourcedata/AD_TAB.xml
        src-db/database/sourcedata/AD_TABLE.xml
        src-db/database/sourcedata/AD_TREENODE.xml
        src-db/database/sourcedata/AD_WINDOW.xml

    Carlos Aristu <carlos.ari...@openbravo.com> null
    related to issue 37734: Protection against ping disabling + minor 
improvements
      
    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Show service name in log message
        src/org/openbravo/cluster/ClusterServiceManager.java

    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Added missing javadoc
        src/org/openbravo/cluster/ClusterService.java
        src/org/openbravo/service/importprocess/ImportEntryManager.java

    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Fix typo
        src/org/openbravo/cluster/ClusterService.java
        src/org/openbravo/cluster/ClusterService_data.xsql

    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Move the start/end processing notification into the 
ImportEntryProcessor
        src/org/openbravo/cluster/ClusterService.java
        src/org/openbravo/service/importprocess/ImportEntryManager.java
        src/org/openbravo/service/importprocess/ImportEntryProcessor.java

    Asier Lostalé <asier.lost...@openbravo.com> null
    [ha] typo
        src/org/openbravo/cluster/ClusterService.java
        src/org/openbravo/cluster/JmxClusterServiceManager.java

    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Added log info when disabling ping while processing
        src/org/openbravo/cluster/ClusterService.java

    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Reset processing attributes when a new node can take the leadership
        src/org/openbravo/cluster/ClusterService.java
        src/org/openbravo/cluster/ClusterServiceManager.java

    Carlos Aristu <carlos.ari...@openbravo.com> null
    [ha] Added protection against ping disabling while processing import entries
        src/org/openbravo/cluster/ClusterService.java
        src/org/openbravo/cluster/ClusterServiceManager.java
        src/org/openbravo/cluster/ClusterService_data.xsql
        src/org/openbravo/cluster/JmxClusterServiceManager.java
        src/org/openbravo/service/importprocess/ImportEntryArchiveManager.java
        src/org/openbravo/service/importprocess/ImportEntryManager.java

Changes for Build #227
No changes

Changes for Build #228

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

Refactor code to improve redability:
* Encapsulated order parameters into a sub class that can be
  shared between methods
* Split code of methods to reduce cognitive complexity
* Removed comments since the code is self explanatory enough
        src/org/openbravo/event/OrderEventHandler.java

    Mark <markm...@gmail.com> null
    Fixes issue 37843: OrderEventHandler does useless and repeated queries when 
not
needed.

OrderEventHandler class has been refactorized:
- Removed unused vars.
- Renamed newWarehouseId to newWarehouse as it store a Warehouse object instead 
of an ID
- Renamed oldWarehouseId to oldWarehouse as it store a Warehouse object instead 
of an ID
- Only executed query to get preferences after verify that values have changed. 
This way
  it avoids to execute unneeded queries.
- Saved the orderLineCriteria.list() in a variable to avoid execute the same 
query more than once.
- Merged the three used loops updating fields in only one that updates fields 
if needed.
- Only loop is executed if at least one of the expected fields is changed and 
it is not
  activated the related preference.
- Created new methods to make code more cleaned and reuse in similar cases.
        src/org/openbravo/event/OrderEventHandler.java

    Armaignac <collazoan...@gmail.com> null
    Fixes issue 37775:Cannot create an outbound picking list for a partially 
shipped
sales order

When the shipment has removed all the stock a new reservation line is created
with 0 reservedqty and releasedqty to correct the product reservation values and
the the reservation line is removed.
        src-db/database/model/functions/M_CREATE_RESERVE_FROM_SOL.xml




Last 20 lines of the console output:

[...truncated 2278 lines...]
BUILD SUCCESSFUL
Total time: 10 minutes 23 seconds

[up-2.50topi-oracle] $ /bin/bash -xe /tmp/hudson539738723434308107.sh
+ OBPROPS=/srv/ci/workspace/up-2.50topi-oracle/config/Openbravo.properties
++ awk -F = '/^bbdd.sid/ {print $2}' 
/srv/ci/workspace/up-2.50topi-oracle/config/Openbravo.properties
+ DBNAME=oborcl
++ awk -F = '/^bbdd.user/ {print $2}' 
/srv/ci/workspace/up-2.50topi-oracle/config/Openbravo.properties
+ DBUSER=up_250topi_oracle
++ awk -F = '/^bbdd.pass/ {print $2}' 
/srv/ci/workspace/up-2.50topi-oracle/config/Openbravo.properties
+ DBPASS=up_250topi_oracle
+ run_ora_command 'delete from C_BP_CUSTOMER_ACCT where C_BP_CUSTOMER_ACCT_id 
in ('\''1000020'\'', '\''1000019'\'', '\''1000024'\'', '\''1000022'\'', 
'\''1000021'\'');'
+ ssh oracle@
Pseudo-terminal will not be allocated because stdin is not a terminal.
ssh: Could not resolve hostname : Name or service not known
Build step 'Execute shell' marked build as failure
Checking console output
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