Do not tap activities twice

[jrmajor]

Fixes a bug introduced in #1031.

[Gummibeer]

That's not a fix - it removes a feature.

[jrmajor]

@Gummibeer What feature is being removed? Is it a feature that the tapActivity() method gets called twice?

[jrmajor]

// tapActivity() is called from LogsActivity (these are the lines I've removed)
if (method_exists($model, 'tapActivity')) {
    $logger->tap([$model, 'tapActivity'], $eventName);
}

// #1031 added a tapActivty() call in this method, so without
// the previous lines being removed, it will be called twice
$logger->log($description);

[krembo]

The solution is good as tapActivity is called in log() method anyway.

[cgnathancoyne]

Hello could you expend on the feature being removed? This fix seems fine to me and tapActivity does get called inside the log()

[chWagnr]

@Gummibeer Is it possible to reopen the request?

tapActivity is definitly called twice