Browse Source

fix: don't crash public classes in private published graph

Public classes don't load their properties currently but something to
consider
Gabriel Horner 1 year ago
parent
commit
96cbac5f47
1 changed files with 4 additions and 2 deletions
  1. 4 2
      src/main/frontend/components/objects.cljs

+ 4 - 2
src/main/frontend/components/objects.cljs

@@ -117,8 +117,10 @@
         [view-entity set-view-entity!] (rum/use-state class)
         [views set-views!] (rum/use-state [class])
         [data set-data!] (rum/use-state objects)
-        columns* (views/build-columns config properties {:add-tags-column? (or (= (:db/ident class) :logseq.class/Root)
-                                                                               (> (count (distinct (mapcat :block/tags objects))) 1))})
+        ;; Properties can be nil for published private graphs
+        properties' (remove nil? properties)
+        columns* (views/build-columns config properties' {:add-tags-column? (or (= (:db/ident class) :logseq.class/Root)
+                                                                                (> (count (distinct (mapcat :block/tags objects))) 1))})
         columns (cond
                   (= (:db/ident class) :logseq.class/Pdf-annotation)
                   (remove #(contains? #{:logseq.property/ls-type} (:id %)) columns*)