Merge pull request #4097 from dalthviz/fixes_issue_4086

PR: Add a validation for editorstack existence
spyder-ide · Feb 4, 2017 · d4def8e · d4def8e
2 parents 0e08506 + a501591
commit d4def8e
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/continuous_integration/travis/build_test.sh b/continuous_integration/travis/build_test.sh
@@ -9,7 +9,7 @@ cd $FULL_SPYDER_CLONE
 if [ $TRAVIS_PULL_REQUEST != "false" ] ; then
     git checkout travis_pr_$TRAVIS_PULL_REQUEST
 else
-    git checkout master
+    git checkout $TRAVIS_BRANCH
 fi
 
 # -- Build package

diff --git a/continuous_integration/travis/modules_test.sh b/continuous_integration/travis/modules_test.sh
@@ -11,7 +11,7 @@ cd $FULL_SPYDER_CLONE
 if [ $TRAVIS_PULL_REQUEST != "false" ] ; then
     git checkout travis_pr_$TRAVIS_PULL_REQUEST
 else
-    git checkout master
+    git checkout $TRAVIS_BRANCH
 fi
 
 # Depth 1

diff --git a/spyder/plugins/editor.py b/spyder/plugins/editor.py
@@ -1475,9 +1475,10 @@ def refresh_file_dependent_actions(self):
     def refresh_save_all_action(self):
         """Enable 'Save All' if there are files to be saved"""
         editorstack = self.get_current_editorstack()
-        state = any(finfo.editor.document().isModified()
-                    for finfo in editorstack.data)
-        self.save_all_action.setEnabled(state)
+        if editorstack:
+            state = any(finfo.editor.document().isModified()
+                        for finfo in editorstack.data)
+            self.save_all_action.setEnabled(state)
 
     def update_warning_menu(self):
         """Update warning list menu"""