ahtlon/deck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Revert "fix: Chunk query for getting labels for cards"

Browse Source 
This reverts commit cb469fb78c.
This commit is contained in:
Julius Knorr
2024-12-19 18:04:42 +01:00
parent a1bd24dbf7
commit 93cdedb6b0
1 changed files with 16 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
lib/Db/LabelMapper.php
View File

@@ -44,40 +44,36 @@ class LabelMapper extends DeckMapper implements IPermissionMapper {
} }
/** /**
* @param numeric $cardId
* @param int|null $limit
* @param int|null $offset
* @return Label[] * @return Label[]
* @throws \OCP\DB\Exception * @throws \OCP\DB\Exception
*/ */
public function findAssignedLabelsForCard(int $cardId): array { public function findAssignedLabelsForCard($cardId, $limit = null, $offset = null): array {
$qb = $this->db->getQueryBuilder(); $qb = $this->db->getQueryBuilder();
$qb->select('l.*', 'card_id') $qb->select('l.*', 'card_id')
->from($this->getTableName(), 'l') ->from($this->getTableName(), 'l')
->innerJoin('l', 'deck_assigned_labels', 'al', 'l.id = al.label_id') ->innerJoin('l', 'deck_assigned_labels', 'al', 'l.id = al.label_id')
->where($qb->expr()->eq('card_id', $qb->createNamedParameter($cardId, IQueryBuilder::PARAM_INT))) ->where($qb->expr()->eq('card_id', $qb->createNamedParameter($cardId, IQueryBuilder::PARAM_INT)))
->orderBy('l.id');
return $this->findEntities($qb);
}
public function findAssignedLabelsForCards(array $cardIds, ?int $limit = null, ?int $offset = null): array {
$qb = $this->db->getQueryBuilder();
$qb->select('l.*', 'card_id')
->from($this->getTableName(), 'l')
->innerJoin('l', 'deck_assigned_labels', 'al', 'l.id = al.label_id')
->where($qb->expr()->in('card_id', $qb->createParameter('ids')))
->orderBy('l.id') ->orderBy('l.id')
->setMaxResults($limit) ->setMaxResults($limit)
->setFirstResult($offset); ->setFirstResult($offset);
$results = []; return $this->findEntities($qb);
foreach (array_chunk($cardIds, 1000) as $chunk) {
$qb->setParameter('ids', $chunk, IQueryBuilder::PARAM_STR_ARRAY);
foreach ($this->findEntities($qb) as $result) {
$results[] = $result;
}
} }
return $results; public function findAssignedLabelsForCards($cardIds, $limit = null, $offset = null): array {
$qb = $this->db->getQueryBuilder();
$qb->select('l.*', 'card_id')
->from($this->getTableName(), 'l')
->innerJoin('l', 'deck_assigned_labels', 'al', 'l.id = al.label_id')
->where($qb->expr()->in('card_id', $qb->createNamedParameter($cardIds, IQueryBuilder::PARAM_INT_ARRAY)))
->orderBy('l.id')
->setMaxResults($limit)
->setFirstResult($offset);
return $this->findEntities($qb);
} }
/** /**