DFSWEMProducts

Complexity

Total Complexity

3

Size/Duplication

Total Lines	51
Duplicated Lines	56.86 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	29	29	1
A	csvcolumns()	0	8	1
A	prepareRow()	0	8	1

<?php

/**
 * @file
 *  Migration for Commerce Products used in DFS WEM.
 */

class DFSWEMProducts extends DemoFrameworkBaseProducts {

  public function __construct($arguments) {

This method seems to be duplicated in your project.

    parent::__construct($arguments);
    $this->description = t('Import DFS WEM products.');

    // Create a map object for tracking the relationships between source rows
    $this->map = new MigrateSQLMap($this->machineName,
      array(
        'sku' => array(
          'type' => 'varchar',
          'length' => 32,
          'not null' => TRUE,
        ),
      ),
      MigrateDestinationEntityAPI::getKeySchema('commerce_product', 'wem_product')
    );

    $this->destination = new MigrateDestinationEntityAPI('commerce_product', 'wem_product');

    // Define a default import path.
    $import_path = drupal_get_path('module', 'dfs_wem') . '/import/';

    // Create a MigrateSource object.
    $this->source = new MigrateSourceCSV($import_path . 'dfs_wem.products.csv', $this->csvcolumns(), array('header_rows' => 1));

    $this->addFieldMapping('field_images:source_dir')->defaultValue($import_path . 'images');

    $this->addFieldMapping('field_wem_product_interests', 'interests');

  }

  function csvcolumns() {

It is generally recommended to explicitly declare the visibility for methods.

    $columns[0] = array('title', 'Title');

$columns was never initialized. Although not strictly required by PHP, it is generally a good practice to add $columns = array(); before regardless.

    $columns[1] = array('sku', 'SKU');
    $columns[2] = array('commerce_price', 'Price');
    $columns[3] = array('images', 'Images');
    $columns[4] = array('interests', 'Interests');
    return $columns;
  }

  function prepareRow($row) {

It is generally recommended to explicitly declare the visibility for methods.

    $row->images = explode(', ', $row->images);
    $row->uid = 1;
    $row->language = LANGUAGE_NONE;
    $row->status = 1;
    $row->interests = explode(", ", $row->interests);
    return TRUE;
  }

}