The fix for issue #2979903 is wrong.

It should be

          'description' => t('You need to <a href="@url">add a SendGrid API Secret Key</a> for Drupal to be able to deliver mail through SendGrid Integration.', [
            '@url' => url('admin/config/services/sendgrid'),
          ]),

instead of


          'description' => t('You need to <a href="@url">add a SendGrid API Secret Key</a> for Drupal to be able to deliver mail through SendGrid Integration.', [
            '@url' => l(t('SendGrid'), 'admin/config/services/sendgrid'),
          ]),

as you are using @url inside of HREF already, the l() function will create a full tag.

CommentFileSizeAuthor
#2 url_fix-13306696-2.patch901 bytesyang_yi_cn

Comments

yang_yi_cn created an issue. See original summary.

yang_yi_cn’s picture

yang_yi_cn commented
StatusFileSize
new url_fix-13306696-2.patch901 bytes

patch here (against 7.x-1.x dev)

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Assigned: Unassigned » i-trokhanenko
Status: Active » Needs review

I get the same issue. Will test with patch #2.

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Assigned: i-trokhanenko » Unassigned
Status: Needs review » Reviewed & tested by the community

Patch #2 works well for me. Making RTBC.
Thanks!

  • Perignon committed 7fd72e3 on 7.x-1.x authored by yang_yi_cn 
    Issue #3087513 by yang_yi_cn: Not fixed properly - "ArgumentCountError:...
perignon’s picture

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

Thanks for the patch and the team effort to test. I just committed the change the dev.

Status: Fixed » Closed (fixed)

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