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 Add Comment Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: mattsch at gmail dot com
New email:
PHP Version: OS:

 

 [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

Add a Patch

Pull Requests

Add a Pull Request

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-2024 The PHP Group
All rights reserved. 		Last updated: Thu Apr 18 14:01:31 2024 UTC