merged branch boombatower/process-status-5453 (PR symfony#5455)

fabpot · fabpot · commit 41cb44bdac1f · 2012-10-27T18:47:38.000+02:00
This PR was squashed before being merged into the master branch (closes symfony#5455). Commits ------- 7ea2f76 [process] expose the process status. Discussion ---------- [process] expose the process status. Pull request for issue symfony#5453. --------------------------------------------------------------------------- by pborreli at 2012-09-07T07:30:01Z :+1: --------------------------------------------------------------------------- by drak at 2012-09-21T18:53:14Z This PR is missing the patch header in the description https://github.com/symfony/symfony-docs/blob/master/contributing/code/patches.rst#make-a-pull-request --------------------------------------------------------------------------- by stof at 2012-10-13T21:25:04Z @boombatower can you update the PR according to my comments and add some tests ?
diff --git a/src/Symfony/Component/Process/Process.php b/src/Symfony/Component/Process/Process.php
@@ -927,6 +927,30 @@ public function setEnhanceSigchildCompatibility($enhance)
         return $this;
     }
 
+    public function getStatus()
+    {
+        $this->updateStatus();
+
+        return $this->status;
+    }
+
+    /**
+     * Checks if the process has been started with no regard to current state.
+     *
+     * @return Boolean true if status is started or terminated, false otherwise
+     */
+    public function isStarted()
+    {
+        return $this->status != self::STATUS_READY;
+    }
+
+    public function isTerminated()
+    {
+        $this->updateStatus();
+
+        return $this->status == self::STATUS_TERMINATED;
+    }
+
     /**
      * Builds up the callback used by wait().
      *
