php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #56909 rename_function should not be be case-sensitive
Submitted: 2006-03-23 17:48 UTC Modified: 2015-02-26 07:34 UTC
Votes:1
Avg. Score:3.0 ± 0.0
Reproduced:0 of 0 (0.0%)
From: mattsch at gmail dot com Assigned:
Status: Suspended Package: apd (PECL)
PHP Version: 5_1 CVS-2006-03-23 OS: Gentoo
Private report: No CVE-ID: None
View Developer Edit
 [2006-03-23 17:48 UTC] mattsch at gmail dot com 
Description:
------------
rename_function should be not case-sensitive as that seems to conflict with php which specifically states that its functions aren't case sensitive.

Reproduce code:
---------------
function testFunc(){
 echo 'test';
}
rename_function('testFunc','testFuncOther');

Outputs this:

PHP Warning:  rename_function(testFunc, testFuncOther) failed: testFunc does not exist!

Workaround:

function testFunc(){
 echo 'test';
}
rename_function('testfunc','testFuncOther');

Expected result:
----------------
Rename the function.

Actual result:
--------------
Only renames if the original function name is specified in lower case.

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2006-03-23 17:54 UTC] mattsch at gmail dot com 
changing summary
 [2008-06-16 21:25 UTC] geoff at spacevs dot com 
Here is a patch to fix this.

--- orig.c	2008-06-17 10:27:26.000000000 +1000
+++ php_apd.c	2008-06-17 11:24:41.000000000 +1000
@@ -663,6 +663,7 @@
 PHP_FUNCTION(rename_function)
 {
 	zval **z_orig_fname, **z_new_fname;
+	char *orig_fname;
 	zend_function *func, *dummy_func;
 
 	if( ZEND_NUM_ARGS() != 2 ||
@@ -671,6 +672,13 @@
 			ZEND_WRONG_PARAM_COUNT();
 		}
 
+	orig_fname = Z_STRVAL_PP(z_orig_fname);
+	/* Convert to to lower case */
+	for(i = 0; i < Z_STRLEN_PP(z_orig_fname); i++)
+		if ((orig_fname[i] > 64) && (orig_fname[i] < 91))
+			orig_fname[i] += 32;
+	Z_STRVAL_PP(z_orig_fname) = orig_fname;
+
 	convert_to_string_ex(z_orig_fname);
 	convert_to_string_ex(z_new_fname);
 [2008-06-16 21:27 UTC] geoff at spacevs dot com 
Whoops, missed a variable, this patch fixes this...

--- orig.c	2008-06-17 10:27:26.000000000 +1000
+++ php_apd.c	2008-06-17 11:26:55.000000000 +1000
@@ -663,6 +663,8 @@
 PHP_FUNCTION(rename_function)
 {
 	zval **z_orig_fname, **z_new_fname;
+	char *orig_fname;
+	int i;
 	zend_function *func, *dummy_func;
 
 	if( ZEND_NUM_ARGS() != 2 ||
@@ -671,6 +673,13 @@
 			ZEND_WRONG_PARAM_COUNT();
 		}
 
+	orig_fname = Z_STRVAL_PP(z_orig_fname);
+	/* Convert to to lower case */
+	for(i = 0; i < Z_STRLEN_PP(z_orig_fname); i++)
+		if ((orig_fname[i] > 64) && (orig_fname[i] < 91))
+			orig_fname[i] += 32;
+	Z_STRVAL_PP(z_orig_fname) = orig_fname;
+
 	convert_to_string_ex(z_orig_fname);
 	convert_to_string_ex(z_new_fname);
 [2015-02-26 07:34 UTC] krakjoe@php.net
-Status: Open +Status: Suspended
 [2015-02-26 07:34 UTC] krakjoe@php.net 
APD hasn't had a release in 10 years, this means it's source code is way out of sync with modern PHP.

I'm going to mark this bug as suspended, the report can still be found if a maintainer for APD comes forward.

Sorry about the wait.
 
PHP Copyright © 2001-2025 The PHP Group
All rights reserved. 		Last updated: Wed Dec 17 23:00:01 2025 UTC