Registration does not honor the email.mention config key

esotalk / esoTalk

Fat-free forum software.

GNU General Public License v2.0

1.47k stars 239 forks source link

Registration does not honor the email.mention config key #386

Open jgknight opened 9 years ago

jgknight commented 9 years ago

When setting the email.mention key = true, if a new user registers the "mention me in a post" setting by default is still disabled.

I think we just need to change config.defaults.php so that email.mention = false, and then in core/models/ETMemberModel.class.php line 60 update it so it contains

$preferences = array("email.privateAdd", "email.post", "starOnReply", "email.mention");

inliquid commented 9 years ago

Strange - everything works fine w/o this fix...

jgknight commented 9 years ago

When I tested without this fix it didn't work as expected. With the key set if I created a new user, they did not have the email.mention preference enabled by default. With this fix, if I had the key set and I created a user, the preference was correctly set by default.

I'll reopen so I can retest this.

inliquid commented 9 years ago

Ok, maybe I didn't get the case...