Cleanup client

Resolves a bunch of issues- still need to test it though.

Resolves #292 #290 #293 (Theoretically...)
Yeah- I definitely don't like it in ok.py since it'll clutter it more.

Also, going off of Albert's point- it doesn't make sense to tag something
as "submit" if the file contents aren't there. I think I'm now sure that it
belongs here. XD
On Thu, Nov 6, 2014 at 17:03 albert12132 <notifications@github.com> wrote:

> In client/protocols/file_contents.py:
>
> > @@ -9,6 +9,8 @@ class FileContents(protocol.Protocol):
> >      def on_start(self):
> >          """Find all source files and return their complete contents."""
> >          contents = {}
> > +        if self.args.submit:
>
> Moving this to ok.py would probably be better, but I'm fine with having
> it here. Submitting is an assignment-wide operation and should work
> independent of analytics.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Cal-CS-61A-Staff/ok/pull/295/files#r19988794>.
>