CKAN: Ticket #1742: Naming of a new package/group can clash with a route

ross — Wed, 01 Feb 2012 17:17:15 GMT

Potentially valid diff, needs test + patch when I have time.

--- a/ckan/controllers/api.py +++ b/ckan/controllers/api.py @@ -592,6 +592,8 @@ class ApiController?(BaseController?):

def is_slug_valid(self):

slug = request.params.get('slug') or slugtype = request.params.get('type') or

+ if slug in ['new', 'edit', 'search']: + return self._finish_ok( dict(valid=False) )

if slugtype==u'package':

response_data = dict(valid=not bool(package_exists(slug))) return self._finish_ok(response_data)

priority changed

dread — Wed, 01 Feb 2012 17:25:35 GMT

Probably better in the ckan/logic/validators.py: name_validator

ross — Mon, 05 Mar 2012 14:10:15 GMT

ross — Mon, 12 Mar 2012 10:50:21 GMT

ross — Mon, 12 Mar 2012 12:26:33 GMT