Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use Plugin class as container for WPDB #26

Closed
rickard2 opened this issue Apr 6, 2013 · 1 comment
Closed

Use Plugin class as container for WPDB #26

rickard2 opened this issue Apr 6, 2013 · 1 comment
Assignees
@rickard2
Labels
code quality
Milestone
2.3

Comments

@rickard2
Copy link
Owner

rickard2 commented Apr 6, 2013

Instead of injecting WPDB into every class that needs it, let Plugin return it instead. Probably every class that needs WPDB also need Plugin
@rickard2
Copy link
Owner Author

Same problem with new Config classes, should be injected into Plugin (or created there?)

@ghost ghost assigned rickard2 May 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rickard2 rickard2

Labels
code quality
Projects
None yet
Milestone
2.3
Development

No branches or pull requests
1 participant
@rickard2