Updating the homepage

Author: weaverryan

app/Resources/views/default/homepage.html.twig

@@ -11,10 +11,24 @@
 
                     <h1>Carson: The Issue Butler</h1>
 
-                    <div>
-                        <a href="{{ needsReviewUrl }}" class="btn btn-primary">Find "Needs Review" issues and pull requests</a>
-                    </div>
-
+                    <table class="table">
+                        <thead>
+                            <tr>
+                                <th>Repository</th>
+                                <th>Needs Review issues & pull requests</th>
+                            </tr>
+                        </thead>
+                        <tbody>
+                            {% for repository in repositories %}
+                                <tr>
+                                    <td>{{ repository.vendor }}/{{ repository.name }}</td>
+                                    <td>
+                                        <a href="{{ repository.needsReviewUrl }}" class="btn btn-xs btn-default">Needs Review Issues</a>
+                                    </td>
+                                </tr>
+                            {% endfor %}
+                        </tbody>
+                    </table>
                 </div>
             </div>
         </div>

src/AppBundle/Controller/DefaultController.php

@@ -12,8 +12,10 @@ class DefaultController extends Controller
      */
     public function homepageAction()
     {
+        $repositories = $this->get('app.repository_provider')->getAllRepositories();
+
         return $this->render('default/homepage.html.twig', [
-            'needsReviewUrl' => $this->get('app.status_api')->getNeedsReviewUrl(),
+            'repositories' => $repositories
         ]);
     }
 }

src/AppBundle/Issues/GitHub/GitHubStatusApi.php

@@ -8,7 +8,7 @@
 
 class GitHubStatusApi implements StatusApi
 {
-    private $statusToLabel = [
+    private static $statusToLabel = [
         Status::NEEDS_REVIEW => 'Status: Needs Review',
         Status::NEEDS_WORK => 'Status: Needs Work',
         Status::WORKS_FOR_ME => 'Status: Works for me',
@@ -25,7 +25,7 @@ class GitHubStatusApi implements StatusApi
     public function __construct(CachedLabelsApi $labelsApi)
     {
         $this->labelsApi = $labelsApi;
-        $this->labelToStatus = array_flip($this->statusToLabel);
+        $this->labelToStatus = array_flip(self::$statusToLabel);
     }
 
     /**
@@ -35,11 +35,11 @@ public function __construct(CachedLabelsApi $labelsApi)
      */
     public function setIssueStatus($issueNumber, $newStatus, Repository $repository)
     {
-        if (!isset($this->statusToLabel[$newStatus])) {
+        if (!isset(self::$statusToLabel[$newStatus])) {
             throw new \InvalidArgumentException(sprintf('Invalid status "%s"', $newStatus));
         }
 
-        $newLabel = $this->statusToLabel[$newStatus];
+        $newLabel = self::$statusToLabel[$newStatus];
         $currentLabels = $this->labelsApi->getIssueLabels($issueNumber, $repository);
         $addLabel = true;
 
@@ -82,13 +82,8 @@ public function getIssueStatus($issueNumber, Repository $repository)
         return;
     }
 
-    public function getNeedsReviewUrl(Repository $repository)
+    public static function getNeedsReviewLabel()
     {
-        return sprintf(
-            'https://github.com/%s/%s/labels/%s',
-            $repository->getVendor(),
-            $repository->getName(),
-            rawurlencode($this->statusToLabel[Status::NEEDS_REVIEW])
-        );
+        return self::$statusToLabel[Status::NEEDS_REVIEW];
     }
 }

src/AppBundle/Issues/NullStatusApi.php

@@ -15,8 +15,4 @@ public function getIssueStatus($issueNumber, Repository $repository)
     public function setIssueStatus($issueNumber, $newStatus, Repository $repository)
     {
     }
-
-    public function getNeedsReviewUrl(Repository $repository)
-    {
-    }
 }

src/AppBundle/Issues/StatusApi.php

@@ -13,6 +13,4 @@ interface StatusApi
     public function getIssueStatus($issueNumber, Repository $repository);
 
     public function setIssueStatus($issueNumber, $newStatus, Repository $repository);
-
-    public function getNeedsReviewUrl(Repository $repository);
 }

src/AppBundle/Repository/Provider/InMemoryRepositoryProvider.php

@@ -44,6 +44,11 @@ public function getRepository($repositoryName)
         return isset($this->repositories[$repositoryName]) ? $this->repositories[$repositoryName] : null;
     }
 
+    public function getAllRepositories()
+    {
+        return array_values($this->repositories);
+    }
+
     private function addRepository(Repository $repository)
     {
         $this->repositories[strtolower($repository->getVendor().'/'.$repository->getName())] = $repository;

src/AppBundle/Repository/Provider/RepositoryProviderInterface.php

@@ -15,4 +15,9 @@ interface RepositoryProviderInterface
      * @return Repository
      */
     public function getRepository($repositoryName);
+
+    /**
+     * @return Repository[]
+     */
+    public function getAllRepositories();
 }

src/AppBundle/Repository/Repository.php

@@ -1,6 +1,8 @@
 <?php
 
 namespace AppBundle\Repository;
+use AppBundle\Issues\GitHub\GitHubStatusApi;
+use AppBundle\Issues\Status;
 
 /**
  * @author Ener-Getick <egetick@gmail.com>
@@ -83,4 +85,14 @@ public function getSecret()
     {
         return $this->secret;
     }
+
+    public function getNeedsReviewUrl()
+    {
+        return sprintf(
+            'https://github.com/%s/%s/labels/%s',
+            $this->getVendor(),
+            $this->getName(),
+            rawurlencode(GitHubStatusApi::getNeedsReviewLabel())
+        );
+    }
 }