impl updateIssue

Author: lesstif

composer.json

@@ -4,7 +4,7 @@
      "type": "library",
     "keywords": ["jira", "jira-php", "jira-rest"],
     "require": {
-        "php": ">=5.3",
+        "php": ">=5.3.1",
         "netresearch/jsonmapper": "0.4.*",
         "monolog/monolog": "~1.12"
     },

src/JiraClient.php

@@ -57,6 +57,13 @@ private function convertLogLevel($log_level) {
 			return Logger::INFO;
 	}
 
+	// serilize only not null field.
+	protected function filterNullVariable ($arr) {		 
+        return array_filter((array) $arr, function ($val) {
+        	return (!is_null($val) && !empty($val) );
+        });
+	}
+
 	public function __construct($config)
     {
     	$this->json_mapper = new \JsonMapper();

src/issue/IssueField.php

@@ -75,6 +75,23 @@ public function addVersion($id, $name) {
     	return $this;
     }
 
+    public function addComment($comment) {
+    	if (is_null($this->comments))
+    	  $this->comments = new \JiraRestApi\Issue\Comments();
+
+    	array_push($this->versions, $v);
+    	return $this;
+    }
+
+    //@TODO
+    public function addLabel($label) {
+    	if (is_null($this->labels))
+    	  $this->labels = array();
+
+    	array_push($this->labels, $label);
+    	return $this;
+    }
+
 	/** @var string */
 	public $summary;
 
@@ -121,7 +138,7 @@ public function addVersion($id, $name) {
 	public $components;
 
 	/** @var Comments */
-	public $comment;
+	public $comments;
 
 	/** @var string */
 	public $votes;

src/issue/IssueService.php

@@ -37,10 +37,7 @@ public function create($issueField) {
         $issue = new Issue();
 
         // serilize only not null field.
-        $issue->fields = array_filter((array) $issueField, function ($val) {
-            return !is_null($val);
-        });
-
+        $issue->fields = $this->filterNullVariable($issueField);
 
         $data = json_encode($issue);
 
@@ -79,26 +76,22 @@ public function addAttachments($issueIdOrKey, $filePath) {
     /**
      * update issue
      * 
-     * @param   $issueKey Issue Key
+     * @param   $issueIdOrKey Issue Key
      * @param   $issueField object of Issue class
      * 
      * @return created issue key
      */
-    public function update($issueKey, $issueField) {
+    public function update($issueIdOrKey, $issueField) {
         $issue = new Issue();
 
-        $issue->key = $issueKey;
-
         // serilize only not null field.
-        $issue->fields = array_filter((array) $issueField, function ($val) {
-            return !is_null($val);
-        });
+        $issue->fields = $this->filterNullVariable($issueField);
 
         $data = json_encode($issue);
 
         $this->log->addInfo("Update Issue=\n" . $data );
 
-        $ret = $this->exec($this->uri . "/$issueIdOrKey/attachments", $data, "PUT");
+        $ret = $this->exec($this->uri . "/$issueIdOrKey", $data, "PUT");
 
         $issue = $this->json_mapper->map(
              json_decode($ret), new Issue()