Add the property 'tabWidth' to the checkstyle rule 'LineLength'

This is my patch for 1_3RC1. This is useful if you start with sonar from scratch, i.e. your database is empty.

Apply

patch -p0 <003_populate_database.rb.patch

when you are in the sonar root directory.

If you have sonar already running and are measuring projects, this option may help you:

1) Connect to your sonar database
2) Issue some commands to get the right ids:

– find rule_id, lets say the result is 52
select id as rule_id from rules where plugin_config_key like '%LineLength%';

– find profile_id, lets say the result is 3
– Attention repeat this for all your profiles
select id as profile_id from rules_profiles where name = 'Sun checks';

– find active_rule_id (result for example: 207), insert propper values for rule_id and profile_id from your queries above
select id as active_rule_id from active_rules where rule_id = 52 and profile_id=3;

3) Now comes the meat:
– this will create you a new rules_parameter_id (lets say: 192)
insert into rules_parameters (rule_id, name, description, param_type, default_value) values(52, 'tabWidth', 'Define the tab width used for line length calculation', 'i', '8');
– finally add it to your profile as active parameter, note that I have changed the tabWidth to 4 for the project.
insert into active_rule_parameters (active_rule_id, rules_parameter_id, value) values(207, 192, '4');

At least, this does the trick.
May be there is a more elegant solution.
Please let me know.

Version 1.3 RC1 is also affected by this issue.

Hi Eris, sorry for the delay.
Sonar does not manage global properties, but only properties for a given rule. It's an improvement that will be planned for next releases.
In the case of the 'tabWith' property, I don't understand why it's defined as a global property on the module TreeWalker. It seems be used only by the 'LineLength' rule. In such a case the workaround is rather simple and we could fix it for the 1.4 release.
Do you confirm it ?

Thanks for the feedback, Simon.
At least the LineLength rule gets upset when using the exported checkstyle.xml in Eclipse.
Thus, from my understanding, it is sufficient to fix it in the LineLength rule and not in the general case (i.e. with a global property).

regards,
Eris

OK, so it will be fixed on the 1.4 release. I rename the title of this issue to be more precise. Thank you for your precious feedback.

The fix does not work when migrating from a 1.3 installation. The property appears only on fresh 1.4 installations.
We fix it again on 1.4.1.