feat: add possibility to disable default redirect for prefix_and_default strategy

docs/content/en/api.md

@@ -30,7 +30,7 @@ All [Vue I18n properties and methods](http://kazupon.github.io/vue-i18n/api/#vue
     - locale: (type: `string`)
   - **Returns**: `string`
 
-  Returns path of the current route for specified `locale`.
+  Returns path of the current route for specified `locale`. 
 
   See also [Basic usage - nuxt-link](../basic-usage#nuxt-link).
 

docs/content/en/options-reference.md

@@ -186,6 +186,16 @@ Set this to `true` when using different domains for each locale. If enabled, no
 
 Whether [custom paths](/routing#custom-paths) are extracted from page files using babel parser.
 
+## `prefixAndDefaultRule`
+
+- type: `object`
+- default: `'default'`
+
+Modification of the standard behavior of the `prefix_and_default` strategy.
+
+By setting the value `prefixAndDefaultRule = 'prefix'` we are no longer being redirected from prefixed path.
+If there is no prefix then the other routers will also have no prefix.
+
 ## `pages`
 
 - type: `object`

docs/content/en/strategies.md

@@ -27,7 +27,9 @@ With this strategy, all routes will have a locale prefix.
 
 ### prefix_and_default
 
-This strategy combines both previous strategies behaviours, meaning that you will get URLs with prefixes for every language, but URLs for the default language will also have a non-prefixed version (though the prefixed version will be preferred when `detectBrowserLanguage` is enabled.
+This strategy combines both previous strategies behaviours, meaning that you will get URLs with prefixes for every language, but URLs for the default language will also have a non-prefixed version (though the prefixed version will be preferred when `detectBrowserLanguage` is enabled.)
+
+The behavior of the strategy can be modified, more - [prefixAndDefaultRule](../options-reference#prefixanddefaultrule)
 
 ### Configuration
 

package.json

@@ -23,6 +23,7 @@
   },
   "scripts": {
     "dev:basic": "nuxt -c ./test/fixture/basic/nuxt.config.js",
+    "dev:redirect": "nuxt -c ./test/fixture/disable-redirect/nuxt.config.js",
     "dev:basic:generate": "nuxt generate -c ./test/fixture/basic/nuxt.config.js",
     "dev:basic:start": "nuxt start -c ./test/fixture/basic/nuxt.config.js",
     "start:dist": "jiti ./test/utils/http-server-internal.js --port 8080 -v dist",

src/helpers/constants.js

@@ -37,6 +37,9 @@ export const DEFAULT_OPTIONS = {
   defaultDirection: 'ltr',
   routesNameSeparator: '___',
   defaultLocaleRouteNameSuffix: 'default',
+  prefixAndDefaultRules: {
+    routing: 'default'
+  },
   sortRoutes: true,
   strategy: STRATEGY_PREFIX_EXCEPT_DEFAULT,
   lazy: false,

src/templates/plugin.main.js

@@ -15,7 +15,7 @@ import {
   loadLanguageAsync,
   resolveBaseUrl,
   registerStore,
-  mergeAdditionalMessages
+  mergeAdditionalMessages, getHelpers
 } from './plugin.utils'
 // @ts-ignore
 import { joinURL } from '~i18n-ufo'
@@ -153,14 +153,18 @@ export default async (context) => {
     let redirectPath = ''
 
     const isStaticGenerate = process.static && process.server
+    const isDifferentLocale = getLocaleFromRoute(route) !== newLocale
+    const { isPrefixAndDefault, isDefaultLocale } = getHelpers(newLocale)
+
     // Decide whether we should redirect to a different route.
     if (
       !isStaticGenerate &&
       !app.i18n.differentDomains &&
       options.strategy !== Constants.STRATEGIES.NO_PREFIX &&
       // Skip if already on the new locale unless the strategy is "prefix_and_default" and this is the default
-      // locale, in which case we might still redirect as we prefer unprefixed route in this case.
-      (getLocaleFromRoute(route) !== newLocale || (options.strategy === Constants.STRATEGIES.PREFIX_AND_DEFAULT && newLocale === options.defaultLocale))
+      // locale, in which case we might still redirect as we prefer unprefixed route in this case, but let user a
+      // possibility to disable this behavior by switching prefixAndDefaultRules.routing option to prefix.
+      (isDifferentLocale || (isPrefixAndDefault && isDefaultLocale && options.prefixAndDefaultRules.routing === 'default'))
     ) {
       // The current route could be 404 in which case attempt to find matching route using the full path since
       // "switchLocalePath" can only find routes if the current route exists.

src/templates/plugin.routing.js

@@ -1,7 +1,8 @@
 import './middleware'
 import Vue from 'vue'
 import { Constants, nuxtOptions, options } from './options'
-import { getDomainFromLocale } from './plugin.utils'
+import { getDomainFromLocale, getHelpers } from './plugin.utils'
+import { removeLocaleFromPath } from './utils-common'
 // @ts-ignore
 import { withoutTrailingSlash, withTrailingSlash } from '~i18n-ufo'
 
@@ -44,7 +45,7 @@ function resolveRoute (route, locale) {
     return
   }
 
-  const { i18n } = this
+  const { i18n, route: currentRoute } = this
 
   locale = locale || i18n.locale
 
@@ -68,19 +69,21 @@ function resolveRoute (route, locale) {
   if (localizedRoute.path && !localizedRoute.name) {
     const resolvedRoute = this.router.resolve(localizedRoute).route
     const resolvedRouteName = this.getRouteBaseName(resolvedRoute)
+    const forceDefaultRoute = shouldForceDefaultRoute(currentRoute.fullPath)
+
     if (resolvedRouteName) {
       localizedRoute = {
-        name: getLocaleRouteName(resolvedRouteName, locale),
+        name: getLocaleRouteName(resolvedRouteName, locale, forceDefaultRoute),
         params: resolvedRoute.params,
         query: resolvedRoute.query,
         hash: resolvedRoute.hash
       }
     } else {
-      const isDefaultLocale = locale === options.defaultLocale
+      const { isDefaultLocale, isPrefixAndDefault, isPrefixExceptDefault } = getHelpers(locale)
       // if route has a path defined but no name, resolve full route using the path
       const isPrefixed =
-          // don't prefix default locale
-          !(isDefaultLocale && [Constants.STRATEGIES.PREFIX_EXCEPT_DEFAULT, Constants.STRATEGIES.PREFIX_AND_DEFAULT].includes(options.strategy)) &&
+          // don't prefix default locale, if not forced
+          !(isDefaultLocale && (isPrefixExceptDefault || (isPrefixAndDefault && forceDefaultRoute))) &&
           // no prefix for any language
           !(options.strategy === Constants.STRATEGIES.NO_PREFIX) &&
           // no prefix for different domains
@@ -95,7 +98,9 @@ function resolveRoute (route, locale) {
       localizedRoute.name = this.getRouteBaseName()
     }
 
-    localizedRoute.name = getLocaleRouteName(localizedRoute.name, locale)
+    const forceDefaultRoute = shouldForceDefaultRoute(currentRoute.fullPath)
+
+    localizedRoute.name = getLocaleRouteName(localizedRoute.name, locale, forceDefaultRoute)
 
     const { params } = localizedRoute
     if (params && params['0'] === undefined && params.pathMatch) {
@@ -123,6 +128,7 @@ function switchLocalePath (locale) {
 
   const { i18n, route, store } = this
   const { params, ...routeCopy } = route
+
   let langSwitchParams = {}
   if (options.vuex && options.vuex.syncRouteParams && store) {
     langSwitchParams = store.getters[`${options.vuex.moduleName}/localeRouteParams`](locale)
@@ -137,6 +143,16 @@ function switchLocalePath (locale) {
   })
   let path = this.localePath(baseRoute, locale)
 
+  const { prefixAndDefaultRules } = options
+  const { isPrefixAndDefault, isDefaultLocale } = getHelpers(locale)
+  const shouldSwitchToPrefix = prefixAndDefaultRules.routing === 'prefix'
+
+  if (isPrefixAndDefault && isDefaultLocale && shouldSwitchToPrefix) {
+    const cleanPath = removeLocaleFromPath(path, [locale])
+    const localizedPath = `/${locale}${cleanPath}`
+    path = nuxtOptions.trailingSlash ? withTrailingSlash(localizedPath) : withoutTrailingSlash(localizedPath)
+  }
+
   // Handle different domains
   if (i18n.differentDomains) {
     const getDomainOptions = {
@@ -167,17 +183,35 @@ function getRouteBaseName (givenRoute) {
 /**
  * @param {string | undefined} routeName
  * @param {string} locale
+ * @param {boolean} forceDefaultName
  */
-function getLocaleRouteName (routeName, locale) {
+function getLocaleRouteName (routeName, locale, forceDefaultName = true) {
+  const { isDefaultLocale, isPrefixAndDefault } = getHelpers(locale)
   let name = routeName + (options.strategy === Constants.STRATEGIES.NO_PREFIX ? '' : options.routesNameSeparator + locale)
 
-  if (locale === options.defaultLocale && options.strategy === Constants.STRATEGIES.PREFIX_AND_DEFAULT) {
+  if (isDefaultLocale && isPrefixAndDefault && forceDefaultName) {
     name += options.routesNameSeparator + options.defaultLocaleRouteNameSuffix
   }
 
   return name
 }
 
+/**
+ * @param {string} currentPath
+ * @return {boolean}
+ */
+function shouldForceDefaultRoute (currentPath) {
+  const { isPrefixAndDefault } = getHelpers()
+  const { defaultLocale, prefixAndDefaultRules } = options
+  const isPrefixedPath = new RegExp(`^/${defaultLocale}(/|$)`).test(currentPath)
+
+  if (!isPrefixAndDefault || prefixAndDefaultRules.routing === 'default') {
+    return true
+  }
+
+  return !isPrefixedPath
+}
+
 /**
  * @template {(...args: any[]) => any} T
  * @param {T} targetFunction

src/templates/plugin.utils.js

@@ -1,5 +1,5 @@
 import isHTTPS from 'is-https'
-import { localeMessages, options } from './options'
+import { Constants, localeMessages, options } from './options'
 import { formatMessage } from './utils-common'
 // @ts-ignore
 import { hasProtocol } from '~i18n-ufo'
@@ -216,6 +216,21 @@ export function mergeAdditionalMessages (i18n, additionalMessages, localeCodes,
   }
 }
 
+/**
+ * @param {string} locale
+ */
+export function getHelpers (locale = '') {
+  const isDefaultLocale = locale === options.defaultLocale
+  const isPrefixAndDefault = options.strategy === Constants.STRATEGIES.PREFIX_AND_DEFAULT
+  const isPrefixExceptDefault = options.strategy === Constants.STRATEGIES.PREFIX_EXCEPT_DEFAULT
+
+  return {
+    isDefaultLocale,
+    isPrefixAndDefault,
+    isPrefixExceptDefault
+  }
+}
+
 /**
  * @param {any} value
  * @return {boolean}

src/templates/utils-common.js

@@ -223,3 +223,14 @@ export function setLocaleCookie (locale, res, { useCookie, cookieDomain, cookieK
     res.setHeader('Set-Cookie', headers)
   }
 }
+
+/**
+ * @param  {string} pathString
+ * @param  {readonly string[]} localeCodes
+ * @return {string}
+ */
+export function removeLocaleFromPath (pathString, localeCodes) {
+  const regexp = new RegExp(`^(\\/${localeCodes.join('|\\/')})(?=\\/|$)`)
+
+  return pathString.replace(regexp, '') || '/'
+}

test/fixture/disable-redirect/components/LangSwitcher.vue

@@ -0,0 +1,15 @@
+<template>
+  <ul class="lang-switcher">
+    <li v-for="locale in $i18n.locales" :key="locale.code">
+      <nuxt-link :to="switchLocalePath(locale.code)">
+        {{ locale.code.toUpperCase() }}
+      </nuxt-link>
+    </li>
+  </ul>
+</template>
+
+<style scoped>
+.lang-switcher {
+  margin: 20px;
+}
+</style>

test/fixture/disable-redirect/components/NavBar.vue

@@ -0,0 +1,37 @@
+<template>
+  <div class="navbar">
+    <nuxt-link to="/">
+      Home Default
+    </nuxt-link>
+    <nuxt-link v-for="link in links" :key="link.label" tag="a" :to="localePath(link.path)">
+      {{ link.label }}
+    </nuxt-link>
+  </div>
+</template>
+
+<script>
+export default {
+  computed: {
+    links () {
+      return [
+        { label: 'Home', path: '/' },
+        { label: 'About', path: '/about' },
+        { label: 'Foo', path: '/foo' },
+        { label: 'FooBar', path: '/foo/bar' }
+      ]
+    }
+  }
+}
+</script>
+
+<style>
+.navbar {
+  display: flex;
+  align-items: center;
+  justify-content: flex-start;
+}
+
+.navbar a {
+  padding: 0 10px;
+}
+</style>

test/fixture/disable-redirect/layouts/default.vue

@@ -0,0 +1,21 @@
+<template>
+  <div>
+    <p><strong>Default Layout</strong></p>
+
+    <NavBar />
+
+    <LangSwitcher />
+
+    <hr>
+
+    <Nuxt />
+  </div>
+</template>
+
+<script>
+import LangSwitcher from '../components/LangSwitcher'
+import NavBar from '../components/NavBar'
+export default {
+  components: { NavBar, LangSwitcher }
+}
+</script>

test/fixture/disable-redirect/nuxt.config.js

@@ -0,0 +1,31 @@
+import { resolve } from 'path'
+import BaseConfig from '../base.config'
+
+/** @type {import('@nuxt/types').NuxtConfig} */
+const config = {
+  ...BaseConfig,
+  router: {
+    trailingSlash: true
+  },
+  i18n: {
+    prefixAndDefaultRule: 'default',
+    strategy: 'prefix_and_default',
+    locales: [
+      {
+        code: 'en',
+        iso: 'en',
+        name: 'English'
+      },
+      {
+        code: 'fr',
+        iso: 'fr-FR',
+        name: 'Français'
+      }
+    ],
+    defaultLocale: 'en'
+  },
+  buildDir: resolve(__dirname, '.nuxt'),
+  srcDir: __dirname
+}
+
+module.exports = config

test/fixture/disable-redirect/pages/about.vue

@@ -0,0 +1,5 @@
+<template>
+  <div>
+    <h1>About</h1>
+  </div>
+</template>

test/fixture/disable-redirect/pages/foo/bar.vue

@@ -0,0 +1,5 @@
+<template>
+  <div>
+    <h1>Bar</h1>
+  </div>
+</template>

test/fixture/disable-redirect/pages/foo/index.vue

@@ -0,0 +1,5 @@
+<template>
+  <div>
+    <h1>Foo</h1>
+  </div>
+</template>

test/fixture/disable-redirect/pages/index.vue

@@ -0,0 +1,5 @@
+<template>
+  <div>
+    <h1>Home</h1>
+  </div>
+</template>