[CRITICAL BUG] Count error

  • Resolved blue11

    (@blue11)


    12 years, 4 months ago

    Hello,

    First of all you wrote a very nice plugin! This one is going to be huge!

    I see that is still in BETA testing so I just want to help you out.I found one problem with your module that is CRITICAL.

    Use case :

    1) You have 2 users.
    2) Vote ‘Like’ with first user -> counter for post gets set to 1
    3) Vote ‘Like’ with second user -> counter for post gets set to 2
    4) Vote ‘Unlike’ with second user -> counter gets set to 0 instead of 1! However I have enabled ‘Show voters after post/page content?’ and I can see the first user. But the counter is 0.

    I will write also a list of futures that I think is very important to be added for the success of this plugin in another topic.

    Many thanks,
    /Blue

    https://www.ads-software.com/extend/plugins/kk-i-like-it/

Viewing 6 replies - 1 through 6 (of 6 total)
  • Thread Starter blue11

    (@blue11)

    *Hint : Don’t forget to hit refresh after step 4!

    True.. I had this issue too, any solutions?

    Still the same problem, anybody found the reason?

    Popescu M

    (@radiourionline)

    in db.php inside function removeLike at line
    $result = update_post_meta($idPost, $this->metaLikes, $val->rating);
    you need to edit it like this
    $result = update_post_meta($idPost, $this->metaLikes, $rating);

    Thank you.

    I agree that this is a critical bug that prevails in the current version (1.7) of the plugin. Sadly, it allows a malicious user to “unlike” the counter on someone else’s post down to 0.

    Thanks @radiourionline for the hack, which solves the problem.

    Since there is no excuse for the rating plugin to err in the math department, this is a serious issue, but fortunately a breeze for the plugin author to fix. I hope this is incorporated ASAP in a forthcoming version of this otherwise brilliant plugin.

    Plugin Author Krzysztof Furtak

    (@krzy-siek)

    Hello,
    This issue fixed in version 1.7.2

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘[CRITICAL BUG] Count error’ is closed to new replies.