Completed
Pull Request — master (#142)
by Robbie
01:43
created
mysite/code/services/PackagistService.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 use Composer\Factory;
4 4
 use Composer\IO\NullIO;
5 5
 use Composer\Package\Loader\ArrayLoader;
6
-use Composer\Repository\ComposerRepository;
7 6
 use Guzzle\Http\Client;
8 7
 
9 8
 /**
Please login to merge, or discard this patch.
mysite/code/dataobjects/Addon.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	/**
105 105
 	 * Remove the effect of code of conduct Helpful Robot measure that we currently don't include in the Supported module definition
106 106
 	 *
107
-	 * @return integer Adjusted Helpful Robot score
107
+	 * @return double Adjusted Helpful Robot score
108 108
 	 */
109 109
 	public function getAdjustedHelpfulRobotScore()
110 110
 	{
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	}
164 164
 
165 165
 	/**
166
-	 * @return array
166
+	 * @return ArrayData
167 167
 	 */
168 168
 	public function HelpfulRobotData()
169 169
 	{
Please login to merge, or discard this patch.
mysite/code/dataobjects/SilverStripeVersion.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
 	);
21 21
 
22 22
 	/**
23
-	 * @return Composer\Package\LinkConstraint\LinkConstraintInterface
23
+	 * @return MultiConstraint
24 24
 	 */
25 25
 	public function getConstraint() {
26 26
 		$next = $this->Major . '.' . ($this->Minor + 1);
Please login to merge, or discard this patch.
mysite/code/services/AddonUpdater.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -1,9 +1,7 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-use Composer\Package\AliasPackage;
4 3
 use Composer\Package\CompletePackage;
5 4
 use Composer\Semver\Constraint\Constraint;
6
-use Guzzle\Http\Exception\ClientErrorResponseException;
7 5
 use SilverStripe\Elastica\ElasticaService;
8 6
 use Packagist\Api\Result\Package;
9 7
 use Composer\Package\Version\VersionParser;
Please login to merge, or discard this patch.
mysite/code/services/AddonBuilder.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -66,6 +66,9 @@  discard block
 block discarded – undo
66 66
         $addon->write();
67 67
     }
68 68
 
69
+    /**
70
+     * @param string $path
71
+     */
69 72
     protected function download(PackageInterface $package, $path) {
70 73
         $this->packagist
71 74
             ->getComposer()
@@ -73,6 +76,9 @@  discard block
 block discarded – undo
73 76
             ->download($package, $path);
74 77
     }
75 78
 
79
+    /**
80
+     * @param string $path
81
+     */
76 82
     private function buildReadme(Addon $addon, $path) {
77 83
         $candidates = array(
78 84
             'README.md',
@@ -107,6 +113,9 @@  discard block
 block discarded – undo
107 113
         }
108 114
     }
109 115
 
116
+    /**
117
+     * @param string $path
118
+     */
110 119
     private function buildScreenshots(Addon $addon, PackageInterface $package, $path) {
111 120
         $extra = $package->getExtra();
112 121
         $screenshots = array();
Please login to merge, or discard this patch.