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

Request #1337 DB::assertExtension should use PEAR::loadExtension
Submitted: 2004-05-04 19:18 UTC
From: jausions Assigned: danielc
Status: Closed Package: DB
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
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 : 11 - 1 = ?

 
 [2004-05-04 19:18 UTC] jausions
Description: ------------ The DB::assertExtension() method should use the PEAR::loadExtension() method instead. Diff file can also be found at: http://www.11abacus.com/cvs/pear/DB.diff Reproduce code: --------------- Index: DB.php =================================================================== RCS file: /repository/pear/DB/DB.php,v retrieving revision 1.58 diff -u -r1.58 DB.php --- DB.php 13 Mar 2004 16:17:19 -0000 1.58 +++ DB.php 4 May 2004 19:23:41 -0000 @@ -704,13 +704,7 @@ */ function assertExtension($name) { - if (!extension_loaded($name)) { - $dlext = OS_WINDOWS ? '.dll' : '.so'; - $dlprefix = OS_WINDOWS ? 'php_' : ''; - @dl($dlprefix . $name . $dlext); - return extension_loaded($name); - } - return true; + return PEAR::loadExtension($name); } // }}} }

Comments

 [2005-02-02 22:48 UTC] danielc
Changed in CVS.