The effects of alcohol on sustanon 250 leucine for – real weight loss & bodybuilding benefits?
Share:
Notifications
Clear all

Limited Support

Our team is currently on holiday, so support will be limited during this period. Response times may be slower than usual, and some inquiries may be delayed.
We appreciate your patience and understanding, and we’ll resume our usual support by the end of August.

 

Issue [Solved] Mail Translation

13 Posts
2 Users
1 Reactions
1,151 Views
Posts: 8
Topic starter
(@skoposconnect)
Active Member
Joined: 2 years ago

Hi

we are running wpDiscuz on multiple sides and some of them are multilingual. Most parts of the feature support the translaion.

 

I asked you to add a filter for the subject within: wpdiscuz/utils/class.WpdiscuzHelperEmail.php

L: 160  $subject = apply_filters("wpdiscuz_email_notification", true, $subject, $comment, $emailData);

And you added the filter call very fast, thanks for that.

But sadly I made a mistake with the position. This line is after the marker replacement magic. So it's translatable now, but I cannot use the marker in my translation. It would be great if you either move the apply_filters call bevor that happens or add another set of filters.

Move suggestion:

confirmEmailSender --> move apply_filters to before line 100
emailSender --> move apply_filters before 161

 

New apply_filters call suggestion:

confirmEmailSender line 100:
wpdiscuz_email_subject_pre_compile
wpdiscuz_email_content_pre_compile
 
emailSender line 161:
wpdiscuz_confirm_email_subject_pre_compile
wpdiscuz_confirm_email_content_pre_compile

 

Thanks in advance and let me know if that is possible. It would improve the translatability of the plugin.

12 Replies
Asti
Posts: 8075
 Asti
Support
(@asti)
Illustrious Member
Joined: 8 years ago

Hi SkoposConnect,

I'll get back to you once I get a response from the developers. 

1 Reply
Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

Hi @skoposconnect,

Please follow the steps below:

1. Deactivate and delete the version you currently have

2. Download and install the version attached below

3 Delete all kinds of caches and find the new hooks  

New version: 

The hooks that are added:

  • wpdiscuz_confirm_email_subject_pre_replace
  • wpdiscuz_confirm_email_content_pre_replace
  • wpdiscuz_email_subject_pre_replace
  • wpdiscuz_email_content_pre_replace

Please test this and let us know soon, so we'll be able to update the wpDiscuz plugin. 

Posts: 8
Topic starter
(@skoposconnect)
Active Member
Joined: 2 years ago

Hi,

 

I found another function, where I can't translate the subject and content.

class.WpdiscuzHelperEmail.php > notifyOnApproving

Line: 355 / 356

$subject = str_replace(["[BLOG_TITLE]", "[POST_TITLE]", "[COMMENT_AUTHOR]"], [$blogTitle, $postTitle, $comment->comment_author], $this->options->subscription["emailSubjectCommentApproved"]);
$message = str_replace($search, $replace, wpautop($this->options->subscription["emailContentCommentApproved"]));

Could you make it so that i can access the subject and message before the replacing is happening.

 
9 Replies
Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

@skoposconnect,

Try the version attached below and let us know ASAP: 

 

 

Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

Also, please confirm that you don't see any other issues. 

(@skoposconnect)
Joined: 2 years ago

Active Member
Posts: 8

@asti 

Im sorry, i checked the hole file now.

There are three more functions which are not translatable.

followConfirmEmail

notifyFollowers

sendMailToMentionedUsers

same as in notifyOnApproving

Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

@skoposconnect,

Again please check and let us know soon as far as we're going to release a new version once you confirm that everything is okay.

 

Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

@skoposconnect,

Please don't forget to let us know, as far as we need a confirmation to release a new version of the wpDiscuz plugin. 

(@skoposconnect)
Joined: 2 years ago

Active Member
Posts: 8

@asti 

I'm currently working on it, it's looking good so far. I probably can give you feedback today.

 

(@skoposconnect)
Joined: 2 years ago

Active Member
Posts: 8

@asti 

Everything is fine 👍 

Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

Thank you for confirmation @skoposconnect,

The new version is ready, so the changes are already included in the latest version of the wpDiscuz plugin. 

Asti
 Asti
Support
(@asti)
Joined: 8 years ago

Illustrious Member
Posts: 8075

We're going to mark this thread as resolved. Feel free to open a new topic if you have further questions.

Share: