Fixed code conventions

[kaine119]

https://codein.withgoogle.com/dashboard/task-instances/5021956311089152/

I'm Kai En, from the above Google Code-in task. I've fixed some spelling mistakes and coding conventions.

[robbyoconnor]

Your branch has conflicts which must be resolved.

[robbyoconnor]

Rebase and Squash your commits please.

[robbyoconnor]

@mogoodrich, could you have a look at this?

[mogoodrich]

@kaine119 @robbyoconnor annoying, this diff is so big github won't show me the whole thing. Kai En, can you summarize what you did? Was it just applying the formatter? Thanks!

[dkayiwa]

Looks like so! :)

[kaine119]

@mogoodrich: Yes, I did just apply the formatter. I fixed indents, staggered long lines, and added curly braces to some if statements, i.e:

if (condition) {
    // do stuff
}

instead of

if (condition)
    //do stuff

Here's a (mostly accurate) list of changes.
module-htmlformentry changes.txt

[mogoodrich]

@kaine119 thanks for the very detailed explanation!

[lluismf]

@ mogoodrich Weird that github can't show you the diff, which browser are you using?

[mogoodrich]

Chrome. I can see most of it, but it tells me it is too big to show all of it:

https://github.com/openmrs/openmrs-module-htmlformentry/pull/91/files

(I think I've got all the information I need)

[lluismf]

ok got the same message with Firefox (I was looking at another link).

[mogoodrich]

@kaine119 I just merged this in, but it resulted in compile errors, so I had to revert. See here:

http://bamboo.pih-emr.org:8085/browse/MIREBALAIS-HFE-467