Skip to content

Commit

Permalink
Don't set last_modified_by if only changing ref to last result
Browse files Browse the repository at this point in the history
  • Loading branch information
arikfr committed Aug 1, 2015
1 parent 189ceaf commit e6f6698
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion controllers.py
Original file line number Diff line number Diff line change
Expand Up @@ -392,7 +392,9 @@ def post(self, query_id):
if 'data_source_id' in query_def:
query_def['data_source'] = query_def.pop('data_source_id')

query_def['last_modified_by'] = self.current_user
# Don't set "last_modified_by" if the user only refreshing this query
if not ('latest_query_data' in query_def and len(query_def.keys()) == 1):
query_def['last_modified_by'] = self.current_user

# TODO: use #save() with #dirty_fields.
models.Query.update_instance(query_id, **query_def)
Expand Down

0 comments on commit e6f6698

Please sign in to comment.