+global\EcommerceTaskMigration::run()

Complexity

Conditions	22
Paths	> 20000

Size

Total Lines	102
Code Lines	56

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function run($request)

run uses the super-global variable $_REQUEST which is generally not recommended.

    {
        set_time_limit(1200);
        increase_memory_limit_to(-1);
        $nextGetStatement = '';
        //can we do the next step?
        //IN general yes, but if the current one is not finished yet, then we do it again.
        $canDoNext = true;
        if (isset($_REQUEST['limit'])) {
            $this->limit = intval($_REQUEST['limit']);
        }
        if (isset($_REQUEST['start'])) {
            $this->start = intval($_REQUEST['start']);
        }

        //what is the current step?
        $currentMethod = isset($_REQUEST['action']) ? $_REQUEST['action'] : '';
        if (in_array($currentMethod, $this->listOfMigrationTasks)) {

            //are we doing the same one for a different limti?
            if ($this->start) {
                $this->DBAlterationMessageNow('this task is broken down into baby steps - we are now starting at .... '.$this->start.' processing '.$this->limit, 'created');
            }
            $nextLimit = $this->$currentMethod();
            if ($canDoNext && $nextLimit) {
                $canDoNext = false;
                //NEXT OPTION 1: do again with new limit
                $nextGetStatement = '?action='.$currentMethod.'&start='.$nextLimit;
                $nextDescription = 'run next batch ...';
            }
        }

        if ($canDoNext && !$nextGetStatement) {
            //NEXT OPTION 2: start from the beginning
            $nextGetStatement = '?fullmigration=1&action='.$this->listOfMigrationTasks[0];
            $nextDescription = "Start Migration by clicking on <i>'Next'</i> (this link) until all tasks have been completed.";
        }
        //retrieve data...
        $this->retrieveInfoOnly = true;
        $html = '';
        $createListOfActions = false;
        if (!$currentMethod) {
            $createListOfActions = true;
        }
        if ($createListOfActions) {
            $html .= '
			<p>Always make a backup of your database before running any migration tasks.</p>
			<ul>';
        }
        foreach ($this->listOfMigrationTasks as $key => $task) {
            $explanation = $this->$task();
            $explanation = str_replace(array('<h1>', '</h1>', '<p>', '</p>'), array('<strong>', '</strong>: ', '<span style="color: grey;">', '</span>'), $explanation);
            if ($task == $currentMethod) {
                if ($canDoNext) {
                    $keyPlusOne = $key + 1;
                    if (isset($this->listOfMigrationTasks[$keyPlusOne]) && isset($_REQUEST['fullmigration'])) {
                        //NEXT OPTION 3: next action!
                        $action = $this->listOfMigrationTasks[$keyPlusOne];
                        $nextGetStatement = '?action='.$action;
                        $nextDescription = $this->$action();
                        $nextDescription = str_replace(array('<h1>', '</h1>', '<p>', '</p>'), array('<strong>', '</strong>: ', '<span style="color: grey;">', '</span>'), $nextDescription);
                    } else {
                        //NEXT OPTION 4: we have done all of them - no more next option...
                        $nextGetStatement = '';
                        $nextDescription = '';
                    }
                }
            }
            if ($createListOfActions) {
                $html .=  '<li><a href="/dev/ecommerce/ecommercetaskmigration/?action='.$task."\">$explanation </a></li>";
            }
        }
        if ($createListOfActions) {
            $html .= '</ul>';
        }
        if ($nextGetStatement) {
            $nextLink = '/dev/ecommerce/ecommercetaskmigration/'.$nextGetStatement;
            if (isset($_REQUEST['fullmigration'])) {
                $nextLink .= '&fullmigration=1';
            }
            echo "
				<hr style=\"margin-top: 50px;\"/>
				<h3><a href=\"$nextLink\">NEXT: $nextDescription</a></h3>";

The variable $nextDescription does not seem to be defined for all execution paths leading up to this point.

            if ($currentMethod) {
                echo "
				<div style=\"width: 400px; height: 20px; padding-top: 20px; font-size: 11px; background: url(/ecommerce/images/loading.gif) no-repeat top left transparent\">
					Next step, if any - will load automatically in ten seconds.
				</div>
				<script type=\"text/javascript\">
					var t = window.setTimeout(
						function(){
							window.location = '$nextLink';
						},
						500
					);
				</script>
				<hr style=\"margin-bottom: 500px;\"/>
			";
            }
        }
        echo $html;
    }