Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: OpenStruct causes NameError in Rake13.2.0 #1110

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

rhiroe
Copy link
Contributor

@rhiroe rhiroe commented Apr 18, 2024

resolved #1109

Loading of "ostruct" depends on Rake, so we need to explicitly load it and resolve the dependency.
Because Rake does not load "ostruct" since version 13.2.0.
ruby/rake#545

@unixmonkey unixmonkey merged commit 5e6b063 into mileszs:master Apr 18, 2024
12 checks passed
@unixmonkey
Copy link
Collaborator

Thank you!

@rhiroe rhiroe deleted the fix/1109 branch April 19, 2024 00:08
@heliocola
Copy link

👋 Maybe this is worthy of a version 2.8.1.

I run into this issue a few weeks ago upgrading a Rails app from 6.1 to 7.0.8.1 and I ended up adding the require 'ostruct' to the application.rb.

This app was already using WickedPdf v2.8.0.

And interesting enough: the OpenStruct name error was only happening in Production (not staging, nor CI)... :-D

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OpenStruct causes NameError in Rake13.2.0
3 participants