From 15f724223eb8ce2a77653e456aed90be46586432 Mon Sep 17 00:00:00 2001
From: Andrew Chen <chenopis@users.noreply.github.com>
Date: Fri, 14 Apr 2017 16:07:46 -0700
Subject: [PATCH] Cleanup, simplify /js/redirect.js (#3417)

---
 js/redirects.js | 84 ++++++++++++++++++++++++++-----------------------
 1 file changed, 45 insertions(+), 39 deletions(-)

diff --git a/js/redirects.js b/js/redirects.js
index d2f2d2a8a0..7a3447c674 100644
--- a/js/redirects.js
+++ b/js/redirects.js
@@ -1,6 +1,20 @@
 $( document ).ready(function() {
+    var doRedirect = false;
+    var notHere = false;
+    var forwardingURL = window.location.href;
+
     var oldURLs = ["/README.md","/README.html","/index.md",".html",".md","/v1.1/","/v1.0/"];
-    var fwdDirs = ["examples/","cluster/","docs/devel","docs/design"];
+
+    /* var:  forwardingRules
+     * type: array of objects
+     * example rule object:
+     * {
+     *   "from":    "/path/from/old/location", //search in incoming forwardingURL for this string
+     *   "pattern": "#([0-9a-zA-Z\-\_]+)",     //[optional] regex to parse out a token of digits, letters, hyphen, or underscore
+     *   "to":      "/path/to/new/location",   //base URL to forward to
+     *   "postfix": "/#<token>"                //[optional] append this to base URL w/ <token> found by "pattern"
+     * }
+     */
     var forwardingRules = [{
         "from":"/docs/api-reference/v1/definitions",
         "pattern":"#_v1_(\\w+)",
@@ -18,46 +32,54 @@ $( document ).ready(function() {
         "pattern":"\/contribute\/([0-9a-zA-Z\-\_]+)",
         "to":"/docs/home/contribute",
         "postfix":"/<token>"
-    }];
-    var doRedirect = false;
-    var notHere = false;
-    var forwardingURL = window.location.href;
-
-    var redirects = [{
-        "from": "/resource-quota",
-        "to": "http://kubernetes.io/docs/admin/resourcequota/"
     },
     {
-        "from": "horizontal-pod-autoscaler",
-        "to": "http://kubernetes.io/docs/user-guide/horizontal-pod-autoscaling/"
+        "from":"/resource-quota",
+        "pattern":"",
+        "to":"/docs/concepts/policy/resource-quotas/",
+        "postfix":""
     },
     {
-        "from": "docs/roadmap",
-        "to": "https://github.com/kubernetes/kubernetes/milestones/"
+        "from":"/horizontal-pod-autoscaler",
+        "pattern":"",
+        "to":"/docs/tasks/run-application/horizontal-pod-autoscale/",
+        "postfix":""
     },
     {
-        "from": "api-ref/",
-        "to": "https://github.com/kubernetes/kubernetes/milestones/"
+        "from":"/docs/roadmap",
+        "pattern":"",
+        "to":"https://github.com/kubernetes/kubernetes/milestones/",
+        "postfix":""
     },
     {
-        "from": "kubernetes/third_party/swagger-ui/",
-        "to": "/docs/reference"
+        "from":"/api-ref/",
+        "pattern":"",
+        "to":"https://github.com/kubernetes/kubernetes/milestones/",
+        "postfix":""
     },
     {
-        "from": "docs/user-guide/overview",
-        "to": "http://kubernetes.io/docs/whatisk8s/"
+        "from":"/kubernetes/third_party/swagger-ui/",
+        "pattern":"",
+        "to":"/docs/reference",
+        "postfix":""
     },
     {
-        "from": "docs/admin/multiple-schedulers",
-        "to": "http://kubernetes.io/docs/tutorials/clusters/multiple-schedulers/"
+        "from":"/docs/user-guide/overview",
+        "pattern":"",
+        "to":"/docs/concepts/overview/what-is-kubernetes/",
+        "postfix":""
+    },
+    {
+        "from": "/docs/admin/multiple-schedulers",
+        "to": "/docs/tutorials/clusters/multiple-schedulers/"
     }];
 
     forwardingRules.forEach(function(rule) {
         if (forwardingURL.indexOf(rule.from) > -1) {
+            var newURL = rule.to;
             var re = new RegExp(rule.pattern, 'g');
             var matchary = re.exec(forwardingURL);
-            var newURL = rule.to;
-            if (matchary !== null) {
+            if (matchary !== null && rule.postfix) {
                 newURL += rule.postfix.replace("<token>", matchary[1]);
             }
             notHere = true;
@@ -66,22 +88,6 @@ $( document ).ready(function() {
 
     });
 
-    for (var i = 0; i < redirects.length; i++) {
-        if (forwardingURL.indexOf(redirects[i].from) > -1){
-            notHere = true;
-            window.location.replace(redirects[i].to);
-        }
-    }
-
-    for (var i = 0; i < fwdDirs.length; i++) {
-        if (forwardingURL.indexOf(fwdDirs[i]) > -1){
-            var urlPieces = forwardingURL.split(fwdDirs[i]);
-            var newURL = "https://github.com/kubernetes/kubernetes/tree/{{page.githubbranch}}/" + fwdDirs[i] + urlPieces[1];
-            notHere = true;
-            window.location.replace(newURL);
-        }
-    }
-
     if (!notHere) {
         for (var i = 0; i < oldURLs.length; i++) {
             if (forwardingURL.indexOf(oldURLs[i]) > -1 &&