Error message when using SymfonyMailer plugin

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Thanks for your work on this issue.

Merge request should always be filed against 11.x to start. Fixes will be backported to the appropriate Drupal branches when the issue is committed.

Sometimes the easiest thing to do is to close the old MRs and create a new one based on 11.x. That will bring us closer to passing automated tests Thanks!

samit.310@gmail.com → made their first commit to this issue’s fork.

samit.310@gmail.com → changed the visibility of the branch 10.2.x to hidden.

samit.310@gmail.com → changed the visibility of the branch 3423775-symfony-mailer-description to hidden.

@samit.310@gmail.com i have checked and verified merge request MR !6768 and found that there is no warning showing.

steps to check
1. Install 11.x-dev on DDEV v1.22.6 and php 8.3.0
2. using git remote add drupal-3423775 https://git.drupalcode.org/issue/drupal-3423775.git
3. git fetch drupal-3423775
4. and then git checkout -b '3423775-symfony-mailer-description-fix' --track drupal-3423775/'3423775-symfony-mailer-description-fix'
5. install webform module and create form for mail sent.
6. Found no warning after going to branch 3423775-symfony-mailer-description-fix

Description is not a required argument in \Drupal\Core\Mail\Attribute\Mail ... I feel this is something that should be fixed in webform.

All it needs to do is something like

  if ($mail_module_name || $mail_plugin_id) {
    $t_args = [
      '@module' => $mail_module_name,
      '@plugin_id' => $mail_plugin_id,
    ];
    $requirements['webform_email'] = [
      'title' => t('Webform: HTML email support'),
      'value' => ($mail_module_name) ? t('Provided by the @module module.', $t_args) : t('Provided by the @plugin_id mail plugin.', $t_args),
      'severity' => REQUIREMENT_OK,
    ];
    if ($mail_plugin_definition) {
      $requirements['webform_email']['description'] = 
        $mail_plugin_definition['label'] .
        // Add the description if available.
        isset($mail_plugin_definition['description']) ? ': ' . $mail_plugin_definition['description'] : '';
    }
  }

Note the use of formattable markup is unnecessary here - we can rely on Twig's auto-escape.

Moving to Webform as per #13

In Drupal 11 this results in a TypeError on the status report page:

TypeError: Drupal\Component\Utility\Html::escape(): Argument #1 ($text) must be of type string, null given, called in /app/web/core/lib/Drupal/Component/Render/FormattableMarkup.php on line 238 in Drupal\Component\Utility\Html::escape() (line 431 of /app/web/core/lib/Drupal/Component/Utility/Html.php).

Adding a patch.

Please put your change into an issue fork and merge request.

Please review the MR based on #17 which tries to keep the changes as simple as possible.