Merge pull request #51 from BlackbitNeueMedien/bugfix/graphviz-instal…

…lation-error [Visualization] Show message when graphviz is not installed
YouweGit · Nov 9, 2020 · f588975 · f588975
2 parents 540c538 + d892e76
commit f588975
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/WorkflowGui/Controller/WorkflowController.php b/src/WorkflowGui/Controller/WorkflowController.php
@@ -333,7 +333,6 @@ public function visualizeImageAction(Request $request)
         $this->isGrantedOr403();
 
         try {
-
             $image = $this->getVisualization($request->get('workflow'), 'png');
 
             $response = new Response();
@@ -363,9 +362,9 @@ public function visualizeImageAction(Request $request)
      */
     private function getVisualization($workflow, $format) {
         try {
-            $dotExecutable = Console::getExecutable('dot');
+            $dotExecutable = Console::getExecutable('dot', true);
         } catch(\Exception $e) {
-            return new Response('Please install graphviz to visualize workflows');
+            throw new \Exception('Please install graphviz to visualize workflows');
         }
 
         $cliCommand = '"'.Console::getPhpCli().'" "'.PIMCORE_PROJECT_ROOT . '/bin/console" pimcore:workflow:dump '.$workflow.' | "'.$dotExecutable.'" -T'.$format;