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

dropped content in idml2xml:XML-Hubformat-extract-frames mode #6

Closed
peppobon opened this issue Nov 21, 2021 · 10 comments
Closed

dropped content in idml2xml:XML-Hubformat-extract-frames mode #6

peppobon opened this issue Nov 21, 2021 · 10 comments
Assignees

Comments

@peppobon
Copy link

Hello,
when converting the attached idml the sidebar content get lost and only anchors are present:



The content is ok in output from previous step in idml2xml.HUB.07a.idml2xml_XML-Hubformat-properties2atts.xml

I think the problem is in mode idml2xml:XML-Hubformat-extract-frames templates in GenerateHubformat.xsl

Tested with latest 2beb2b8..b6ce1ad
Attaches the idml test doc and the generated hub.

Thanks
test.zip

@peppobon
Copy link
Author

Forgot to mention that if you anchor the group to a para the conversion is ok.

@gimsieke gimsieke self-assigned this Nov 21, 2021
@gimsieke
Copy link
Contributor

gimsieke commented Nov 21, 2021

If I restore the behavior before this fix, I think the lost content will be retained. Now checking how to reconcile this behavior with the duplicate-content issue that the previous fix was meant to address.

@gimsieke
Copy link
Contributor

I hope that I was able to fix it while still avoiding the other issue that made me change the template in the first place. Please have a try.

@peppobon
Copy link
Author

Sorry, but now I get the content twice.
See the same file converted after your patch.
test-after-patch.idml.hub.zip
.

@peppobon
Copy link
Author

the two inner sidebars hub/sidebar[1]/sidebar are repeated after hub/sidebar[1].
The right content should be hub/sidebar[1] only

@gimsieke
Copy link
Contributor

hmm. Investigating…

@gimsieke
Copy link
Contributor

I didn’t mean to mark it as resolved. It was just an attempt at resolving the issue. All tests of the other projects that use this library need to pass.

@gimsieke gimsieke reopened this Nov 21, 2021
@gimsieke
Copy link
Contributor

But you may already test whether this would work in your case, @peppobon.

@peppobon
Copy link
Author

Yes! my tests now pass.
Thanks Gerrit (as always).

@gimsieke
Copy link
Contributor

To be honest, I didn’t exactly know what I was doing. We need to monitor this. It may also be that we will detect side effects with other projects. Due to intermittent issues with Github’s svn bridge, we cannot test some of our projects yet.

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