ID: 27659
Comment by: shelever at ukrlink dot net
Reported By: js at enradia dot com
Status: Bogus
Bug Type: Unknown/Other Function
Operating System: linux 2.6.3
PHP Version: 5.0.0RC1
New Comment:
osCommerce 2.2-MS2 gets same error when open admin folder. Error in
file includes/classes/upload.php.
On Windows/IIS5/ISAPI.
Resolwed by commenting:
// self destruct
// $this = null;
Worked at PHP4
Good Luck!
Previous Comments:
------------------------------------------------------------------------
[2004-07-22 15:28:08] mschulznospam at qualserv dot net
Expected behavior is if a script that works in PHP 4 also works in PHP
5. This is definitely a bug, whether it's a bug in PHP or a bug in the
script. But this broke a whole bunch of scripts on our servers so I'm
blaming PHP.
(remove the nospam in my email address if u want to reply)
------------------------------------------------------------------------
[2004-07-21 18:00:33] drew at estara dot com
Anyone who knows PHP objects better than me, as a simple replacement
for:
$this = $foo;
will something like this:
foreach (get_object_vars($foo) as $key => $value)
$this->$key = $value;
work in all cases? It's worked in all the cases I've tested it on so
far (including Gallery), but I figure there must be a problem with it
since no one's mentioned it so far.
Drew
------------------------------------------------------------------------
[2004-07-15 23:26:06] mlemos at acm dot org
There is no assignment of $this in Metabase code. It was a
misunderstanding of the original poster. Metabase works seeminglessly
in PHP 3, PHP 4 and PHP 5.
------------------------------------------------------------------------
[2004-07-15 02:49:27] jbeall at heraldic dot us
Here is a potential solution. Where you would have reassigned $this,
do something like this:
$this->Object = new Whatever();
Or, in the case of unserializing session data
$this->Session = unserialize($data);
Now you have a choice. Your first option is to delegate every method
that needs to be delegated. So, $this would need methods for every
method in $Object. Each method would look something like:
function nameOfFunction()
{
return $this->Object->nameOfFunction();
}
The other option is to intercept all method calls, and then decide on
the fly if they belong to $this or to $this->Object.
If you have properties, I think you are going to have to intercept with
__get() and __set().
You can read about overloading object functions with __get(), __set(),
and __call() here:
http://us2.php.net/manual/en/migration5.oop.php#migration5.oop.overload
I chose the delegation model. If possible, I would recommend that. If
you have properties to deal with, you will have to intercept the __get()
and __set() calls and handle them appropriately.
------------------------------------------------------------------------
[2004-03-23 14:40:09] [EMAIL PROTECTED]
Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
http://www.php.net/manual/ and the instructions on how to report
a bug at http://bugs.php.net/how-to-report.php
Actually this is expected behavior. We explicitly decided to have $this
being readonly because of interna problems with the new engine.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/27659
--
Edit this bug report at http://bugs.php.net/?id=27659&edit=1
