Got an error while deleting the account.

Yeah, I noticed that some days ago… I was thinking to fork the repo in order to fix it, but unfortunately I haven’t got the time yet :(

I suppose it is something related to the current session, it should be easily fixed with a cleanup of the current user session (like on logout).

Could very well be. Don’t see any problems with the delete itself and the error is talking about ‘add or update’. The session isn’t invalidated and also not verified on loading a page. Looking at the code, it is probably trying to add a veteran medal, because null < strtotime(‘-1 year’).

@flabbyrabbit!!! shakes fist

Hmm I see…
There is also a quick fix, by editing account.php at this location:

``` else if (isset($GET[ ‘delete’])) {
$status = $app->user->delete($POST[ ‘delete’], $_POST[ ‘token’]);

if ($status === true) {
header(‘Location: /?deleted’);
die();
}```

Before redirecting to ?deleted you can destroy the session as in logout.php

There might be a better way, what do you think?

Oops sorry, formatting of the code is ugly… it must be because I’m posting from my phone :s

Downside is that ?deleted is exactly the place where it tries to add the medal before even getting to the ?deleted part. It’s perhaps nicer to split the logout function in the user class so you have an invalidate_session function and use that function either in the delete function itself or after the call to it.

It turns out:

$this->logout();

Is fairly important. I’ve added it to the script things are happily destroyed now. @dloser was right that the problem was with the medal being added rather than being deleted, as you were still logged in but with no user details the medal script got confused and tried to award you medals for no reason.

Yup, now I see… I missed this piece of the puzzle :)

Got the same error, just delete cookies and it’ll work