#18949 [Csd->Opn]: ITX: Minor mistake in 'placeholderExists()' ?

From: Date: Fri, 23 Aug 2002 12:38:45 +0000
Subject: #18949 [Csd->Opn]: ITX: Minor mistake in 'placeholderExists()' ?
References: 1  Groups: php.pear.dev 
Request: Send a blank email to pear-dev+get-8620@lists.php.net to get a copy of this message
ID: 18949 User updated by: heino@gehlsen.dk Reported By: heino@gehlsen.dk -Status: Closed +Status: Open Bug Type: PEAR related Operating System: Debian 3.0 -PHP Version: 4.2.1 +PHP Version: CVS (rev. 1.5) New Comment: This bug was kind of a double-bug, since the same typo was made two times in the same function – both when $block is empty (fixed) and when it is set (not fixed). Sorry for the lack of detail... Previous Comments: ------------------------------------------------------------------------ [2002-08-23 06:58:06] mj@php.net This bug has been fixed in CVS. In case this was a PHP problem, snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. In case this was a documentation problem, the fix will show up soon at http://www.php.net/manual/. In case this was a PHP.net website problem, the change will show up on the PHP.net site and on the mirror sites in short time. Thank you for the report, and for helping us make PHP better. ------------------------------------------------------------------------ [2002-08-16 16:46:01] heino@gehlsen.dk I can't get placeholderExists() to work without making minor changes to the code: [...] while (list($k, $variable) = each($variables)) - if ($variable == $placeholder) { + if ($k == $placeholder) { $found = $block; [...] (boolean) $variable != (string) $placeholder. (string) $k = (string) $placeholder. PS. I don't have the time to check if it's placeholderExists() that is buggy or if it is $this->blockvariables' - sorry... ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=18949&edit=1

« previous php.pear.dev (#8620) next »