Fixed Copying/Moving Posts is always logged as to an existing thread

[Xon]

Affected version

2.0.2

Both XF:post\Copier and XF:post\Mover call setExistingTarget like so:

$copier->setExistingTarget($options['thread_type']);

However: $options['thread_type'] is the string "new" or "existing", to casting to a boolean is always true.

public function setExistingTarget($existing)
{
   $this->existingTarget = (bool)$existing;
}

 

[Chris D]

Fair point, thanks.