0004619: Allow hooks to include a file

(0013705)
groundup (Team Member)
2011-02-07 18:57

You'd need to add it to that array. What would you use to split it? I'm thinking a colon would be fine. The column, in MySQL, would look like: func1,func2,func3,file1:func4,file1:func5,file2:func6

(0013707)
[SiNaN] (Developer)
2011-02-08 01:56

Colon looks fine.

(0014146)
AngelinaBelle (Team Member)
2011-07-29 07:51

In the further future, will it be advantageous to take hooks completely out of $modSettings so they can have a better data structure?

(0014171)
Suki (Team Member)
2011-10-03 12:17

Agree, $modSettings is way too bloated already.

(0014243)
groundup (Team Member)
2011-10-29 16:01

An alternative to even having this is to use an autoloader. It would be very easy to do and you could use the autoloader to increase the internal security of your functions.

(0014664)
emanuele (Developer)
2012-08-14 11:33

commit 1f94fbe8ad83523a960a094fb519d370b9a104c1
Author: emanuele
Date: Tue Jul 10

That should allow people to define a file to include when they add a function to a hook

Plus few other commits to fix the bugs :P

News:

Simple Machines Bug and Feature Tracker

Notes
(0013705) groundup (Team Member) 2011-02-07 18:57	You'd need to add it to that array. What would you use to split it? I'm thinking a colon would be fine. The column, in MySQL, would look like: func1,func2,func3,file1:func4,file1:func5,file2:func6

(0013707) [SiNaN] (Developer) 2011-02-08 01:56	Colon looks fine.

(0014146) AngelinaBelle (Team Member) 2011-07-29 07:51	In the further future, will it be advantageous to take hooks completely out of $modSettings so they can have a better data structure?

(0014171) Suki (Team Member) 2011-10-03 12:17	Agree, $modSettings is way too bloated already.

(0014243) groundup (Team Member) 2011-10-29 16:01	An alternative to even having this is to use an autoloader. It would be very easy to do and you could use the autoloader to increase the internal security of your functions.

(0014664) emanuele (Developer) 2012-08-14 11:33	commit 1f94fbe8ad83523a960a094fb519d370b9a104c1 Author: emanuele Date: Tue Jul 10 That should allow people to define a file to include when they add a function to a hook Plus few other commits to fix the bugs :P

Relationships