Created attachment 347039 [details] [diff] [review] v1 evaluate to undefined rather than dereferencing When gloda is started with a clean database and has not made any contacts be popular, the query for popular contacts will return with 0 contacts. Accordingly, there will be no identity sub-collection associated, which means that glautocomp's cleverness in attempting to fish out the identity collection will fail. I had already dealt with this partially, but was in a rush and didn't get it all. This fix is the one I discussed with davida. In a nutshell, it's okay to pass undefined to concat, and we use && in order to short-circuit return undefined rather than attempting to dereference it.
Comment on attachment 347039 [details] [diff] [review] v1 evaluate to undefined rather than dereferencing r=dmose, conditional on adding a comment explaining what's going on
Created attachment 348852 [details] [diff] [review] v2 evaluate to undefined rather than dereferencing added requested comment, carrying forward r=dmose.