From 384b627b5ef652c4ee31ba4a6340468237442ae6 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Sun, 4 May 2014 20:03:00 -0700 Subject: [PATCH] ui: reduce describe depth, using recursive describe instead Updates issue https://code.google.com/p/camlistore/issues/detail?id=319 Change-Id: Ie02b0f565c6ff4c9582cecc78914392a60bf9502 --- server/camlistored/ui/search_session.js | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/server/camlistored/ui/search_session.js b/server/camlistored/ui/search_session.js index feca397a6..b3c5d58f1 100644 --- a/server/camlistored/ui/search_session.js +++ b/server/camlistored/ui/search_session.js @@ -62,11 +62,27 @@ cam.SearchSession.DESCRIBE_REQUEST = { // TODO(aa): This needs to die: https://code.google.com/p/camlistore/issues/detail?id=321 thumbnailSize: 1000, - // TODO(aa): This is not great. The describe request will return tons of data we don't care about: - // - Children of folders + // TODO(aa): This is not perfect. The describe request will return some data we don't care about: // - Properties we don't use // See: https://code.google.com/p/camlistore/issues/detail?id=319 - depth: 4 + + depth: 1, + rules: [ + { + attrs: ['camliContent', 'camliContentImage'] + }, + { + ifCamliNodeType: 'foursquare.com:checkin', + attrs: ['foursquareVenuePermanode'] + }, + { + ifCamliNodeType: 'foursquare.com:venue', + attrs: ['camliPath:photos'], + rules: [ + { attrs: ['camliPath:*'] } + ] + } + ] }; cam.SearchSession.instanceCount_ = 0;