Two Bugs That Should Be Fixed

  • Resolved ljmac

    (@ljmac)


    7 years, 11 months ago

    IMHO this is the best plug-in of its type, but I found two serious bugs:

    1) The login and register links are reversed – I had to swap around the following piece of code in voter_functions.php:

    ?redirect_to=<?php echo urlencode($redirect_to); ?>

    2) Your plug-in overwrites the user description field, which it most certainly should not do – I had to comment out the following piece of code in voter.php:

    add_filter('get_the_author_description', array('VoterBpMembers','az_display_author_total_votes'), 3, 10);

    • This topic was modified 7 years, 11 months ago by ljmac.
Viewing 5 replies - 1 through 5 (of 5 total)
  • Thread Starter ljmac

    (@ljmac)

    Actually, I also had to swap $register_link with $login_link on line 705 to ensure the login redirect works correctly.

    Additionally, with BuddyPress I had to change line 391 to the following to ensure all voters display correctly:

    $name = bp_core_get_user_displayname( $uid );

    • This reply was modified 7 years, 10 months ago by ljmac.
    Plugin Author aheadzen

    (@aheadzen)

    thanks for the feedback. applied all changes, to be pushed live after testing.

    Thread Starter ljmac

    (@ljmac)

    Great! Good to know this plug-in is still alive. ??

    Hi aheadzen,

    I agree, that this is a great plugin, but I had to deactivate it because it overwrites the user biography field with the value ‘1’ – same bug as @ljmac reported , I suppose?

    Please release new version soon. ??

    Thread Starter ljmac

    (@ljmac)

    Yep, that;s the same bug. ??

    I sometimes get the following error in my logs too:

    PHP Warning: Missing argument 2 for VoterPostComments::aheadzen_comment_voting_links() in [URL REMOVED]/wp-content/plugins/voter-plugin/voter_functions.php on line 1210

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Two Bugs That Should Be Fixed’ is closed to new replies.