Favour "forEach" over "map"

Most of the instances of .map were now either returning the value, or
assigning it to a variable. These were the remaining cases where we
were just using it to touch every item in an array
Use lowercase for namespaces

$ grep -r @namespace src/

src//helpers/functions.js: * @namespace insight.helpers.functions
src//Insight.js: * @namespace insight
src//mda/Correlation.js: * @namespace insight.correlation
src//utils/insight.Formatters.js: * @namespace insight.formatters
Update private, "internal" and public members to meet conventions.

Comments to separate each section (private, "internal", public).

A single var declaration for all simple private class variables.
Merge pull request #132 from tpkelly/develop

Favour "forEach" over "map"
Moved addSeries to internal functions and removed it from jsdocs.
Merge branch 'master' of https://github.com/ScottLogic/insight into dev

Conflicts:
	src/charts/Chart.js
Merge pull request #133 from niksoper/dev

Update private, "internal" and public members to meet conventions.