I been working on a script since yesterday and I think all was well until I commented the Redirect::to index line after my script has gone through. I didn’t noticed any errors and all the data in my forms gets uploaded to mysql as it should.
What I just noticed by staying on the same page after the update is this message: the ( if($user->data()->group == 1) {) line
Fatal error: Call to a member function data() on boolean in C:\xampp\htdocs\database\includes\title.php on line 6
I tried removing the require_once title.php in my script and it removed the fatal error problem but also noticed that my form no longer completely refresh itself. it stops right after the first field which is a select tag for [siteid] and the option tag doesn’t reload the content that was populated by mysql prior to updating. When I first load the page everything is fine and it update correctly to all columns in my database. I been looking for an explanation but after so many hours looking at this I just can’t see why this is happening? any idea?
this is the script that causes the error:
<?php
if(Session::exists('home')){
echo '<span class="flash">' . Session::flash('home') . '</span>';
}
// could say if user ! permission than redirect to another page
if($user->data()->group == 1) {
echo '<br /><p class="title">User: <a href="profile.php?user='
. escape($user->data()->id) . '">' . escape($user->data()->firstname) . ' ' . escape($user->data()->lastname) .'</a></p>';
}else{
echo '<br /><p class="title" >Administrator: <a class="title a" href="profile.php?user='
. escape($user->data()->id) . '">' . escape($user->data()->firstname) . ' ' . escape($user->data()->lastname) .'</a></p>';///changed email to ID in user->data()->id
}
?>
this is my html form:
<form action="" method="post" enctype="multipart/form-data" id="uploadImage">
<div class="fluid stretch">
<label for="FSSiteid" class="labelStretch">Where was the drill was performed? </label>
<br />
<select name="FSSiteid" required class="input" id="FSSiteid">
<option > Select a Site </option>
<?php
$result_site = DB::getInstance()->query("SELECT id, site FROM sites");
if($result_site->count()){
foreach($result_site->results() as $row){
$selected = $row->id == $user->data()->FSSiteid ? ' selected="selected"' : null;?>
<option value="<?php echo $row->id ?>"<?php echo $selected?> ><?php echo $row->site ?></option>
<?php
}
}
?>
</select>
</div> ...
That error says there is boolean value (true of false) in your $user variable, but you’re trying to use it as an object (calling $user->data()). Check where value for that variable come from.
yes I know??? it was working before without a problem. the value is not a bolean at all, print_r returns the content of the variable and it is not a boolean it is an object. I can actually call this object by itself with an echo statement and I get the right value returned. I just re-tested. whats weird is I don’t get this error message when I first load the page; the script that is giving problem is working fine. Only after I hit submit button on my form. the page won’t reload totally and I get this error boolean message
the only clue that I had that something wasn’t working properly is the Flash message I am supposed to get in my index.php file once the firesafety.php has sucessfully executed it’s script. When I look at my DB all fields were updated properly. I brought this Flash message into the firesafety.php file to check if it was set and it echos perfectly on the page, but won’t show when I redirect to index.php with the same session variable set in flash?
here is the validation and redirect to index script from firesafety.php
well I;m pretty sure that this is not the culprit.
I actually have another question on the board regarding this piggy, I;m new to PDO and was wondering about creating new instances I have tried to give that line a different variable name but when I do so the whole script stops working. When I var-dump $user I get all the instances that I created with DB::getInstance() method. The insert() method is the actual query to mysql. Everything from $user=DB::getInstance()->insert(… works ok
I’ve echoed $user->data()->group before and after the if statement and get the value 2 which is what I would expect.
<div id="main" class="fluid ">
<div class="pageTitle">
<?php include ("includes/title.php");
echo $user->data()->group;////////////////////////// value returned = 2 which is the correct value in my database for this field and current user.
?>
</div pageTitle>
<form action="" method="post" enctype="multipart/form-data" id="uploadImage">
<div class="fluid stretch">
<label for="FSSiteid" class="labelStretch">Where was the drill was performed? </label>
<br />
<select name="FSSiteid" required class="input" id="FSSiteid">
<option > Select a Site </option>
<?php
$result_site = DB::getInstance()->query("SELECT id, site FROM sites");
if($result_site->count()){
foreach($result_site->results() as $row){
$selected = $row->id == $user->data()->FSSiteid ? ' selected="selected"' : null;?>
<option value="<?php echo $row->id ?>"<?php echo $selected?> ><?php echo $row->site ?></option>
<?php
}
}
?>
</select>
</div>
Te more I think about it, the more I think it is my local server that is giving me grief. I’ve had this problem before.
So, I just got out of bed, made coffee and checked my page again and guest what? Now, it’s working ok without a fatal error and I didn’t do nothing! The only thing that is still not working is the flash message that I am passing from this page to my index page notifying me that the record was successfully updated.
What is even weirder, now it’s my Update Profile page that is screwing with me. When I try to update a record of a user the page returns an error saying that this email address already exist(I have set a rule throughout my site for email to be unique) but this shouldn’t matter since I am updating the user’s own current record???
When I had a similar problem about a week ago, I talked to someone who said I might be having a latent server problem. I guess my next question is, how do you check if your local server is getting lazy lol?
It’s entirely possible that when your script attempted to fill the $user variable, it failed, and thus $user was set to FALSE (the default return of many database functions that suffer an error).
This would then cause $user to be a boolean (FALSE), and $user->data() to generate the OP error.
You may want to check with your host about their database server stability.
User::find() has the possibility of returning FALSE and your constructor doesnt check for this state…might want to catch that in the find function (finding no data should be an error!)
public function find($user = null) {
if($user){
$field = (is_numeric($user)) ? 'id' : 'email';
$data = $this->_db->get('associates', array($field, '=', $user));
//echo '<pre>' . print_r($data, 1) . '</pre>';/////////////////////////////////////////////////////////////////////////////
if($data->count()) {
$this->_data = $data->first();
return true;
}
//If you get to this point, something has gone wrong!!!!
}
return false;
}
Incidentally, this will also fail if user’s ID = 0, or email = “0” or “” (because then something gets passed into $user that evaluates as FALSE for purposes of IF)