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.

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

ResultSet.pm: Permit user defined target classes for low-level use. #20

Closed
wants to merge 1 commit into from

Conversation

kentfredric
Copy link
Contributor

This makes it easier in cases where you want to return a different type
of object that may have specific extra features added that may not be
useful for MetaCPAN::Client.

This avoids the need for downstream users to do re-blessing in strange
places to get the same result.

@oalders
Copy link
Member

oalders commented Oct 15, 2014

This is an intriguing way of moving past some of the limitations of MetaCPAN::Client.

@kentfredric
Copy link
Contributor Author

I suspect this commit introduces a subtle bug if the constructor fails to pass either class or type value, then it will fail in some mysterious way, or not fail at all, instead of correctly telling the user they have to provide one of the two. I'll see if I can fix that.

This makes it easier in cases where you want to return a different type
of object that may have specific extra features added that may not be
useful for MetaCPAN::Client.

This avoids the need for downstream users to do re-blessing in strange
places to get the same result.
@mickeyn
Copy link
Contributor

mickeyn commented Nov 18, 2017

hi, this is not ready for merging. if relevant - please reopen.

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.

None yet

3 participants