Jump to content
  • mangosdLoadDB.sql INSERT issue (linux)


    Chucksta
    • Status: Completed
      Main Category: Database
      Sub-Category: sqlupdate
      Version: 2.0.11 Milestone: 20 Priority: High
      Implemented Version: 0.20

    Column count does not match value count error (spell_bonus_data table)

    Tested and found under Ubuntu Linux. Not tested under MS Windows.

    Line 6513 of the mangosdLoadDB.sql is attempting to INSERT records made up of just 6 values into a table whose records consist of 14 values:

    LOCK TABLES `spell_bonus_data` WRITE;
    /*!40000 ALTER TABLE `spell_bonus_data` DISABLE KEYS */;
    INSERT INTO `spell_bonus_data` VALUES
    /* Druid */
    (339, 0, 0.1, 0, 0, 'Druid - Entangling Roots'),
    (5185, 1.6104, 0, 0, 0, 'Druid - Healing Touch'),
    (5570, 0, 0.127, 0, 0, 'Druid - Insect Swarm'),
    (8921, 0.1515, 0.13, 0, 0, 'Druid - Moonfire'),
    (8936, 0.3, 0.1, 0, 0, 'Druid - Regrowth'),
    (18562, 0, 0, 0, 0, 'Druid - Swiftmend'),
    (5176, 0.5714, 0, 0, 0, 'Druid - Wrath'),
    /* Mage */
    (7268, 0.2857, 0, 0, 0, 'Mage - Arcane Missiles Triggered Spell Rank 1'),


    User Feedback

    Recommended Comments

    Related to:
    [url]https://github.com/mangoszero/database/blob/Rel20/World/Updates/Rel20/20003_01_mangos_spell_bonus_data.sql[/url]

    [url]https://github.com/mangoszero/server/commit/20d7f814a607578e4d4571079b1b36d913ec9e43[/url]

    Antz's comments re this:
    "looks like the core side of that patch is missing"

    "would you mind checking the code segments to see whether the code has been reverted somehow, especially SpellMgr.cpp"

    ----------------------

    I'll now do as Antz has requested ... looking into this now!

    Link to comment
    Share on other sites

    Issue found to be due to update not being applied to the mangosdLoadDB.sql file, which changed the structure of the table (added more fields). The adding of the fields was put in place, but the following code that adds the data was not altered to match the change.

    I'll fix this and PR it.

    [B][COLOR="#008000"]FIX HAS NOW BEEN PR'd[/COLOR][/B]
    [url]https://github.com/Chuck5ta/database/commit/0af9bc5ee7af53b255d4b1ad98223d58e6e3431a[/url]

    Link to comment
    Share on other sites



    Create an account or sign in to comment

    You need to be a member in order to leave a comment

    Create an account

    Sign up for a new account in our community. It's easy!

    Register a new account

    Sign in

    Already have an account? Sign in here.

    Sign In Now

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