search

WPTT / theme-sniffer

Theme Sniffer plugin using sniffs.
MIT License
269 stars 3 forks source link

Add a dependency injection container #182

Closed dingo-d closed 4 years ago

dingo-d commented 5 years ago

Added a dependency injection container and removed the Assets_* classes, as they complicated and coupled the code for no reason.

This is the first step towards refactoring the plugin.

dingo-d commented 4 years ago

Hmmm, I'll have to check this and see if I can just cache the container in the 'production' version. This is a significant slowdown and it shouldn't happen.

dingo-d commented 4 years ago

I couldn't reproduce the slowness issue locally. I've added a cached DI container so it should be faster. I'll also update the documentation regarding developing with it