A list of things you could check yourself before anyone else reviews your code

gistfile1.txt

A list of things you could check yourself before anyone else reviews your code:

- all files must have an empty newline at OEF (use .editorconfig for that)
- check for proper indentation (smarty indentations are often wrong after copy-paste)
- remove unnecessary newlines
- check that docblocks are valid and up-to-date
- remove unnecessary if ($foo == true). if ($foo) should be enough
- check that no merge conflict markers (>>>>>>>) are left behind
- check for unused variables
- extract long ifs to own method if possible
- prefer '' over ""
- remove commented code
- .prop('disabled', true) instead of .attr('disabled', 'disabled')
- run unit tests
- run acceptance tests
- run jshint
- run severity tests