Add new command to clean old log entries, fixes issue #622

[rochdi80tn]

What are you changing/introducing

Adding new console command implementing the log cleaning feature
This new command will do it (via a cron job on a monthly bases for instance) where it will delete log files older than YYY years but only if we have more more then XXX rows. This ensures log files won’t be deleted if there just a few log entries.

What is the reason for changing/introducing

Overtime log tables such as admin_ngrest_log and cms_log will contain large amount of data. To keep the database cleaner and if we don not need these logs anymore we have to manually perform a cleanup.

QA

Q	A
Is bugfix?	no
New feature?	yes
Breaks BC?	no
Tests pass?	n/a
Fixed issues	622

[nadar]

@rochdi80tn thanks for your PR. Please add your @author name, also it would be very nice to have some examples in the php doc header with how to use the command, showing the different options. Cause this is what will be rendered on luya.io in the api docs.

I will adjust the code a little, also to meet the psr2 standards (we usually take https://github.com/FriendsOfPHP/PHP-CS-Fixer to fix coding standards)

[rochdi80tn]

@nadar , looking at what you altered helped me a lot better understanding the coding standards. Thanks.
I believe will add some documentation, but this shall be done in a different PR right ?

[nadar]

I just added one more usage example. Yes, more documentation and usage is welcome, maybe also add your mail adresse as author, i just had your username.

Your commands works perfectly and is a really good tool when i comes to cleanup the tables. thanks again

[rochdi80tn]

Hi @nadar, Is there a place where we can check the master branch API documentation builds (if any)? Want to see how the documentation looks like. Best -- Rochdi

…

On Tue, Mar 9, 2021 at 2:58 PM Basil ***@***.***> wrote: I just added one more usage example. Yes, more documentation and usage is welcome, maybe also add your mail adresse as author, i just had your username. Your commands works perfectly and is a really good tool when i comes to cleanup the tables. thanks again — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#623 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABNP4WGROEMF4CANBI454TDTCYSRHANCNFSM4Y2BAWVA> .

[nadar]

No sadly we don't have a preview, sorry. But looking at the other controllers might help https://luya.io/api/luya-admin-commands-ProxyController