Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
These messages are sent to watchdog, so they are available to the admin and users with 'administer users' could just be confused by them (they appear to be errors when the account doesn't exist on the slave).
Patch to follow will simply remove them, but they could be hidden behind a variable as well ('Show Bakery debug messages' or something similar).
Comments
Comment #1
glennpratt CreditAttribution: glennpratt commentedD6 patch.
Comment #2
gregglesI think they are useful to be notified of them. Either way - better status.
Comment #3
coltraneHow about leaving them for roles with permission 'administer bakery' ?
I don't think they should be removed entirely.
Comment #4
glennpratt CreditAttribution: glennpratt commentedMakes sense to me.
7.x was missing the watchdog and a comment was out of place... I just made it like 6.x.
Comment #5
bcmiller0 CreditAttribution: bcmiller0 commentedre-rolled for d7 for alpha4 release to apply cleanly.
Comment #6
bcmiller0 CreditAttribution: bcmiller0 commentedversion d6 for alpha4, clean patch
Comment #7
drummThis needs to go into 7.x-2.x, as discovered at #2321431: Site functionality problem - New account holder receives Account Not Found message.
Comment #8
drummComment #9
mallezieLet's try
Comment #10
mallezieCould this be the right approach?
Comment #11
drummNot quite, the D6 approach was good and the same should be forward ported.
Comment #12
zaporylieLike this?
Comment #13
drummLooks good. I haven't fully tested this, so I won't set it to RTBC quite yet.
Comment #14
drummComment #15
drummLooks good. I've added this to Drupal.org's build: https://bitbucket.org/drupalorg-infrastructure/drupal.org-sites-common/c...
Comment #16
drummComment #17
drummCommitted to 7.x.