Reporting progress on this issue, counting the occurrences of "variable={}" in "def" lines :
Good news:
Addons: trunk has 350, down from 493 in 5.0
Web client: trunk has 29, down from 32 in 5.0
Bad news:
Server: trunk has 114, up from 100 in 5.0 (on the server! shame!)
Client: trunk has 70, up from 58 in 5.0
Extra: trunk has 868, up from 702 in 5.0
That means there is still a LOT of mindless coding and silly copy/paste going on.
That the partners didn't improve was expected - some clearly said they didn't give a damn - too bad.
But OpenERP SA said they would fix this over time, one function at a time as other improvements were being done. Obviously that doesn't work.
Dear maintainers, please either step in and have the coding practices improve at OpenERP SA, or consider again making a massive one-time fix across the whole code base. The script I wrote does not fix it all, but it sure will help.
Lionel.
Reporting progress on this issue, counting the occurrences of "variable={}" in "def" lines :
Good news:
Addons: trunk has 350, down from 493 in 5.0
Web client: trunk has 29, down from 32 in 5.0
Bad news:
Server: trunk has 114, up from 100 in 5.0 (on the server! shame!)
Client: trunk has 70, up from 58 in 5.0
Extra: trunk has 868, up from 702 in 5.0
That means there is still a LOT of mindless coding and silly copy/paste going on.
That the partners didn't improve was expected - some clearly said they didn't give a damn - too bad.
But OpenERP SA said they would fix this over time, one function at a time as other improvements were being done. Obviously that doesn't work.
Dear maintainers, please either step in and have the coding practices improve at OpenERP SA, or consider again making a massive one-time fix across the whole code base. The script I wrote does not fix it all, but it sure will help.
Lionel.