[5.4] Log entry format

[brianteeman]

Pull Request for Issue #46271 .

Summary of Changes

Changes a space to a tab to ensure that all columns are separated by a tab.

Testing Instructions

Code Review

Actual result BEFORE applying this Pull Request

The priotiy and ip address columns have a space as a separator when all the other columns are tab separated

Expected result AFTER applying this Pull Request

All columns are tab separated

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed

[alikon]

I have tested this item ✅ successfully on c46fe44

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46273.}

[heelc29]

The same here?

joomla-cms/plugins/system/webauthn/src/Extension/Webauthn.php

Line 126 in 8abfcff

'text_entry_format' => '{DATETIME} {PRIORITY} {CLIENTIP} {MESSAGE}',

[brianteeman]

Not at my pc but if that's also mixed spaces and tabs then please open a new issue

[heelc29]

I do a quick test ... yes it looks like

then please open a new issue

Isn't it easier to integrate this into your PR since it's the same mistake

[brianteeman]

added webauthn log

[heelc29]

I have tested this item ✅ successfully on 16d123c

added webauthn log

thanks

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46273.}

[sergeytolkachyov]

I have tested this item ✅ successfully on 16d123c

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46273.}

[richard67]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46273.}

[muhme]

Final check before merge: ✅ All spaces are replaces by tabs.

@brianteeman And is it not better to use \t as in other places? This would be clearly visible when reading the code in any case (independed from editor and configuration). After applying the PR, we could search and find:

find . -name \*.php | xargs grep "{MESSAGE}" | sed 's/\t/→/g'           
./plugins/system/webauthn/src/Extension/Webauthn.php:            'text_entry_format' => '{DATETIME}→{PRIORITY}→{CLIENTIP}→{MESSAGE}',
./libraries/src/Log/Logger/FormattedtextLogger.php:    protected $format = '{DATETIME}→{PRIORITY}→{CLIENTIP}→{CATEGORY}→{MESSAGE}';
./libraries/src/Log/Logger/W3cLogger.php:    protected $format = '{DATE}→{TIME}→{PRIORITY}→{CLIENTIP}→{CATEGORY}→{MESSAGE}';
./installation/includes/app.php:            'format'    => '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}',
./installation/includes/cli.php:            'format'    => '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}',
./administrator/components/com_scheduler/src/Scheduler/Scheduler.php:        $options['text_entry_format'] = '{DATE}→{TIME}→{PRIORITY}→{MESSAGE}';
./administrator/components/com_scheduler/src/Task/Task.php:            $options['text_entry_format'] = '{DATE}→{TIME}→{PRIORITY}→{MESSAGE}';
./administrator/components/com_joomlaupdate/src/Model/UpdateModel.php:            'format'    => '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}',
./administrator/components/com_finder/src/Response/Response.php:            $options['format']    = '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}';
./administrator/components/com_finder/src/Controller/IndexerController.php:            $options['format']    = '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}';
./administrator/components/com_finder/src/Controller/IndexerController.php:            $options['format']    = '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}';
./administrator/components/com_finder/src/Controller/IndexerController.php:            $options['format']    = '{DATE}\t{TIME}\t{LEVEL}\t{CODE}\t{MESSAGE}';

[richard67]

Thanks all (author and testers).