Use getURI instead in place of document URIs

Author: carlbennett

src/templates/Document/Index.phtml

@@ -41,7 +41,7 @@ require("./header.inc.phtml");
         }
 ?>
               <tr>
-                <td><a href="<?php echo Common::relativeUrlToAbsolute("/document/" . urlencode($document->getId()) . "/" . Common::sanitizeForUrl($document->getTitle(), true)); ?>"><?php echo htmlspecialchars($document->getTitle(), ENT_HTML5, "UTF-8"); ?></a></td>
+                <td><a href="<?php echo $document->getURI(); ?>"><?php echo htmlspecialchars($document->getTitle(), ENT_HTML5, "UTF-8"); ?></a></td>
                 <td><?php if ($user) { ?><a href="<?php echo $user_url; ?>"><img class="avatar" src="<?php echo $avatar_url; ?>"/> <?php echo htmlspecialchars($user->getName(), ENT_HTML5, "UTF-8"); ?></a><?php } else { ?>Anonymous<?php } ?></td>
               </tr>
 <?php } ?>

src/templates/Document/View.phtml

@@ -18,9 +18,8 @@ $this->opengraph->attach(new Pair("type", "article"));
 $url = Common::relativeUrlToAbsolute("/document/" . urlencode($object_id));
 if ($object) {
 
-  $url .= "/" . Common::sanitizeForUrl($object->getTitle(), true);
-  $this->opengraph->attach(new Pair("url", "/document/" . urlencode($object_id)
-    . "/" . Common::sanitizeForUrl($object->getTitle(), true)));
+  $url = $object->getURI();
+  $this->opengraph->attach(new Pair("url", $url));
 
   $title       = htmlspecialchars($object->getTitle(), ENT_HTML5, "UTF-8");
   $description = Common::stripUpTo(trim(filter_var(

src/templates/User/View.phtml

@@ -92,7 +92,7 @@ require("./header.inc.phtml");
         <section>
           <ul>
 <?php foreach ($this->getContext()->documents as $document) { ?>
-            <li><a href="<?php echo Common::relativeUrlToAbsolute("/document/" . urlencode($document->getId()) . "/" . Common::sanitizeForUrl($document->getTitle(), true)); ?>"><?php echo htmlspecialchars($document->getTitle()); ?></a></li>
+            <li><a href="<?php echo $document->getURI(); ?>"><?php echo htmlspecialchars($document->getTitle()); ?></a></li>
 <?php } ?>
           </ul>
         </section>