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

stylesheet is broken as of 2.2 #378

Closed
robot-ranger opened this issue Dec 11, 2023 · 7 comments · Fixed by #407
Closed

stylesheet is broken as of 2.2 #378

robot-ranger opened this issue Dec 11, 2023 · 7 comments · Fixed by #407

Comments

@robot-ranger
Copy link
Contributor

robot-ranger commented Dec 11, 2023

the stylesheet appears to be broken for me since 2.2. anyone else have this problem?

image

When i change the agent.cfg and devices.xml back to 2.0, it works as expected. 2.2 and it appears like above.

no issues with client-end apps; the xml appears to be in good shape, so i think its just the stylesheet

@wsobel
Copy link
Member

wsobel commented Dec 12, 2023

This usually has to do with the namespace in the stylesheet. I'll check.

@wsobel wsobel added the bug label Dec 12, 2023
@robot-ranger
Copy link
Contributor Author

robot-ranger commented Dec 12, 2023

This usually has to do with the namespace in the stylesheet. I'll check.

I was trying my best to act like i know stylesheets and dove straight into the transform logic, right past the header things haha. looks like youre right!

image

is it possible to make it read the schema version from the probe?

@robot-ranger
Copy link
Contributor Author

or maybe it could be an env var? i dont know much about stylesheets, so i dont know how dynamic it could be or how it would work. but it would be nice to be able to roll up and down schema versions without editing the file

@robot-ranger
Copy link
Contributor Author

oops - accidentally close it

@wsobel
Copy link
Member

wsobel commented Dec 12, 2023 via email

@wsobel wsobel added the feature label Jan 7, 2024
@robot-ranger
Copy link
Contributor Author

fixed in #403

@robot-ranger
Copy link
Contributor Author

fyi - if anyone needs to fix this on an existing container, the following command works:

docker exec -it agent sed -i 's/2.0/2.2/g' /mtconnect/data/styles/styles.xsl

(replace agent with your contianer name if not agent)

@wsobel wsobel linked a pull request Jan 26, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants