-
Notifications
You must be signed in to change notification settings - Fork 56
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
Cannot sync .xml files, such as cq_dialog fields #19
Comments
Usually this should not be a problem as in most cases the entire dialog is represented by one |
Thanks for responding. I don't have the immediate code in front of me right now, but what were trying to do is very similar to this: http://experience-aem.blogspot.com/2015/02/aem-6-sp2-touch-ui-multi-field-component.html. I should note that we are on AEM 6.2, focusing on the Touch UI dialogs. Where our project differs from the the example above is on line 42 of their 'step 4', our .content.xml just has We use these multi-fields in a number of places, and use this method to split up the code and make it more manageable as they grow in size. Editing that final |
This is an example of the actual tree:
|
It seems that the .content.xml files are not added to the folders on the path. I'm working on a fix. |
Thats awesome - thank you. Let me know if I can provide any more information. |
@gavoja just wanted to see if you had any thoughts. I could take a stab at it, but with my so far limited experience i'm not sure what the filter was meant to prevent well enough to be confident in working around it again. |
I found a fix, though it's not very nice and I didn't have time to polish it. I'm currently flat out with other projects I do for living, but I'll definitely come back to this as soon as I can. Basically for all the folders up to the root under jcr_content, the corresponding .content.xml files need to be added to the package - otherwise they will all be treated as |
Can you please verify if this works properly with 2.0.0? |
Closing. |
Dialog options within the _cq_dialog folders are often contained within nested folders and an XML file. It appears that aemsync is removing the .xml from these files, creating incorrect filters during package sync.
Using debug mode, I pulled the following filter settings:
VS a standard css file
Note the difference between "main.css" and simply "field", where the latter should be "field.xml"
Code that performs this here:
https://github.com/gavoja/aemsync/blob/master/src/package.js#L128-L132
What is this filter meant to prevent? As a result of it, it seems that no edits to .xml files can be synced correctly. Is that purposeful?
The text was updated successfully, but these errors were encountered: