bug with e-mails sender address in Arabic website

[cupid4]

I just started Arabic website using buddypress, and found this problem, the: noreply @ domainname . com is wrong when Arabic language is set as site language in wp-config, the e-mails sender show like this:

???.????@domainname.com ( in hotmail )

. @domainname.com ( in yahoo )

and of course, however you try to add this address to your safe list or contacts it never happen, it can't even be removed from Hotmail spam box for this reason.

• another side note, is it just my website or you can't choose Arabic letters username in buddypress? :)

[cupid4]

just to make sure..that bug report is not hidden or anything? :)

[johnjamesjacoby]

Not hidden. Do have any idea if this happens with the 1.2 branch of BuddyPress, or if this is new? Something tells me this is an old bug that we may not have time for in 1.5 unless someone can volunteer time with a fix.

[cupid4]

well, I never tried Arabic language with 1.2 branch, just when the 1.5 beta stage started, I though of doing that, as 1.5 looked more solid to me to build a community, I wish I can help with the bug, but I can help testing any patch on my website, it's semi-live, I just advertised it for an hour before I discovered this bug so I paused.

ar.cupidcalled.com

[cnorris23]

This is not an issue with BP per se. This is an issue with translating 'noreply' into Arabic. According to RFC 5322 (part of the email standard) only approved ASCII characters may be used in email addresses. Non-latin (which includes Arabic) characters are forbidden. Attached patch adds context to the translation of 'noreply', to give translators some clarity on this matter.

As far as Arabic usernames... again, it's not a BP issue. BP uses WP functions during username creation. You may want to add your voice to ​#WP5918. The ticket appears to have traction, just no momentum.

I almost think it might be better to remove the i18n from 'noreply'. WP doesn't even have this as translatable.

[cnorris23]

I decided to go ahead and add a patch for the alternative solution.

[cupid4]

Thank you so much Cnorris23 for the patch, I tested it now, it give me that error:

Parse error: syntax error, unexpected T_RETURN in /home/content/.../wp-content/plugins/buddypress/bp-core/bp-core-filters.php on line 56

is this fix for allowing translating "noreply" or for making it not translatable? I think the second option will be the best, as the e-mail address is usually English itself as the domain name, so the translation of "noreply" won't be any useful.

Thanks for the ticket link, I'm amazed it has been there since 4 years without a fix, I added a ​comment let's hope it will change anything :)

[cnorris23]

I made an important comma disappear. For my final act I made it reappear, so everything should work now.

[cupid4]

sorry, but it give this error now:

Parse error: syntax error, unexpected '.' in /home/content/..../wp-content/plugins/buddypress/bp-core/bp-core-filters.php on line 56

[cnorris23]

remove i18n

[cnorris23]

After reviewing it again, the original patch (3498.002.diff) was correct. I've changed it back. I'm not a my dev computer, so I can't test. How are you applying the patch?

[cupid4]

I don't know exactly what I was doing wrong, I did the same and now no error, but the activation e-mail still the same:

???.????@domainname.com

Edit: well, I thought of something and wanted to check, I found the problem was that the translation file of BuddyPress that I got from GlotPress have the "noreply" there and translated to Arabic, I changed that to English and everything worked, I think any fix for this should include removing the " noreply" from buddypress POT file and making it fixed term, or include a warning there for translator to not change it from English.
for now, I just submitted a change in GlotPress for 'noreply' reverting back to English.

​http://translate.wordpress.org/projects/buddypress/dev/ar/default?filters[status]=either&filters[original_id]=4691&filters[translation_id]=1178166

[boonebgorges]

I agree with cupid4 that the best course here is probably to hardcode 'noreply@'. Any objections?

[cnorris23]

No objections. That's what 3498.002 does, and what WP does also.

[cupid4]

Yes, just as cnorris23 explained in ​comment4 ,I should have thought to check the translation file from the point.
For the Arabic usernames, the current ​ticket seems to be totally ignored as it was reported 4 years ago, You think I should add new ticket about current WordPress version?

[cnorris23]

No don't start another ticket. It will just get closed as it will be a duplicate. That's why I suggested you comment on the existing ticket. Any help you can add there will help get it added. You can help by testing the existing patch. Update the patch if needed and you're capable.

[boonebgorges]

(In [5123]) Hardcode default replyto email address in bp_core_email_from_address_filter(), to avoid l18n problems in non-Latin charsets. Fixes #3498. Props cnorris23