elkarte/Elkarte — pull request #1460 by Spuds (2014-03-04 17:53:19+00:00)
Title: Fix some more errors

- revert part of an earlier commit which breaks the output of attachment errors from the secondary checks.
- combine class updates,
- improper IIFE constructs could break the compilation when certain min functions were run. This fixes that error when it finds it (only exists on pretty code that I found)
- deferred output always assumed local files, it checks if this is the case now
- added support for pre-minimized files. As it was if a file was already min-ified it would just try again. Now it will use filename matching .min.js to skip these file and simply add them as is to the compilation. saves some process. These files are also always placed at the start of the compilation, reason being generally they are library files (sceditor, sceditor.bbcode) that other scripts need in place, so adding them first is safer.
- broke down the js min method in to 3, simply because there were 3 ways it would try to minimize so having that all in one method was to procedural, so this is a bit cleaner.

To support the above the minimized files were renamed to .min.js

- Fix (maybe) error with the editor localization file not finding sceditor, this was an load order problem so I've set the localization to deferred so its after sceditor and that seems to work, although the docs says the opposite which makes no sense, at least given the example code.