emliron
Forum Replies Created
-
Forum: Plugins
In reply to: [bbp style pack] E-mail translationAll work, the warning for the role is clear. Thank you!
Forum: Plugins
In reply to: [bbp style pack] E-mail translationHi Robin,
I’ve tested it and it’s exactly that.
On my site, I’ve hidden (css) the registration link. Registration is done using a script for members of the association, and only for a forum. For the other forums, I’ve replaced the email notification with a push notification (Onesignal).Forum: Plugins
In reply to: [bbp style pack] E-mail translationSorry, I’m not making myself clear.
In the Mail subscriptions options, option 3 allows you to limit sending to certain roles. For example, I haven’t ticked Spectator. A spectator won’t be able to subscribe. However, I can subscribe him or her using Subscription Management.
But the function bsp_get_active_subscribers_to_email defined at the end of the functions_email.php script seems to exclude him from the list of subscribers.
Is it a good idea to subscribe a user if they don’t receive the emails in the end?Forum: Plugins
In reply to: [bbp style pack] E-mail translationJust a comment: even if you prevent a user (by role) from subscribing, the plugin allows the administrator to subscribe them. But I’m not sure that the message will then be sent, if there’s a filter when it’s sent. (bbp_forum_subscription_user_ids)
Forum: Plugins
In reply to: [bbp style pack] E-mail translationHi Robin,
I’ve tested it and everything works. One note, one can register members, even if their role does not normally allow it in the plugin options. In my essay, visitors can’t, but I was able to do it nonetheless as an administrator.
In the functions_email.php file, don’t forget to add the missing ‘\’ in the three functions (lines 103, 173 and 240). In the bsp_test_email function, you could also change the reply email (line 307).
Thanks for the update!
Forum: Plugins
In reply to: [bbp style pack] E-mail translationThanks a lot, I’ll do the tests!
Forum: Plugins
In reply to: [bbp style pack] E-mail translationOn my personal site, I’ve deactivated all plugins except bbpress and bbp-style-pack, and chosen the twentytwentyfour theme. I have the same error …
There is no php error (in errors.log) and no javascript error in the console.
I changed the language (French->English(US)), and it works. There must be an error there.
(on my personal site, there is no WPML).Forum: Plugins
In reply to: [bbp style pack] E-mail translationThanks, I’ll check it out and let you know.
Forum: Plugins
In reply to: [bbp style pack] E-mail translationI tick the chosen forum, select the option (subscribe or unsubscribe) and validate. And nothing changes! ( (I also check in the forum subscriptions section)
Forum: Plugins
In reply to: [bbp style pack] E-mail translationHi Robin,
I have another request!. For testing, I used subscriptions management. It works very well from a forum, but not at all from a user: when you validate with the button, it seems to run but there are no changes. I tested on two sites, with the same error (I’m using the Enfold theme).Forum: Plugins
In reply to: [bbp style pack] E-mail translationThanks for the update. It works “almost”. You forgot a \ in the lines:
$message = str_replace( “\r\n”, ‘<br>’, $message );
before the r. If not, that’s great!Forum: Plugins
In reply to: [bbp style pack] E-mail translationThe “/n”, “/r”, “/r/n” should be replaced by a single “<br>”.
I propose to replace:$message = str_replace( “\r”, ‘<br>’, $message );
by the lines :
$message = str_replace( “\r\n”, ‘<br>’, $message );
$message = str_replace( “\r”, ‘<br>’, $message );
$message = str_replace( “\n”, ‘<br>’, $message );Forum: Plugins
In reply to: [bbp style pack] E-mail translationSorry, I’ve realised my mistake: the correct syntax is __( $text, $domain), not __e($text, $domain)!
I’ve corrected it and it works…Forum: Plugins
In reply to: [bbp style pack] E-mail translationThanks for the update.
Since you updated, the __e() function is undefined, I don’t know why and it’s probably nothing to do with that. I don’t understand.I replaced __e(…) with translate(…) in my initial script and in the plugin, and it works.
You didn’t add the line:
$message = str_replace( “\r\n”, ‘<br>’, $message );In the native version (the backend), line breaks contain ‘\r\n’. So the conversion, without this line, adds two line breaks.
In the text translated by WPML, there is only ‘\n’, hence the interest in the other lines.
This is what I’m seeing in the messages sent. In the English version, without this line, there are double line breaks.Forum: Plugins
In reply to: [bbp style pack] E-mail translationI think the WPML interface writes ‘/n’ instead of ‘/r’ or ‘/r/n’.
Because of this, the initial line is not enough to create line breaks. At least, that’s the problem I found, and I corrected it this way.
- This reply was modified 1 year, 2 months ago by emliron.