Fatal error: Using $this when not in object context

Question

here is the part if having error.

Fatal error: Using $this when not in object context in /pb_events.php on line 6

line 6 is: $jpp = $this->vars->data["jpp"];

function DoEvents($this) {

    global $_CONF, $_PAGE, $_TSM , $base;

    $jpp = $this->vars->data["jpp"];

    $cache["departments"] = $this->db->QFetchRowArray("SELECT * FROM {$this->tables[job_departments]}");
    $cache["locations"] = $this->db->QFetchRowArray("SELECT * FROM {$this->tables[job_location]}");
    $cache["names"] = $this->db->QFetchRowArray("SELECT * FROM {$this->tables[job_names]}");
    $cache["categories"] = $this->db->QFetchRowArray("SELECT * FROM {$this->tables[job_categories]}");

Thanks a lot! appreciate!

user187291 · Accepted Answer · 2009-10-29T14:26:12.847

15

$this only makes sense in methods, not in functions

this is ok

class Foo {
     function bar() {
          $this->...

this is not

function some() {
    $this->

// edit: didn't notice he passes "$this" as parameter

advice: simply replace "$this" with "$somethingElse"

edited Oct 29 '09 at 14:26

answered Oct 29 '09 at 14:00

user187291

52,425
19
93
127

thanks for reply! this is an opensource softwares code which was working well on PHP4. now i try on PHP5 and has errors :/ I'm not very good at php :/ trying to sort out the issue according to your advise... – designer-trying-coding Oct 29 '09 at 14:06
so how can i edit this? :/ if i put a class on top, the other pages are not working... big mess – designer-trying-coding Oct 29 '09 at 14:15
wowowowow man!! I LOVE YOU@@@@!!!! LIFE SAVER!!!! THANKS A LOT MAN!! :) )woaaaa so happy!! – designer-trying-coding Oct 29 '09 at 14:40
You will also get this error if you use self::bar() even inside Foo. – gcb Jan 19 '11 at 22:41
You will also get this error if you declare you method as static. All static methods are considered to be called outside the class scope – gcb Jan 19 '11 at 22:42

score 7 · Answer 2 · answered Oct 29 '09 at 14:01

7

You cannot pass $this to a procedural function. $this is a reserved variable.

answered Oct 29 '09 at 14:01

Jeff Ober

4,857
19
15

hey, thanks! so what is your advise for solution? can you be a bit more clear? Im not very good at php :/ – designer-trying-coding Oct 29 '09 at 14:16
3

just change the name of the variable passed. check my answer. – Michal M Oct 29 '09 at 14:30
Michael is absolutely correct. Just change the name of the variable. – Jeff Ober Oct 29 '09 at 14:54

score 5 · Answer 3 · answered Oct 29 '09 at 14:31

As per my comments. You want to use $this as passed variable and php doesn't allow it outside class methods body.

function DoEvents($obj) {

    global $_CONF, $_PAGE, $_TSM , $base;

    $jpp = $obj->vars->data["jpp"];

    $cache["departments"] = $obj->db->QFetchRowArray("SELECT * FROM {$obj->tables[job_departments]}");
    $cache["locations"] = $obj->db->QFetchRowArray("SELECT * FROM {$obj->tables[job_location]}");
    $cache["names"] = $obj->db->QFetchRowArray("SELECT * FROM {$obj->tables[job_names]}");
    $cache["categories"] = $obj->db->QFetchRowArray("SELECT * FROM {$obj->tables[job_categories]}");

yes ! thats the issue! how silly im! sorted just before your comment... you life saver too!! thanks a lot!!! super you guys are! — designer-trying-coding, Oct 29 '09 at 14:42

score 0 · Answer 4 · answered Oct 29 '09 at 14:21

0

You have to make the object first.

   $object=new Myobject;
   DoEvents($object);

answered Oct 29 '09 at 14:21

dnagirl

19,786
13
77
119

Fatal error: Using $this when not in object context

4 Answers4

Linked

Related