Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MergeTagger doesn't respect field order (in contrast to v. 2.x) #76

Open
bes-shutok opened this issue Jan 29, 2024 · 1 comment
Open

Comments

@bes-shutok
Copy link

Hey!
Thank you for your work on the project.

We are currently moving from 2.x to 3.x version. In this process I found that MergeTagger doesn't keep the provided order of the fields it merges. In reality the fields are merged in the order of appearance in metadata (Property object).

For example

                <merge toField="asset.name.tds" singleValue="true" singleValueSeparator=" - ">
                    <fieldMatcher method="csv">name,name.tds</fieldMatcher>
                </merge>

can now resolves either in ${name} - ${name.tds} or in ${name.tds} - ${name}

Is there any plans /possibilities to fix that in the future?
Do you accept external pull requests?

Best regards,
Andrey

@ohtwadi
Copy link

ohtwadi commented Jan 29, 2024

Hi Andrey,

We very much appreciate external pull requests :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants