Function preprocess has a Cognitive Complexity of 139 (exceeds 5 allowed). Consider refactoring.

Open

    public function preprocess()
    {
        $col_select = \FormLib::get_form_value('cs','');
        $this->finishPreviewOpts();

• Read up
  Read up

File FannieUploadPage.php has 606 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/*******************************************************************************

    Copyright 2013 Whole Foods Co-op

Method preprocess has 147 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function preprocess()
    {
        $col_select = \FormLib::get_form_value('cs','');
        $this->finishPreviewOpts();

Function processUpload has a Cognitive Complexity of 33 (exceeds 5 allowed). Consider refactoring.

Open

    protected function processUpload() 
    {
        /* use a dedicated temp directory */
        $tpath = sys_get_temp_dir().'/fannie/';
        if (!is_dir($tpath)) {

• Read up
  Read up

Method processUpload has 98 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function processUpload() 
    {
        /* use a dedicated temp directory */
        $tpath = sys_get_temp_dir().'/fannie/';
        if (!is_dir($tpath)) {

Function basicPreview has a Cognitive Complexity of 25 (exceeds 5 allowed). Consider refactoring.

Open

    protected function basicPreview()
    {
        $ret = '<h3>Select columns</h3>';
        /* show any errors */
        if ($this->error_details != 'n/a' && $this->error_details != '') {

• Read up
  Read up

FannieUploadPage has 28 functions (exceeds 20 allowed). Consider refactoring.

Open

class FannieUploadPage extends \FanniePage 
{

    public $required = true;

Method basicPreview has 61 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function basicPreview()
    {
        $ret = '<h3>Select columns</h3>';
        /* show any errors */
        if ($this->error_details != 'n/a' && $this->error_details != '') {

Method javascript_content has 39 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function javascript_content()
    {
        ob_start();
        ?>
        $(document).ready(function(){

Function finishPreviewOpts has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    private function finishPreviewOpts()
    {
        foreach ($this->preview_opts as $k=>$v) {
            if (!isset($this->preview_opts[$k]['name'])) {
                $this->preview_opts[$k]['name'] = $k;

• Read up
  Read up

Function ajaxContent has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function ajaxContent()
    {
        $ret = '<div id="progressDiv">
            <span id="uploadingSpan"></span><span id="numLines"></span><br />
            <span id="progressSpan"></span><span id="numRecords"></span><br />

• Read up
  Read up

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

                return false;

Avoid too many return statements within this method.

Open

                        return true;

Avoid too many return statements within this method.

Open

        return true;

Avoid too many return statements within this method.

Open

                            $url .= array_reduce($files, function($carry, $item){ return $carry . 'f[]=' . $item . '&'; }, '');

Avoid too many return statements within this method.

Open

                            return false;

Avoid too many return statements within this method.

Open

        return true;

Avoid too many return statements within this method.

Open

            return array();

Avoid too many return statements within this method.

Open

                            $url .= array_reduce($col_select, function($carry, $item){ return $carry . 'cs[]=' . $item . '&'; }, '');

Avoid too many return statements within this method.

Open

            return false;