archive/pillar
0
0
Fork 0
Code Issues 42 Pull Requests Projects Releases Wiki Activity

Don't fetch entire project just to fetch a node type.

Browse Source 
The query can probably be made more specific by including the node type
beforehand, and only fetching that.
This commit is contained in:
Sybren A. Stüvel 2016-07-26 17:20:03 +02:00
parent 336ba7d6fe
commit 83bad3d88d
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
pillar/application/__init__.py
View File

@ -54,10 +54,13 @@ class ValidateCustomFields(Validator):
projects_collection = app.data.driver.db['projects'] projects_collection = app.data.driver.db['projects']
lookup = {'_id': ObjectId(self.document['project'])} lookup = {'_id': ObjectId(self.document['project'])}
project = projects_collection.find_one(lookup) project = projects_collection.find_one(lookup, {
'node_types.name': 1,
'node_types.dyn_schema': 1,
})
if project is None: if project is None:
log.warning('Unknown project %s, declared by node %s', log.warning('Unknown project %s, declared by node %s',
project, self.document.get('_id')) lookup, self.document.get('_id'))
self._error(field, 'Unknown project') self._error(field, 'Unknown project')
return False return False