Allow custom attribute comparison in LogsActivity trait

[jargoud]

In the LogsActivity trait's attributeValuesToBeLogged method, the comparison logic is currently hardcoded in a closure:

function ($new, $old) {
    // Strict check for php's weird behaviors
    if ($old === null || $new === null) {
        return $new === $old ? 0 : 1;
    }

    // Handles Date interval comparisons since php cannot use spaceship
    // Operator to compare them and will throw ErrorException.
    if ($old instanceof DateInterval) {
        return CarbonInterval::make($old)->equalTo($new) ? 0 : 1;
    } elseif ($new instanceof DateInterval) {
        return CarbonInterval::make($new)->equalTo($old) ? 0 : 1;
    }

    return $new <=> $old;
}

The Issue: I encountered a case where an attribute's "old" value is an integer and the "new" value is an object (Value Object). For instance, a "Free Shipping" threshold that can be either a number of items (int) or a currency amount (Money object).

In this scenario, the spaceship operator or strict null checks aren't flexible enough and might lead to incorrect logging or ErrorExceptions.

The Proposal: It would be very convenient to have a way to override this comparison logic in the Model, similar to how tapActivity allows custom behavior. We could introduce a method like compareAttributes (or similar) that the trait would check for before falling back to the default closure.