Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style: explicit React imports #1525

Merged
merged 1 commit into from Jun 26, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
26 changes: 13 additions & 13 deletions src/Trans.js
@@ -1,4 +1,4 @@
import React, { useContext } from 'react';
import { useContext, isValidElement, cloneElement, createElement } from 'react';
import HTML from 'html-parse-stringify';
import { unescape } from 'html-escaper';
import { getI18n, I18nContext, getDefaults } from './context';
Expand All @@ -18,7 +18,7 @@ function getChildren(node) {

function hasValidReactChildren(children) {
if (Object.prototype.toString.call(children) !== '[object Array]') return false;
return children.every((child) => React.isValidElement(child));
return children.every((child) => isValidElement(child));
}

function getAsArray(data) {
Expand Down Expand Up @@ -49,7 +49,7 @@ export function nodesToString(children, i18nOptions) {
// actual e.g. lorem
// expected e.g. lorem
stringNode += `${child}`;
} else if (React.isValidElement(child)) {
} else if (isValidElement(child)) {
const childPropsCount = Object.keys(child.props).length;
const shouldKeepChild = keepArray.indexOf(child.type) > -1;
const childChildren = child.props.children;
Expand Down Expand Up @@ -124,7 +124,7 @@ function renderNodes(children, targetString, i18n, i18nOptions, combinedTOpts, s
childrenArray.forEach((child) => {
if (typeof child === 'string') return;
if (hasChildren(child)) getData(getChildren(child));
else if (typeof child === 'object' && !React.isValidElement(child))
else if (typeof child === 'object' && !isValidElement(child))
Object.assign(data, child);
});
}
Expand All @@ -145,7 +145,7 @@ function renderNodes(children, targetString, i18n, i18nOptions, combinedTOpts, s

function pushTranslatedJSX(child, inner, mem, i, isVoid) {
if (child.dummy) child.children = inner; // needed on preact!
mem.push(React.cloneElement(child, { ...child.props, key: i }, isVoid ? undefined : inner));
mem.push(cloneElement(child, { ...child.props, key: i }, isVoid ? undefined : inner));
}

// reactNode (the jsx root element or child)
Expand All @@ -171,7 +171,7 @@ function renderNodes(children, targetString, i18n, i18nOptions, combinedTOpts, s
const child =
Object.keys(node.attrs).length !== 0 ? mergeProps({ props: node.attrs }, tmp) : tmp;

const isElement = React.isValidElement(child);
const isElement = isValidElement(child);

const isValidTranslationWithChildren =
isElement && hasChildren(node, true) && !node.voidElement;
Expand Down Expand Up @@ -203,22 +203,22 @@ function renderNodes(children, targetString, i18n, i18nOptions, combinedTOpts, s
node.children,
rootReactNode,
);
mem.push(React.cloneElement(child, { ...child.props, key: i }, inner));
mem.push(cloneElement(child, { ...child.props, key: i }, inner));
} else if (Number.isNaN(parseFloat(node.name))) {
if (isKnownComponent) {
const inner = renderInner(child, node, rootReactNode);
pushTranslatedJSX(child, inner, mem, i, node.voidElement);
} else if (i18nOptions.transSupportBasicHtmlNodes && keepArray.indexOf(node.name) > -1) {
if (node.voidElement) {
mem.push(React.createElement(node.name, { key: `${node.name}-${i}` }));
mem.push(createElement(node.name, { key: `${node.name}-${i}` }));
} else {
const inner = mapAST(
reactNodes /* wrong but we need something */,
node.children,
rootReactNode,
);

mem.push(React.createElement(node.name, { key: `${node.name}-${i}` }, inner));
mem.push(createElement(node.name, { key: `${node.name}-${i}` }, inner));
}
} else if (node.voidElement) {
mem.push(`<${node.name} />`);
Expand All @@ -242,17 +242,17 @@ function renderNodes(children, targetString, i18n, i18nOptions, combinedTOpts, s
} else if (node.children.length === 1 && translationContent) {
// If component does not have children, but translation - has
// with this in component could be components={[<span class='make-beautiful'/>]} and in translation - 'some text <0>some highlighted message</0>'
mem.push(React.cloneElement(child, { ...child.props, key: i }, translationContent));
mem.push(cloneElement(child, { ...child.props, key: i }, translationContent));
} else {
mem.push(React.cloneElement(child, { ...child.props, key: i }));
mem.push(cloneElement(child, { ...child.props, key: i }));
}
} else if (node.type === 'text') {
const wrapTextNodes = i18nOptions.transWrapTextNodes;
const content = shouldUnescape
? unescape(i18n.services.interpolator.interpolate(node.content, opts, i18n.language))
: i18n.services.interpolator.interpolate(node.content, opts, i18n.language);
if (wrapTextNodes) {
mem.push(React.createElement(wrapTextNodes, { key: `${node.name}-${i}` }, content));
mem.push(createElement(wrapTextNodes, { key: `${node.name}-${i}` }, content));
} else {
mem.push(content);
}
Expand Down Expand Up @@ -339,5 +339,5 @@ export function Trans({
// and override `defaultTransParent` if is present
const useAsParent = parent !== undefined ? parent : reactI18nextOptions.defaultTransParent;

return useAsParent ? React.createElement(useAsParent, additionalProps, content) : content;
return useAsParent ? createElement(useAsParent, additionalProps, content) : content;
}
4 changes: 2 additions & 2 deletions src/context.js
@@ -1,4 +1,4 @@
import React from 'react';
import { createContext } from 'react';

let defaultOptions = {
bindI18n: 'languageChanged',
Expand All @@ -14,7 +14,7 @@ let defaultOptions = {

let i18nInstance;

export const I18nContext = React.createContext();
export const I18nContext = createContext();

export function setDefaults(options = {}) {
defaultOptions = { ...defaultOptions, ...options };
Expand Down
4 changes: 2 additions & 2 deletions src/withSSR.js
@@ -1,4 +1,4 @@
import React from 'react';
import { createElement } from 'react';
import { useSSR } from './useSSR';
import { composeInitialProps } from './context';
import { getDisplayName } from './utils';
Expand All @@ -8,7 +8,7 @@ export function withSSR() {
function I18nextWithSSR({ initialI18nStore, initialLanguage, ...rest }) {
useSSR(initialI18nStore, initialLanguage);

return React.createElement(WrappedComponent, {
return createElement(WrappedComponent, {
...rest,
});
}
Expand Down
8 changes: 4 additions & 4 deletions src/withTranslation.js
@@ -1,4 +1,4 @@
import React from 'react';
import { createElement, forwardRef as forwardRefReact } from 'react';
import { useTranslation } from './useTranslation';
import { getDisplayName } from './utils';

Expand All @@ -18,7 +18,7 @@ export function withTranslation(ns, options = {}) {
} else if (!options.withRef && forwardedRef) {
passDownProps.forwardedRef = forwardedRef;
}
return React.createElement(WrappedComponent, passDownProps);
return createElement(WrappedComponent, passDownProps);
}

I18nextWithTranslation.displayName = `withI18nextTranslation(${getDisplayName(
Expand All @@ -28,8 +28,8 @@ export function withTranslation(ns, options = {}) {
I18nextWithTranslation.WrappedComponent = WrappedComponent;

const forwardRef = (props, ref) =>
React.createElement(I18nextWithTranslation, Object.assign({}, props, { forwardedRef: ref }));
createElement(I18nextWithTranslation, Object.assign({}, props, { forwardedRef: ref }));

return options.withRef ? React.forwardRef(forwardRef) : I18nextWithTranslation;
return options.withRef ? forwardRefReact(forwardRef) : I18nextWithTranslation;
};
}