Fix user_delete() return value when user_delete() is successful. issue#85

[iwashi]

As I wrote issue #85, user_delete() should return true when it's successful.

[WanWizard]

It can even be condensed to

if ($user)
{
    return (bool) $user->delete();
}

return false;

[iwashi]

return (bool) $user->delete(); looks good. :+1:

[WanWizard]

You update the PR, or I close it and change it myself?

[kenjis]

@WanWizard He updated this PR already. See "File changed"

[Art4]

@kenjis I think, @WanWizard wants to have the else statement removed. Find this:

if ($user)
{
    return (bool) $user->delete();
}
else
{
    return false;
}

and replace it with this:

if ($user)
{
    return (bool) $user->delete();
}

return false;

[WanWizard]

Exactly. The else is redundant there.

[iwashi]

Okay, I'm going to update PR from now.

[WanWizard]

Thanks.