Fatal error: Call to a member function setFlags() on a non-object in /class_appcontroller.php

[FounderSim]

When I access myadopts/add I get this error:
[spoiler]
its driving me crazy!!
[/spoiler]

Fatal error: Call to a member function setFlags() on a non-object in full_path_hidden/class_appcontroller.php on line 192;


So I added two functions.

myadopts.php

	
//sim edit
	public function add(){
	
		//die("EAT ME");
		$mysidia = Registry::get("mysidia");	
	}
	
//end sim edit

[/code]

my adoptsview.php

	//
	// Sim Edit
	//
	public function add(){
		
		$mysidia = Registry::get("mysidia");
		
		$document = $this->document;	
		
		$document->setTitle("Do you want to eat me");	
		
		$document->add(new Comment("<br><br>Welcome earthlings...<br>"));

		$eatForm = new Form("eaters", "eater", "post");	


		$eatForm->add(new Button("Eat me if you can", "submit", "submit"));	
		
		$document->add($eatForm);	
	}
	
	//
	// End Sim Edit
	//

[Hwona]

Oookay... I'm not exactly great with v1.3.4 or coding, but just to check...
in your adoptsview.php:
$eatForm = new Form("eaters", "eater", "post");
It looks like you're trying to post the form to a page called 'eater' - at least that's how it works with my version.

Try this:
replace $eatForm = new Form("eaters", "eater", "post"); with $eatForm = new Form("eaters", "", "post");

In adoptsview.php, add this to the add function right below $document->setTitle("Do you want to eat me");

if($mysidia->input->post("submit")){
put your "eater" code here?
}

I'm not sure if this is what you were looking for, but I hope it works out for you!

[FounderSim]

Don't believe that was the issue:

Believe the issue is in the constructor of myadopts.php

if( $this->action != "index"){
				try{
					$this->adopt = new OwnedAdoptable($mysidia->input->get("aid"));	
					if($this->adopt->getOwner() != $mysidia->user->username) throw new NoPermissionException("permission");		
					$this->image = $this->adopt->getImage("gui");
				}

if( $this->action != "index"){, if the page isn't index page of myadopts/, it tries to fetch a pet id ? =)

[Hwona]

Originally Posted by FounderSim

Don't believe that was the issue:

Believe the issue is in the constructor of myadopts.php

if( $this->action != "index"){
				try{
					$this->adopt = new OwnedAdoptable($mysidia->input->get("aid"));	
					if($this->adopt->getOwner() != $mysidia->user->username) throw new NoPermissionException("permission");		
					$this->image = $this->adopt->getImage("gui");
				}

if( $this->action != "index"){, if the page isn't index page of myadopts/, it tries to fetch a pet id ? =)

Uh... does v.1.3.4 need the try{}?

[FounderSim]

Originally Posted by Wallie987

Uh... does v.1.3.4 need the try{}?

I don't know. Your asking the wrong guy that question. The try block could be removed, but might turn into bigger issues down the road.

I will either be creating a if elseif then with the $this->action or or just use a separate page. =0

[Hwona]

Other such pages Ive seen dont use try.... even though they aim for the same effect.

[FounderSim]

The try statement is better then using IF statements for error checking/handling. Especially when you need a few blocks of code to check for errors opposed to checking each line 1 by 1 for errors.

Maybe HOF hasn't gotten around to rest of pages.

[Hwona]

Hi! I think I know why this is happening. Are the two codes you posted the entire files, or just a section?
I was a bit confused with this quite some time ago. Did you construct myadopts like this? I suspect that your were missing the "catch" statement in the _construct function.

PHP Code:

<?php

use Resource\Native\Integer;
use Resource\Native\String;

class MyadoptsController extends AppController{

    const PARAM = "aid";
    const PARAM2 = "confirm";
    private $adopt;
    private $image;

    public function __construct(){
        parent::__construct("member");
        $mysidia = Registry::get("mysidia");
        if($this->action != "index"){
            try{
                $this->adopt = new OwnedAdoptable($mysidia->input->get("aid"));    
                if($this->adopt->getOwner() != $mysidia->user->username) throw new NoPermissionException("permission");        
                $this->image = $this->adopt->getImage("gui");
            }
            catch(AdoptNotfoundException $pne){
                $this->setFlags("nonexist_title", "nonexist");
            }                          
        }
    }
    
    public function index(){
        $mysidia = Registry::get("mysidia");
        $total = $mysidia->db->select("owned_adoptables", array("aid"), "owner = '{$mysidia->user->username}'")->rowCount();
        $pagination = new Pagination($total, 10, "myadopts");
        $pagination->setPage($mysidia->input->get("page"));    
        $stmt = $mysidia->db->select("owned_adoptables", array("aid"), "owner = '{$mysidia->user->username}' ORDER BY totalclicks LIMIT {$pagination->getLimit()},{$pagination->getRowsperPage()}");        
        $this->setField("pagination", $pagination);
        $this->setField("stmt", new DatabaseStatement($stmt));
    }
    
    public function add(){
        $mysidia = Registry::get("mysidia");    
    }
    
}
?>

[FounderSim]

See previous post, what I posted about Wallie.

Code I replaced with mine. I added a $safelist array and checked if in it.

class MyadoptsController extends AppController{

    const PARAM = "aid";
    const PARAM2 = "confirm";
	private $safelist = array("page1", "page2", "page3");
	private $adopt;
	private $image;

    public function __construct(){
        parent::__construct("member");
		$mysidia = Registry::get("mysidia");

		if(!in_array($this->action, $this->safelist))
		{
			if( $this->action != "index"){
				try{
					$this->adopt = new OwnedAdoptable($mysidia->input->get("aid"));	
					if($this->adopt->getOwner() != $mysidia->user->username) throw new NoPermissionException("permission");		
					$this->image = $this->adopt->getImage("gui");
				}
				catch(AdoptNotfoundException $pne){
					$this->setFlags("nonexist_title", "nonexist");
				}              			
			}
		}
    }

[Hwona]

Originally Posted by FounderSim

See previous post, what I posted about Wallie.

Code I replaced with mine. I added a $safelist array and checked if in it.

class MyadoptsController extends AppController{

    const PARAM = "aid";
    const PARAM2 = "confirm";
	private $safelist = array("page1", "page2", "page3");
	private $adopt;
	private $image;

    public function __construct(){
        parent::__construct("member");
		$mysidia = Registry::get("mysidia");

		if(!in_array($this->action, $this->safelist))
		{
			if( $this->action != "index"){
				try{
					$this->adopt = new OwnedAdoptable($mysidia->input->get("aid"));	
					if($this->adopt->getOwner() != $mysidia->user->username) throw new NoPermissionException("permission");		
					$this->image = $this->adopt->getImage("gui");
				}
				catch(AdoptNotfoundException $pne){
					$this->setFlags("nonexist_title", "nonexist");
				}              			
			}
		}
    }

Umm, have you tried just checking the page action in the 'if' condition? So... if($this->action != "index" && $this->action != "Page 1"....)
Other than that, I really have no idea as to what could possibly have gone wrong. I'm assuming everything works fine if you revert the file to its original state? :L