Skip to content

multiple values for keyword argument 'use_datetime' #3

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

Merged
merged 1 commit into from
Aug 14, 2015
Merged

multiple values for keyword argument 'use_datetime' #3

merged 1 commit into from
Aug 14, 2015

Conversation

tokas
Copy link
Contributor

@tokas tokas commented Aug 13, 2015

This should fix the "TypeError: ServerProxy constructor got multiple values for keyword argument 'use_datetime'" you get, if you provide use_datetime as a keyword argument to DokuWiki()

…eyword argument 'use_datetime'" if you provide use_datetime kwarg to DokWiki()
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling bec3447 on tokas:master into 39a3fe5 on fmenabe:master.

fmenabe added a commit that referenced this pull request Aug 14, 2015
multiple values for keyword argument 'use_datetime'
@fmenabe fmenabe merged commit 1ee06af into fmenabe:master Aug 14, 2015
@fmenabe
Copy link
Owner

fmenabe commented Aug 14, 2015

Thanks for correcting the bug! I reupload the package on Pypi with the correction.

@tokas
Copy link
Contributor Author

tokas commented Aug 14, 2015

Thanks for the prompt update!
Maybe you should take this out completely and leave the default to False because as of the new DokuWiki version 'Detritus' their xmlrpc service is returning date strings in a format which the python xmlrpclib (or xmlrpc.client) isn't expecting and therefore throws an error while trying to parse it.

@fmenabe
Copy link
Owner

fmenabe commented Aug 16, 2015

I don't really remember why I forced this parameter (probably because I prefer to manipulate datetime.datetime object)! But if forcing this parameter breaks things by default with the new DokuWiki version, I'll remove it.

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