Commit Graph
100 Commits
Author SHA1 Message Date
Trazom62 e0e8d6f6ee Avoid crash in TurretAI if spell1=0 and issue an error log instead.
Fixes issue #1360.
Fixes issue #1032.

(same for ArchorAI and AOEAI).

--HG--
branch : trunk
2010-04-09 19:45:06 +02:00
teacher fcd660b240 Changed recent spell_difficulty_dbc table creation to CHARSET=utf8.
Added recent sql updates to world.sql...

--HG--
branch : trunk
2010-04-09 15:41:01 +02:00
Rat 4bd2379c6f *add example difficulty spells for magus telestra
--HG--
branch : trunk
2010-04-09 10:57:42 +02:00
Rat cbbe587817 *added code for loading Spelldifficulty.dbc + Custom Spelldifficulty from Database
*added spell searcher based on creature's map difficulty, usage is optional and only for scipts for now
*added new table spelldifficulty_dbc

--HG--
branch : trunk
2010-04-09 10:47:11 +02:00
Rat 1f46c1150d *fix an sql-dbc load bug, patch by Qaston
--HG--
branch : trunk
2010-04-08 22:40:14 +02:00
Trazom62 d93e4079f7 Script Instance Forge of Souls.
Script based on proposal of Synric on trinitycore forum (submitted by Skarabex on GC).
Fix EventMap::DelayEvents
Fixes issue #1310.

--HG--
branch : trunk
2010-04-08 21:03:26 +02:00
teacher 6f3996fd11 Added serverside spell as place holders for future development.
Added some vehicleIDs and creature_template spells (by Xanadu and Mikebeck).
Some fixes for proper sql format.

--HG--
branch : trunk
2010-04-08 16:29:29 +02:00
Spp 61faf0cd10 SOTA: Fix graveyard respawn location.
Thanks to AntiDiavol for pointing this bug... twice
Closes issue 1516

--HG--
branch : trunk
2010-04-08 11:49:34 +02:00
Spp 4635fbc020 Code Style (game + scripts only):
"!=" --> " != " (when needed)
"  !=" --> " !="
"!=  " --> "!= "

--HG--
branch : trunk
2010-04-08 08:20:08 +02:00
Spp 46f0674e23 Code Style (game + scripts only):
">=" --> " >= " (when needed)
"  >=" --> " >="
">=  " --> ">= "
"<=" --> " <= " (when needed)
"  <=" --> " <="
"<=  " --> "<= "
" ==" --> " =="
"==  " --> "== "

--HG--
branch : trunk
2010-04-07 23:56:35 +02:00
Spp 2454c290b8 Code Style (game + scripts only):
"==" --> " == " (when needed)

--HG--
branch : trunk
2010-04-07 23:25:02 +02:00
Spp 49d05ba9aa Code style (game + scripts only):
"( " --> "("
" )" --> ")"

--HG--
branch : trunk
2010-04-07 22:59:46 +02:00
Spp d19e127080 Code style (game + scripts only):
"if(" --> "if ("

--HG--
branch : trunk
2010-04-07 19:14:10 +02:00
Spp 2e127f7a30 Code style (game + scripts only):
"while(" --> "while ("

--HG--
branch : trunk
2010-04-07 19:13:19 +02:00
Spp 182e9a20b1 Code style (game + scripts only):
"for(" --> "for ("

--HG--
branch : trunk
2010-04-07 19:12:44 +02:00
Spp b27ce42704 Code Style: Remove trailing spaces
--HG--
branch : trunk
2010-04-07 18:09:10 +02:00
Spp f490ad5ac2 Code Style: tab to spaces
--HG--
branch : trunk
2010-04-07 17:24:07 +02:00
teacher 502fc82fb8 Little sql cleanup and full update.
--HG--
branch : trunk
2010-04-07 18:21:31 +02:00
Spp 7afdb17ed3 Fix a typo in SOTA that prevented players to respawn in one of the graveyards
--HG--
branch : trunk
2010-04-07 16:27:17 +02:00
[email protected]> 9923df7745 remove CRLF from blackrock spire scripts.
--HG--
branch : trunk
2010-04-07 10:30:46 -02:30
Tartalo acc426951c Clean SQLs from last commit
--HG--
branch : trunk
2010-04-07 14:56:48 +02:00
Tartalo aaa340fa25 Blackrock Spire: Implements Leroy Jenkins achievement, by Spichy.
Fixes issue #1073.
Please, post your contribution following our coding style, otherwise I have to
waste a lot of time cleaning up the code

--HG--
branch : trunk
2010-04-07 14:55:54 +02:00
Anubisss 3316adb5a2 Merge.
--HG--
branch : trunk
2010-04-07 13:20:30 +02:00
Anubisss fa3aa746ac Remove some pointers and use GUIDs in borean_tundra.
--HG--
branch : trunk
2010-04-07 13:17:24 +02:00
raczman bcb67eb490 Include ACE headers in case they are not in system path.
Thanks for spotting the bug guys:
GL.AntiDiavol, next3x, Aokromes

--HG--
branch : trunk
2010-04-07 13:09:07 +02:00
Tartalo 599ffd8bb9 Boran Tundra, Quest 11648, The Art of Persuassion: Fix pCaster pointer, by microbius51
Cleanup the script

--HG--
branch : trunk
2010-04-07 10:33:58 +02:00
Tartalo 0ae812b7bd Gundrak, Drakkari Colossus: Fix logic, by [email protected]
Fixes issue #1313.

--HG--
branch : trunk
2010-04-07 10:18:54 +02:00
[email protected]> e316869ee4 remove CRLF from alterac_valley scripts.
--HG--
branch : trunk
2010-04-06 21:32:35 -02:30
Tartalo b58dc98b2b Utgarde Keep, Utgarde Pinnacle: Cleanup Svala Sorrowgrave
Better handling of sacrifice target storing his GUID on ScriptedInstance to
share it between npcs.

--HG--
branch : trunk
2010-04-07 01:04:38 +02:00
Tartalo 668dd91368 Ulduar, Halls of Lightning: Ionar disperses only once.
3.3.2 Blizzlike change

--HG--
branch : trunk
2010-04-07 00:21:31 +02:00
Tartalo 7c389f4c39 Ulduar, Halls of Lightning: Cleanup Ionar's script
--HG--
branch : trunk
2010-04-07 00:18:53 +02:00
Rat c39f8bf7c7 *fix bug "cant cast or logout till you move a bit if someone dismounted near you from a flying mount"
--HG--
branch : trunk
2010-04-06 23:52:08 +02:00
Tartalo f871e12fa4 Eastern Kingdoms, Alterac Valley: Register scripts on db
--HG--
branch : trunk
2010-04-06 21:39:16 +02:00
Tartalo 827751ce0c Branch merge
--HG--
branch : trunk
2010-04-06 21:36:27 +02:00
Tartalo 81127a5df4 Eastern Kingdoms, Alterac Valley: Add forgotten file from previous commit
--HG--
branch : trunk
2010-04-06 21:35:11 +02:00
Tartalo 48b1ecf08f Eastern Kingdoms, Alterac Valley: Scripts for Marshalls, Warmasters and bosses
Unknown author code contribution from issue #954.

--HG--
branch : trunk
2010-04-06 21:34:37 +02:00
raczman e30e9a45cb Cleaned up include directories in cmakes.
--HG--
branch : trunk
2010-04-06 15:53:27 +02:00
[email protected]> 971d617e9e Add Spells to Coldarra Warmages to target Tansisus Shield Bunnys same as Violet Stand.
Also required: http://trinitydatabase.org/index.php?/topic/15320-fix-for-transis-shield-bunnys-coldarra/

--HG--
branch : trunk
2010-04-05 22:09:36 -02:30
[email protected]> 9629a340c6 Fix script to have less calls
--HG--
branch : trunk
2010-04-05 11:34:30 -02:30
XTZGZoReX d3e30bab72 Fix pet leveling logic. Patch by click.
--HG--
branch : trunk
2010-04-05 13:45:35 +02:00
thenecromancer 1fa2fbc658 Fix wrong condition in previous commit, sorry for that
--HG--
branch : trunk
2010-04-05 13:18:14 +02:00
thenecromancer a1db8825d4 Use unit target by default (script targets will send self flag usualy)
Fixes quite some new issues like #1466

--HG--
branch : trunk
2010-04-05 13:15:21 +02:00
thenecromancer fc3a1a3a21 Merge
--HG--
branch : trunk
2010-04-05 13:09:57 +02:00
thenecromancer 0218e45565 Always add CC unit state to allow restoring CC effects if one expires (stun in case of root/confuse, and fear in case of everything else)
Fixes issue #1468

--HG--
branch : trunk
2010-04-05 13:09:28 +02:00
XTZGZoReX 7c277c87d8 * Only call Database::ThreadStart()/ThreadEnd() once per thread.
* Some cleanups in MapUpdater/DelayExecutor.

--HG--
branch : trunk
2010-04-05 13:08:47 +02:00
silverice a795a1110c probably fix crash with boss_paletressAI::MovementInform
to our script writers: please, turn on the brain

--HG--
branch : trunk
2010-04-05 13:50:13 +03:00
[email protected]> ce1280627d Add option maxPercentHealth to item_required_target to allow setting health percentage a target must be at before item use on item_required_target is allowed. This will allow better scripting of quest item use on some creatures.
--HG--
branch : trunk
2010-04-05 02:05:26 -02:30
[email protected]> 2e6ca81365 Fix typo
--HG--
branch : trunk
2010-04-03 22:48:11 -02:30
[email protected]> 848dafd3ac Add MaxAffectedTargets=1 to shoot spells 25425,25761,42611
--HG--
branch : trunk
2010-04-03 22:40:07 -02:30
silverice 5dfcccdd85 correct movement packet broadcasting
mover should be source of movement packets, not player
   resolves problem that movements of controlled unit not visible for nearby players
   (you can detect it only when controlled unit too far from player-controller)

--HG--
branch : trunk
2010-04-04 03:46:38 +03:00
Trazom62 917193eda5 Fix Fishing skill in low level zones, where the zone_skill is < 0. The formula was not correct for those cases.
Fixes isssue #1387.

Note: junk loot is still not yet implemented.

--HG--
branch : trunk
2010-04-04 00:30:11 +02:00
[email protected]> 85d3dc8b71 Valiance keep Cannoneer script to activate cannon.
--HG--
branch : trunk
2010-04-03 14:01:00 -02:30
Trazom62 bcfd44f978 Fix Script HoL/Ionar.
Fixes issue #1132.

--HG--
branch : trunk
2010-04-03 16:51:56 +02:00
silverice 1f477b49c3 handle CMSG_REQUEST_VEHICLE_SWITCH_SEAT opcode
thanks to havenard for packet structure, closes issue #1439
also added vehicle guid check

--HG--
branch : trunk
2010-04-03 16:59:52 +03:00
Trazom62 6917c057de Fix compile.
Fixes issue #1443.

--HG--
branch : trunk
2010-04-03 14:17:33 +02:00
Machiavelli 2708b757e8 Fix SMSG_GMTICKET_GETTICKET structure, by havenard. Closes issue #1419
--HG--
branch : trunk
2010-04-03 13:41:28 +02:00
thenecromancer 18b30b5f18 Fix periodic heal log (correctly display critical heal tick)
--HG--
branch : trunk
2010-04-03 12:43:36 +02:00
thenecromancer 71082bd48b Change pull effects speed calculations to work more or less correctly ( no more flying really high )
--HG--
branch : trunk
2010-04-03 12:41:12 +02:00
thenecromancer aa31f350b3 Fix waypoint movement removing flying animation for creatures
Fix flying creatures always moving forward ( maybe use flag only when really moving? )

--HG--
branch : trunk
2010-04-03 12:39:02 +02:00
thenecromancer e50b04f657 Fix issue #1437, don't remove passive auras if they're triggering spell that fails for whatever reason
--HG--
branch : trunk
2010-04-03 12:32:11 +02:00
Trazom62 6b6ee726dd Script Instance Pit of Saron.
First basic script based on proposal of Synric on trinitycore forum (submitted by Skarabex on GC).
Still WiP with lots of work. In particular, the faction based npc/mob will have to be fixed when they will be spawned in TDB.
Fixes issue #1308.

--HG--
branch : trunk
2010-04-03 09:04:02 +02:00
Trazom62 c6b8602619 Fix Hymn of Hope. Thanks Proof.
Fixes issue #1322.
Fixes issue #1428.

--HG--
branch : trunk
2010-04-03 08:22:04 +02:00
Machiavelli 6dedf65e3e Fix calculation of partial resistance cases, by Tlexii.
--HG--
branch : trunk
2010-04-03 03:59:41 +02:00
Anubisss 08b4748e62 Use GUIDs instead of "global" pointers in zone scripts.
This commit is like that: 7714 / rce505237e4aa

And use const_iterators where it is possible.

--HG--
branch : trunk
2010-04-02 15:41:37 +02:00
[email protected]> 7c4f2f520f Fix Go type 10 spell handling by QAston
--HG--
branch : trunk
2010-04-01 20:08:22 -02:30
[email protected]> 3b389b8aa2 Spells 42442 & 45863 should be limited to 1 target.
--HG--
branch : trunk
2010-04-01 18:14:04 -02:30
Anubisss f78b766705 Use ToTempSummon() in a case instead of a C style cast.
Use correct Unit::GetCreature() instead of Unit::GetUnit() with casting the object to Creature pointer.
Remove some usless casts.
Fix a possible crash in pyrewood_ambushAI.

TODO: Remove ugly cast macros(like CAST_PLR(), CAST_CRE()) from scripts and use the correct ToX() functions(like ToPlayer(), ToCreature()).

--HG--
branch : trunk
2010-04-01 19:09:56 +02:00
Anubisss ab4c7cbd98 Fix a typo in friends list and ignore list loading.
This fixes the bugs which appear when you have more than 50 entries on friends or ignore list.
For example: You have 40 friends and 20 ignores, it's 60. But the core loads just 50 entries so some of them not loaded.
Fixes issue #134.

Patch by johnholiver.

--HG--
branch : trunk
2010-04-01 17:28:59 +02:00
Trazom62 e280d4e45b Fix Script Grand Magus Telestra to reappear when the 3 adds are killed.
Fixes issue #1382.

--HG--
branch : trunk
2010-03-31 00:12:49 +02:00
Trazom62 7cc192e973 Fix Skinning where there is no loot on creature.
Fixes issue #1174.

--HG--
branch : trunk
2010-03-30 23:39:18 +02:00
[email protected]> 83500cb1e4 Added SCRIPT_COMMAND_MODEL 32 Allows changing of npc model in db scripts.
--HG--
branch : trunk
2010-03-30 19:01:25 -02:30
Anubisss 82686437fb Fix a typo in Keristrasza's script. Patch by havenard, #1373.
--HG--
branch : trunk
2010-03-30 21:38:03 +02:00
Anubisss b66f312888 Break spells on damage in some aura types.
This commit fixes spell Repentance(20066).

Patch by QAston, thanks to him.

--HG--
branch : trunk
2010-03-30 15:49:59 +02:00
[email protected]> 66f5d749c0 Added script command 31 SCRIPT_COMMAND_EQUIP inables an npc to change weapons in TC scripts.
--HG--
branch : trunk
2010-03-29 17:45:08 -02:30
XTZGZoReX d3fdb24f1e * Remove an old dumb hack.
--HG--
branch : trunk
2010-03-29 21:47:07 +02:00
Trazom62 c1678a9ab8 Fix crash in Mal'Ganis script + cleanup.
Fixes issue #1347. Thanks Johnholiver.

--HG--
branch : trunk
2010-03-28 11:06:19 +02:00
Tartalo 756e3e3bec Merge
--HG--
branch : trunk
2010-03-27 23:51:33 +01:00
Tartalo 691a1f3974 Fix typo, thanks [email protected]
Fixes issue #1341.
Fixes issue #1344.

--HG--
branch : trunk
2010-03-27 23:50:44 +01:00
Tartalo 69579e38a5 Nexus, The Oculus: Cleanup
--HG--
branch : trunk
2010-03-27 22:32:17 +01:00
Tartalo 63bfba39a8 Nexus, Nexus: Cleanup
--HG--
branch : trunk
2010-03-27 22:10:18 +01:00
XTZGZoReX 2c3aaee502 * Remove unused config option Rate.XP.PastLevel70.
--HG--
branch : trunk
2010-03-27 20:26:35 +01:00
Trazom62 d02f303b8d Implement Script CoS. Thanks Johnholiver.
Fixes issue #336.

Requires DB update.
Still WiP but functional.

--HG--
branch : trunk
2010-03-27 18:39:22 +01:00
Trazom62 eba6ae5ffd Fix crash in Player::DuelComplete.
Fixes issue #1243.

--HG--
branch : trunk
2010-03-27 16:23:43 +01:00
Anubisss d9de0874ae Use ToTotem() in a case.
--HG--
branch : trunk
2010-03-27 16:21:37 +01:00
Anubisss ce8cc85dfa Make a function(ToTempSummon()) which can convert Creature to TempSummon with a C++ cast(reinterpret_cast) and with a type check(isSummon()).
This function is like ToPlayer(), ToTotem(), etc.

This commit is like ede831bdd6f0cff481acc33f269fa7f8c78befd4 / r7496:
http://code.google.com/p/trinitycore/source/detail?r=ede831bdd6f0cff481acc33f269fa7f8c78befd4
As a result, this commit has the advantages and the "disadvantages" too, like r7496.

Please try to be polite if this commit causes some crashes.

--HG--
branch : trunk
2010-03-27 16:08:14 +01:00
Machiavelli 769f54eedf Fix compiler warning in mob_crystal_handlerAI
--HG--
branch : trunk
2010-03-27 15:11:14 +01:00
Machiavelli b9927ca559 Implement usage of RewardArenaPoints field in quest_template
--HG--
branch : trunk
2010-03-27 15:10:26 +01:00
Machiavelli 9ce6094dce Partial implementation of instance system patch by scarymovie87, fix max players check in instance. Cause of error was a pointer of class Player was casted to class InstanceMap. Refs issue #1323
--HG--
branch : trunk
2010-03-27 15:09:26 +01:00
Anubisss be739e9900 Use GUIDs instead of "global" pointers in ~all boss/instance scripts.
GUIDs are most safer than pointers and in some cases can avoid segmentation faults.

== - ==

Let me show a simple example:

We have a pointer to a creature: Creature *pCreature;

and try to search a creature and store it in the pointer:
pCreature = SearchCreature(ID);

We can check simply if the creature found or not:
if(pCreature)
- if this give true: creature found and we have a memory address which points to the creature
- if this give false: creature not found and this is a NULL pointer

Suppose that we have a valid pointer, so creature found.
We wanna use that pointer somewhere, for example in script phase 3.
But we delete the creature(from world) in phase 1.
In phase 3:
if(pCreature)
    pCreature->DoSomeShit();
The if condition gives true, but pCreature points to an _invalid_ address.
In that address maybe have "nothing" or another creature's object or a player or anything else, dont know that.

~~ - ~~

Let me show the correct way:
Creature's GUID: uint64 CreatureGUID;

search the creature and set the GUID:
Creature *pCreature = SearchCreature(ID);
CreatureGUID = pCreature ? pCreature->GetGUID() : 0; // if pCreature is NULL set the GUID to 0

So we have a GUID and not a pointer.
We can get the creature from that GUID:
Creature *pCreature = GetCreature(CreatureGUID);
and we can simply check it is valid or not(found or not):
if(pCreature)

So we deleted the creature in phase 1 and try to use it in phase 3:
Creature *pCreature = GetCreature(CreatureGUID);
And this gives NULL because GetCreature() can't find the creature which GUID is CreatureGUID.

if(pCreature) // pCreature is NULL
    pCreature->DoSomeShit(); // DoSomeShit() not called

== - ==

Remove some not used variables.
Some clean.

TODO: Should search/fix these in zones scripts.

--HG--
branch : trunk
2010-03-27 13:54:38 +01:00
Trazom62 099ef2a9b6 Fix InstanceData::DoCompleteAchievement (and crash in AchievementMgr::CompletedAchievement).
Remove support of no longer existing achievement "The Party is Over". Thanks JuliuSZS.
Fixes issue #1318.

--HG--
branch : trunk
2010-03-27 13:45:39 +01:00
Trazom62 eedd43efa5 Fix UT/Ingvar the plunderer script.
Fixes issue #1034.

- Remove setDisplayID hack and use UpdateEntry to fix Ingvar Kill Credit related achievements.
- code style cleaning.

Requires DB update to set loot from Ingvar Human (23954) to Undead (23980) (in normal and heroic).

--HG--
branch : trunk
2010-03-27 13:14:13 +01:00
Xanadu 180651d95e A visually better way to handle Avenger Wrath blocking of immunity shields.
--HG--
branch : trunk
2010-03-26 22:21:33 +01:00
Spp cd372966af * Add support to save and load worldstates.
will be mainly used by outdoorPvP system
* Update next arena auto distribute points time to custom worldstate

--HG--
branch : trunk
2010-03-26 16:48:33 +01:00
teacher afbb5f2e3c Moved sql update files to sql\updates\3.3.2_old.
--HG--
branch : trunk
2010-03-26 14:40:09 +01:00
Xanadu 68a19906c8 Renamed the SQLs to reflect correct revision after the merges.
--HG--
branch : trunk
rename : sql/updates/7692_world_creature_template.sql => sql/updates/7694_world_creature_template.sql
rename : sql/updates/7692_world_gameobject_template.sql => sql/updates/7694_world_gameobject_template.sql
rename : sql/updates/7692_world_item_template.sql => sql/updates/7694_world_item_template.sql
rename : sql/updates/7692_world_npc_text.sql => sql/updates/7694_world_npc_text.sql
rename : sql/updates/7692_world_page_text.sql => sql/updates/7694_world_page_text.sql
rename : sql/updates/7692_world_quest_template.sql => sql/updates/7694_world_quest_template.sql
rename : sql/updates/7694_world_npc_trainer.sql => sql/updates/7697_world_npc_trainer.sql
rename : sql/updates/7694_world_npc_vendor.sql => sql/updates/7697_world_npc_vendor.sql
2010-03-25 17:19:19 +01:00
Xanadu d0c10d44b8 Merge
--HG--
branch : trunk
2010-03-25 17:09:45 +01:00
Xanadu 20de79f6ba Fixed Forbearance. Also self-cast LoH will now correctly prevent Avenging Wrath for 30 s. Closes issue #1270. Thanks to _manuel_ for research.
--HG--
branch : trunk
2010-03-25 17:09:05 +01:00
Tartalo 7d6923d8c7 BlackRock Depths: Some improvements, by antihrists
Fix Tomb of Seven event
Fix Shadowforge braziers
GO handling for Magmus

--HG--
branch : trunk
2010-03-25 13:33:49 +01:00
Tartalo 5bd9bfcdda Merge
--HG--
branch : trunk
2010-03-25 10:34:41 +01:00
Tartalo 080229fec3 Gundrak, Gal'Darah: Script improvements by pivonroll
Fix some Spell IDs & timers
Implement transformation

--HG--
branch : trunk
2010-03-25 10:32:07 +01:00