Create plugins as a TGMPA_Plugin object #49

Closed
GaryJones opened this Issue Oct 17, 2011 · 2 comments

Comments

Projects
None yet
2 participants
@GaryJones
Owner

GaryJones commented Oct 17, 2011

Although the registration of plugins (as found in example.php) can stay the same, I think internally, the plugins property of the main class should contain an associative array of objects, rather than an indexed array of arrays.

For this, we'll need a new class (suggested as TGMPA_Plugin).

@ghost ghost assigned GaryJones Oct 17, 2011

@thomasgriffin

This comment has been minimized.

Show comment Hide comment
@thomasgriffin

thomasgriffin Oct 17, 2011

Contributor

Not doubting that there is a good reason for this, but what is the main benefit of going this way?

Contributor

thomasgriffin commented Oct 17, 2011

Not doubting that there is a good reason for this, but what is the main benefit of going this way?

@thomasgriffin

This comment has been minimized.

Show comment Hide comment
@thomasgriffin

thomasgriffin Dec 26, 2011

Contributor

Coming back to this now, I really like this idea. Definitely a 3.0 thing, but I am all for going this way. This should make referencing outside of the class much easier.

Contributor

thomasgriffin commented Dec 26, 2011

Coming back to this now, I really like this idea. Definitely a 3.0 thing, but I am all for going this way. This should make referencing outside of the class much easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment