Problem/Motivation

In function smtp_install() it uses the route smtp.config, which is not available yet.

see hook_install documentation https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Extension...

Proposed resolution

Rebuild the routes in the install hook of smtp-module.

Remaining tasks

Implement a patch.

CommentFileSizeAuthor
#2 3092942-rebuild-routes-in-hook-install.patch794 bytespetar_basic

Comments

petar_basic created an issue. See original summary.

petar_basic’s picture

petar_basic commented
StatusFileSize
new 3092942-rebuild-routes-in-hook-install.patch794 bytes
petar_basic’s picture

petar_basic commented
Status: Needs work » Needs review
diego_mow’s picture

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

Work as expected.

diego_mow’s picture

diego_mow commented
Assigned: petar_basic » Unassigned
rivimey’s picture

rivimey
she/her
commented
Parent issue: » #3114671: Plan for SMTP 8.x-1.0 release
rivimey’s picture

rivimey
she/her
commented
Parent issue: #3114671: Plan for SMTP 8.x-1.0 release » #3081980: Plan for full 8.x-1.0 release of SMTP
wundo’s picture

wundo commented

  • wundo committed 1b7d08c on 8.x-1.x authored by petar_basic 
    Issue #3092942 by petar_basic, rivimey, wundo: Route "smtp.config" not...
  • wundo committed 25f4365 on 8.x-1.x
    Issue #3092942 by wundo: Fixes trailing whitespace
wundo’s picture

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

Status: Fixed » Closed (fixed)

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