Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.1.14

Bug #18586 Register Action Handle only accepts one class
Submitted: 2011-06-05 01:16 UTC
From: renownedmedia Assigned: garrettw
Status: Closed Package: Net_SmartIRC (version 1.0.2)
PHP Version: 5.3.1 OS: Linux
Roadmaps: 1.1.0    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 41 + 5 = ?

 
 [2011-06-05 01:16 UTC] renownedmedia (Renowned Media)
Description: ------------ When running multiple $irc->registerActionhandler() function calls, the class argument (third parameter), only accepts one class. It appears that later declarations overwrite previous ones. Test script: --------------- $irc->registerActionhandler(SMARTIRC_TYPE_QUERY, '!a1', $class1, 'message_handle'); $irc->registerActionhandler(SMARTIRC_TYPE_QUERY, '!a2', $class2, 'message_handle'); Expected result: ---------------- The method in $class1 and the method in $class2 to be executed. Actual result: -------------- Jun 04 16:00:12 SmartIRC.php(2309) DEBUG_MESSAGEHANDLER: calling internal method "Net_SmartIRC->_event_privmsg" (by string) Jun 04 16:00:12 SmartIRC.php(2349) DEBUG_ACTIONHANDLER: actionhandler match found for id: 1 type: 2 message: "!a2" regex: "/!a2/" Jun 04 16:00:12 SmartIRC.php(2358) DEBUG_ACTIONHANDLER: method doesn't exist! "NameSpace\a1->message_handle"

Comments

 [2014-09-03 20:26 UTC] garrettw (Garrett Whitehorn)
-Status: Open +Status: Feedback
Thank you for taking the time to report a problem with the package. This problem may have been already fixed by a previous change that is in the SVN of the package. Please checking out the SVN repository of this package and upgrade svn checkout svn.php.net/repository/pear/packages/Net_SmartIRC/trunk pear upgrade package2.xml or pear upgrade package.xml If you are able to reproduce the bug with the latest SVN, please change the status back to "Open". Again, thank you for your continued support of PEAR. I've made a lot of changes since I took over this package, so I may have fixed this without realizing it. In any case, I just tried to reproduce this issue with my latest code and I couldn't. It works properly now.
 [2014-09-03 20:28 UTC] garrettw (Garrett Whitehorn)
-Status: Feedback +Status: Closed -Assigned To: +Assigned To: garrettw
Well of course we aren't using SVN anymore but rather Github, so go there. https://github.com/pear/Net_SmartIRC Should have marked this closed.