chore: Separate schemaPointer and uniquePointer better

[lassopicasso]

Description

This PR is an extension of the work done in PR #13406

• Adding a prefix 'uniquePointer-' to unique pointers, improving the gap between schemaPointer and uniquePointer.

• Fixed a bug where adding a reference to a non-existing type caused a console error, AddPropertyMenu.

Related Issue(s)

• Opening a node that is an object type with multiple usages, will open all #13240

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

• User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.21%. Comparing base (27cd365) to head (673142a).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #13538      +/-   ##
==========================================
- Coverage   94.23%   94.21%   -0.02%     
==========================================
  Files        1527     1527              
  Lines       20790    20803      +13     
  Branches     2509     2509              
==========================================
+ Hits        19591    19600       +9     
- Misses        950      954       +4     
  Partials      249      249              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Jondyr]

Tested

Bug not in production:

• Moving a selected property twice errors
  • Moving nonselected properties seems to not fail
  • Also seems to only happen when moving out of an object reference, never from top level
  • Property also loses highlight when moving into an object reference, but is still editable

schemapointer-bug.mp4

[TomasEng]

Moving a selected property twice errors

I tried to do exactly as in the video, but I was not able to reproduce this. Maybe it was fixed by #13635? Could you test again, @Jondyr?

Property also loses highlight when moving into an object reference, but is still editable

This seems to be present in production as well. I suggest we create a separate issue for it.

[Jondyr]

@TomasEng

I tried to do exactly as in the video, but I was not able to reproduce this. Maybe it was fixed by #13635? Could you test again, @Jondyr?

I managed to replicate it locally on b240351, clip recreating this from an empty datamodel:

20240926-1111-23.9104554.mp4

[TomasEng]

Thank you for retesting, @Jondyr. Now I managed to replicate it. I'll look into it.

[TomasEng]

Fixed the bug. This can now be retested. Not sure why Codecov fails on project coverage, as we've added more code with 100% diff hit.

[Jondyr]

Retested without any errors ✅ The changes also keeps the selection indication when moving the nodes, well done 👏