ENHANCEMENT Default archive year

If no year is passed, rather than 404 it should grab the latest years posts.
silverstripe · Nov 9, 2015 · 43bb224 · 43bb224
1 parent 336a9b4
commit 43bb224
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/code/model/Blog.php b/code/model/Blog.php
@@ -727,13 +727,17 @@ public function archive() {
 	/**
 	 * Fetches the archive year from the url.
 	 *
-	 * @return null|int
+	 * @return int
 	 */
 	public function getArchiveYear() {
-		$year = $this->request->param('Year');
 
-		if(preg_match('/^[0-9]{4}$/', $year)) {
-			return (int) $year;
+		if($this->request->param('Year')){
+
+			if(preg_match('/^[0-9]{4}$/', $year = $this->request->param('Year'))) {
+				return (int) $year;
+			}
+		} else {
+			return SS_Datetime::now()->Year();
 		}
 
 		return null;

diff --git a/tests/BlogTest.php b/tests/BlogTest.php
@@ -109,8 +109,8 @@ public function testArchiveLinks() {
 		$this->assertEquals(200, $this->getStatusOf($link), 'HTTP Status should be 200');
 
 		$link = Controller::join_links($blog->Link('archive'));
-
-		$this->assertEquals(404, $this->getStatusOf($link), 'HTTP Status should be 404');
+		$this->assertEquals(200, $this->getStatusOf($link), 'HTTP Status should be 200');
+		$this->assertEquals(SS_Datetime::now()->Year(), ModelAsController::controller_for($blog)->getArchiveYear(), 'Defaults to current year');
 
 		$link = Controller::join_links($blog->Link('archive'), 'invalid-year');
 
@@ -123,6 +123,7 @@ public function testArchiveLinks() {
 		$link = Controller::join_links($blog->Link('archive'), '2013', '10', '99');
 
 		$this->assertEquals(404, $this->getStatusOf($link), 'HTTP Status should be 404');
+
 	}
 
 	/**