Skip to content

Conversation

@MalikMAlna
Copy link

Not sure if this mod makes an externally visible change in this case, but it should fix the issue I encountered in #5519

Also, I ended up adding my name to Authors.rst for the fix. I hope that's fine.

local path = ol.object_load_order_src_dir[idx]
local path = nil
if ol.object_load_order_src_dir[idx] then
path = ol.object_load_order_src_dir[idx].value
Copy link
Member

Choose a reason for hiding this comment

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

src_dir is a vector<path> in 52.02 and so its elements do not have the .value metamethod. This code will not work with 52.02, or against develop

@ab9rf
Copy link
Member

ab9rf commented Jul 26, 2025

Please confirm that the bug you're trying to fix here hasn't already been fixed in dev.

@MalikMAlna
Copy link
Author

Will do, if it's fixed in dev then I'll close out the PR here.

@MalikMAlna
Copy link
Author

Alright, it's resolved in dev, gonna close out this PR and the Issue.

May I ask when you all plan to drop the change in as a release?

@MalikMAlna MalikMAlna closed this Jul 26, 2025
@SilasD
Copy link
Contributor

SilasD commented Jul 26, 2025

rome said this in Discord # planning
rome of oxtrot — 1:46 PM
at this point i'm planning on releasing 52.02-r2 with the fixes already committed. i have adequate UAT on everything that's been touched since 52.02-r1 (as far as i can tell) so i feel confident that it's safe to release. gonna wait till morning so as not to hit the saturday evening peak

@MalikMAlna
Copy link
Author

Cool, thank you both for your time and best of luck with the release to you all! 👋 😄

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