Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Fix undefined memoized_methods error raised when a parent class has not call memoize #68

Merged
merged 1 commit into from
Jun 20, 2017
Merged

Fix undefined memoized_methods error raised when a parent class has not call memoize #68

merged 1 commit into from
Jun 20, 2017

Conversation

PikachuEXE
Copy link
Contributor

Fixes #51

The method memoized_methods is only defined when memoize method is called
Thus memoized_methods method is absent in any class that contains no method call to memoize

So this PR just define the method on module extend

@PikachuEXE PikachuEXE mentioned this pull request Jun 20, 2017
Closed
@matthewrudy matthewrudy merged commit fb82fd1 into matthewrudy:master Jun 20, 2017
@PikachuEXE PikachuEXE deleted the fix/flush-cache branch June 21, 2017 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PikachuEXE @matthewrudy