X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;f=app%2FController%2FGithub.php;h=364f0e584424fa2a1438605f746f14ecf655c5b6;hb=eb76e9bb9a39fe2725301f6cf6fc3cf29bbc4e00;hp=49b14885be7eb986f88e88e73a884ac0a67ee961;hpb=ebc0d017c8a24bd16ca1f4347b39b07ba4349135;p=pharext%2Fpharext.org

diff --git a/app/Controller/Github.php b/app/Controller/Github.php
index 49b1488..364f0e5 100644
--- a/app/Controller/Github.php
+++ b/app/Controller/Github.php
@@ -8,7 +8,7 @@ use app\Session;
 use app\Web;
 
 use http\QueryString;
-use http\Url;
+use http\Header;
 
 abstract class Github implements Controller
 {
@@ -26,12 +26,25 @@ abstract class Github implements Controller
 	 * @var \app\Session
 	 */
 	protected $session;
-
-	function __construct(Web $app, API $github) {
+	
+	function __construct(Web $app, API $github, Session $session) {
 		$this->app = $app;
 		$this->github = $github;
-		$this->app->getView()->addData(["location" => "github"]);
+		$this->session = $session;
+		$this->app->getView()->addData(compact("session") + [
+			"location" => "github", 
+			"title" => "Github"
+		]);
 		$this->app->getView()->registerFunction("check", [$this, "checkRepoHook"]);
+		
+		if (($header = $this->app->getRequest()->getHeader("Cache-Control", Header::class))) {
+			$params = $header->getParams();
+			if (!empty($params["no-cache"])) {
+				$this->github->setMaxAge(0);
+			} elseif (!empty($params["max-age"])) {
+				$this->github->setMaxAge($params["max-age"]["value"]);
+			}
+		}
 	}
 
 	protected function checkToken() {
@@ -45,34 +58,19 @@ abstract class Github implements Controller
 		return false;
 	}
 
-	function setSession(Session $session) {
-		$this->session = $session;
-	}
-
+	/**
+	 * Check if the pharext webhook is set for the repo and return it
+	 * @param object $repo
+	 * @return int hook id
+	 */
 	function checkRepoHook($repo) {
 		if ($repo->hooks) {
 			foreach ($repo->hooks as $hook) {
 				if ($hook->name === "web" && $hook->config->url === $this->github->getConfig()->hook->url) {
-					return true;
+					return $hook;
 				}
 			}
 		}
-		return false;
+		return null;
 	}
-
-	function createLinkGenerator($links) {
-		return function($which) use($links) {
-			if (!isset($links[$which])) {
-				if ($which !== "next" || !isset($links["last"])) {
-					return null;
-				} else {
-					$which = "last";
-				}
-			}
-			$url = new Url($links[$which], null, 0);
-			$qry = new QueryString($url->query);
-			return $qry->getInt("page", 1);
-		};
-	}
-
 }