Jump to content

altronrain

Members
  • Posts

    74
  • Joined

  • Last visited

  • Donations

    5.00 GBP 

Bug Comments posted by altronrain

  1. And what is the purpose of pools? Not to allow whole the nodes spawn simultaneously?
    And how is the spawn should be done in classic? I'm lack of info. 

    Btw, it's good idea to bring other locations to the "wowhead" state one by one slo-o-owly.
    No need to be rush and I even wanna help with TomCat too. (Cause I'm fond of Gatherer addon and wanted to create Mangos Zero full Gatherer_db --> but it could be easier to make it from mangos db info, who knows)
    If modern wowhead could be a reliable source of info. 

     

  2. According to info around the internet my suggested solution has wrong chances (But I can't figure out what they should be). Needed an explanation of Dice system and .dbc params.

    Also my solution overall  seems "barbaric", cause mangos code has an opportunity to operate this params, so -> It should be more suitable way to fix this problem without switch/case construction. 

    Both topics require understanding of Dice system params and chances in Spell.dbc + understanding of their handling on server side.

    It's hard due to shortage of info (could find only 3.3.5 Spell.dbc short explanation of params without any further progress/discussion).
    Requires a dev review (who knows spell handling server side) 

  3. @antz fyi

    If I am right and the food itself should trigger poison at the beginning (not durig process|per sec), so that may fix this (based on chances and Dice params DBC arithmetic for spell 6410):

    	// Scorpid Surprise, special case because chance is set 101% in DBC but has 2 Effects. Wea are using Dice DBC params?
            // This is the fix attempt that makes this food proc poison (not on tick|per sec) at the beginning with chance based on DBC
            case 6411:
                if (urand(1,71) == 1)
                {
                    m_caster->CastSpell(unitTarget, triggered_spell_id, true);
                }
                return;

    placed into SpellEffects.cpp here

    But I'm not sure that this is the righ way to fix the problem cause I orient in source code not well at all. 

    After that during tests poison procs not with 100% chance. I could say seldom enough to suit the chance. 

    The main question is it should proc on every sec (too huge chance) or every process start (pretty low). And also: is it right way to fix? 
    Wanted to know about Dice params handling on the server side too. 

    UPD: seems like I was wrong about the chances. Looking for additional info.
     

  4. It seems that More Dense Grinding Stones (quest=7939) works (You need 2500+ rep to unlock it, same for othe Tier 5 quests that is stated like "More...".

    suggested fix:

    MariaDB [zero_world]> update quest_template set RequiredRaces=0 where entry=7891;
    MariaDB [zero_world]> update quest_template set RequiredRaces=0 where entry=7892;
    MariaDB [zero_world]> update quest_template set RequiredRaces=0 where entry=7893;
    

    Screen after fix > (Green bracers out of rep condition but also should work)

    darkmoon_fix.png

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use