PythonConfluenceAPI icon indicating copy to clipboard operation
PythonConfluenceAPI copied to clipboard

#16 Broke python2 support

Open wojciechwroblewski opened this issue 8 years ago • 2 comments

Thanks to #16 we've got python3 support, which is great, but unfortunately it also broke support for python2. Tested on:

$ python3 --version
Python 3.5.1
$ python2 --version
Python 2.7.11
# python --version
Python 2.7.5

Using sample code provided in README.md:

#!/usr/bin/env python

from PythonConfluenceAPI import ConfluenceAPI

conf = ConfluenceAPI('admin', '12345', 'http://localhost/confluence/')

confluence_recent_data = conf.get_content()
print(len(confluence_recent_data))

Results:

$ python2 test.py
5
5
$ python3 test.py
5

It's definitely something related to future lib, my guess is that ConfluenceAPI object is somehow created twice. I decided to stick with 62c95b8d2cc9334600a8ea5fa732fdead226e5b1 as I don't really need python3 support.

wojciechwroblewski avatar Mar 22 '16 10:03 wojciechwroblewski

Noted. I'll take a look.

On Tue, Mar 22, 2016 at 4:59 AM, s1gnalrunner [email protected] wrote:

Thanks to #16 https://github.com/pushrodtechnology/PythonConfluenceAPI/pull/16 we've got python3 support, which is great, but unfortunately it also broke support for python2. Tested on:

$ python3 --version Python 3.5.1 $ python2 --version Python 2.7.11

python --version

Python 2.7.5

Using sample code provided in README.md:

#!/usr/bin/env python

from PythonConfluenceAPI import ConfluenceAPI

conf = ConfluenceAPI('admin', '12345', 'http://localhost/confluence/')

confluence_recent_data = conf.get_content() print(len(confluence_recent_data))

Results:

$ python2 test.py 5 5 $ python3 test.py 5

It's definitely something related to future lib, my guess is that ConfluenceAPI object is somehow created twice. I decided to stick with 62c95b8 https://github.com/pushrodtechnology/PythonConfluenceAPI/commit/62c95b8d2cc9334600a8ea5fa732fdead226e5b1 as I don't really need python3 support.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/pushrodtechnology/PythonConfluenceAPI/issues/18

rpcope1 avatar Mar 22 '16 15:03 rpcope1

+1

astewart-twist avatar Apr 03 '16 00:04 astewart-twist