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:

its driving me crazy!!


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]

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]

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

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]

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

[FounderSim]

Post #9 is the fix to my problem. I am sharing answers to questions or problems I run across as I edit things in the mysidia script to help other people.