Thread: Show Thread Enhancements - [AJAX] Post Thank You Hack
View Single Post
  #1892  
Old 01-17-2008, 07:20 PM
liwo liwo is offline
 
Join Date: Apr 2006
Location: Germany
Posts: 21
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by Abe1 View Post
Can you please explain more.
I think I can explain that problem. In reputation.php, not only the reputation points of a user are updated, but also the reputationlevelid. That way, the reputation level of a user is updated instantly. You just give the reputation points, but do not update the reputationlevelid. So if a user gets 1000 reputation points by post thank you hack, but does not get any reputation the normal way, his reputationlevel will not update.

To fix this problem, a minor change to function add_thanks() is sufficent i think

PHP Code:
    if ($vbulletin->options['post_thanks_reputation'])
    {
        
$vbulletin->db->query_write("
            INSERT IGNORE INTO "
TABLE_PREFIX ."reputation
                (postid, userid, reputation, whoadded, reason, dateline)
            VALUES
                ('"
.$postinfo['postid']."', '".$postinfo['userid']."', '" $vbulletin->options['post_thanks_reputation'] . "', '" $vbulletin->userinfo['userid'] . "', '$vbphrase[post_thanks_thanked_post]', " TIMENOW ")
        "
);

        if (
$vbulletin->db->affected_rows() != 0)
        {
            
////////////////////////////////////////////////
            // BEGIN FIX
            ////////////////////////////////////////////////
            // Query copied from reputation.php
            // Determine this user's reputationlevelid.
            
$reputationlevel $vbulletin->db->query_first_slave("
                SELECT reputationlevelid
                FROM " 
TABLE_PREFIX "reputationlevel
                WHERE " 
. ($postinfo['reputation'] + $vbulletin->options['post_thanks_reputation']) . "  >= minimumreputation
                ORDER BY minimumreputation
                DESC LIMIT 1
            "
);
            
// I do not store the new reputation calculated for the query above and use it here again to avoid a potential race condition
            
$set_user_got .= ", reputation = " $vbulletin->options['post_thanks_reputation'] . " + reputation, reputationlevelid = " intval($reputationlevel['reputationlevelid']);
            
////////////////////////////////////////////////
            // END FIX
            ////////////////////////////////////////////////

        
}
    } 
I hope that small note helps you improving that great hack further!
Reply With Quote
 
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01173 seconds
  • Memory Usage 1,795KB
  • Queries Executed 11 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD_SHOWPOST
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)bbcode_php
  • (1)bbcode_quote
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)post_thanks_box
  • (1)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (1)post_thanks_postbit_info
  • (1)postbit
  • (1)postbit_onlinestatus
  • (1)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • reputationlevel
  • showthread
Included Files:
  • ./showpost.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_postinfo_query
  • fetch_postinfo
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showpost_start
  • bbcode_fetch_tags
  • bbcode_create
  • postbit_factory
  • showpost_post
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • showpost_complete