ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pierre Smits (Jira)" <j...@apache.org>
Subject [jira] [Commented] (OFBIZ-11030) Convert FactServices.xml minilang to groovy
Date Wed, 26 Feb 2020 21:22:00 GMT

    [ https://issues.apache.org/jira/browse/OFBIZ-11030?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17045911#comment-17045911

Pierre Smits commented on OFBIZ-11030:

Thanks [~sberg],

I did some aceptance testing with the changes, and it seems to look good. Except for the loading
of the InventoryItemFact entity. Compare OFBIZ-11030-InventoryItemFact-test to OFBIZ-11030-InventoryItemFact-DemoTrunk.
Maybe you could have a look at this?

And there are some flaws implemented previously (e.g. compare OFBIZ-11030-Order-SalesOrder-overview
to OFBIZ-11030-SalesOrderItemFact-test, where order DEMO10090 is not included in the SalesOrderItemFact,
or the flaws in OFBIZ-11030-SalesInvoiceItemFact-test)

With the GIGO (Garbage In Garbage Out) regarding the pre-change code the project should decide
wether or not accept the migration. I suggest it should, and deal with the bug fixes at a
later date.

> Convert FactServices.xml minilang to groovy
> -------------------------------------------
>                 Key: OFBIZ-11030
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11030
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: bi
>    Affects Versions: Trunk
>            Reporter: Pierre Smits
>            Assignee: Sebastian Berg
>            Priority: Major
>              Labels: Fact, dwh, services
>         Attachments: OFBIZ-11030-FactServices.xml-minilang-to-groovy.patch, OFBIZ-11030-FactServices.xml-minilang-to-groovy.patch,
OFBIZ-11030-InventoryItemFact-DemoTrunk.png, OFBIZ-11030-InventoryItemFact-test.png, OFBIZ-11030-Order-SalesOrder-overview.png,
OFBIZ-11030-SalesInvoiceItemFact-test.png, OFBIZ-11030-SalesOrderItemFact-test.png, OFBIZ-11030-applyPatch-error.png
> With the purpose to deprecate mini-lang OFBIZ-9350,  convert FactServices.xml

This message was sent by Atlassian Jira

View raw message