Completed
Pull Request — master (#193)
by
unknown
01:45
created
src/Sprint/SprintService.php 3 patches
Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -26,14 +26,14 @@  discard block
 block discarded – undo
26 26
         $this->path = $path;
27 27
         $this->restClient = $jiraClient;
28 28
         if (!$this->restClient) {
29
-          $this->setRestClient();
29
+            $this->setRestClient();
30 30
         }
31 31
 
32 32
     }
33 33
 
34 34
     public function setRestClient(){
35
-      $this->restClient = new JiraClient($this->configuration, $this->logger, $this->path);
36
-      $this->restClient->setAPIUri('');
35
+        $this->restClient = new JiraClient($this->configuration, $this->logger, $this->path);
36
+        $this->restClient->setAPIUri('');
37 37
     }
38 38
 
39 39
 
@@ -61,26 +61,26 @@  discard block
 block discarded – undo
61 61
     }
62 62
 
63 63
     public function getVelocityForSprint($sprintID){
64
-      try {
64
+        try {
65 65
         $sprint = $this->getSprint($sprintID);
66 66
         if (!is_null($sprint->originBoardId)){
67
-          $ret = $this->restClient->exec('/rest/greenhopper/1.0/rapid/charts/velocity.json?rapidViewId='.$sprint->originBoardId.'&sprintId='.$sprint->id);
68
-          $velocityObject = json_decode($ret);
69
-          $velocityStats = $velocityObject->{'velocityStatEntries'};
70
-          if (property_exists($velocityStats,$sprint->id)) {
67
+            $ret = $this->restClient->exec('/rest/greenhopper/1.0/rapid/charts/velocity.json?rapidViewId='.$sprint->originBoardId.'&sprintId='.$sprint->id);
68
+            $velocityObject = json_decode($ret);
69
+            $velocityStats = $velocityObject->{'velocityStatEntries'};
70
+            if (property_exists($velocityStats,$sprint->id)) {
71 71
             $sprint->estimatedVelocity = $velocityStats->{$sprint->id}->{'estimated'}->value;
72 72
             $sprint->completedVelocity = $velocityStats->{$sprint->id}->{'completed'}->value;
73
-          } else {
73
+            } else {
74 74
             $sprint->estimatedVelocity = null;
75 75
             $sprint->completedVelocity = null;
76
-          }
76
+            }
77 77
         }
78 78
         return $sprint;
79
-      }
80
-      catch (JiraException $e) {
79
+        }
80
+        catch (JiraException $e) {
81 81
         print("Error Occured! " . $e->getMessage());
82 82
         return null;
83
-      }
83
+        }
84 84
     }
85 85
 
86 86
 }
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 
32 32
     }
33 33
 
34
-    public function setRestClient(){
34
+    public function setRestClient() {
35 35
       $this->restClient = new JiraClient($this->configuration, $this->logger, $this->path);
36 36
       $this->restClient->setAPIUri('');
37 37
     }
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
     {
42 42
         $json_mapper = new \JsonMapper();
43 43
         $json_mapper->undefinedPropertyHandler = [new \JiraRestApi\JsonMapperHelper(), 'setUndefinedProperty'];
44
-        return $json_mapper->map($json, new Sprint() );
44
+        return $json_mapper->map($json, new Sprint());
45 45
     }
46 46
 
47 47
     /**
@@ -60,14 +60,14 @@  discard block
 block discarded – undo
60 60
         return $this->getSprintFromJSON(json_decode($ret));
61 61
     }
62 62
 
63
-    public function getVelocityForSprint($sprintID){
63
+    public function getVelocityForSprint($sprintID) {
64 64
       try {
65 65
         $sprint = $this->getSprint($sprintID);
66
-        if (!is_null($sprint->originBoardId)){
66
+        if (!is_null($sprint->originBoardId)) {
67 67
           $ret = $this->restClient->exec('/rest/greenhopper/1.0/rapid/charts/velocity.json?rapidViewId='.$sprint->originBoardId.'&sprintId='.$sprint->id);
68 68
           $velocityObject = json_decode($ret);
69 69
           $velocityStats = $velocityObject->{'velocityStatEntries'};
70
-          if (property_exists($velocityStats,$sprint->id)) {
70
+          if (property_exists($velocityStats, $sprint->id)) {
71 71
             $sprint->estimatedVelocity = $velocityStats->{$sprint->id}->{'estimated'}->value;
72 72
             $sprint->completedVelocity = $velocityStats->{$sprint->id}->{'completed'}->value;
73 73
           } else {
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
         return $sprint;
79 79
       }
80 80
       catch (JiraException $e) {
81
-        print("Error Occured! " . $e->getMessage());
81
+        print("Error Occured! ".$e->getMessage());
82 82
         return null;
83 83
       }
84 84
     }
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -76,8 +76,7 @@
 block discarded – undo
76 76
           }
77 77
         }
78 78
         return $sprint;
79
-      }
80
-      catch (JiraException $e) {
79
+      } catch (JiraException $e) {
81 80
         print("Error Occured! " . $e->getMessage());
82 81
         return null;
83 82
       }
Please login to merge, or discard this patch.
src/Board/Board.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -25,8 +25,8 @@
 block discarded – undo
25 25
 
26 26
     public function jsonSerialize()
27 27
     {
28
-      return array_filter(get_object_vars($this), function ($var) {
29
-          return !is_null($var);
30
-      });
28
+        return array_filter(get_object_vars($this), function ($var) {
29
+            return !is_null($var);
30
+        });
31 31
     }
32 32
 }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
 
26 26
     public function jsonSerialize()
27 27
     {
28
-      return array_filter(get_object_vars($this), function ($var) {
28
+      return array_filter(get_object_vars($this), function($var) {
29 29
           return !is_null($var);
30 30
       });
31 31
     }
Please login to merge, or discard this patch.