Problem/Motivation

I get an error trying to send a test email

Steps to reproduce

Send the test email

Proposed resolution

Add a "use" statetement for Drupal\Component\Utility\HTML;

Remaining tasks

Code

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#16 3318852-16.patch1.06 KBwilfred waltman
#8 3318852-8.patch869 bytesfenstrat

Issue fork symfony_mailer-3318852

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

rodrigoaguilera created an issue. See original summary.

rodrigoaguilera opened merge request !34

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented
Status: Active » Needs review
maacl’s picture

maacl commented
Status: Needs review » Reviewed & tested by the community

Encountered the same problem and came to the same solution as in the patch.

adamps’s picture

adamps commented
Status: Reviewed & tested by the community » Fixed

Thanks

adamps’s picture

adamps commented

Hmm it keeps failing to merge. I'll try again later.

adamps’s picture

adamps commented
Status: Fixed » Reviewed & tested by the community
fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
commented
StatusFileSize
new 3318852-8.patch869 bytes

Also hit this, confirming this is RTBC.

Attached is the MR in patch form for anyone else who also needs this for composer patches. @AdamPS you could use it if the MR still won't merge.

adamps’s picture

adamps commented
adamps’s picture

adamps commented

  • AdamPS committed 3dd633a on 1.x 
    Issue #3318852 by rodrigoaguilera, fenstrat: HTML class not found
adamps’s picture

adamps commented
Status: Reviewed & tested by the community » Fixed

Thanks committed manually using the patch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented

Sorry, but, is not the correct way to use:

use Drupal\Component\Utility\Html;

Instead:

use Drupal\Component\Utility\HTML;
rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented

@rpayanm
Where do you see the incorrect use?

wilfred waltman’s picture

wilfred waltman commented
StatusFileSize
new 3318852-16.patch1.06 KB

I think it should be Html.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented

@rodrigoaguilera Here:
https://git.drupalcode.org/project/symfony_mailer/-/commit/3dd633a#b2c99...
And it is here:
https://git.drupalcode.org/project/symfony_mailer/-/blob/1.1.0-beta3/src...

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented

I think it should be "Html".

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented
Version: 1.1.0-beta2 » 1.x-dev

I see now. I can't reopen this one a new issue should be opened

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented

https://www.drupal.org/project/symfony_mailer/issues/3325123

AdamPS closed merge request !34