Skip to content

Incorrect Property Mapping in Action Model Migration Generation #454

Description

@fabiSwDevBeg

File Affected: collective.easyform.migration.actions

During property mapping in the generation of action models, the property xinfo_headers should use append_list_node instead of append_node.

Expected Behavior:
The current implementation appears to handle it as:

<xinfo_headers>
    <element>HTTP_X_FORWARDED_FOR</element>
</xinfo_headers>

Current Behavior:
The XML definition for xinfo_headers is generated from the migration script as:

<xinfo_headers>
    HTTP_X_FORWARDED_FOR
</xinfo_headers>

Since this is a list-based property (with <element> nodes), append_list_node should be used instead of append_node for correct parsing.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions