There is a missing "use drupal" in the code. See patch attached.

CommentFileSizeAuthor
#2 0001-Fix-missing-use.patch971 bytesjehon
email_auto_login-multi_use_token-2629142-2-7.patch1.23 KBjehon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jehon created an issue. See original summary.

jehon’s picture

jehon CreditAttribution: jehon as a volunteer commented
FileSize
0001-Fix-missing-use.patch971 bytes
jehon’s picture

jehon CreditAttribution: jehon as a volunteer commented
jehon’s picture

jehon CreditAttribution: jehon as a volunteer commented

Now with the correct file :-)

hansfn’s picture

hansfn CreditAttribution: hansfn commented
Status: Active » Postponed (maintainer needs more info)

I would have liked to see the warning/error that is fixed by the use statement.

Which version of Drupal 8 are you using?

jehon’s picture

jehon CreditAttribution: jehon as a volunteer commented

Hello

I am using Drupal 8.0.4.

As far as I remember (it was this morning after all :-) ), it was at this line:
if (VIEWS_SEND_MIMEMAIL && Drupal::currentUser()->hasPermission('attachments with views_send') &&

The "Drupal" is not prefixed with "\" and is not found in the current scope.

I could not reactivate the mime mail extension now, so I could not reproduce the problem. But I am pretty sure it was there...

jehon’s picture

jehon CreditAttribution: jehon as a volunteer commented

See our other bug https://www.drupal.org/node/2699779 that explain why I could not enable that extension anymore :-)

And thank you for your help !!

hansfn’s picture

hansfn CreditAttribution: hansfn commented
Status: Postponed (maintainer needs more info) » Closed (duplicate)

Yes, I consider this a duplicate of #2699779: Problem when using Mime Mail so I'm just closing this one.