Skip to content
Snippets Groups Projects
Commit e6c042e1 authored by Cam Saül's avatar Cam Saül Committed by GitHub
Browse files

Merge pull request #2917 from metabase/fix-sammys-card-fave-500

Make sure we have cards before we try to fetch favorite status for them
parents 7d9dd942 33d8afeb
No related branches found
No related tags found
No related merge requests found
......@@ -30,9 +30,10 @@
(defn- hydrate-favorites
"Efficiently add `favorite` status for a large collection of `Cards`."
[cards]
(let [favorite-card-ids (set (db/select-field :card_id CardFavorite, :owner_id *current-user-id*, :card_id [:in (map :id cards)]))]
(for [card cards]
(assoc card :favorite (contains? favorite-card-ids (:id card))))))
(when (seq cards)
(let [favorite-card-ids (set (db/select-field :card_id CardFavorite, :owner_id *current-user-id*, :card_id [:in (map :id cards)]))]
(for [card cards]
(assoc card :favorite (contains? favorite-card-ids (:id card)))))))
(defn- cards:all
"Return all `Cards`."
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment