[jira] Closed: (SLING-1409) ignoreImportProviders doesn't allow multiple values.

["Justin Edelson (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/SLING-1409?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson closed SLING-1409.
---------------------------------


> ignoreImportProviders doesn't allow multiple values.
> ----------------------------------------------------
>
>                 Key: SLING-1409
>                 URL: https://issues.apache.org/jira/browse/SLING-1409
>             Project: Sling
>          Issue Type: Bug
>          Components: JCR
>    Affects Versions: JCR Contentloader 2.0.6
>            Reporter: Simon Gaeremynck
>            Assignee: Carsten Ziegeler
>             Fix For: JCR ContentLoader 2.1.0
>
>
> The ignoreImportProviders property as defined on [1] states that you can specify multiple readers you wish to ignore.
> For example:
> SLING-INF/content;overwrite=true;ignoreImportProviders:=xml,json
> AFAICT it only chooses the first one.
> I've traced this down to the ManifestHeader class in o.a.s.commons.osgi. 
> It looks like it interprets the comma between xml and json as a seperator of Attributes instead of a seperator of Values.
> The following test in ManifestHeaderTest identifies this issue:
> public void testMultiple() {
>       String header = "SLING-INF/content/etc;overwrite=true;ignoreImportProviders=json,xml";
>       final ManifestHeader entry = ManifestHeader.parse(header);
>       assertEquals("json,xml", entry.getEntries()[0].getAttributes()[1].getValue());
>     }
> As pointed out by Carsten on the list a comma cannot be used to seperate values for an attribute.
> Using another character would solve this.
> [1] http://sling.apache.org/site/content-loading-jcrcontentloader.html#ContentLoading(jcr.contentloader)-Extractors

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.