Call to a member function show() on string on line 15 - php

Question

It worked a week before, I check if the objects are being defined with function class_exists() and they are both fine.
My error:
Call to a member function show() on string on line 15
Which is a function of class $logged_user in profile.php
echo $logged_user->show("Email");
My object $logged_user looks like this:
class LOGGED_USER
{
private $DB;
private $ID;
private $Firstname;
private $Surname;
private $DisplayName;
private $Gender;
private $Birth;
private $Email;
private $Level;
private $FolderPath;
private $LastLogin;
private $LastIP;
private $LevelWord;
private $ProfilePicture;
private $ProfilePictureID;
function __construct($DB_con, $ID, $Fname, $Srname, $Email, $Level, $FolderPath, $LastLogin, $LastIP, $Gender, $Birth, $DisplayName, $ProfilePicture, $ProfilePictureID){
$BirthDay = strtotime($Birth);
$this->DB = $DB_con;
$this->ID = $ID;
$this->Firstname = $Fname;
$this->Surname = $Srname;
$this->Gender = $Gender;
$this->Birth = date("d-m-Y", $BirthDay);
$this->Email = $Email;
$this->Level = $Level;
$this->FolderPath = $FolderPath;
$this->LastLogin = $LastLogin;
$this->LastIP = $LastIP;
$this->LevelWord = $this->sayLevel($Level);
$this->DisplayName = $DisplayName;
$this->ProfilePicture = $ProfilePicture;
$this->ProfilePictureID = $ProfilePictureID;
}
//Function for calling "private" data
public function show($atribut){
return $this->$atribut;
}
This object in $logged_user is beeing created in this object $user which works for registration and stuff for trying things as annonymous:
class USER
{
private $DB;
function __construct($DB_con)
{
$this->DB = $DB_con;
}
//Function for creating LOGGED_USER
public function useFactory($id){
require_once "logged_user.php";
$sql = $this->DB->prepare("SELECT us.ID, us.Firstname, us.Surname, us.Email, us.Level, us.Gender, us.Email, us.DisplayName, us.Birth, um.FolderHash, um.LastLogin, um.LastIP, uc.ProfilePictureID, up.Name, up.ID FROM Users AS us JOIN UsersMeta AS um ON us.ID = um.UsersID JOIN UsersConfig AS uc ON us.ID = uc.UsersID JOIN UsersPictures AS up ON up.ID = uc.ProfilePictureID WHERE us.ID = :id LIMIT 1");
$sql->execute(array(':id' => $id));
$sqlResult = $sql->fetchAll(PDO::FETCH_ASSOC);
if(!empty($sqlResult)){
foreach($sqlResult as $data){
return new LOGGED_USER($this->DB, $id, $data["Firstname"], $data["Surname"], $data["Email"], $data["Level"], $data["FolderHash"], $data["LastLogin"], $data["LastIP"], $data["Gender"], $data["Birth"], $data["DisplayName"], $data["Name"], $data["ID"]);
}
}else{
return "Prázdno!";
}
}
And here I am creating the $logged_userand trying to call function show():
session_start();
require_once "../config.php";
require_once "../objects/user.php";
$userFactory = new USER($db);
$userFactory->is_loggedin();
$logged_user = $userFactory->useFactory($_SESSION["user_session"]); //Returns new class (LOGGED_USER)
echo $logged_user->show("Email");

Answer 1

There is no need to fetchAll in your factory, as your criteria are id (the result always should be zero or one records). If you have no user with such id return null, so replace this:
$sqlResult = $sql->fetchAll(PDO::FETCH_ASSOC);
if(!empty($sqlResult)){
foreach($sqlResult as $data){
return new LOGGED_USER($this->DB, $id, $data["Firstname"], $data["Surname"], $data["Email"], $data["Level"], $data["FolderHash"], $data["LastLogin"], $data["LastIP"], $data["Gender"], $data["Birth"], $data["DisplayName"], $data["Name"], $data["ID"]);
}
}else{
return "Prázdno!";
}
with this:
$data = $sql->fetch(PDO::FETCH_ASSOC);
if (empty($data)) {
return null;
}
return new LOGGED_USER(
$this->DB,
$id, $data["Firstname"],
$data["Surname"],
$data["Email"],
$data["Level"],
$data["FolderHash"],
$data["LastLogin"],
$data["LastIP"],
$data["Gender"],
$data["Birth"],
$data["DisplayName"],
$data["Name"],
$data["ID"]
);
Then in your code check if logged_user is null:
if ($logged_user = $userFactory->useFactory($_SESSION["user_session"])) {
echo $logged_user->show("Email");
}