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

Make Redmine class picklable #65

Merged
merged 1 commit into from
Nov 5, 2014

Conversation

rconradharris
Copy link
Contributor

The Redmine class couldn't be pickled because the gettatr would attempt to
turn any attribute access into a Resource, even __getstate__.

The proposed fix treats attributes that begin with an underscore (and
double-underscore) as non-Resource attributes and returns their value as-is.

Fixes #64

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 925a7bc on rconradharris:make-redmine-picklable into * on maxtepkeev:master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 925a7bc on rconradharris:make-redmine-picklable into * on maxtepkeev:master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling bececf5 on rconradharris:make-redmine-picklable into * on maxtepkeev:master*.

The `Redmine` class couldn't be pickled because the `gettatr` would attempt to
turn any attribute access into a `Resource`, even `__getstate__`.

The proposed fix treats attributes that begin with an underscore (and
double-underscore) as non-`Resource` attributes.

Fixes maxtepkeev#64
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 34b081a on rconradharris:make-redmine-picklable into * on maxtepkeev:master*.

maxtepkeev added a commit that referenced this pull request Nov 5, 2014
@maxtepkeev maxtepkeev merged commit ed1d835 into maxtepkeev:master Nov 5, 2014
@rconradharris rconradharris deleted the make-redmine-picklable branch November 5, 2014 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the Redmine class picklable
3 participants