Merge pull request #1126 from getredash/fix_669

Fix #669: save fails when doing partial save of new query

bin/pre_compile

@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-# heroku pre_compile script
+# Heroku pre_compile script
 
 DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 pushd $DIR/..
@@ -8,8 +8,9 @@ pushd $DIR/..
 # https://github.com/heroku/heroku-buildpack-python/blob/master/bin/steps/cryptography
 # to avoid making it a requirement for other build systems, we'll inject it now
 # into the requirements.txt file
-# remove pymssql, it doesn't compile on heroku
+
-grep -v '^pymssql' requirements_all_ds.txt >> requirements.txt
+# Remove Heroku unsupported Python packages:
+grep -v -E "^(pymssql|thrift|sasl|pyhive)" requirements_all_ds.txt >> requirements.txt
 
 # make the heroku Procfile the active one
 cp Procfile.heroku Procfile

rd_ui/app/scripts/controllers/query_source.js

@@ -53,6 +53,10 @@
     $scope.saveQuery = function(options, data) {
       var savePromise = saveQuery(options, data);
 
+      if (!savePromise) {
+        return;
+      }
+
       savePromise.then(function(savedQuery) {
         queryText = savedQuery.query;
         $scope.isDirty = $scope.query.query !== queryText;

rd_ui/app/scripts/controllers/query_view.js

@@ -123,6 +123,10 @@
 
     $scope.saveQuery = function(options, data) {
       if (data) {
+        // Don't save new query with partial data
+        if ($scope.query.isNew()) {
+          return;
+        }
         data.id = $scope.query.id;
       } else {
         data = _.pick($scope.query, ["schedule", "query", "id", "description", "name", "data_source_id", "options"]);