Enforce name/title in URL for viewing docs/pkts

This helps with SEO so that pages don't appear as being "/packet/123"
but instead will appear as being "/packet/123/bnls-cdkey".

Author: carlbennett

controllers/Document/View.php

@@ -2,6 +2,7 @@
 
 namespace BNETDocs\Controllers\Document;
 
+use \BNETDocs\Controllers\Redirect as RedirectController;
 use \BNETDocs\Libraries\Common;
 use \BNETDocs\Libraries\Controller;
 use \BNETDocs\Libraries\Document;
@@ -25,6 +26,26 @@ public function __construct($document_id) {
   }
 
   public function run(Router &$router) {
+    $model              = new DocumentViewModel();
+    $model->document_id = $this->document_id;
+    try {
+      $model->document  = new Document($this->document_id);
+    } catch (DocumentNotFoundException $e) {
+      $model->document  = null;
+    }
+    $pathArray = $router->getRequestPathArray();
+    if ($model->document && (
+      !isset($pathArray[3]) || empty($pathArray[3]))) {
+      $redirect = new RedirectController(
+        Common::relativeUrlToAbsolute(
+          "/document/" . $model->document->getId() . "/"
+          . Common::sanitizeForUrl(
+            $model->document->getTitle(), true
+          )
+        ), 302
+      );
+      return $redirect->run($router);
+    }
     switch ($router->getRequestPathExtension()) {
       case "htm": case "html": case "":
         $view = new DocumentViewHtmlView();
@@ -35,13 +56,6 @@ public function run(Router &$router) {
       default:
         throw new UnspecifiedViewException();
     }
-    $model = new DocumentViewModel();
-    $model->document_id  = $this->document_id;
-    try {
-      $model->document   = new Document($this->document_id);
-    } catch (DocumentNotFoundException $e) {
-      $model->document   = null;
-    }
     $model->user_session = UserSession::load($router);
     ob_start();
     $view->render($model);

controllers/Packet/View.php

@@ -2,6 +2,7 @@
 
 namespace BNETDocs\Controllers\Packet;
 
+use \BNETDocs\Controllers\Redirect as RedirectController;
 use \BNETDocs\Libraries\Common;
 use \BNETDocs\Libraries\Controller;
 use \BNETDocs\Libraries\Exceptions\PacketNotFoundException;
@@ -26,6 +27,26 @@ public function __construct($packet_id) {
   }
 
   public function run(Router &$router) {
+    $model = new PacketViewModel();
+    $model->packet_id = $this->packet_id;
+    try {
+      $model->packet  = new Packet($this->packet_id);
+    } catch (PacketNotFoundException $e) {
+      $model->packet  = null;
+    }
+    $pathArray = $router->getRequestPathArray();
+    if ($model->packet && (
+      !isset($pathArray[3]) || empty($pathArray[3]))) {
+      $redirect = new RedirectController(
+        Common::relativeUrlToAbsolute(
+          "/packet/" . $model->packet->getId() . "/"
+          . Common::sanitizeForUrl(
+            $model->packet->getPacketName(), true
+          )
+        ), 302
+      );
+      return $redirect->run($router);
+    }
     switch ($router->getRequestPathExtension()) {
       case "htm": case "html": case "":
         $view = new PacketViewHtmlView();
@@ -36,13 +57,6 @@ public function run(Router &$router) {
       default:
         throw new UnspecifiedViewException();
     }
-    $model = new PacketViewModel();
-    $model->packet_id    = $this->packet_id;
-    try {
-      $model->packet     = new Packet($this->packet_id);
-    } catch (PacketNotFoundException $e) {
-      $model->packet     = null;
-    }
     if ($model->packet) {
       $model->used_by    = $this->getUsedBy($model->packet);
     } else {