extractFromUrl API is awkward to use about qwik-speak HOT 4 CLOSED

@intellix I don't understand what the problem is:

• you don't like the extractFromUrl function? don't use it, it's just a helper method, you can easily create the method you prefer, and if you think it might be useful to others, contribute to this library
• or is your redirect not working because it doesn't contain the localized URL? In this case, as in any case of navigation, you must localize the URL, for example:

export const onRequest: RequestHandler = ({ redirect, url }) => {
  let lang: string | undefined = undefined;
  const prefix = extractFromUrl(url);

  if (prefix && validateLocale(prefix)) {
    lang = config.supportedLocales.find(value => value.lang === prefix)?.lang;
  }

  const getPath = translatePath();
  throw redirect(302, getPath('/somewhere', lang || config.defaultLocale.lang));
};

from qwik-speak.

I think because it's implied that you're extracting the locale from the URL, but in fact it just extracts the first segment which always requires the same additional work afterwards (validateLocale). Can't think of any reason not to always call validateLocale first.

Maybe a more useful export:

export function extractLocaleFromUrl(url: string) {
  const localeMaybe = extractFromUrl(url);
  return validateLocale(localeMaybe) ? localeMaybe : null;
}

Then it would look like:

Get locale using URL:

const locale = extractLocaleFromUrl(url) ?? config.defaultLocale.lang;

Get prefix for redirecting:

const locale = extractLocaleFromUrl(url);
const prefix = locale ? `/${locale}` : '';
throw redirect(302, `${prefix}/somewhere);

from qwik-speak.

I initially created validateLocale because there were Qwik bugs that assigned the lang parameter the wrong value. It is used in the solution without url rewriting: https://github.com/robisim74/qwik-speak/blob/main/docs/tutorial-routing.md#routing

So a method was simply added to help extract the locale from the URL in the case of url rewriting.

If you want to make your own method that uses the two methods (which are used in two different solutions), just do it.

Then if you are using url rewriting, throw redirect(302, `${prefix}/somewhere); will not work, because the URL is not rewritten (the path is not translated), so you need to use:

throw redirect(302, getPath('/somewhere', lang || config.defaultLocale.lang));

from qwik-speak.

The two functions will remain separate, because each has a different responsibility.

See also https://github.com/robisim74/qwik-speak/blob/main/docs/tutorial-routing-rewrite.md#routing

from qwik-speak.