doofinder / doofinder-wordpress

Integrate Doofinder in your WordPress site with (almost) no effort.
GNU General Public License v3.0
2 stars 3 forks source link

Wordpress / WooCommerce Refactor: Data migration #70

Closed pedromcp90 closed 1 year ago

pedromcp90 commented 1 year ago

Required by: doofinder-wordpress/issues/65

Aitor-Corrales commented 1 year ago

I don't know how much can I dive into this, but I was wondering about constructing a dependency injection container in order to support this technique. Wordpress hasn't got this container by default but there are several ways to implement this and to improve the code by using it:

https://carlalexander.ca/dependency-injection-wordpress/

Aitor-Corrales commented 1 year ago

I think we should avoid this massive usage of static functionality as it is not a recommended practice