pacificclimate/climate-explorer-frontend

Function render has 79 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  render() {
    const mapLegend = (
      <MapLegend
        {...this.props}
        {...this.state}

File DualMapController.js has 265 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/*************************************************************************
 * DualMapController - displays two variables on the map at once
 *
 * This controller coordinates a map displaying two layers at once:
 *   * A raster layer with colour shading reflecting values

Function loadMap has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  loadMap(props, dataSpec, newVariable = false, newComparand = false) {
    // update state with all the information needed to display
    // maps for specific dataspecs.
    // A 'dataspec' is a variable + emissions + model + period + run combination.
    // Timestamps for an dataspec may be spread across up to three files

Function constructor has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  constructor(props) {
    super(props);

    this.state = {
      run: undefined,

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

Open

            <DataMap
              raster={{
                ...getDatasetIdentifiers(
                  this.props,
                  this.state,

• Read up
  Read up

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

Open

        <Panel.Heading>
          <Panel.Title>
            <Row>
              <Col lg={2}>{mapPanelLabel}</Col>
              <Col lg={10}>{mapLegend}</Col>

• Read up
  Read up

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

Open

  static propTypes = {
    model_id: PropTypes.string.isRequired,
    experiment: PropTypes.string.isRequired,
    variable_id: PropTypes.string.isRequired,
    meta: PropTypes.array.isRequired,

• Read up
  Read up

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

Open

  shouldComponentUpdate(nextProps, nextState) {
    // This guards against re-rendering before we have required data
    // TODO: Make more efficient?
    // Currently doing deep comparison on big objects (meta, comparandMeta).
    // Deep comparison matters on rasterRange, isolineRange, but not on

• Read up
  Read up

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

Open

      if (!newVariable) {
        rasterParams.palette = this.state.raster.palette;
        rasterParams.logscale = this.state.raster.logscale;
      }

• Read up
  Read up

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

Open

                  isoline={{
                    ...this.state.isoline,
                    onChangeTime: this.handleChangeComparandTime,
                    onChangePalette: this.handleChangeIsolinePalette,
                    onChangeScale: this.handleChangeIsolineScale,

• Read up
  Read up

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

Open

                  raster={{
                    ...this.state.raster,
                    onChangeTime: this.handleChangeVariableTime,
                    onChangePalette: this.handleChangeRasterPalette,
                    onChangeScale: this.handleChangeRasterScale,

• Read up
  Read up