PHP Fatal Error Call to Undefined Method...that I'm 99% sure is defined

Question

session.php

include("database.php");

function addPOTW($subweek, $subtitle, $subcaption, $subsubmittedby)
{
    global $database, $form;
    /* Errors exist, have user correct them */
    if ($form->num_errors > 0) {
        return 1; // Errors with form
    }
    /* No errors, add the new POTW to the database */
    else {
        if ($database->addNewPOTW($subweek, $subtitle, $subcaption, $subsubmittedby, $subfile)) {
            return 0; //Event signup added succesfully
        } else {
            return 2; //Event signup attempt failed
        }
    }
}

This is my function, "addPOTW", located in the file session.php (with useless parts redacted). For some reason, I keep getting the error message: "Fatal error: Call to undefined method MySQLDB::addNewPOTW()" even though it's defined right here:

database.php

class MYSQLDB {
    function addNewPOTW($date, $title, $caption, $submitter, $filepath)
        {
            $q = "INSERT INTO `" . TBL_POTW . "` VALUES ('','$date','$title','$caption','$submitter','$filepath')";
            return mysql_query($q, $this->connection);
        }
}

I have other functions in session.php accessing functions in database.php using the $database variable in the exact same way, and they work perfectly fine. Any ideas why only this one function is being reported as undefined??

Are you sure the version of database.php being used in the application is the same one you're looking at? — Phil
– Phil, Commented Dec 22, 2011 at 5:31
Also try var_dump($database) right before you call the method. — Francis Avila
– Francis Avila, Commented Dec 22, 2011 at 5:32

alex · Accepted Answer · 2011-12-22 05:33:02Z

2

Because you are referring to $this, you would need to instantiate an object from that class and then call the method.

Something like this should get it working...

$database = new MYSQLDB;

Make sure you have it in scope before your addPOTW() function.

edited Dec 22, 2011 at 5:33

answered Dec 22, 2011 at 5:28

alex

492k205 gold badges891 silver badges992 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Phil Over a year ago

Can't use static as it makes use of $this

alex Over a year ago

@Phil: Thanks, didn't notice that. Fixed.

Phil Over a year ago

I believe the global (eww) $database is meant to be an instance of MYSQLDB. FYI, unless indicated by public, protected or private keyword, class methods (functions) are implicitly public

alex Over a year ago

@Phil: Yeah, that's what I assumed. Everything public except noted otherwise is a relic from PHP4's OO implementation I believe.

Collectives™ on Stack Overflow

PHP Fatal Error Call to Undefined Method...that I'm 99% sure is defined

session.php

database.php

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

session.php

database.php

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related