Spell: fixed modelviewer and spellEffect-listing, when spell had multiple models set (e.g. aura:shapeshift AND aura:transform)

This commit is contained in:
Sarjuuk
2015-06-26 18:54:12 +02:00
parent a6e49e3643
commit 61bc839d2f
2 changed files with 84 additions and 52 deletions

View File

@@ -408,12 +408,12 @@ class SpellList extends BaseType
return $this->tools; return $this->tools;
} }
public function getModelInfo($targetId = 0) public function getModelInfo($spellId = 0, $effIdx = 0)
{ {
$displays = [0 => []]; $displays = [0 => []];
foreach ($this->iterate() as $id => $__) foreach ($this->iterate() as $id => $__)
{ {
if ($targetId && $targetId != $id) if ($spellId && $spellId != $id)
continue; continue;
for ($i = 1; $i < 4; $i++) for ($i = 1; $i < 4; $i++)
@@ -425,14 +425,18 @@ class SpellList extends BaseType
// GO Model from MiscVal // GO Model from MiscVal
if (in_array($this->curTpl['effect'.$i.'Id'], [50, 76, 104, 105, 106, 107])) if (in_array($this->curTpl['effect'.$i.'Id'], [50, 76, 104, 105, 106, 107]))
{ {
$displays[TYPE_OBJECT][$id] = $effMV; if (isset($displays[TYPE_OBJECT][$id]))
break; $displays[TYPE_OBJECT][$id][0][] = $i;
else
$displays[TYPE_OBJECT][$id] = [[$i], $effMV];
} }
// NPC Model from MiscVal // NPC Model from MiscVal
else if (in_array($this->curTpl['effect'.$i.'Id'], [28, 90, 134]) || in_array($this->curTpl['effect'.$i.'AuraId'], [56, 78])) else if (in_array($this->curTpl['effect'.$i.'Id'], [28, 90, 134]) || in_array($this->curTpl['effect'.$i.'AuraId'], [56, 78]))
{ {
$displays[TYPE_NPC][$id] = $effMV; if (isset($displays[TYPE_NPC][$id]))
break; $displays[TYPE_NPC][$id][0][] = $i;
else
$displays[TYPE_NPC][$id] = [[$i], $effMV];
} }
// Shapeshift // Shapeshift
else if ($this->curTpl['effect'.$i.'AuraId'] == 36) else if ($this->curTpl['effect'.$i.'AuraId'] == 36)
@@ -446,17 +450,16 @@ class SpellList extends BaseType
if ($st = DB::Aowow()->selectRow('SELECT *, displayIdA as model1, displayIdH as model2 FROM ?_shapeshiftforms WHERE id = ?d', $effMV)) if ($st = DB::Aowow()->selectRow('SELECT *, displayIdA as model1, displayIdH as model2 FROM ?_shapeshiftforms WHERE id = ?d', $effMV))
{ {
foreach ([1, 2] as $i) foreach ([1, 2] as $j)
if (isset($subForms[$st['model'.$i]])) if (isset($subForms[$st['model'.$j]]))
$st['model'.$i] = $subForms[$st['model'.$i]][array_rand($subForms[$st['model'.$i]])]; $st['model'.$j] = $subForms[$st['model'.$j]][array_rand($subForms[$st['model'.$j]])];
$displays[0][$id] = array( $displays[0][$id][$i] = array(
'npcId' => 0, 'typeId' => 0,
'displayId' => [$st['model1'], $st['model2']], 'displayId' => $st['model2'] ? $st['model'.rand(1, 2)] : $st['model1'],
'creatureType' => $st['creatureType'], 'creatureType' => $st['creatureType'],
'displayName' => Util::localizedString($st, 'name') 'displayName' => Util::localizedString($st, 'name')
); );
break;
} }
} }
} }
@@ -466,34 +469,48 @@ class SpellList extends BaseType
if (!empty($displays[TYPE_NPC])) if (!empty($displays[TYPE_NPC]))
{ {
$nModels = new CreatureList(array(['id', $displays[TYPE_NPC]])); $nModels = new CreatureList(array(['id', array_column($displays[TYPE_NPC], 1)]));
foreach ($nModels->iterate() as $nId => $__) foreach ($nModels->iterate() as $nId => $__)
{ {
$srcId = array_search($nId, $displays[TYPE_NPC]); $srcId = 0;
$results[$srcId] = array( foreach ($displays[TYPE_NPC] as $srcId => $set)
'typeId' => $nId, if ($set[1] == $nId)
'displayId' => $nModels->getRandomModelId(), break;
'displayName' => $nModels->getField('name', true)
); foreach ($set[0] as $idx)
{
$results[$srcId][$idx] = array(
'typeId' => $nId,
'displayId' => $nModels->getRandomModelId(),
'displayName' => $nModels->getField('name', true)
);
}
} }
} }
if (!empty($displays[TYPE_OBJECT])) if (!empty($displays[TYPE_OBJECT]))
{ {
$oModels = new GameObjectList(array(['id', $displays[TYPE_OBJECT]])); $oModels = new GameObjectList(array(['id', array_column($displays[TYPE_OBJECT], 1)]));
foreach ($oModels->iterate() as $oId => $__) foreach ($oModels->iterate() as $oId => $__)
{ {
$srcId = array_search($oId, $displays[TYPE_OBJECT]); $srcId = 0;
$results[$srcId] = array( foreach ($displays[TYPE_OBJECT] as $srcId => $set)
'typeId' => $oId, if ($set[1] == $oId)
'displayId' => $oModels->getField('displayId'), break;
'displayName' => $oModels->getField('name', true)
); foreach ($set[0] as $idx)
{
$results[$srcId][$idx] = array(
'typeId' => $oId,
'displayId' => $oModels->getRandomModelId(),
'displayName' => $oModels->getField('name', true)
);
}
} }
} }
if ($targetId) if ($spellId && $effIdx)
return !empty($results[$targetId]) ? $results[$targetId] : 0; return !empty($results[$spellId][$effIdx]) ? $results[$spellId][$effIdx] : 0;
return $results; return $results;
} }
@@ -736,7 +753,7 @@ class SpellList extends BaseType
if (isset($$var)) if (isset($$var))
{ {
$eval = eval('return @$'.$var.';'); // attention: error suppression active here (will be logged anyway) $eval = eval('return @$'.$var.';'); // attention: error suppression active here (will be logged anyway)
if (getType($eval) == 'object') if (getType($eval) == 'object')
continue; continue;
else if (is_numeric($eval)) else if (is_numeric($eval))
@@ -769,6 +786,7 @@ class SpellList extends BaseType
} }
// since this function may be called recursively, there are cases, where the already evaluated string is tried to be evaled again, throwing parse errors // since this function may be called recursively, there are cases, where the already evaluated string is tried to be evaled again, throwing parse errors
// todo (med): also quit, if we replaced vars with non-interactive text
if (strstr($formula, '</dfn>')) if (strstr($formula, '</dfn>'))
return $formula; return $formula;
@@ -1112,7 +1130,7 @@ class SpellList extends BaseType
if (isset($formula[++$formCurPos]) && $formula[$formCurPos] == '.') if (isset($formula[++$formCurPos]) && $formula[$formCurPos] == '.')
{ {
$formPrecision = (int)$formula[++$formCurPos]; $formPrecision = (int)$formula[++$formCurPos];
++$formCurPos; // for some odd reason the precision decimal survives if wo dont increment further.. ++$formCurPos; // for some odd reason the precision decimal survives if we dont increment further..
} }
$formOutStr = $this->resolveFormulaString($formOutStr, $formPrecision, $scaling); $formOutStr = $this->resolveFormulaString($formOutStr, $formPrecision, $scaling);
@@ -1281,13 +1299,30 @@ class SpellList extends BaseType
// step 2: resolving conditions // step 2: resolving conditions
// aura- or spell-conditions cant be resolved for our purposes, so force them to false for now (todo (low): strg+f "know" in aowowPower.js ^.^) // aura- or spell-conditions cant be resolved for our purposes, so force them to false for now (todo (low): strg+f "know" in aowowPower.js ^.^)
/* sequences
a) simple - $?cond[A][B] // simple case of b)
b) elseif - $?cond[A]?cond[B]..[C] // can probably be repeated as often as you wanted
c) recursive - $?cond[A][$?cond[B][..]] // can probably be stacked as deep as you wanted
only case a) can be used for KNOW-parameter IF, AND ONLY IF it is not containing further variables ($) AND it has a spimple spell-condition ( \(?!?[as]\d+\)? )
_[100].tooltip_enus = '<table><tr><td><b>Charge</b><br />8 - 25 yd range<table width="100%"><tr><td>Instant</td><th>20 sec cooldown</th></tr></table>Requires Warrior<br />Requires level 3</td></tr></table><table><tr><td><span class="q">Charge to an enemy, stunning it <!--sp58377:0--><!--sp58377-->for <!--sp103828:0-->1 sec<!--sp103828-->. Generates 20 Rage.</span></td></tr></table>';
_[100].buff_enus = '';
_[100].spells_enus = {"58377": [["", "and 2 additional nearby targets "]], "103828": [["1 sec", "3 sec and reducing movement speed by 50% for 15 sec"]]};
_[100].buffspells_enus = {};
Turns the Shaman into a Ghost Wolf, increasing speed by $s2%$?s59289[ and regenerating $59289s1% of your maximum health every 5 sec][].
Lasts 5 min. $?$gte($pl,68)[][Cannot be used on items level 138 and higher.]
*/
// \1: full pattern match; \2: any sequence, that may include an aura/spell-ref; \3: any other sequence, between "?$" and "[" // \1: full pattern match; \2: any sequence, that may include an aura/spell-ref; \3: any other sequence, between "?$" and "["
while (preg_match('/\$\?(([\W\D]*[as]\d+)|([^\[]*))/i', $data, $matches)) while (preg_match('/\$\?(([\W\D]*[as]\d+)|([^\[]*))/i', $data, $matches))
{ {
$condBrktCnt = 0; $condBrktCnt = 0;
$targetPart = 3; // we usually want the second pair of brackets $targetPart = 3; // we usually want the second pair of brackets
$curPart = 0; // parts: $? 0 [ 1 ] 2 [ 3 ] 4 $curPart = 0; // parts: $? 0 [ 1 ] 2 [ 3 ] 4
$relSpells = []; // see spells_enus $relSpells = []; // see spells_enus
$condOutStr = ''; $condOutStr = '';
@@ -1301,14 +1336,6 @@ class SpellList extends BaseType
$condCurPos = $condStartPos; $condCurPos = $condStartPos;
} }
/*
_[100].tooltip_enus = '<table><tr><td><b>Charge</b><br />8 - 25 yd range<table width="100%"><tr><td>Instant</td><th>20 sec cooldown</th></tr></table>Requires Warrior<br />Requires level 3</td></tr></table><table><tr><td><span class="q">Charge to an enemy, stunning it <!--sp58377:0--><!--sp58377-->for <!--sp103828:0-->1 sec<!--sp103828-->. Generates 20 Rage.</span></td></tr></table>';
_[100].buff_enus = '';
_[100].spells_enus = {"58377": [["", "and 2 additional nearby targets "]], "103828": [["1 sec", "3 sec and reducing movement speed by 50% for 15 sec"]]};
_[100].buffspells_enus = {};
Turns the Shaman into a Ghost Wolf, increasing speed by $s2%$?s59289[ and regenerating $59289s1% of your maximum health every 5 sec][].
Lasts 5 min. $?$gte($pl,68)[][Cannot be used on items level 138 and higher.]
*/
else if (!empty($matches[2])) else if (!empty($matches[2]))
{ {
@@ -1780,11 +1807,16 @@ Lasts 5 min. $?$gte($pl,68)[][Cannot be used on items level 138 and higher.]
if ($addInfoMask & ITEMINFO_MODEL) if ($addInfoMask & ITEMINFO_MODEL)
{ {
if (!empty($modelInfo[$this->id])) // may have multiple models set, in this case i've no idea what should be picked
for ($i = 1; $i < 4; $i++)
{ {
$data[$this->id]['npcId'] = $modelInfo[$this->id]['typeId']; if (!empty($modelInfo[$this->id][$i]))
$data[$this->id]['displayId'] = $modelInfo[$this->id]['displayId']; {
$data[$this->id]['displayName'] = $modelInfo[$this->id]['displayName']; $data[$this->id]['npcId'] = $modelInfo[$this->id][$i]['typeId'];
$data[$this->id]['displayId'] = $modelInfo[$this->id][$i]['displayId'];
$data[$this->id]['displayName'] = $modelInfo[$this->id][$i]['displayName'];
break;
}
} }
} }
} }

View File

@@ -1639,9 +1639,9 @@ class SpellPage extends GenericPage
case 90: // Kill Credit case 90: // Kill Credit
case 134: // Kill Credit2 case 134: // Kill Credit2
$_ = Lang::game('npc').' #'.$effMV; $_ = Lang::game('npc').' #'.$effMV;
if ($summon = $this->subject->getModelInfo($this->typeId)) if ($summon = $this->subject->getModelInfo($this->typeId, $i))
{ {
$_ = '(<a href="?npc='.$summon['typeId'].'">'.$summon['displayName'].'</a>)'; $_ = $summon['typeId'] ? ' (<a href="?npc='.$summon['typeId'].'">'.$summon['displayName'].'</a>)' : ' (#0)';
$redButtons[BUTTON_VIEW3D] = ['type' => TYPE_NPC, 'displayId' => $summon['displayId']]; $redButtons[BUTTON_VIEW3D] = ['type' => TYPE_NPC, 'displayId' => $summon['displayId']];
} }
@@ -1691,9 +1691,9 @@ class SpellPage extends GenericPage
case 106: // Summon Object (slot 3) case 106: // Summon Object (slot 3)
case 107: // Summon Object (slot 4) case 107: // Summon Object (slot 4)
$_ = Util::ucFirst(Lang::game('gameObject')).' #'.$effMV; $_ = Util::ucFirst(Lang::game('gameObject')).' #'.$effMV;
if ($summon = $this->subject->getModelInfo($this->typeId)) if ($summon = $this->subject->getModelInfo($this->typeId, $i))
{ {
$_ = '(<a href="?object='.$summon['typeId'].'">'.$summon['displayName'].'</a>)'; $_ = $summon['typeId'] ? ' (<a href="?object='.$summon['typeId'].'">'.$summon['displayName'].'</a>)' : ' (#0)';
$redButtons[BUTTON_VIEW3D] = ['type' => TYPE_OBJECT, 'displayId' => $summon['displayId']]; $redButtons[BUTTON_VIEW3D] = ['type' => TYPE_OBJECT, 'displayId' => $summon['displayId']];
} }
@@ -1810,11 +1810,11 @@ class SpellPage extends GenericPage
break; break;
case 36: // Shapeshift case 36: // Shapeshift
if ($st = $this->subject->getModelInfo($this->typeId)) if ($st = $this->subject->getModelInfo($this->typeId, $i))
{ {
$redButtons[BUTTON_VIEW3D] = array( $redButtons[BUTTON_VIEW3D] = array(
'type' => TYPE_NPC, 'type' => TYPE_NPC,
'displayId' => $st['displayId'][1] ? $st['displayId'][rand(0, 1)] : $st['displayId'][0] 'displayId' => $st['displayId']
); );
if ($st['creatureType'] > 0) if ($st['creatureType'] > 0)
@@ -1966,10 +1966,10 @@ class SpellPage extends GenericPage
break; break;
case 78: // Mounted case 78: // Mounted
case 56: // Transform case 56: // Transform
if ($transform = $this->subject->getModelInfo($this->typeId)) if ($transform = $this->subject->getModelInfo($this->typeId, $i))
{ {
$redButtons[BUTTON_VIEW3D] = ['type' => TYPE_NPC, 'displayId' => $transform['displayId']]; $redButtons[BUTTON_VIEW3D] = ['type' => TYPE_NPC, 'displayId' => $transform['displayId']];
$bar = ' (<a href="?npc='.$transform['typeId'].'">'.$transform['displayName'].'</a>)'; $bar = $transform['typeId'] ? ' (<a href="?npc='.$transform['typeId'].'">'.$transform['displayName'].'</a>)' : ' (#0)';
} }
else else
$bar = Lang::main('colon').Lang::game('npc').' #'.$effMV;; $bar = Lang::main('colon').Lang::game('npc').' #'.$effMV;;