Merge pull request #95 from phpcr/exit-ask-confirmation

Ask confirmation before exiting when there are pending changes

Author: dantleech

CHANGELOG.md

@@ -8,6 +8,10 @@ dev-master
 
 - [mixin] Node mxin remove does not accept a path
 
+### Enhancements
+
+- [exit] Ask for confirmation before logging out when there are pending changes
+
 alpha-5
 -------
 

features/shell_exit.feature

@@ -0,0 +1,12 @@
+Feature: Exit the shell
+    In order to quit this damned shell
+    As a user
+    I want to be able to execute a command which does that.
+
+    Background:
+        Given that I am logged in as "testuser"
+
+    Scenario: Make a change and attempt to exist (default to no exit)
+        Given I execute the "node:create foo" command
+        And I execute the "shell:exit --no-interaction" command
+        Then the command should not fail

src/PHPCR/Shell/Console/Command/Shell/ExitCommand.php

@@ -16,7 +16,23 @@ public function configure()
 
     public function execute(InputInterface $input, OutputInterface $output)
     {
-        $this->getHelper('phpcr')->getSession()->logout();
+        $dialog = $this->getHelper('dialog');
+        $session = $this->getHelper('phpcr')->getSession();
+        $noInteraction = $input->getOption('no-interaction');
+
+        if ($session->hasPendingChanges()) {
+            $res = false;
+
+            if ($input->isInteractive()) {
+                $res = $dialog->askConfirmation($output, '<question>Session has pending changes, are you sure you want to quit? (Y/N)</question>', false);
+            }
+
+            if (false === $res) {
+                return;
+            }
+        }
+
+        $session->logout();
         $output->writeln('<info>Bye!</info>');
         exit(0);
     }