Problem/Motivation

Hi Matthew, when I try to save the personal digest settings with the box checked for "Include 'change settings' link in the email." I get the following error:

Error: Call to undefined function getComponent() in personal_digest_user_settings_form_mode() (line 138 of /var/www/html/okhighlandtrades/web/modules/personal_digest/personal_digest.module)

It works just fine when I leave that box unchecked.

I'm also trying to find a way to send emails with HTML formatting via SMTP. Do you know if there are any distro-specific requirements that would affect this?

Thanks!

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

amysragbag created an issue. See original summary.

matslats’s picture

matslats commented

if you were using beta1 that was very old.
In the current version I don't see how that error was possible.
Anyway I've now just pushed beta2 for you

amysragbag’s picture

amysragbag commented

Ok thank you. For personal digest, I now have Version: 2.0.0-beta2.

There is a new error message when I try to include the 'change my digest settings' link.

It says: Error: Class 'Drupal\personal_digest\Link' not found in Drupal\personal_digest\Generator->loginLink() (line 91 of /var/www/html/okhighlandtrades/web/modules/personal_digest/src/Generator.php)

It's not urgent for us, just thought you might want to know.

matslats’s picture

matslats commented

thanks that is fixed in HEAD

amysragbag’s picture

amysragbag commented

Ok. I think I should have HEAD. Is there a way to check and make sure?

I get that same error when I check the box to include the link for changing the digest settings and hit 'submit and test' but not when I hit 'submit.'

Message Error: Class 'Drupal\personal_digest\Link' not found in Drupal\personal_digest\Generator->loginLink() (line 91 of /var/www/html/okhighlandtrades/web/modules/personal_digest/src/Generator.php)

matslats’s picture

matslats commented

Sounds like you haven't got the version I just pushed.
To get HEAD you need to set your composer version to @dev and of course run
composer update --no-dev

amysragbag’s picture

amysragbag commented

That worked! Thank you.

amysragbag’s picture

amysragbag commented
Status: Active » Fixed
amysragbag’s picture

amysragbag commented

Hi Matthew, there's just one more bug. When you follow the link in the email to change your digest settings, you get:

Symfony\Component\Routing\Exception\RouteNotFoundException: Route "entity.entity_form_display.user.user.user.default" does not exist. in Drupal\Core\Routing\RouteProvider->getRouteByName() (line 206 of /var/www/html/okhighlandtrades/web/core/lib/Drupal/Core/Routing/RouteProvider.php).

matslats’s picture

matslats commented

Please note this is not a hamlets issue but a personal_digest issue.
Fixed and pushed rc1

amysragbag’s picture

amysragbag commented

Ok, sorry. The link now goes to admin/config/people/accounts/form-display?user=1

matslats’s picture

matslats commented

ok this is fixed and tested in HEAD

Status: Fixed » Closed (fixed)

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