Removed test as no error handling is needed for ident generation. Also
removed all related downstream uses of catch for
db-ident/create-db-ident-from-name that are no longer needed
- Graphs with description page failed to import
- Graphs that had a page with a built-in class name mangled the block
types. Reverted fix for logseq/db-test#17
- Also updated test for datom count as some parser changes were made
last week to alter it
by export rather than listing db-graph properties which would
change a lot. This is a follow up to fixing built-in property conflicts
as part of LOG-3039
New data model of allowing a single :block/type failed a lot of import tx
because properties and class types could be overwritten by a single page ref.
Also this fixes page-properties-tx which wasn't just transacting the
page properties. Also fixes importing pages like 'Tags' which were
incorrectly getting associated to built-in properties with the same name
Fixes LOG-3039. This was noticed because new description built-in
conflicted with file graphs that had the same name. This fixes the issue
for :description but also any of the other new db built-in property
names
* refactor: don't store :checkbox as ref type
* fix: :property dsl query
* fix: task rule uses :property-ref-type
* enhance: dynamic query in :property rule depends on db/valueType
* enhance: use identity to avoid multiple vals binding error
* fix: replace outdated :page type with :node type
* fix: query-dsl tests
* fix: more outdated :page type
* fix: db test
* fix: db rules tests
* fix: move [?p ?prop ?pv] to the front
* fix: outliner tests
* fix: graph-parser tests
* Migrate checkbox ref type to non-ref type
Only class ident name that wasn't capitalized and didn't match its
title. Considered renaming :logseq.task but unsure if it even makes
sense for their names to match since they aren't sharing a common ns.
:logseq.task may be better off under :logseq.property.*
rRemoving this test as the differences between a file and db graph are
enough that it's not worth testing. This test was helpful when we were
first developing db version
Some block/path-refs were invalid because importer was manually setting
this instead of relying on listeners. Confirmed relying on listeners
works in the frontend and CLI. Part of LOG-3176