[16.0][FIX] edi_stock_oca: fixing missing string field information from the stock.picking model

[QuocDuong1306]

The error starts from: here

Throw error: here

Solutions:

• if add more string attrb in the fields in the template, it will throw new error (because viewFieldsInfo is still undefined)
• so we make viewFieldsInfo is not undefined ( it hasn't been undefined when the fieldName appears in the form stock.picking) --> So we make edi_config and edi_has_form_config appear in stock.picking form view)

[simahawk]

Wondering: we should probably add these fields automatically as we do with buttons here https://github.com/OCA/edi-framework/blob/16.0/edi_oca/models/edi_exchange_consumer_mixin.py#L118

[santostelmo]

Wondering: we should probably add these fields automatically as we do with buttons here https://github.com/OCA/edi-framework/blob/16.0/edi_oca/models/edi_exchange_consumer_mixin.py#L118

I tried something like that

for node in doc.xpath("//group[@name='edi_main']"):
                node.addprevious(etree.Element(
                    "field",
                    {
                        "name": "edi_config",
                        "invisible": "1",
                    },
                ))
                node.addprevious(etree.Element(
                    "field",
                    {
                        "name": "edi_has_form_config",
                        "invisible": "1",
                    },
                ))

But is not working

[simahawk]

/ocabot merge patch

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-68-by-simahawk-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at ca558c0f22229cb3084931ceea4f3ab4ae0327f9. Thanks a lot for contributing to OCA. ❤️