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

Language of send emails and messages (Trac #2211) #2211

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 4 comments
Closed

Language of send emails and messages (Trac #2211) #2211

elgg-gitbot opened this issue Feb 16, 2013 · 4 comments
Assignees

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/2211 on 40381758-01-15 by trac user BvdH, assigned to unknown.

Elgg version: 1.7

If a user A with language english send a message to another user B with language german the user B get the notifications in english. I think thats not good.

Someone should add the second language parameter from user B to elgg_echo('groups:invite:subject') in alle concerned notify_user-cases.

Unfortunatly these changes should be done at many places in many files, below
mod/riend_request/start.php
actions/comments/add.php
actions/useradd.php
mod/messages/start.php
mod/messageboard/ajax_endpoint/load.php
mod/messageboard/actions/add.php
mod/groups/actions/joinrequest.php
mod/groups/actions/addtogroup.php
mod/group/actions/invite.php
mod/friend_request/actions/friendrequest/approve.php
mod/friend_request/actions/friendrequest/decline.php
mod/friend_request/actions/friendrequest/removefriend.php
engine/lib/relationships.php
engine/lib/users.php

Another solution is to change the "notify_user"-function.

@elgg-gitbot
Copy link
Author

cash wrote on 40381913-09-01

See also #392 and #1320

@elgg-gitbot
Copy link
Author

Milestone changed to Elgg 1.9 by cash on 40888599-07-18

@ghost ghost assigned cash Jul 9, 2013
@cash
Copy link
Contributor

cash commented Sep 27, 2013

Pushing this off to 1.10

@ewinslow ewinslow removed this from the Elgg 1.10.0 milestone Jun 13, 2014
@juho-jaakkola
Copy link
Member

Duplicate of #7241

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

No branches or pull requests

4 participants