From f9f0b2ce860df2bdca8d0f00367b8db541deca4c Mon Sep 17 00:00:00 2001
From: Matt Pollard <mattpollard@users.noreply.github.com>
Date: Wed, 23 Jun 2021 09:11:51 +0200
Subject: [PATCH] Revert "Fix homepage versions dropdown" (#20087)

---
 components/SidebarNav.tsx                    |  8 ++++----
 components/article/ArticleVersionPicker.tsx  |  4 ++++
 components/context/MainContext.tsx           |  2 +-
 components/landing/HomepageVersionPicker.tsx | 13 +++++++++++--
 includes/head.html                           |  2 +-
 includes/header-version-switcher.html        |  6 ++++++
 includes/landing.html                        |  2 +-
 lib/path-utils.js                            |  2 +-
 lib/permalink.js                             | 15 +++++++++++++++
 middleware/context.js                        |  7 +++++++
 pages/index.tsx                              |  4 ++--
 tests/unit/page.js                           |  2 +-
 tests/unit/permalink.js                      |  3 ++-
 tests/unit/versions.js                       |  4 ++--
 14 files changed, 58 insertions(+), 16 deletions(-)

diff --git a/components/SidebarNav.tsx b/components/SidebarNav.tsx
index 86e917ddaf..1c95af0509 100644
--- a/components/SidebarNav.tsx
+++ b/components/SidebarNav.tsx
@@ -10,7 +10,7 @@ import { useVersion } from './hooks/useVersion'
 
 export const SidebarNav = () => {
   const router = useRouter()
-  const { error, relativePath } = useMainContext()
+  const { error, relativePath, isHomepageVersion } = useMainContext()
   const { t } = useTranslation('header')
 
   return (
@@ -39,7 +39,7 @@ export const SidebarNav = () => {
       <nav>
         {error === '404' || relativePath === 'index.md' ? (
           <ul className="sidebar-products mt-4">
-            {<AllProductsLink />}
+            {!isHomepageVersion && <AllProductsLink />}
             <SidebarHomepage />
           </ul>
         ) : (
@@ -66,12 +66,12 @@ export const SidebarNav = () => {
 const SidebarHomepage = () => {
   const router = useRouter()
   const { currentVersion } = useVersion()
-  const { activeProducts } = useMainContext()
+  const { activeProducts, isHomepageVersion } = useMainContext()
 
   return (
     <>
       {activeProducts.map((product) => {
-        if (!product.versions?.includes(currentVersion) && !product.external) {
+        if (!product.versions?.includes(currentVersion) && !isHomepageVersion) {
           return null
         }
 
diff --git a/components/article/ArticleVersionPicker.tsx b/components/article/ArticleVersionPicker.tsx
index 9d10332f2c..989fd4d745 100644
--- a/components/article/ArticleVersionPicker.tsx
+++ b/components/article/ArticleVersionPicker.tsx
@@ -31,6 +31,10 @@ export const ArticleVersionPicker = () => {
       </summary>
       <Dropdown.Menu direction="sw">
         {(page.permalinks || []).map((permalink) => {
+          if (permalink.pageVersion === 'homepage') {
+            return null
+          }
+
           return (
             <Dropdown.Item key={permalink.href}>
               <Link href={permalink.href}>{permalink.pageVersionTitle}</Link>
diff --git a/components/context/MainContext.tsx b/components/context/MainContext.tsx
index 7fef5de9c6..0a624b0ccf 100644
--- a/components/context/MainContext.tsx
+++ b/components/context/MainContext.tsx
@@ -115,7 +115,7 @@ export const getMainContextFromRequest = (req: any): MainContextT => {
     activeProducts: req.context.activeProducts,
     currentProduct: req.context.productMap[req.context.currentProduct] || null,
     currentLayoutName: req.context.currentLayoutName,
-    isHomepageVersion: req.context.page?.documentType === 'homepage',
+    isHomepageVersion: req.context.currentVersion === 'homepage',
     error: req.context.error ? req.context.error.toString() : '',
     data: {
       ui: req.context.site.data.ui,
diff --git a/components/landing/HomepageVersionPicker.tsx b/components/landing/HomepageVersionPicker.tsx
index 8171a7001f..a2b68043af 100644
--- a/components/landing/HomepageVersionPicker.tsx
+++ b/components/landing/HomepageVersionPicker.tsx
@@ -6,6 +6,7 @@ import { ChevronDownIcon } from '@primer/octicons-react'
 import { Link } from 'components/Link'
 import { useMainContext } from 'components/context/MainContext'
 import { useVersion } from 'components/hooks/useVersion'
+import { useTranslation } from 'components/hooks/useTranslation'
 
 type Props = {
   variant?: 'inline'
@@ -14,13 +15,14 @@ export const HomepageVersionPicker = ({ variant }: Props) => {
   const router = useRouter()
   const { currentVersion } = useVersion()
   const { getDetailsProps } = useDetails({})
-  const { allVersions, page, enterpriseServerVersions } = useMainContext()
+  const { allVersions, page, enterpriseServerVersions, isHomepageVersion } = useMainContext()
+  const { t } = useTranslation('homepage')
 
   if (page.permalinks && page.permalinks.length <= 1) {
     return null
   }
 
-  const label = allVersions[currentVersion].versionTitle
+  const label = isHomepageVersion ? t('version_picker') : allVersions[currentVersion].versionTitle
 
   if (variant === 'inline') {
     return (
@@ -33,6 +35,9 @@ export const HomepageVersionPicker = ({ variant }: Props) => {
         </summary>
         <div>
           {(page.permalinks || []).map((permalink) => {
+            if (permalink.pageVersion === 'homepage') {
+              return null
+            }
             return (
               <Link
                 key={permalink.href}
@@ -73,6 +78,10 @@ export const HomepageVersionPicker = ({ variant }: Props) => {
       </summary>
       <Dropdown.Menu direction="sw">
         {(page.permalinks || []).map((permalink) => {
+          if (permalink.pageVersion === 'homepage') {
+            return null
+          }
+
           return (
             <Dropdown.Item key={permalink.href}>
               <Link href={permalink.href}>{permalink.pageVersionTitle}</Link>
diff --git a/includes/head.html b/includes/head.html
index 99425bbd70..0cd6b63fad 100644
--- a/includes/head.html
+++ b/includes/head.html
@@ -1,7 +1,7 @@
 <head>
   {% comment %} For human readers {% endcomment %}
   <meta charset="utf-8" />
-  <title>{% if error == '404' %}{% data ui.errors.oops %}{% elsif page.documentType == 'homepage' and currentVersion == 'free-pro-team@latest' %}GitHub Documentation{% elsif page.fullTitle %}{{ page.fullTitle }}{% else %}GitHub Documentation{% endif %}</title>
+  <title>{% if error == '404' %}{% data ui.errors.oops %}{% elsif currentVersion == 'homepage' %}GitHub Documentation{% elsif page.fullTitle %}{{ page.fullTitle }}{% else %}GitHub Documentation{% endif %}</title>
   <meta name="viewport" content="width=device-width, initial-scale=1">
   <link rel="alternate icon" type="image/png" href="/assets/images/site/favicon.png">
   <link rel="icon" type="image/svg+xml" href="/assets/images/site/favicon.svg">
diff --git a/includes/header-version-switcher.html b/includes/header-version-switcher.html
index 4462c7d344..b0b6b847a7 100644
--- a/includes/header-version-switcher.html
+++ b/includes/header-version-switcher.html
@@ -6,17 +6,23 @@
     <details class="dropdown-withArrow position-relative details details-reset mr-md-3 close-when-clicked-outside">
       <summary class="py-2 color-text-primary" role="button" aria-label="Toggle versions list">
         <div class="d-flex flex-items-center flex-justify-between">
+          {% if currentVersion == 'homepage' %}
+          {% data ui.homepage.version_picker %}
+          {% else %}
           {{ allVersions[currentVersion].versionTitle }}
+          {% endif %}
           <svg class="arrow ml-md-1" width="14px" height="8px" viewBox="0 0 14 8" xml:space="preserve" fill="none" stroke="currentColor"><path d="M1,1l6.2,6L13,1"></path></svg>
         </div>
       </summary>
       <div id="versions-selector" class="position-md-absolute nav-desktop-langDropdown p-md-4 right-md-n4 top-md-6" style="z-index: 6;">
       {% for permalink in page.permalinks %}
+      {% unless permalink.pageVersion == 'homepage' %}
       <a
       href="{{ permalink.href }}"
       class="d-block py-2 no-underline {% if currentPath == permalink.href %}active{% else %}Link--primary{% endif %}"
       style="white-space: nowrap"
       >{{ allVersions[permalink.pageVersion].versionTitle }}</a>
+      {% endunless %}
       {% endfor %}
       {% include all-enterprise-releases-link %}
       </div>
diff --git a/includes/landing.html b/includes/landing.html
index 44c406a178..fa2cac4fff 100644
--- a/includes/landing.html
+++ b/includes/landing.html
@@ -20,7 +20,7 @@
       <h2 class="text-mono f5 text-normal color-text-secondary text-md-center mb-4">{% data ui.homepage.explore_by_product %}</h2>
       <div class="d-flex flex-wrap gutter gutter-xl-spacious">
       {% for product in activeProducts %}
-      {% if product.versions contains currentVersion %}
+      {% if product.versions contains currentVersion or currentVersion == 'homepage' %}
       <div class="d-flex flex-column col-12 col-sm-6 col-lg-3 pb-4">
         <a class="btn-mktg flex-auto d-flex flex-items-center btn-outline-mktg btn-large-mktg ws-normal " href="{% unless product.external %}/{{ currentLanguage }}{% endunless %}{% if product.versions contains currentVersion %}/{{currentVersion}}/{{product.id}}{% else %}{{product.href}}{% endif %}" {% if product.external %}target="_blank"{% endif %}>{{ product.name }}
       {% if product.external %}
diff --git a/lib/path-utils.js b/lib/path-utils.js
index e8fb6b6f3e..16da0545cf 100644
--- a/lib/path-utils.js
+++ b/lib/path-utils.js
@@ -36,7 +36,7 @@ function getVersionStringFromPath (href) {
 
   // Return immediately if this is a link to the homepage
   if (href === '/') {
-    return nonEnterpriseDefaultVersion
+    return 'homepage'
   }
 
   // Get the first segment
diff --git a/lib/permalink.js b/lib/permalink.js
index e15afe8989..8eb391fb04 100644
--- a/lib/permalink.js
+++ b/lib/permalink.js
@@ -2,6 +2,7 @@ const assert = require('assert')
 const path = require('path')
 const patterns = require('./patterns')
 const allVersions = require('./all-versions')
+const nonEnterpriseDefaultVersion = require('./non-enterprise-default-version')
 const removeFPTFromPath = require('./remove-fpt-from-path')
 
 class Permalink {
@@ -26,10 +27,24 @@ class Permalink {
     assert(languageCode, 'languageCode is required')
 
     const permalinks = applicableVersions
+      // skip the Dotcom homepage here because a special homepage permalink is added below
+      .filter(pageVersion => !(pageVersion === nonEnterpriseDefaultVersion && relativePath === 'index.md'))
       .map(pageVersion => {
         return new Permalink(languageCode, pageVersion, relativePath, title)
       })
 
+    // special permalink for homepage
+    if (relativePath === 'index.md') {
+      const homepagePermalink = {
+        ...permalinks[0],
+        href: '/' + languageCode,
+        pageVersion: 'homepage',
+        pageVersionTitle: permalinks[0].title,
+        homepage: true
+      }
+      permalinks.push(homepagePermalink)
+    }
+
     return permalinks
   }
 
diff --git a/middleware/context.js b/middleware/context.js
index 35fc667425..42cfcfe168 100644
--- a/middleware/context.js
+++ b/middleware/context.js
@@ -52,6 +52,13 @@ module.exports = async function contextualize (req, res, next) {
   req.context.siteTree = siteTree
   req.context.pages = pageMap
 
+  // TODO we should create new data directories for these example files instead of using variable files
+  if (productMap[req.context.currentProduct]) {
+    req.context.productCodeExamples = req.context.site.data.variables[`${productMap[req.context.currentProduct].id}_code_examples`]
+    req.context.productCommunityExamples = req.context.site.data.variables[`${productMap[req.context.currentProduct].id}_community_examples`]
+    req.context.productUserExamples = req.context.site.data.variables[`${productMap[req.context.currentProduct].id}_user_examples`]
+  }
+
   // JS + CSS asset paths
   req.context.builtAssets = builtAssets
 
diff --git a/pages/index.tsx b/pages/index.tsx
index f129a0c682..2e6d466d1f 100644
--- a/pages/index.tsx
+++ b/pages/index.tsx
@@ -44,7 +44,7 @@ type LandingPageProps = {
 function LandingPage(props: LandingPageProps) {
   const router = useRouter()
   const { gettingStartedLinks, popularLinks } = props
-  const { activeProducts } = useMainContext()
+  const { activeProducts, isHomepageVersion } = useMainContext()
   const { currentVersion } = useVersion()
   const { t } = useTranslation(['homepage', 'search', 'toc'])
   return (
@@ -80,7 +80,7 @@ function LandingPage(props: LandingPageProps) {
           </h2>
           <div className="d-flex flex-wrap gutter gutter-xl-spacious">
             {activeProducts.map((product) => {
-              if (!product.versions?.includes(currentVersion) && !product.external) {
+              if (!product.versions?.includes(currentVersion) && !isHomepageVersion) {
                 return null
               }
 
diff --git a/tests/unit/page.js b/tests/unit/page.js
index 0e30b51372..bdb23cdee8 100644
--- a/tests/unit/page.js
+++ b/tests/unit/page.js
@@ -268,7 +268,7 @@ describe('Page class', () => {
         basePath: path.join(__dirname, '../../content'),
         languageCode: 'en'
       })
-      expect(page.permalinks.find(permalink => permalink.pageVersion === nonEnterpriseDefaultVersion).href).toBe('/en')
+      expect(page.permalinks.find(permalink => permalink.pageVersion === 'homepage').href).toBe('/en')
       expect(page.permalinks.find(permalink => permalink.pageVersion === `enterprise-server@${enterpriseServerReleases.oldestSupported}`).href).toBe(`/en/enterprise-server@${enterpriseServerReleases.oldestSupported}`)
     })
 
diff --git a/tests/unit/permalink.js b/tests/unit/permalink.js
index a3dd0653b8..9699d75d2a 100644
--- a/tests/unit/permalink.js
+++ b/tests/unit/permalink.js
@@ -7,6 +7,7 @@ const getApplicableVersions = require('../../lib/get-applicable-versions')
 // Permalink.derive requires: languageCode, relativePath, title, versions (<- FM prop)
 
 describe('Permalink class', () => {
+  // We can only use Permalink.derive to get the special 'homepage' permalink
   test('derives info for unversioned homepage', () => {
     const versions = {
       'free-pro-team': '*',
@@ -14,7 +15,7 @@ describe('Permalink class', () => {
     }
     const permalinks = Permalink.derive('en', 'index.md', 'Hello World', getApplicableVersions(versions))
     expect(permalinks.length).toBeGreaterThan(1)
-    const homepagePermalink = permalinks.find(permalink => permalink.pageVersion === nonEnterpriseDefaultVersion)
+    const homepagePermalink = permalinks.find(permalink => permalink.pageVersion === 'homepage')
     expect(homepagePermalink.href).toBe('/en')
   })
 
diff --git a/tests/unit/versions.js b/tests/unit/versions.js
index 7ff049e2b0..45b136115f 100644
--- a/tests/unit/versions.js
+++ b/tests/unit/versions.js
@@ -30,10 +30,10 @@ describe('versions middleware', () => {
 
   test('adds res.context.currentVersion string', async () => {
     let currentVersion = await getJSON('/en?json=currentVersion')
-    expect(currentVersion).toBe(nonEnterpriseDefaultVersion)
+    expect(currentVersion).toBe('homepage')
 
     currentVersion = await getJSON(`/en/${nonEnterpriseDefaultVersion}?json=currentVersion`)
-    expect(currentVersion).toBe(nonEnterpriseDefaultVersion)
+    expect(currentVersion).toBe('homepage')
 
     currentVersion = await getJSON(`/en/enterprise-server@${latest}?json=currentVersion`)
     expect(currentVersion).toBe(`enterprise-server@${latest}`)