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

<output> inside repeat has absolute path instead of relative #446

Closed
MartijnR opened this issue May 26, 2020 · 3 comments
Closed

<output> inside repeat has absolute path instead of relative #446

MartijnR opened this issue May 26, 2020 · 3 comments
Labels

Comments

@MartijnR
Copy link
Contributor

type name label::English calculation
begin repeat member    
calculate pos   position(..)
text member_name Name of ${pos}  
end repeat      

xlsform

The above form contains <output value=" /data/member/pos "/> instead of the expected <output value=" ../pos "/>.

This is only the case when the output is placed as a descendant of <itext>. The above form with a label column instead of label::English works fine.

@MartijnR MartijnR changed the title <output> inside repeat has absolute path instead of relative <output> inside repeat has absolute path instead of relative May 26, 2020
@MartijnR MartijnR added the bug label Jun 4, 2020
@MartijnR
Copy link
Contributor Author

MartijnR commented Jun 8, 2020

Would be great to wait for this fix before the next release if we get a PR for this fix soon (say this week). Am discussing with major user about perhaps supporting this as a priority.

@gushil
Copy link
Contributor

gushil commented Jun 10, 2020

Hi @MartijnR I've created PR #450 to address this issue. Waiting for the review.

Thanks.

@lognaturel
Copy link
Contributor

Thanks again, @gushil! Quick Github tip -- if you use a closing word before tagging the corresponding issue in a PR description, merging that PR will automatically close the issue.

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

No branches or pull requests

3 participants