Skip to content

Conversation

@Banane9
Copy link
Contributor

@Banane9 Banane9 commented Mar 20, 2025

@Banane9 Banane9 requested a review from Nytra March 20, 2025 18:11
@Nytra
Copy link
Contributor

Nytra commented Mar 21, 2025

From Claude AI: The overall effect is that any check using Slot.IsChildOf() in the patched method will always return true, regardless of what the original method would have returned. This effectively bypasses parent-child relationship validation, making the code behave as if any slot is always a child of whatever slot it's being compared to.

@Banane9
Copy link
Contributor Author

Banane9 commented Mar 22, 2025

Did you test this? The transpiler just changes the second argument of IsChildOf from false to true so that it considers the slot itself a child too

@Nytra
Copy link
Contributor

Nytra commented Mar 22, 2025

I did not test it but now I see what it does

Copy link
Contributor

@Nytra Nytra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good

@Banane9 Banane9 merged commit de2e54a into master Mar 22, 2025
2 checks passed
@Banane9 Banane9 deleted the fix/duplicate-slot-crash branch March 22, 2025 10:04
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.

3 participants