About LineNumbers displaying on Issue

[ptupakul]

HI,

I am Uisng the SonarQube to check my Mule Code Quality,
But for Java,C++... other languages it is displaying the Line number at right side corner but it is not for MuleCode see below fig.

I think should be more useful for quick understanding and fixing the issue if it has that feature.

Thanks,

[fperezpa]

Hi,
The plugin doesn't set the line number. It evaluates XPath expressions and uses Jdom 2 for that. I haven't found yet a solution to evaluate the expression and get the line number.

[ca-stefan-cordes]

The xml plugin from sonar has some nice helpers to find the line numbers (XmlTextRange).
With the original xpath boolean the location is not possible to find.
I added the XmlFile from
sonar-xml-parsing
and now it looks like this.

• I needed to add "the same" xpath to find the node, see locationHint="//*[local-name()='listener-config'] in the rules-3/4.xml files
• I needed to parse the file another time as this plugin depends on org.jdom2 and sonar is org.w3c.dom
• I just added 2 exemplary locationHint

[abhayagarwalonline]

hello

Does Mule sonar plugin codebase has this change of showing line numbers in file where bug is ?
I took latest sonar plugin codebase , built up sonar and ran code analysis.
I am not seeing line number of file.
please help

thanks
abhay

[ca-stefan-cordes]

Hi @abhayagarwalonline ,
just a few rules are prepared to show line numbers:
"HTTP Listener should use HTTPS protocol" and "HTTP Listener should use a specific port property".

All others currently does not have an locationHint xpath and so still are showing on top of file.

[abhayagarwalonline]

thanks for the reply ... please let me know how to add locationHint xpath for other Rules

[ca-stefan-cordes]

Just edit the rules-4.xml (and create a pull-request) ;-)

Each body of the nodes has a boolean xpath.
The locationHint="" needs to have "the same" rule but returning the first matching node.

Example:
value=

			count(//mule:mule/http:listener-config)=0
			or
			//mule:mule/http:listener-config/http:listener-connection/@port='${https.port}'
			or
			//mule:mule/http:listener-config/http:listener-connection/@port='${https.private.port}'

locationHint="//*[local-name()='listener-config']"

Maybe just the "count(" parameter is working as well, but I did not check it:
locationHint="//mule:mule/http:listener-config"
(as currently two different xml implementations are used #23 ) the root note for locationHint probably needs to be whithout namespace:)
locationHint="//mule/http:listener-config"

[abhayagarwalonline]

Thanks for the reply, I will check and revert if it works for me

[gurupradeep088]

The locationHint without the name space locationHint="//mule/http:listener-config" doesn't seems to be working. I have used locationHint="//http:request" which will return the all http:request nodes. But it isn't working

[ca-stefan-cordes]

Did you try
locationHint="//*[local-name()='listener-config']"
?