Skip to content
This repository was archived by the owner on Jan 7, 2021. It is now read-only.

Commit 8912e92

Browse files
committed
Fixes for the StringIO import make it 2to3 friendly.
1 parent 52a4abd commit 8912e92

3 files changed

Lines changed: 9 additions & 7 deletions

File tree

documentcloud/MultipartPostHandler.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@
4848
from os import SEEK_END
4949
from six.moves import urllib
5050
try:
51-
import io
52-
except ImportError:
5351
import cStringIO as io
52+
except ImportError:
53+
import io
5454

5555

5656
class Callable:

documentcloud/__init__.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
"""
1414
from __future__ import absolute_import
1515
import os
16+
import sys
1617
import six
1718
import copy
1819
import base64
@@ -70,11 +71,12 @@ def _make_request(self, url, params=None, opener=None):
7071
# Make the request
7172
try:
7273
response = request_method(request)
73-
except urllib.error.HTTPError, e:
74-
if e.code == 404:
74+
except:
75+
e = sys.exc_info()[1]
76+
if getattr(e, 'code', None) == 404:
7577
raise DoesNotExistError("The resource you've requested does \
7678
not exist or is unavailable without the proper credentials.")
77-
elif e.code == 401:
79+
elif getattr(e, 'code', None) == 401:
7880
raise CredentialsFailedError("The resource you've requested \
7981
requires proper credentials.")
8082
else:

test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@
1313
import textwrap
1414
import unittest
1515
try:
16-
import io
17-
except ImportError:
1816
import cStringIO as io
17+
except ImportError:
18+
import io
1919
from copy import copy
2020
from documentcloud import DocumentCloud
2121
from documentcloud.toolbox import DoesNotExistError

0 commit comments

Comments
 (0)