mirror of
https://github.com/wallabag/wallabag.git
synced 2024-11-26 11:01:04 +00:00
Removed comment
And move the SQL query inside the php doc
This commit is contained in:
parent
4059a061c0
commit
6be9750155
1 changed files with 3 additions and 9 deletions
|
@ -185,7 +185,9 @@ class EntryRepository extends EntityRepository
|
||||||
* Remove a tag from all user entries.
|
* Remove a tag from all user entries.
|
||||||
*
|
*
|
||||||
* We need to loop on each entry attached to the given tag to remove it, since Doctrine doesn't know EntryTag entity because it's a ManyToMany relation.
|
* We need to loop on each entry attached to the given tag to remove it, since Doctrine doesn't know EntryTag entity because it's a ManyToMany relation.
|
||||||
* It could be faster with one query but I don't know how to retrieve the table name `entry_tag` which can have a prefix.
|
* It could be faster with one query but I don't know how to retrieve the table name `entry_tag` which can have a prefix:
|
||||||
|
*
|
||||||
|
* DELETE et FROM entry_tag et WHERE et.entry_id IN ( SELECT e.id FROM entry e WHERE e.user_id = :userId ) AND et.tag_id = :tagId
|
||||||
*
|
*
|
||||||
* @param int $userId
|
* @param int $userId
|
||||||
* @param Tag $tag
|
* @param Tag $tag
|
||||||
|
@ -203,14 +205,6 @@ class EntryRepository extends EntityRepository
|
||||||
}
|
}
|
||||||
|
|
||||||
$this->getEntityManager()->flush();
|
$this->getEntityManager()->flush();
|
||||||
|
|
||||||
// An other solution can be to use raw query but I can't find a way to retrieve the `entry_tag` table name since it can be prefixed....
|
|
||||||
// $sql = 'DELETE et FROM entry_tag et WHERE et.entry_id IN ( SELECT e.id FROM entry e WHERE e.user_id = :userId ) AND et.tag_id = :tagId';
|
|
||||||
// $stmt = $this->getEntityManager()->getConnection()->prepare($sql);
|
|
||||||
// $stmt->execute([
|
|
||||||
// 'userId' => $userId,
|
|
||||||
// 'tagId' => $tag->getId(),
|
|
||||||
// ]);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
Loading…
Reference in a new issue