http://localhost/ticket/293 <p> This is caused by editing a package with resources over the REST interface. </p> <p> Exception: </p> <pre class="wiki">URL: http://de.ckan.net/api/rest/package/destatis Module weberror.errormiddleware:162 in __call__ &lt;&lt; __traceback_supplement__ = Supplement, self, environ sr_checker = ResponseStartChecker(start_response) app_iter = self.application(environ, sr_checker) return self.make_catching_iter(app_iter, environ, sr_checker) except: &gt;&gt; app_iter = self.application(environ, sr_checker) Module repoze.who.middleware:107 in __call__ &lt;&lt; wrapper = StartResponseWrapper(start_response) app_iter = app(environ, wrapper.wrap_start_response) # The challenge decider almost(?) always needs information from the &gt;&gt; app_iter = app(environ, wrapper.wrap_start_response) Module beaker.middleware:73 in __call__ &lt;&lt; self.cache_manager) environ[self.environ_key] = self.cache_manager return self.app(environ, start_response) &gt;&gt; return self.app(environ, start_response) Module beaker.middleware:152 in __call__ &lt;&lt; headers.append(('Set-cookie', cookie)) return start_response(status, headers, exc_info) return self.wrap_app(environ, session_start_response) def _get_session(self): &gt;&gt; return self.wrap_app(environ, session_start_response) Module routes.middleware:130 in __call__ &lt;&lt; environ['SCRIPT_NAME'] = environ['SCRIPT_NAME'][:-1] response = self.app(environ, start_response) # Wrapped in try as in rare cases the attribute will be gone already &gt;&gt; response = self.app(environ, start_response) Module pylons.wsgiapp:125 in __call__ &lt;&lt; controller = self.resolve(environ, start_response) response = self.dispatch(controller, environ, start_response) if 'paste.testing_variables' in environ and hasattr(response, &gt;&gt; response = self.dispatch(controller, environ, start_response) Module pylons.wsgiapp:324 in dispatch &lt;&lt; if log_debug: log.debug("Calling controller class with WSGI interface") return controller(environ, start_response) def load_test_env(self, environ): &gt;&gt; return controller(environ, start_response) Module ckan.lib.base:51 in __call__ &lt;&lt; # available in environ['pylons.routes_dict'] try: return WSGIController.__call__(self, environ, start_response) finally: model.Session.remove() &gt;&gt; return WSGIController.__call__(self, environ, start_response) Module pylons.controllers.core:221 in __call__ &lt;&lt; return response(environ, self.start_response) response = self._dispatch_call() if not start_response_called: self.start_response = start_response &gt;&gt; response = self._dispatch_call() Module pylons.controllers.core:172 in _dispatch_call &lt;&lt; req.environ['pylons.action_method'] = func response = self._inspect_call(func) else: if log_debug: &gt;&gt; response = self._inspect_call(func) Module pylons.controllers.core:107 in _inspect_call &lt;&lt; func.__name__, args) try: result = self._perform_call(func, args) except HTTPException, httpe: if log_debug: &gt;&gt; result = self._perform_call(func, args) Module pylons.controllers.core:60 in _perform_call &lt;&lt; """Hide the traceback for everything above this method""" __traceback_hide__ = 'before_and_this' return func(**args) def _inspect_call(self, func): &gt;&gt; return func(**args) Module ckan.controllers.rest:145 in update &lt;&lt; fs = ckan.forms.group_fs_combined fs = fs.bind(entity, data=request_fa_dict) validation = fs.validate_on_edit(entity.name, entity.id) if not validation: response.status_int = 409 &gt;&gt; validation = fs.validate_on_edit(entity.name, entity.id) Module ckan.forms.package:385 in validate_on_edit &lt;&lt; temp_name = orig_pkg_name self.data['Package-%s-name' % record_id] = u'something_unique' validation = self.validate() if temp_name: # restore it &gt;&gt; validation = self.validate() Module formalchemy.forms:175 in validate &lt;&lt; if self.readonly: raise Exception('Cannot validate a read-only FieldSet') return AbstractFieldSet.validate(self) def sync(self): &gt;&gt; return AbstractFieldSet.validate(self) Module formalchemy.forms:104 in validate &lt;&lt; success = True for field in self.render_fields.itervalues(): success = field._validate() and success # run this _after_ the field validators, since each field validator # resets its error list. we want to allow the global validator to add &gt;&gt; success = field._validate() and success Module formalchemy.fields:763 in _validate &lt;&lt; try: value = self._deserialize() except validators.ValidationError, e: self.errors.append(e) &gt;&gt; value = self._deserialize() Module formalchemy.fields:1354 in _deserialize &lt;&lt; if self.is_composite_foreign_key: return self.query(self.relation_type()).get(python_pk(self.renderer.deserialize())) return self.renderer.deserialize() &gt;&gt; return self.renderer.deserialize() Module formalchemy.fields:183 in deserialize &lt;&lt; if self.field.is_collection: return [self._deserialize(subdata) for subdata in self._serialized_value()] return self._deserialize(self._serialized_value()) def _deserialize(self, data): &gt;&gt; return self._deserialize(self._serialized_value()) Module formalchemy.fields:167 in _serialized_value &lt;&lt; if self.field.is_collection: return self._params.getall(self.name) return self._params.getone(self.name) def deserialize(self): &gt;&gt; return self._params.getone(self.name) Module formalchemy.base:63 in getone &lt;&lt; if v is None or isinstance(v, basestring) or isinstance(v, cgi.FieldStorage): return v return v[0] def getall(self, key): v = dict.get(self, key) &gt;&gt; return v[0] TypeError: 'int' object is unsubscriptable </pre><p> Thanks to Friedrich for spotting this. </p> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/293 Trac 0.12.3 dread Tue, 20 Apr 2010 10:51:47 GMT http://localhost/ticket/293#comment:1 http://localhost/ticket/293#comment:1 <ul> <li><strong>owner</strong> changed from <em>dread</em> to <em>johnbywater</em> </li> </ul> Ticket johnbywater Wed, 21 Apr 2010 21:30:57 GMT http://localhost/ticket/293#comment:2 http://localhost/ticket/293#comment:2 <p> Can't reproduce this exception. Have added tests covering adding, removing and updating resources, and it all seems to work. </p> Ticket johnbywater Thu, 22 Apr 2010 12:41:23 GMT http://localhost/ticket/293#comment:3 http://localhost/ticket/293#comment:3 <ul> <li><strong>status</strong> changed from <em>new</em> to <em>closed</em> </li> <li><strong>resolution</strong> set to <em>fixed</em> </li> </ul> <p> With the metastable revision of CKAN, the package resource data structure in ckanclient scripts must have all four keys set in the Resource-Dict. Setting 'hash' in the resource data cures this issues with "metastable". </p> <p> The "bad" code was: </p> <blockquote> <p> resources.append((res<a class="missing wiki">url?</a>, res<a class="missing wiki">format?</a>, res<a class="missing wiki">description?</a>, res<a class="missing wiki">hash?</a>)) </p> </blockquote> <p> <a class="missing wiki">KeyError?</a>: 'hash' </p> <p> This code was adjusted in revision 40c4fe04038d, to default unspecified resource attributes to the empty string. It was changed further in subsequent revisions, to use parameters of the Package.add_resource method to default unspecified values. </p> <p> However, the API doc doesn't mention the resource hash, although it is required, but I just fixed that in the source (in revision 0f20bfb45d13). <a class="ext-link" href="http://knowledgeforge.net/ckan/doc/ckan/api.html"><span class="icon">​</span>http://knowledgeforge.net/ckan/doc/ckan/api.html</a> </p> Ticket