Can you please update the PR title so that it is clearer what is being changed here.

@brianteeman what's the issue when removing (C) ? the @copyright already says that it's about the copyright isn't it?
as alternative we could use © can't we? just asking

@brianteeman what's the issue when removing (C) ? the @copyright already says that it's about the copyright isn't it? as alternative we could use © can't we? just asking

1. We should only use one form of copyright statement throughout Joomla
2. In many countries - including the USA - the @copyright "element" does not match the legal requirements of a copyright statement
3. Using © instead of (c) is ok but we must be consistent

I can provide more legal references if required

@brianteeman using "Copyright" as word should also be ok, right? But I am looking what is the best way in terms of IDE configuration.

@brianteeman using "Copyright" as word should also be ok, right? But I am looking what is the best way in terms of IDE configuration.

In the USA it would be ok - not sure about elsewhere. We should be consistent though.

I did it again on another way, so anything should be fine now and the PR can be tested

I am confused about this codestyle as it doesnt appear to be consistent

        if (!empty($extension)) {
            $extension .= '%';
            $query
                ->where($db->quoteName('a.extension') . ' LIKE :extension')
                ->bind(':extension', $extension);
        }

                $query->where(
                    $db->quoteName('a.log_date') . ' BETWEEN :dstart AND :dnow',
                );
                $query->bind(':dstart', $dStart);
                $query->bind(':dnow', $dNow);

In the second one shouldnt the two bind queries be part of the first query - so something like

                $query
                ->where($db->quoteName('a.log_date') . ' BETWEEN :dstart AND :dnow')
                ->bind(':dstart', $dStart);
                ->bind(':dnow', $dNow);

I missed the whole file.

About the code style for multiline chaining. Honestly I am not 100% sure what it better readable. The reformat applies when you have something like

$var->action1()
  ->action2()
  ->action3()

then it is converted into

$var
  ->action1()
  ->action2()
  ->action3()

It might be better to let it as it is, the difference is not so big