Repository: couchdb-fauxton
Updated Branches:
  refs/heads/master 25ea5bdf0 -> 12f372ffd


Fix for Query Options search

Closes COUCHDB-2451


Project: http://git-wip-us.apache.org/repos/asf/couchdb-fauxton/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb-fauxton/commit/12f372ff
Tree: http://git-wip-us.apache.org/repos/asf/couchdb-fauxton/tree/12f372ff
Diff: http://git-wip-us.apache.org/repos/asf/couchdb-fauxton/diff/12f372ff

Branch: refs/heads/master
Commit: 12f372ffd09c18f49baa85ce951ab056dea79d98
Parents: 25ea5bd
Author: Benjamin Keen <[email protected]>
Authored: Mon Nov 10 12:42:41 2014 -0800
Committer: Benjamin Keen <[email protected]>
Committed: Tue Nov 11 13:07:41 2014 -0800

----------------------------------------------------------------------
 app/addons/documents/views-queryoptions.js | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/couchdb-fauxton/blob/12f372ff/app/addons/documents/views-queryoptions.js
----------------------------------------------------------------------
diff --git a/app/addons/documents/views-queryoptions.js 
b/app/addons/documents/views-queryoptions.js
index dd1e536..69b5c02 100644
--- a/app/addons/documents/views-queryoptions.js
+++ b/app/addons/documents/views-queryoptions.js
@@ -25,8 +25,6 @@ define([
     var defaultOptions = {
       showStale: false,
       hasReduce: false,
-      viewName: null,
-      ddocName: null,
 
       // all the possible query search params. Ultimately these should 
probably be moved higher-up (route object?),
       // because they also apply to the actual search results. Seems better to 
place them there, then use them in
@@ -120,15 +118,9 @@ define([
         // this may be empty. That's ok! Perhaps the user just did a search 
with params, then removed them & wants the default
         var params = this.getQueryParams();
 
-        // all looks good! Close the tray and publish the message
+        // all looks good! Use navigate() to update the page URL and trigger 
the appropriate route
         var url = app.utils.replaceQueryParams(params);
         FauxtonAPI.navigate(url);
-
-        if (this.options.viewName !== null && this.options.ddocName !== null) {
-          FauxtonAPI.triggerRouteEvent('updateAllDocs', { ddoc: 
this.options.ddocName, view: this.options.viewName });
-        } else {
-          FauxtonAPI.triggerRouteEvent("updateAllDocs", { allDocs: true });
-        }
       },
 
       // if the screen is so small there isn't space for the full tray height 
we manually shrink the height to allow scrolling.

Reply via email to