ComponentDefinition#save_copy changes the guid if #path is not empty

[Whaat]

This feels like a bug - was definitely a surprising discovery. Firstly, I was surprised that ComponentDefinition#save_as modifies the guid until I realized that it changes the path associated with the component. OK fine, I guess this makes some sense.

But why would calling save_copy also change the guid? (But only for components that have an associated path set)

SketchUp 2024 24.0.579 Windows

To reproduce, select a component (eg. Teddy) in the model using the Select Tool. Then run:

path = File.join(Sketchup.temp_dir, "test.skp")
defn = Sketchup.active_model.selection.first.definition
puts defn.guid
puts defn.path
defn.save_copy(path)
puts defn.guid
puts defn.path

If you run the above snippet when selecting a component with no associated path, the guid will NOT change.

[thomthom]

But why would calling save_copy also change the guid? (But only for components that have an associated path set)

That doesn't sound right to me either. 🤔

[Eneroth3]

Both of these sounds like bugs to me. Saving out the component does change the Loaded From Path but it doesn't make it a separate revision of the same component. Pasting the component from the clipboard or dragging it in from the folder should not create another "#1" version of the component.

…

On Tue, Dec 10, 2024 at 10:59 AM Thomas Thomassen ***@***.***> wrote: But why would calling save_copy also change the guid? (But only for components that have an associated path set) That doesn't sound right to me either. 🤔 — Reply to this email directly, view it on GitHub <#1019 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABK653KL6SXDPQ5SVXR74FD2E23P3AVCNFSM6AAAAABTJYM2ZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZRGA3DKMJWHA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

-- http://julia-christina-eneroth.se

[sketchup]

Logged as: SKEXT-4436