Merge pull request #5 from azijul1/master

change return logic in dial, menu, and filter tags

Author: sankarsuda

src/Voice/Tag/Dial.php

@@ -19,32 +19,35 @@ public function __construct($to, $from = null)
 
     public function onAnswer($tag, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($tag, $attribs);
         if($isSequential){
-            $this->attributes['onanswer'] = array_merge($this->attributes['onanswer'], array($this->append($tag, $attribs)) );
-            return $this->attributes;
+            $this->attributes['onanswer'] = array_merge($this->attributes['onanswer'], array($newTag) );
         }else {
-            return $this->setAttribute('onanswer', array($this->append($tag, $attribs)), true);
+            $this->setAttribute('onanswer', array($newTag), true);
         }
+        return $newTag;
     }
 
     public function onNoAnswer($tag, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($tag, $attribs);
         if($isSequential){
-            $this->attributes['onnoanswer'] = array_merge($this->attributes['onnoanswer'], array($this->append($tag, $attribs)) );
-            return $this->attributes;
+            $this->attributes['onnoanswer'] = array_merge($this->attributes['onnoanswer'], array($newTag) );
         }else {
-            return $this->setAttribute('onnoanswer', array($this->append($tag, $attribs)), true);
+            $this->setAttribute('onnoanswer', array($newTag), true);
         }
+        return $newTag;
     }
 
     public function noAnswer($tag, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($tag, $attribs);
         if($isSequential){
-            $this->attributes['onnoanswer'] = array_merge($this->attributes['onnoanswer'], array($this->append($tag, $attribs)) );
-            return $this->attributes;
+            $this->attributes['onnoanswer'] = array_merge($this->attributes['onnoanswer'], array($newTag) );
         }else {
-            return $this->setAttribute('onnoanswer', array($this->append($tag, $attribs)), true);
+            $this->setAttribute('onnoanswer', array($newTag), true);
         }
+        return $newTag;
     }
 
     public function getDefaultAttributes()

src/Voice/Tag/Filter.php

@@ -32,22 +32,26 @@ public function __construct($data = null, $type = 'frequency')
 
     public function onFail($tag, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($tag, $attribs);
         if($isSequential){
-            $this->attributes['onfail'] = array_merge($this->attributes['onfail'], array($this->append($tag, $attribs)) );
-            return $this->attributes;
+            $this->attributes['onfail'] = array_merge($this->attributes['onfail'], array($newTag) );
         }else {
-            return $this->setAttribute('onfail', array($this->append($tag, $attribs)), true);
+            $this->setAttribute('onfail', array($newTag), true);
         }
+
+        return $newTag;
     }
 
     public function onPass($tag, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($tag, $attribs);
         if($isSequential){
-            $this->attributes['onpass'] = array_merge($this->attributes['onpass'], array($this->append($tag, $attribs)) );
-            return $this->attributes;
+            $this->attributes['onpass'] = array_merge($this->attributes['onpass'], array($newTag) );
         }else {
-            return $this->setAttribute('onpass', array($this->append($tag, $attribs)), true);
+            $this->setAttribute('onpass', array($newTag), true);
         }
+
+        return $newTag;
     }
 
     public function getDefaultAttributes()

src/Voice/Tag/Menu.php

@@ -20,22 +20,25 @@ public function __construct($prompt, $attribs = [])
 
     public function onFail($verb, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($verb, $attribs);
         if($isSequential){
-            $this->attributes['onfail'] = array_merge($this->attributes['onfail'], array($this->append($verb, $attribs)) );
-            return $this->attributes;
+            $this->attributes['onfail'] = array_merge($this->attributes['onfail'], array($newTag) );
         }else {
-            return $this->setAttribute('onfail', array($this->append($verb, $attribs)), true);
+            $this->setAttribute('onfail', array($newTag), true);
         }
+        return $newTag;
     }
 
     public function onTimeout($verb, $attribs = [], $isSequential = false)
     {
+        $newTag = $this->append($verb, $attribs);
         if($isSequential){
-            $this->attributes['timeout'] = array_merge($this->attributes['timeout'], array($this->append($verb, $attribs)) );
-            return $this->attributes;
+            $this->attributes['timeout'] = array_merge($this->attributes['timeout'], array($newTag) );
         }else {
-            return $this->setAttribute('timeout', array($this->append($verb, $attribs)), true);
+            $this->setAttribute('timeout', array($newTag), true);
         }
+
+        return $newTag;
     }
 
     public function getDefaultAttributes()