Complete port of CacheLogger, clean up loggers

1. +++ b/src/Logger/CacheLogEntry.php
   @@ -15,16 +20,14 @@ class CacheLogEntry extends LogEntry {
   +    $expire = $settings['timeout'] > 0 ? time() + $settings['timeout'] : $settings['timeout'];
    
   -    $settings = $job->getSettings('logger');
   +    \Drupal::cache($settings['bin'])
   +      ->set('uc-name:' . $this->name, $this->lid, $expire);
   +    \Drupal::cache($settings['bin'])
   
   +++ b/src/Plugin/ultimate_cron/Logger/CacheLogger.php
   @@ -94,7 +93,7 @@ class CacheLogger extends LoggerBase {
          '#title' => t('Cache timeout'),
   -      '#description' => t('Seconds before cache entry expires (0 = never, -1 = on next general cache wipe).'),
   +      '#description' => t('Seconds before cache entry expires (0 = never).'),
   

   I don't think 0 means what it thinks.

2. +++ b/src/Plugin/ultimate_cron/Logger/CacheLogger.php
   @@ -38,18 +39,16 @@ class CacheLogger extends LoggerBase {
   -
   -    $job = ultimate_cron_job_load($name);
   -    $settings = $job->getSettings('logger');
   +    $settings = $log_entry->logger->configuration;
    
   

   I don't think this is the same? before, we loaded settings from the job, now we access the just-created log entry object?

The two log entry classes are removed in #2731683: LogEntry::save() should move to LoggerPlugin. The save method is moved to CacheLogger.

Also not seeing any database related changes here (anymore), so this is just about CacheLogger now.

Shouldn't be necessary to depend on #2692781: Make global setting ui dynamic and use it for those fixes here.

Ok, restarted this. Ended up doing a bunch of cleanup in the database logger as well.

Also added tests for the cache logger.

Passed, committed.