Begin refactoring Node model for better maintainability

[jywarren]

Please describe the problem (or idea)

CodeClimate gives us a "C" rating -- see the README at https://github.com/publiclab/plots2

The full report is here: https://codeclimate.com/github/publiclab/plots2

The two biggest "smells" as they call them, are overcomplexity and length of node.rb: https://github.com/publiclab/plots2/blob/master/app/models/node.rb

Can we break some of these methods up or clean them up or DRY them up?

Could we break this into subclasses like Wiki and Note and Feature and Map, that inherit from Node? How would that work?

Could we move stats-related methods into a subclass model called Stats or something? like self.weekly_tallies

Just collecting some ideas here! Chime in!

[jywarren]

We could remove the add_xxxx methods for adding comments, and use the more built-in Rails methods for that...

https://github.com/publiclab/plots2/blob/master/app/models/node.rb#L567

We could try to finish deprecating the Drupal legacy data like Images -- that would help by ~80 lines! #956

In general, let's come up with small projects to chip away at this piece by piece, easiest first, then we can consider bigger changes. Thanks!

[jywarren]

@Souravirus this might be of some interest to you... you're good at this sort of thing! I'm sure you're busy with school but would love to hear your thoughts on it when you have a moment sometime. 😄

[Souravirus]

Yeah sure I will work on it @jywarren