Skip to content
Snippets Groups Projects

FIX: use identifiable instead of record

Merged Henrik tom Wörden requested to merge f-identifiable into dev

The crawler has internal checks in order to decide whether a Record can be checked for existence with the server. For this, all identifying properties need to be existent and all references of those need to be resolved or also records that can be checked themselves (so that they can be checked first).

Previously, the crawler required that ALL reference Properties can be checked or are resolved in order decide that the Record at hand can be checked.

Example: RT A with properties b and c

The registered identifiable is A with b

Record A
  b = 1234

can be checked (but the old code assumed it cannot.)

Record A
  c = 2344

cannot be checked since the identifying prop is missing.

Please consider !56 (merged)

Edited by Henrik tom Wörden

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Henrik tom Wörden marked this merge request as draft

    marked this merge request as draft

  • added 12 commits

    Compare with previous version

  • Henrik tom Wörden resolved all threads

    resolved all threads

  • Henrik tom Wörden added 239 commits

    added 239 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • @henrik I think we need to review this once more carefully.

  • Henrik tom Wörden added 191 commits

    added 191 commits

    Compare with previous version

  • Henrik tom Wörden
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading