Move contact deleted check from RemoveContent to Remove task

This commit is contained in:
Hypolite Petovan 2021-10-20 09:43:43 -04:00
parent 9f411b0c76
commit d30955aa14
2 changed files with 13 additions and 17 deletions

View file

@ -29,19 +29,21 @@ use Friendica\Database\DBA;
*/
class Remove extends RemoveContent
{
public static function execute(int $id): array
public static function execute(int $id): bool
{
$contact = parent::execute($id);
// Only delete if the contact is to be deleted
$contact = DBA::selectFirst('contact', ['id', 'uid', 'url', 'nick', 'name'], ['deleted' => true, 'id' => $id]);
if (!DBA::isResult($contact)) {
return false;
}
if (!empty($contact)) {
return [];
if (!parent::execute($id)) {
return false;
}
$ret = DBA::delete('contact', ['id' => $id]);
Logger::info('Deleted contact', ['id' => $id, 'result' => $ret]);
$contact['id'] = null;
return $contact;
return true;
}
}

View file

@ -33,19 +33,13 @@ use Friendica\Model\Post;
*/
class RemoveContent
{
public static function execute(int $id): array
public static function execute(int $id): bool
{
if (empty($id)) {
return [];
return false;
}
// Only delete if the contact is to be deleted
$contact = DBA::selectFirst('contact', ['id', 'uid', 'url', 'nick', 'name'], ['deleted' => true, 'id' => $id]);
if (!DBA::isResult($contact)) {
return [];
}
Logger::info('Start deleting contact content', ['contact' => $contact]);
Logger::info('Start deleting contact content', ['cid' => $id]);
// Now we delete the contact and all depending tables
DBA::delete('post-tag', ['cid' => $id]);
@ -87,6 +81,6 @@ class RemoveContent
DBA::delete('group_member', ['contact-id' => $id]);
DI::intro()->delete(DI::introFactory()->createDummy($id));
return $contact;
return true;
}
}