DISQUS

baron · 5 months ago

Works great, thank you.

dreinavarro · 5 months ago

and thank you for downloading it!

Toxic brain · 5 months ago

nice work. 2 minor issues I've noticed.

in the installation notes its being instructed to copy to a folder named 'YmStatusPlugin'.alright, but if we are installing using WP's internal plugin installer, it fetch the plugin from the wp site and installs automatically, most importantly it cretaed the plugin folder name by slugging the plugin name. So in ur case the folder will be named as 'yahoo-messenger-status-plugin' instead of 'YmStatusPlugin'.

In the 'ym_status_settings.phtml' file lines 24 & 25 the hardcoded folder nam produces error.

another one is regarding the directory separator used allover the code.
'/' is for *nix and '\' is for Win platforms. So in Windows installations the code throws an error in 'ym_status.php'.
I fixed it by replacing all occurance of '/' with DIRECTORY_SEPARATOR.
for example

require(Y_STATUS_BASE_DIR. '/YmStatus.php');

becomes

require(Y_STATUS_BASE_DIR. DIRECTORY_SEPARATOR . 'YmStatus.php');



thats it. :)

dreinavarro · 5 months ago

thanks for pointing those out. i'll work out those fixes for the next release.