When XMLSyntaxSupport is used to create a DOM Element impl for the following markup
<element id="id" foo bar="value"/>
the Element doesn't contain the value of the bar attribute. This happens because the XML lexer returns an error token
starting at the bar attribute (although that attribute is valid -- the error is in the foo attribute). Moreover, the XML
lexer returns a text token for the test of the bar="value", which is visible in the syntax highlighting.
Although the lexer may be behaving as designed, I consider this a valid issue, because our competitors handle this case
better. E5e, for example, marks foo as an error.
I do not think I can get to this for 6.1.
Are you planning to get it done for the 6.5 release? 'Coz one of issues in Spring # 129027 depends on it.
The underlying parser for this is not lexer and the (old) editor syntax APIs. I'll look into it and get back. To me,
this and issue 129027 are nice-to-have fixes.
Did you get a chance to look into this issue? Are you still planning to take it up for 6.5 release?
I'll look into it and get back.
Did you get time to look into this issue? Can you please let me know your plans to solve this? I wanted to reprioritize
the dependent IZ#129027 accordingly.
I agree it is an issue but it should be addressed with Lexer migration.
The problem is that, this piece of code uses editor's syntax API which is old. All of this (at some point) should be
migrated to Lexer but I'm not sure when. This is really the core to XML parsing and potentially impacts a lot of code
that depend on it. The lack of unit tests in this area makes it even more difficult to fix. 6.5 being around the corner,
I suggest you re-prioritize and push for these changes in 7.0.
Still valid issue in trunk (of nb71), changing the target milestone to Next (due to comment #7 which indicate much of work for fixing that), but please feel free and update the target milestone if you know when it could be probably really fixed. Thanks a lot.
Issue is still valid in trunk (of nb73).
This is because XML Syntax uses the old Lexer API/impl, which is semi-deprecated now. See the syntax highlighting, it already correctly reports attribute+value for the rest of the tag, and an error tag instead of whitespace. Highlighting is already based on new Lexer.
I don't want to rewrite the SyntaxSupport into 7.3 release. Planning for the next release.
(In reply to comment #10)
> I don't want to rewrite the SyntaxSupport into 7.3 release. Planning for the
> next release.
Thanks, that's definitely good news.