Skip to content

Conversation

@ElGrenier
Copy link

No description provided.

@FuzzyGamesOn
Copy link
Owner

Did you test this PR? Because it looks like it doesn't work. (Like using a variable that doesn't exist in itemPos.)

@ElGrenier
Copy link
Author

Oops My bad, i mixed the RE7 part with RE2, let me reupdate it

local pos = transform:call("get_Position")
local posx, posxdec = Inspect._Round(pos.x)
local posy, posydec = Inspect._Round(pos.y)
local posz, poszdec = Inspect._Round(pos.z)
Copy link
Owner

Choose a reason for hiding this comment

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

You're setting variables that you're not using here, so those should be removed.

Inspect.Log("Folder Path", InspectItem.GetFolderPath(parentOfComponent))
Inspect.Log("Player Position", InspectPlayer.GetPositionString())
-- Inspect.Log("Player Position", InspectPlayer.GetPositionString())
Inspect.Log("Item Position", "[" .. tostring(table.concat(posvalue, ",")) .. "]")
Copy link
Owner

Choose a reason for hiding this comment

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

Why are you commenting out Player Position? You'd need that for typewriter teleports, if you ever get them working in RE7.

Copy link
Author

Choose a reason for hiding this comment

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

Soo yeah, for the second variable i coul delete them, and for the player pos, for some reason my game just throw an error (even with the old non-modified script), but I could try to check it if you want

Copy link
Owner

Choose a reason for hiding this comment

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

Yeah, if you can share the error you're getting, that might give me a lead on what's broken. Would rather make it work than just comment it out broken. 😄

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.

2 participants