correctly handle facet queries that have no matches

GMOD · May 23, 2012 · 3f6916b · 3f6916b
1 parent 8f2b8c3
commit 3f6916b
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/js/Model/TrackMetaData.js b/js/Model/TrackMetaData.js
@@ -426,8 +426,13 @@ dojo.declare( 'JBrowse.Model.TrackMetaData', null,
                 dojo.map( dojof.keys( query ), function( facetName ) {
                     var values = query[facetName];
                     var items = [];
+                    if( ! this.facetIndexes.byName[facetName] ) {
+                        console.error( "No facet defined with name '"+facetName+"'." );
+                        throw "No facet defined with name '"+facetName+"', faceted search failed.";
+                    }
                     dojo.forEach( values, function(value) {
-                        items.push.apply( items, this.facetIndexes.byName[facetName].byValue[value].items );
+                        var idx = this.facetIndexes.byName[facetName].byValue[value] || {};
+                        items.push.apply( items, idx.items || [] );
                     },this);
                     items.facetName = facetName;
                     items.sort( dojo.hitch( this, '_itemSortFunc' ));