Forum OpenACS Development: Re: wiki code metrics for packages, add standards metrics?

If you have tool that potentially is able to identify hot spots, issues, code smells and the like... this tool is useless unless it is run regularly. On top of that the produced results have to me made available to the developers... who is willing to do that for OpenACS?

I do that for Naviserver, it is a matter of time and energy... of how much effort one wants to dedicate to this activity, if any...

When I see something like http://sonarsrv.spazioit.com/overview?id=42350 done also for the TCL code in OpenACS, something executed and maintained every single day, then we can talk...

About OpenACS the mere HTML analysis, as shown in http://sonarsrv.spazioit.com/overview?id=myOpenACS, the codebase uses plenty of deprecated HTML constructs. I believe it is time to get rid of these deprecated elements/attributes.

Maurizio

Hi Maurizio,
Yes, html should be updated, but it looks like that OpenACS HTML analysis mainly points to issues with html created in the docbook documentation. Docbook writes its own html, so maybe docbook has become more of a maintenance burden than benefit.

Html in adp pages are not so simple to analyze. Wonderful work has gone into upgrading html/css partly by invoking per page html analysis at time of rendering. You can see this in the latest OpenACS release 5.9.

To me, at least 80% of that documentation should be removed, because it no longer fits the code, or is off topic etc. Some of it should be put directly in the code at the points of discussion to provide needed context, even if it is just historical. The remaining 20% should be integrated into the wiki.

As for Tcl testing, when I write a linter for my own code, then we'll see if it is applicable for acs-core --I think not. Because I write code as if I just learned the language. Much of acs-core uses syntax and terms I have little experience with.

I see that SonarQube has a ruleset for Python. Maybe that can be adjusted for Tcl with your experience and knowledge of naviserver?

cheers,
Ben