Comment 25 for bug 525808

Revision history for this message
Numérigraphe (numerigraphe) wrote :

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.