Function definitions within a condition do not function as expected thanks to JS hoisting, so it is better to avoid it, and keep ESLint happy as well.
See discussion at #8622 with @pmario
* fix breaking bug in image tiddler attachment
* fix comments
* fix code format
* refactor processIncomingTiddler flow
* remove whitespaces after if statements
* refactor attachment_blob persistence flow
* refactor process tiddler to support different attachments
* add tests for attachment
* add more attachement test cases
* working on adding instanbul for test coverage report
* code coverage report generation
* remove unnecessary packages
* fix comments
We create a system bag to contain each plugin/theme/language. It seems wasteful because it results in lots of bags, but the semantics are exactly right and so it seems like the right approach
Thank you @PotOfCoffee2Go I ended up taking some of your code from #8101 to get this up and running. There's still some stuff missing (like the tests!) but it gets things moving.