View GitHub Repository chapi

Schedule Inspection
Subscribe

GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Completed
Pull Request — master (#68)
by Bidesh
03:32
created
Status
Category
Toggle Patches
src/Entity/Marathon/MarathonAppEntity.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch 
@@ -33,7 +33,7 @@  discard block
 
 block discarded – undo
33 33 
     /**
34 34 
      * @var PortDefinition[]
35 35 
      */
36 
-    public $portDefinitions =[];
36 
+    public $portDefinitions = [];
37 37
38 38 
     public $requirePorts = false;
39 39 
 
@@ -51,7 +51,7 @@  discard block
 
 block discarded – undo
51 51 
     /**
52 52 
      * @var array
53 53 
      */
54 
-    public $constraints  = [];
54 
+    public $constraints = [];
55 55
56 56
57 57 
     public $acceptedResourceRoles = [];
 
@@ -103,40 +103,40 @@  discard block
 
 block discarded – undo
103 103 
         {
104 104 
             foreach ($aData['portDefinitions'] as $portDefinition)
105 105 
             {
106 
-                $this->portDefinitions[] = new PortDefinition((array)$portDefinition);
106 
+                $this->portDefinitions[] = new PortDefinition((array) $portDefinition);
107 107 
             }
108 108 
         }
109 109
110 110 
         if (isset($aData['container']))
111 111 
         {
112 
-            $this->container = new Container((array)$aData['container']);
112 
+            $this->container = new Container((array) $aData['container']);
113 113 
         }
114 114
115 115 
         if (isset($aData['healthChecks']))
116 116 
         {
117 
-            foreach($aData['healthChecks'] as $healthCheck)
117 
+            foreach ($aData['healthChecks'] as $healthCheck)
118 118 
             {
119 
-                $this->healthChecks[] = new HealthCheck((array)$healthCheck);
119 
+                $this->healthChecks[] = new HealthCheck((array) $healthCheck);
120 120 
             }
121 121 
         }
122 122
123 123 
         if (isset($aData['upgradeStrategy']))
124 124 
         {
125 
-            $this->upgradeStrategy = new UpgradeStrategy((array)$aData['upgradeStrategy']);
125 
+            $this->upgradeStrategy = new UpgradeStrategy((array) $aData['upgradeStrategy']);
126 126 
         } else {
127 127 
             $this->upgradeStrategy = new UpgradeStrategy();
128 128 
         }
129 129
130 130 
         if (isset($aData['ipAddress']))
131 131 
         {
132 
-            $this->ipAddress = new IpAddress((array)$aData['ipAddress']);
132 
+            $this->ipAddress = new IpAddress((array) $aData['ipAddress']);
133 133 
         }
134 134
135 135 
         if (isset($aData['env']))
136 136 
         {
137 
-            $this->env =  (object) $aData['env'];
137 
+            $this->env = (object) $aData['env'];
138 138 
         } else {
139 
-            $this->env = (object)[];
139 
+            $this->env = (object) [];
140 140 
         }
141 141
142 142 
         if (isset($aData['labels']))
Please login to merge, or discard this patch.
src/BusinessCase/Comparison/CompositeJobComparisonBusinessCase.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -34,10 +34,10 @@  discard block
 
 block discarded – undo
34 34 
     public function getLocalMissingJobs()
35 35 
     {
36 36 
         $_aMissingJobs = array();
37 
-        foreach($this->aComposites as $jobComparers)
37 
+        foreach ($this->aComposites as $jobComparers)
38 38 
         {
39 39 
             $missing = $jobComparers->getLocalMissingJobs();
40 
-            $_aMissingJobs = array_merge($_aMissingJobs,  $missing);
40 
+            $_aMissingJobs = array_merge($_aMissingJobs, $missing);
41 41 
         }
42 42 
         return $_aMissingJobs;
43 43 
     }
 
@@ -48,7 +48,7 @@  discard block
 
 block discarded – undo
48 48 
     public function getRemoteMissingJobs()
49 49 
     {
50 50 
         $_aChronosMissingJobs = array();
51 
-        foreach($this->aComposites as $jobComparers)
51 
+        foreach ($this->aComposites as $jobComparers)
52 52 
         {
53 53 
             $_aChronosMissingJobs = array_merge($_aChronosMissingJobs, $jobComparers->getRemoteMissingJobs());
54 54 
         }
 
@@ -61,7 +61,7 @@  discard block
 
 block discarded – undo
61 61 
     public function getLocalJobUpdates()
62 62 
     {
63 63 
         $_aLocalJobUpdates = array();
64 
-        foreach($this->aComposites as $jobComparers)
64 
+        foreach ($this->aComposites as $jobComparers)
65 65 
         {
66 66 
             $_aLocalJobUpdates = array_merge($_aLocalJobUpdates, $jobComparers->getLocalJobUpdates());
67 67 
         }
 
@@ -75,7 +75,7 @@  discard block
 
 block discarded – undo
75 75 
     public function getJobDiff($sJobName)
76 76 
     {
77 77 
         $_aJobDiffs = array();
78 
-        foreach($this->aComposites as $jobComparers)
78 
+        foreach ($this->aComposites as $jobComparers)
79 79 
         {
80 80 
             // assuming same name won't be in all subsystems.
81 81 
             // TODO: add support to handle the duplicate names in different subsystems
 
@@ -101,7 +101,7 @@  discard block
 
 block discarded – undo
101 101 
         }
102 102 
         /** @var JobComparisonInterface $comparer */
103 103 
         $comparer = null;
104 
-        foreach($this->aComposites as $child)
104 
+        foreach ($this->aComposites as $child)
105 105 
         {
106 106 
             if ($child->isJobAvailable($oJobEntityA->getKey()))
107 107 
             {
 
@@ -125,7 +125,7 @@  discard block
 
 block discarded – undo
125 125 
      */
126 126 
     public function isJobAvailable($sJobName)
127 127 
     {
128 
-        foreach($this->aComposites as $child)
128 
+        foreach ($this->aComposites as $child)
129 129 
         {
130 130 
             if ($child->isJobAvailable($sJobName))
131 131 
             {
Please login to merge, or discard this patch.