Code: Select all
drop table wat_cache;
drop table armory_proxy_cache;
CREATE TABLE `wat_cache` (
`cache_id` VARCHAR (100) NOT NULL,
`tooltip` TEXT,
`last_update` INT (11) UNSIGNED NOT NULL,
PRIMARY KEY `cache_id` (`cache_id`)
);
CREATE TABLE `armory_proxy_cache` (
`cache_id` VARCHAR (256) NOT NULL UNIQUE,
`data` BLOB,
`last_update` INT (11) UNSIGNED NOT NULL,
PRIMARY KEY `cache_id` (`cache_id`)
);
Excellent work, Furii - just one small thing I'd like to check with you before I start debugging. Characters who don't have dual talent specialisation aren't showing any talents on my setup (the line in the tooltip is blank). Is that something you've encountered and patched?furii wrote:I had to fix a minor formatting issue so I repackaged it without the config.php
Code: Select all
if ($info && !($info[0])) {
$info[0] = $info;
}
Code: Select all
private function generate_talent_info ()
{
$info = $this->armory_data['charactertab']['talentspecs']['talentspec'];
if ($info && !($info[0])) {
$info[0] = $info;
}
for ($i = 0; $i <= 1; $i++) {
if ($info[$i]) {
Weird - I checked and the changes you made were in the code. I cleared the cache and repopulated it with a single-spec character and it looks as if the talent itself isn't being populated (code below). Since it's only on my setup I'll delve further into what I'm getting from the server. Many thanks for your help and improvements to the code!furii wrote:Actually it is. I thought I had uploaded that version. The XML parser will automatically create an array instead of a scalar variable if there is more then one talent spec. The problem is that my code is just looking for that array. My simple work around is to create the array with a single element from the scalar if the array doesn't exist. I added:
Code: Select all
<...bits chopped out...>
<td class="wat_tt_char_info">
<span class="wat_tt_char_name">Longbraz, Champion of the Naaru</span>
<span class="wat_tt_char_guild"><Argent Dawn Exiles></span>
<span class="wat_tt_char_stuff">Level 80 Night Elf Druid</span>
<div class="wat_tt_talents">
</div>
</td>
<...bits chopped out...>
mmags wrote:furii,
does your update support Death Knights (link color and complete tooltips)?
That's odd. I believe I only modified three files for dual spec support. You may want to grab my v2 tarball and replace those files or diff them to see if I changed anything between the time you downloaded and now.MarcusWatson wrote:furii wrote:
Weird - I checked and the changes you made were in the code. I cleared the cache and repopulated it with a single-spec character and it looks as if the talent itself isn't being populated (code below). Since it's only on my setup I'll delve further into what I'm getting from the server. Many thanks for your help and improvements to the code!