Dev49.net

Hi,
Ah, this makes sense 🙂
Yes, Cyrillic letters are now supported.
I guess I can mark this one as resolved 🙂

DW

Hi,
I just checked the the Polygon Recent Comments With Avatar plugin you are using and unfortunately it is not possible to do it without code modification of this plugin. The author has not implemented any filters I could hook into. The reason why it worked for you earlier is because I was using a very different approach to get the user name. And while it may have worked for you, it was very buggy and caused a lot of trouble.

To make it work, you need to go to wp-content/plugins/polygon-recent-comments-with-avatar/polygon-recent-comments-with-avatar.php and navigate to line 119. It says:
$avatar = get_avatar($email, $avatar_size);//functions only default gravatar;
You need to delete this line and paste this instead:
$avatar = get_avatar($comment, $avatar_size);

Author has not updated the plugin for over 10 months, so I doubt he will update it in the future. However, if you want to make sure you don’t accidentally update it in future, you can change the version number in the same PHP file. Currently line 6 says:
Version: 1.0.2
You can change it to, for example:
Version: 99.0.2
To make sure you will not receive any update for it.

Sorry for that, but unfortunately it is the only way to make it work. The plugin is not passing a comment object, which I could use to get the user name. Instead it gives an email, which is not useful to me at all.

Cheers,
Dan

Hi @aminima, any update on this? 🙂 I could release the update any time now, I just need to know it works properly for Arabic letters 🙂

Cheers,
Dan

Hi there,
I’m glad it worked out. The filter priority basically specifies the execution order of the plugin. Setting it to high value causes it to be triggered last in chain. In plugin settings this is described as:

If you are using multiple avatar plugins, you can increase or decrease execution priority of this plugin. If WP First Letter Avatar is overriding your other plugins, try changing this to a lower value (for example 9).

But you are right, this is what FAQ is for. I will add this as another question.

I think I have already left a good review however I will leave one if I haven’t.

I don’t see your review, so I would appreciate if you left one 😀

I’m marking this thread as resolved.

Cheers,
Daniel

Thanks for the link, I may take a look sometime. I just Googled how Persian actually sounds like and I love it, it sounds soooo smooth.

Huge thanks for the png files. I have just integrated the Arabic alphabet into the plugin. Would you mind giving it a quick test before I make the release? Here is the link – http://dev49.net/buddypress-first-letter-avatar.zip
It has the same version number as the latest release (2.2.1), so if you are using the plugin right now, you will have to replace the files on your FTP or delete the plugin and install it from the zip I’m sending you.

Thanks,
Dan

Actually, it would be great if you could rename the files. This would make it much easier for me to implement.

Just to make sure – these are all letters that appear in Persian script? Also, are they lowercase? They should be lowercase.

If these are all characters and they are lowercase, then it would be absolutely perfect for me if the files were named with their Unicode number. According to the converter I have written, these are the unicode numbers for each of the chars you have sent in your last post:

آ => 1570
ا => 1575
ب => 1576
پ => 1662
ت => 1578
ث => 1579
ج => 1580
چ => 1670
ح => 1581
خ => 1582
د => 1583
ذ => 1584
ر => 1585
ز => 1586
ژ => 1688
س => 1587
ش => 1588
ص => 1589
ض => 1590
ط => 1591
ظ => 1592
ع => 1593
غ => 1594
ف => 1601
ق => 1602
ک => 1705
گ => 1711
ل => 1604
م => 1605
ن => 1606
و => 1608
ه => 1607
ی => 1740

So, for example, آ.png should be named 1570.png. If you could do this, it would be totally perfect. Maybe just do the 512 folder, I can do the rest. But these chars need to be all lowercase (even if the avatar picture shows an uppercase png letter). All filenames should be lowercase and their Unicode number should also be for lowercase. So in case you have sent me uppercase, please send me the lowercase list and I will send you Unicode values for lowercase letters.

Forgive my ignorance, but is Persian alphabet the same as Arabic? It all looks the same to my eyes, so it’s difficult for me to judge 🙁

Thank you for your help.

Daniel