Ticket #1437 (closed defect: fixed)

Opened 10 months ago

Last modified 10 months ago

Call to $this->dispatcher->notify() instead of $this->notify() in LoadData

Reported by: ggoodman Owned by: jwage
Priority: minor Milestone: 1.0.1
Component: Tasks Version: 1.0.0
Severity: Keywords:
Cc: Has Test: no
Status: Pending Core Response Has Patch: no

Description

Using version 1.0

In Doctrine/Task/LoadData.php on line 45, a call is made directly to the parent class's $dispatcher member that is not guaranteed to exist. This line should instead call $this->notify('Data was successfully loaded'); instead.

To reproduce:

$task = Doctrine_Task_LoadData;
$task->setArguments(array('data_fixture_path' => '/data/fixture/path'));

if ($task->validate())
    $task->execute();

Change History

Changed 10 months ago by jwage

  • milestone set to 1.0.1

Changed 10 months ago by jwage

  • status changed from new to closed
  • resolution set to fixed

(In [4899]) fixes #1437

Note: See TracTickets for help on using tickets.