Currently in the D8 version of the module, SMTP defines it's permission using hook_permission().

function smtp_permission() {
  return array(
    'administer smtp module' => array(
      'title' => t('Administer SMTP Authentication Support module'),
      'description' => t('Perform administration tasks for SMTP Authentication Support module.')
    ),
  );
}

However, permissions should be defined in the smtp.permissions.yml file.

More information about this change can be found here.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

donutdan4114 created an issue. See original summary.

donutdan4114’s picture

Example yml file:

# In smtp.permissions.yml file.
administer smtp module:
  title: 'Administer SMTP Authentication Support module'
  description: 'Perform administration tasks for SMTP Authentication Support module.'
  restrict access: TRUE
jorgik’s picture

Assigned: Unassigned » jorgik
Status: Active » Needs review
FileSize
939 bytes

Hi, I added patch with permissions.yml file and removed smtp_permission() function. Needs review
http://joxi.net/823kV8KC6RvOz2.png

naveenvalecha’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

+++ b/smtp.permissions.yml
@@ -0,0 +1,4 @@
\ No newline at end of file

This will be taken care at the time of commit.

wundo’s picture

Status: Reviewed & tested by the community » Fixed
wundo’s picture

  • wundo committed b41b0ea on 8.x-1.x authored by jorgik
    Issue #2792495 by jorgik, donutdan4114: Permission "administer smtp...
  • wundo committed f1a11c4 on 8.x-1.x authored by jorgik
    Issue #2792495 by jorgik, donutdan4114, wundo, naveenvalecha: Permission...

Status: Fixed » Closed (fixed)

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