Merge pull request #161 from ckan/unused-imports
Remove unused imports. PEP8.
This commit is contained in:
commit
5a923ba307
|
@ -1,24 +1,23 @@
|
|||
import urllib2
|
||||
|
||||
from pylons.i18n import _
|
||||
|
||||
import ckan.lib.helpers as h, json
|
||||
from ckan.lib.base import BaseController, c, g, request, \
|
||||
response, session, render, config, abort
|
||||
from ckan.lib.base import BaseController, c, request, \
|
||||
response, render, abort
|
||||
|
||||
from ckan.model import Package
|
||||
|
||||
|
||||
class ViewController(BaseController):
|
||||
|
||||
def wms_preview(self,id):
|
||||
#check if package exists
|
||||
def wms_preview(self, id):
|
||||
# check if package exists
|
||||
c.pkg = Package.get(id)
|
||||
if c.pkg is None:
|
||||
abort(404, 'Dataset not found')
|
||||
|
||||
for res in c.pkg.resources:
|
||||
if res.format.lower() == 'wms':
|
||||
c.wms_url = res.url if not '?' in res.url else res.url.split('?')[0]
|
||||
c.wms_url = res.url \
|
||||
if '?' not in res.url else res.url.split('?')[0]
|
||||
break
|
||||
if not c.wms_url:
|
||||
abort(400, 'This dataset does not have a WMS resource')
|
||||
|
@ -26,7 +25,7 @@ class ViewController(BaseController):
|
|||
return render('ckanext/spatial/wms_preview.html')
|
||||
|
||||
def proxy(self):
|
||||
if not 'url' in request.params:
|
||||
if 'url' not in request.params:
|
||||
abort(400)
|
||||
try:
|
||||
server_response = urllib2.urlopen(request.params['url'])
|
||||
|
|
Loading…
Reference in New Issue