ElkArte Community

Extending Elk => Addons => Addons ideas and questions => Topic started by: emanuele on June 18, 2016, 05:33:50 pm

Title: To addon developers, question about integrate_actions
Post by: emanuele on June 18, 2016, 05:33:50 pm
Code: [Select]
		call_integration_hook('integrate_actions', array(&$actionArray, &$adminActions));

Did anyone use $adminActions in this hook?
I'm playing with the dispatcher and this parameter just became useless (at least in core, not sure in addons), so I was wondering if anyone used it in any addon.
Title: Re: To addon developers, question about integrate_actions
Post by: ahrasis on June 19, 2016, 03:07:56 am
May be in multi forum addon but I have to check.
Title: Re: To addon developers, question about integrate_actions
Post by: Spuds on June 19, 2016, 11:11:09 am
I think I used it in SimplePortal, Misery and one I can't talk about :P  May not have been needed in any of those but there since  they were converted, they remained in some cases.
Title: Re: To addon developers, question about integrate_actions
Post by: emanuele on June 19, 2016, 11:28:06 am
Thanks!
I'll check those and see if it breaks a lot of stuff. :D

Basically, I realized that in 1.1 check for file_exist is redundant, because of the autoloader, and it's enough to test for is_callable. The autoloader already knows where to look for classes picking up by itself ADMINDIR or CONTROLLERDIR, so the $adminActions become pointless.