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

Link resolving breaks sometimes when editing content with XPM #78

Open
willprice76 opened this issue Sep 21, 2017 · 4 comments
Open

Link resolving breaks sometimes when editing content with XPM #78

willprice76 opened this issue Sep 21, 2017 · 4 comments

Comments

@willprice76
Copy link
Contributor

It appears that sometime when editing content with XPM that the published content ends up with version information in component links (eg tcm:12-345-v0). This will cause an exception to be thrown when DXA tries to resolve the link using the com.sdl.webapp.tridion.linking.AbstractTridionLinkResolver as this expects an integer in the 3rd segment of the tcm uri (see line 66 from version 1.7)

@bkoopman
Copy link
Contributor

Can you share a bit more details about this, like what version of SDL Web are you experiencing this on, and which hotfixes do you have installed? Also you mention it appears sometimes, is there any reproducible scenario or is it completely random?

@willprice76
Copy link
Contributor Author

willprice76 commented Sep 21, 2017

Its Web 8.5 with DXA Java 1.7. Unfortunately I don't have a reproduction scenario - its sporadic, but I notice that the issue has been around for longer: https://tridion.stackexchange.com/questions/15847/problem-using-xpm-to-edit-image/15849#comment23104_15849

@bkoopman
Copy link
Contributor

bkoopman commented Sep 21, 2017

I'm wondering if it might be an XPM or CIS/CIL issue possibly, hence my questions. It will be difficult to classify if we can't reproduce it.

@rpannekoek
Copy link
Contributor

rpannekoek commented Sep 21, 2017

There were changes in SDL Web 8.5 which make it possible that you get a TCM URI with -v0 suffix (referring to the "dynamic version") where you wouldn't get that in earlier versions. TCM URI parsing logic should be able to deal with a version number in the TCM URI, but apparently the DXA Java TCM URI parsing logic chokes on it.

rpannekoek pushed a commit that referenced this issue Jan 10, 2018
…983 to develop

* commit '90edf083f27886678e9ef8c5b2b2ffef45c297a8':
  TSI-2985 grammar fixes :) (TSI-2983)
  TSI-2985 Preserve comments in properties and add properties to the last line (TSI-2983)
  TSI-2985 Document framework main properties (TSI-2983)
  TSI-2983 Configuration readme file
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

3 participants