Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User approval - default text is not transferred #4630

Closed
Semir1212 opened this issue Nov 12, 2020 · 7 comments
Closed

User approval - default text is not transferred #4630

Semir1212 opened this issue Nov 12, 2020 · 7 comments

Comments

@Semir1212
Copy link
Contributor

HH 1.7.0

'Default content of approval emails' (Administration --> Users --> Settings) is not transferred to approval form which is displayed when an admin approves a user. Although the form is empty, the email is sent out with content.

Besides that the loginURL is displayed incorrectly in the email.

image_2020_11_12T15_42_22_574Z

@fae7em
Copy link
Contributor

fae7em commented Nov 13, 2020

Have you changed the default message in the setting form or it's generated by code? Because it suppose to be {loginLink} instead of {loginURL}.

public static function getDefaultApprovalMessage($userDisplayName = '{displayName}', $adminDisplayName = '{AdminName}', $loginLink = '{loginLink}')
{
return Yii::t('AdminModule.user', "Hello {displayName},<br><br>\nYour account has been activated.<br><br>\n" .
"Click here to login:<br>\n{loginLink}<br><br>\n\n" .
"Kind Regards<br>\n{AdminName}<br><br>",
[
'{displayName}' => $userDisplayName,
'{AdminName}' => $adminDisplayName,
'{loginLink}' => $loginLink,
]);
}

@buddh4
Copy link
Contributor

buddh4 commented Nov 13, 2020

It seems I changed it in bc62148#diff-50786b2517fe78c0e6d6e2da1e1a20acb43afe83066ab033fd535e02eaa235a7 can we just replace both cases? I think the idea was to simplify the message text.

@buddh4
Copy link
Contributor

buddh4 commented Nov 13, 2020

So loginUrl = only url and loginLink is the rendered link, just as fallback for old texts.

@paviro
Copy link

paviro commented Nov 22, 2020

I am also having this issue. Good tipp that it was changed to {loginLink}. To form is also empty for me on the sending screen.

@luke-
Copy link
Contributor

luke- commented Nov 23, 2020

A fix will be available with 1.7.1

#4631

@luke- luke- closed this as completed Nov 23, 2020
luke- added a commit that referenced this issue Nov 23, 2020
Transfer default content of user approve/decline email to approve/decline form #4630
@Cpt-KeksMonster
Copy link

Hello, is it possible this issue has returned in v1.11.1? I am experiencing the exact issue.

What steps will reproduce the problem?

Deny a user in Administration > "User Approval".

What is the expected result?

User gets an e-mail informing him, that he was not allowed to access this humhub instance.

What do you get instead?

An e-mail is send to the denied user telling him that he was approved.

Additional info

Q A
HumHub version 1.11.1
PHP version 8.1

grafik

@luke-
Copy link
Contributor

luke- commented May 4, 2022

Can you please create a new issue for it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants