Cron fails on amazon_ses when not configured

I have a patch for a work-around, but I'm not sure if this isn't a bad idea :-)

class AmazonSesMailQueue extends QueueWorkerBase implements ContainerFactoryPluginInterface {
  use HandlerTrait;

  /**
   * {@inheritdoc}
   */
  public static function create(ContainerInterface $container, array $configuration, $plugin_id, $plugin_definition) {
    $instance = new static(
      $configuration,
      $plugin_id,
      $plugin_definition
    );

    // Only set the handler if queueing is enabled to avoid an error when
    // trying to run without config.
    $enabled = \Drupal::config('amazon_ses.settings')->get('queue');
    if ($enabled) {
      $instance->setHandler($container->get('amazon_ses.handler'));
    }

    return $instance;
  }

davisben → changed the visibility of the branch 3417090 to hidden.

I took a different approach by implementing hook_queue_info_alter() to remove the mail queue from cron when it's not enabled. Could you give this a test and see if it works as you expect it to?

@davisben

Thanks for the proposed change. It's currently not working because `amazon_ses_hook_queue_info_alter()` should be `amazon_ses_queue_info_alter()`.

With the proper function name, the queue is indeed not processed during cron when queueing is not enabled and processed otherwise.

Thanks! This has been committed.

Hmm, does this need to get ported to 3.1.x as well?

Sure does. I just cherry picked it to 3.x. Thanks!