views:

39

answers:

2

I'm creating my class like I always do. I create the member variables, I setup the setter and getter and the I use $this to call the values of the member variables. but for some reason now I'm get the fatal error: Using $this when not in object context

Here is the regular php file which should populates the member variables with their value

try
    {
        $add_advert = new Site();
        $add_advert->add_time = mysqli_real_escape_string($link, $_POST['time']);
        $add_advert->add_url = mysqli_real_escape_string($link, $_POST['URL']);
        $add_advert->add_advertisement($_FILES);
        $feedback = "<div class='succes'>Advertisement succesfully added!</div>";
    }
    catch(Exception $e)
    {
        $feedback = "<div class='error'>";
        $feedback .= $e->getMessage();
        $feedback .= "</div>";
    }

the member variables:

class Site
{   

    protected $add_url;
    protected $add_time;
    protected $extend_id;

the setter and getter:

public function __set($p_sProperty, $p_sValue)
{
    switch($p_sProperty)
    {
        case "add_url":
            $this->add_url = $p_sValue;
        break;
        case "add_time":
            $this->add_time = $p_sValue;
        break;
        case "extend_id":
            $this->extend_id = $p_sValue;
        break;
    }
}//end of setter


public function __get($p_sProperty)
{
    $vResult = "";
    switch($p_sProperty)
    {
        case "add_url":
            $vResult = $this->add_url;
        break;
        case "add_time":
            $vResult = $this->add_time;
        break;
        case "extend_id":
            $vResult = $this->extend_id;
        break;
    }
    return $vResult;
}// end of getter

and the function where I use the $this object (it's only a part of the function), it's the $this->add_time that is causing the problem:

public static function add_advertisement($files)
    {
        global $link, $userid, $useridentify;

        if(!isset($files['image']['tmp_name']) || empty($files['image']['tmp_name'])):
            throw new exception("No image given, enter an image please.");
        endif;

        if($_POST['URL'] == NULL || $_POST['URL'] == "http://"):
            throw new exception("No url given, enter a valid url please.");
        endif;

        if($_POST['time'] == NULL):
            throw new exception("You did not select any option to advertise. Select one of the available advertising options.");
        endif;

        if(!$path = Uploader::upload_file($files, 'images/', 'gif|jpg|jpeg|jpe|png', 300000)):
            throw new exception("Could not upload the image.");
        endif;

        if(self::get_balance() >= $this->add_time):
+3  A: 

You cannot use $this in a static function because there is no instantiation to work off of.

thetaiko
+7  A: 

You cannot use $this in a static method. Static methods, by definition, have no associated instance.

Artefacto
oooh of course! I don't understand why I typed static, I never use that... weird
krike