Jump to content

H0zen

Senior getMaNGOS Developer
  • Posts

    114
  • Joined

  • Last visited

  • Days Won

    11
  • Donations

    0.00 GBP 

Bug Comments posted by H0zen

  1. DoSpellHitOnUnit is called when spell succeeded ([URL="https://github.com/mangoszero/server/blob/develop21/src/game/WorldHandlers/Spell.cpp#L957"]Spell.cpp@957[/URL]). In this case, Mind Soothe will not aggro.
    But for the resist case, the target will aggro ([URL="https://github.com/mangoszero/server/blob/develop21/src/game/WorldHandlers/Spell.cpp#L979"]Server.cpp@L979[/URL])

  2. If you take a close look to the [I]player_classlevelstats[/I] table in the world database, you'll notice that the warriors (class = 1) at level 10 have a base hp of 101 and at level 11 base hp = 100. Indeed, there is something wrong in the database. You can investigate closer and do a PR when you have some spare time.

    Edit

    Not only warriors loose hp when they grow up. Look closer at class 2, level 1 & 2 :)

  3. [quote=Xenithar]I found the issues. You can't do safe updates without specifying an index. I removed all commenting, coding, everything except SQL from the update and ran it. It ran until it hit the following lines.
    [code]
    update dbdocsfields SET FieldComment = replace(FieldComment, 'table', 'table');
    update dbdocsfields SET fieldNotes = replace(fieldNotes, 'table', 'table');
    [/code]
    There is no index given so it was failing due to not being a safe operation. This was not being shown in the errors log, but it was causing the rollback flag to trip. All updates MUST specify an index! I now have to delete everything up to this line and figure out how to correct it so I can complete the update.[/quote]

    To make these lines safe add an "always true where clause", involving an index, like you did in a previous PR (21_1_16, if I remeber right). Like:

    update dbdocsfields SET FieldComment = replace(FieldComment, 'table', 'table') where `fieldId`> 0;
    update dbdocsfields SET fieldNotes = replace(fieldNotes, 'table', 'table') where `fieldId`> 0;


    This is a standard procedure to calm down the angry MySQL.

    PS: What is the meaning of those 2 updates? Maybe I`m blind, but I don't get it. It replaces all occurences of "table" with.. "table"? In the comments I see something about "fixing a typo"...

×
×
  • 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