bbc/simorgh

Function writeCsvFile has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const writeCsvFile = data => {
  let csvContents =
    'Dependency,Type,Most Recent Version,Most Recent Version Date,Our Version,Our Version Date,Our Version Freshness in Days';

  data

Function Image has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const Image = props => {
  const { children = null, src, useLargeImages = false, ...rest } = props;
  const isProgrammeImage = src.startsWith(
    'https://ichef.bbci.co.uk/images/ic/',
  );

Function animationStory has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const animationStory = (dir, script, service) => {
  const isOpen = boolean('Open', false);
  return (
    <Navigation script={script} service={service} dir={dir}>
      <CanonicalDropdown isOpen={isOpen}>

Function inputs has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const inputs = () => {
  // capitalization is only for presentation purpose on the knob
  const options = Object.keys(svgs)
    .filter(key => key !== 'BBC_BLOCKS')
    .map(key => key.charAt(0).toUpperCase() + key.slice(1));

Function inputs has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const inputs = (service = 'news') => {
  const options = Object.keys(svgs).filter(key => key !== 'BBC_BLOCKS');
  const svgChoice = select('Service SVG', options, service).toLowerCase();
  const productInput = text('Product', 'BBC News');
  const serviceLocalisedNameInput = text('Localised service name', 'Yoruba');

Function getCampaignID has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const getCampaignID = (pageType: CampaignPageTypes) => {
  const campaignID = {
    [ARTICLE_PAGE]: 'article',
    [MEDIA_ARTICLE_PAGE]: 'article-sfv',
    [FRONT_PAGE]: 'index-home',

Function extractHeaders has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const extractHeaders = (headers: IncomingHttpHeaders) => {
  let isUK = null;
  let showCookieBannerBasedOnCountry = true;
  if (headers['x-bbc-edge-isuk']) {
    isUK = headers['x-bbc-edge-isuk'] === 'yes';

Consider simplifying this complex logical expression.

Open

    if (Array.isArray(group.items)) {
      newGroup.items = group.items.filter(item => {
        const itemType = item.assetTypeCode || item.cpsType;
        const isValidItemType = whitelist.includes(itemType);
        const isValidContentType =

Consider simplifying this complex logical expression.

Open

            if (!F && !(F = "undefined" != typeof crypto && crypto.getRandomValues && crypto.getRandomValues.bind(crypto) || "undefined" != typeof msCrypto && "function" == typeof msCrypto.getRandomValues && msCrypto.getRandomValues.bind(msCrypto)))
                throw new Error("crypto.getRandomValues() not supported. See https://github.com/uuidjs/uuid#getrandomvalues-not-supported");

Identical blocks of code found in 4 locations. Consider refactoring.

Open

    const { container } = render(
      <ContextWrap pageType={ARTICLE_PAGE} service="news" isAmp={false}>
        <OptimizelyArticleCompleteTracking />
      </ContextWrap>,
    );

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withContexts', () => Component => {
  return props => (
    <div id="ContextsContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withContexts', () => Component => {
  return props => (
    <div id="ContextsContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withContexts', () => Component => {
  return props => (
    <div id="ContextsContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withError', () => Component => {
  return props => (
    <div id="ErrorContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withError', () => Component => {
  return props => (
    <div id="ErrorContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Identical blocks of code found in 4 locations. Consider refactoring.

Open

    const { container } = render(
      <ContextWrap pageType={ARTICLE_PAGE} service="news" isAmp={false}>
        <OptimizelyArticleCompleteTracking />
      </ContextWrap>,
    );

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withPageWrapper', () => Component => {
  return props => (
    <div id="PageWrapperContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withPageWrapper', () => Component => {
  return props => (
    <div id="PageWrapperContainer">
      <Component {...props} />
    </div>

• Read up
  Read up

Identical blocks of code found in 4 locations. Consider refactoring.

Open

    const { container } = render(
      <ContextWrap pageType={ARTICLE_PAGE} service="news" isAmp={false}>
        <OptimizelyArticleCompleteTracking />
      </ContextWrap>,
    );

• Read up
  Read up

Similar blocks of code found in 10 locations. Consider refactoring.

Open

jest.mock('#containers/PageHandlers/withContexts', () => Component => {
  return props => (
    <div id="ContextsContainer">
      <Component {...props} />
    </div>

• Read up
  Read up