We should do a bit of cleanup in the code. Here are the problems that need to be addressed:

  • All non-API functions should be prefixed with an underscore and should be at the end
  • Make sure all electoral list functions are together and are prefixed by electoral_list_
  • Put theme functions together in a separate section
  • Verify that comments are formatted properly

That's all I can think out for now, I'll start working on that.

Comments

tatien’s picture

I've commited three patches that fix these issues for decisions.module and decisions_datetime.inc. However, we should still do this work on the modes (poll/ranking).

anarcat’s picture

Status: Active » Fixed

I think this is mostly done at this point (modes look fine).

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.