From df5671927fb3ab924179a69b1afa4fef9563b27c Mon Sep 17 00:00:00 2001 From: Tyler Hallada Date: Fri, 26 Jan 2018 15:49:50 -0500 Subject: [PATCH] Handle no assets results case properly --- common/lib/xmodule/xmodule/contentstore/mongo.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/common/lib/xmodule/xmodule/contentstore/mongo.py b/common/lib/xmodule/xmodule/contentstore/mongo.py index 0f2274ed03..019f506f79 100644 --- a/common/lib/xmodule/xmodule/contentstore/mongo.py +++ b/common/lib/xmodule/xmodule/contentstore/mongo.py @@ -297,9 +297,14 @@ class MongoContentStore(ContentStore): }) items = self.fs_files.aggregate(pipeline_stages) - result = items['result'][0] - count = result['count'] - assets = list(result['results']) + if items['result']: + result = items['result'][0] + count = result['count'] + assets = list(result['results']) + else: + # no results + count = 0 + assets = [] # We're constructing the asset key immediately after retrieval from the database so that # callers are insulated from knowing how our identifiers are stored.