Data Importer - not able to handle property names such as "cq:tags@[]"

[ncordeiro]

Please note that the fix for the issue is to update the DataImporter class in the package com.adobe.acs.commons.mcp.impl.processes at line 61 setting the enableHeaderNameConversion to true
Alternative way would be to avoid passing it in the constructor since the Spreadsheet java constructor sets it to true in the class

Required Information

• [6.4 ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: sp2
• ACS AEM Commons Version: 3.18.3
• Reproducible on Latest? yes/no- yes

Expected Behavior

I am using the data import feature to update asset's properties,

property names in the header of an excel, of the format propertyname@[] such as cq:tags@[] should be stored in aem as cq:tags and not cq:tags%5...

The value stored in aem is not cq:tags or any propertyname but a representation of(maybe creating a valid property name of @[]) "@[]"

Links: https://adobe-consulting-services.github.io/acs-aem-commons/features/mcp-tools/data-importer/index.html

Actual Behavior

The aem property name should be stored as cq:tags without parsing the "@[]"

Steps to Reproduce

1. Create an excel with property name(test@[] or cq:tags@[]) in the header
2. Import the excel using data import in the MCP uploading the excel file and choosing create/overwrite property
3. Check the metadata property name

Attaching the sample excel sheet data

[badvision]

Good sleuthing! Sorry about that. I'll get something in for 4.0.0, but if you beat me to it I'll approve the pull request post-haste.

[ncordeiro]

@badvision : done . thanks Brendan

[badvision]

Let me know if my patch is acceptable @ncordeiro. I think the approach I took in #1579 allows some flexibility as well as addressing the root issue.