Change Execute Action Parameter

Summary: `'Execute'` action was previously used only for execute tab which assumed the input will come from the state. This changes so that it can take any input attached to parameters (more explanation about the design in <insert diff number>)

Reviewed By: jknoxville

Differential Revision: D21788244

fbshipit-source-id: 8fb8ecb8f8df84f4854f70432e7a1fdb531403e9

desktop/plugins/databases/index.tsx

@@ -219,6 +219,7 @@ type PreviousPageEvent = {
 
 type ExecuteEvent = {
   type: 'Execute';
+  query: string;
 };
 
 type RefreshEvent = {
@@ -602,18 +603,12 @@ export default class DatabasesPlugin extends FlipperPlugin<
       'Execute',
       (
         state: DatabasesPluginState,
-        _results: ExecuteEvent,
+        event: ExecuteEvent,
       ): DatabasesPluginState => {
         const timeBefore = Date.now();
-        if (
+        const {query} = event;
-          this.state.query !== null &&
-          typeof this.state.query !== 'undefined'
-        ) {
         this.databaseClient
-            .getExecution({
+          .getExecution({databaseId: state.selectedDatabase, value: query})
-              databaseId: state.selectedDatabase,
-              value: this.state.query.value,
-            })
           .then((data) => {
             this.setState({
               error: null,
@@ -640,7 +635,6 @@ export default class DatabasesPlugin extends FlipperPlugin<
           .catch((e) => {
             this.setState({error: e});
           });
-        }
         let newHistory = this.state.queryHistory;
         const newQuery = this.state.query;
         if (
@@ -964,7 +958,9 @@ export default class DatabasesPlugin extends FlipperPlugin<
   };
 
   onExecuteClicked = () => {
-    this.dispatchAction({type: 'Execute'});
+    if (this.state.query !== null) {
+      this.dispatchAction({type: 'Execute', query: this.state.query.value});
+    }
   };
 
   onQueryTextareaKeyPress = (event: KeyboardEvent) => {