This version fixes the version number problem reported in 2.3.6.

All constants have been moved into a single table (PVPLOG). This was done primarily to reduce global namespace polution and any potential name conflicts.

Regards,

Brad

any chance the new "eye of the storm" Bg kills are recorded? eye of the storm doesn't appear to be in the wowroster 1.7.2 but i was just wondering if the coding in PvPlog was there yet?

I believe PvPLog will correctly track all of Blizzard's Battlegrounds with the following code:

Code: Select all

function PvPLogInBG()
    bg_found = false;
    bg_indx = 0;
    for i=1, MAX_BATTLEFIELD_QUEUES do
        bg_status, bg_mapName, bg_instanceId = GetBattlefieldStatus(i);
        if (bg_status == "active" ) then
            bg_found = true;
            bg_indx = i;
            return true;
        end
    end
    return false;
end


The PvPLog.lua savedvariables file records the bg_indx. Translating this index into a name would be Roster's responsibility.

Regards,

Brad

Actually it's quite simple.

Open lib/pvp3.php
Search for

Code: Select all

    $bg_array = array(
        'alterac_valley',
        'arathi_basin',
        'warsong_gulch',
    ); 

and add the string for the Eye of the Storm:

Code: Select all

    $bg_array = array(
        'alterac_valley',
        'arathi_basin',
        'warsong_gulch',
        'eye_of_the_storm',
    ); 

Then add the translation in your localization/xxXX.php (deDE.php for german in my case):

Code: Select all

// These need to be EXACTLY what PvPLog stores them as
$wordings['deDE']['alterac_valley']='Alteractal';
$wordings['deDE']['arathi_basin']='Arathibecken';
$wordings['deDE']['warsong_gulch']='Warsongschlucht';
$wordings['deDE']['eye_of_the_storm']='Auge des Sturms'; 

Works fine on my site.

Mogo