Config method about tgm-plugin-activation

[thomasgriffin]

Could we add all of these as parameters to tgmpa_register_plugins? Like this:

tgmpa_register_plugins( $plugins, $domain, $menu, $default_path, $strings );

I don't see any conflicts of doing this. Only plugins takes an array of plugins..everything else is just a single string.

[GaryJones]

We could, but, if the developer only wanted to specify one particular string, he'd need to add values for everything preceding it.

Better would be:

tgmpa_register_plugins( $plugins, $config );

as then $config could be an array with keys for domain, menu, default_path and strings (with array values) or just whatever they want to overwrite. In that case, I'd suggest changing the function name to something simple like tgmpa() since this would then appear to be the only function that developers would ever need to call.

[thomasgriffin]

Good point. And I like the idea of just having developers make one function call. tgmpa() makes sense, I say we go with that. I'll see what I can come up with.

[GaryJones]

See pull request #22 :-)

[thomasgriffin]

Well that was quick :)

[thomasgriffin]

Closing this issue - fixed in dev.