[johnmont]

Figured it out guys. The error caused by: if ($ret->isError()) is caused -- among other things -- that if things go correctly when doing the embed, nothing is returned by the $ret= statements. So what you need to do is:

Change:

$retcreate = GalleryEmbed :: createUser($uid, $args);
if (!$retcreate->isSuccess())

To:

$retcreate = GalleryEmbed :: createUser($uid, $args);
if ($retcreate)

First, I believe that $retcreate->isSuccess() is no longer used. In the new paradigm of this version, if you do something and its successful, you get nothing returned back to $ret. Only when there is an error, then you get something back in $ret.

So my replacement says, "if you get something back"...which is essentially what the previous statment said: "if what you get back is not successful"


Change:

$ret = GalleryEmbed::checkActiveUser($uid);
if ($ret->isError()) {

To:

$ret = GalleryEmbed::checkActiveUser($uid);
if ($ret) {

Basically the same dealio. In the original, you were saying "if what is returned is an error then....". Now, this says "if you get anything back the...". And since you only get something back when there is an error, then its doing the same thing.