Skip to content

Save each framework's own CFDefinitions, matched by identifier (CGE-184) - #50

Merged
alail-ims merged 1 commit into
mainfrom
task/CGE-184
Oct 19, 2026
Merged

alail-ims merged 1 commit into
mainfrom
task/CGE-184

Conversation

@alail-ims

Copy link
Copy Markdown
Contributor

The editor built CFDefinitions on save from the tenant-wide catalogue, matching definitions by identifier or by title. When two frameworks had a definition with the same title, saving either one pulled in the other's. Bad data in one framework could then make another framework's saves fail validation.

  • Match definitions only by URI identifier. Items that name a type or subject by text alone, with no URI, no longer pick up definitions with a matching title.
  • Keep each framework's own definitions from its CFPackage and put them ahead of the tenant catalogue, for both the save and the pickers.
  • Carry CFDefinitions-level extensions through on save. The server keeps the stored ones when a save leaves them out.

The editor built CFDefinitions on save from the tenant-wide catalogue,
matching definitions by identifier or by title. When two frameworks had a
definition with the same title, saving either one pulled in the other's.
Bad data in one framework could then make another framework's saves fail
validation.

- Match definitions only by URI identifier. Items that name a type or
  subject by text alone, with no URI, no longer pick up definitions with
  a matching title.
- Keep each framework's own definitions from its CFPackage and put them
  ahead of the tenant catalogue, for both the save and the pickers.
- Carry CFDefinitions-level extensions through on save. The server keeps
  the stored ones when a save leaves them out.
@alail-ims
alail-ims merged commit b588828 into main Oct 19, 2026
3 checks passed
@alail-ims
alail-ims deleted the task/CGE-184 branch October 19, 2026 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant