[Bug]: Drawing and Template regression

[Baaaaaz]

Describe the Bug

PR #5926 introduces a regression which breaks the drawing and template improvements in develop and due in 1.19. i.e.

• no longer able to drop drawings/templates to a new location after selecting and dragging them
• no longer able to resize templates
• holding down ctrl when dragging a drawing causes a NPE error.

Originally posted in #5926 (comment)

To Reproduce

1. Run a Develop or a current Nightly Build.
2. Create a new map.
3. Create a new drawing or template.
4. Select the Drawing Pointer Tool or Template Pointer Tool.
5. Try either:
   5.1 dragging a drawable and dropping -> drawable does not drop.
   5.2 resizing a template -> template does not change.
   5.3 holding down ctrl when dragging a drawing -> causes a NPE error.

Expected Behaviour

Able to drag and drop drawings and templates.
Able to resize templates.
Able to drag drawings whilst snapping to grid.

Screenshots

e.g. NPE

MapTool Info

Develop or current Nightly Build

Desktop

No response

Additional Context

No response

[Baaaaaz]

Noting that the commit 6ae4b6a which introduced protected AbstractDrawing(AbstractDrawing other) has the following comment:

To enable this, Drawable now has a copy() method, which is consistently implemented in child classes as delegating
to each implementation's copy constructor. The copy constructor always keeps the same ID, but callers can now modify the
ID through a new method Drawable.setId(GUID) for those cases where the ID needs to be given a different ID from the
original.

Unknown impact of PR #5926 on other Drawable features beyond what has been identified so far in this issue, but seems we either need to revert it, or fix it so it follows the commit guidance above, or something else... Given ShapeFunctions still have an experimental warning, such things should not break non-experimental stuff!