Skip to content

Commit

Permalink
No need to pass context to action because it is implicitly added
Browse files Browse the repository at this point in the history
  • Loading branch information
@domoritz
domoritz committed Jul 3, 2013
1 parent cca66d9 commit 77c7846
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ckanext/datastore/plugin.py
View file
Expand Up @@ -79,7 +79,7 @@ def configure(self, config):

@logic.side_effect_free
def new_resource_show(context, data_dict):
new_data_dict = resource_show(context, data_dict)
new_data_dict = resource_show(data_dict=data_dict)

This comment has been minimized.

Sign in to view

Copy link
@tobes

tobes Jul 3, 2013

Contributor

we should pass the context we were given. It might contain something wanted. It is fine to call with no context were we would have to create one but if we have one we should use that. Maybe we had a 'ignore_auth': True or the like

This comment has been minimized.

Sign in to view

Copy link
@domoritz

domoritz Jul 3, 2013

Author Contributor

Okay, reverted.
try:
connection = read_engine.connect()
result = connection.execute(
Expand Down

0 comments on commit 77c7846

Please sign in to comment.