Summary:	ASTERISK-06495: [patch] IFMODULE() - report if module is loaded or not
Reporter:	Olle Johansson (oej)	Labels:
Date Opened:	2006-03-07 13:35:12.000-0600	Date Closed:	2006-10-30 15:49:14.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Functions/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 6671.func.IFMODULE.patch ( 1) func_module.txt
Description:	Reports 1 if module is loaded in Asterisk, otherwise NULL
Comments:	By: Tilghman Lesher (tilghman) 2006-03-07 13:41:44.000-0600 A function returning a true or false value should return "0" not "" for a false value. By: Olle Johansson (oej) 2006-03-07 14:14:19.000-0600 Ok, I was a bit unclear of that. Will fix. By: Olle Johansson (oej) 2006-03-07 14:25:17.000-0600 New version uploaded. By: Clod Patry (junky) 2006-04-23 14:04:29 Returns \"1\" if module exists in memory, why just just say Returns \"1\" if module is loaded. and im getting: polux:/usr/src/asterisk# patch -p0 < func_module.txt patching file loader.c Hunk #1 succeeded at 986 (offset 409 lines). patching file include/asterisk/module.h Hunk #1 succeeded at 112 (offset -77 lines). patching file funcs/func_module.c patch: **** malformed patch at line 145: when trying to apply the patch. By: Jeffrey C. Ollie (jcollie) 2006-04-23 23:18:19 oej has an up-to-date copy of the code in his test branch: http://svn.digium.com/svn/asterisk/team/oej/test-this-branch/funcs/func_module.c By: Olle Johansson (oej) 2006-06-02 02:00:31 Any feedback on this function? Anyone other than myself that sees a need for it? By: Leif Madsen (lmadsen) 2006-06-03 14:02:37 Hrmmm... I could use it to check if func_odbc is loaded (or res_odbc too) to check to see if I should be using it -- if not, then I could fail back onto a local method such as GROUP() (I'm using func_odbc for a lot of things, but one of them is to keep track of the number of calls for a trunk or a vpbx in a remote database so that everyone in the cluster knows how many calls are being used throughout the system). Seems handy. By: Clod Patry (junky) 2006-08-08 21:10:52 i personnaly think this should go in trunk. i would just put the .so optional, so cdr_custom.so or cdr_custom would be the same thing. By: Denis Smirnov (mithraen) 2006-08-21 04:27:27 Updated to last trunk By: Olle Johansson (oej) 2006-10-30 15:49:02.000-0600 Committed to svn trunk, revision 46513.