Add Wowhead tooltips to your forums or CMS.

Post by Talonos

]I have changed the guild tooltips to display quite a bit more information. Just some guild stats, nothing major, but it looks good. ;-)

i love the new addition to the guild tooltip m8 :)

Post by 12875

This post was from a user who has deleted their account.

Post by Talonos

Getting these errors when either using the armory tags for a char that isn't cached yet (I think, after a refresh I don't get them anymore) or using them on a char that doesn't exist.
http://www.vexillum-guild.com/forum/tooltiperrors.txt
This with version 3.4.4.

Lymion what version of php and phpbb are you using ?

Post by 12875

This post was from a user who has deleted their account.

Post by Lockslap

Your error reporting is too sensitive.

Post by Lockslap

I have released a minor fix to the armory and guild modules. We'll call this version 3.4.5. Apostrophes weren't properly being handled in the guild and armory modules, specifically in the realm names. Files to overwrite are listed below.

I also fixed a possible security issue where HTML was being allowed in the bbcode tags. All text will be sent through PHP's strip_tags() before being sent to the modules.

Download: Universal or Joomla

Files to Overwrite:

./parse.php
./includes/wowhead_guild.php
./includes/wowhead_armory.php
./includes/wowhead_cache.php
./patterns/armory_tooltip.php
./patterns/armory_guild.php

You do not need to overwrite config.php nor run create_table.php. The SQL tables are the same.

Post by 12875

This post was from a user who has deleted their account.

Post by Lockslap

Your error reporting is too sensitive.
Hmm, ok.. Any idea how I can change that?

Before we go changing anything let me see if I can figure out why it is reporting the notices.

Post by xodium

Hello,

I get the following error when trying

PHP Notice: in file /wowhead/includes/wowhead_armory_guild.php on line 126: Invalid argument supplied for foreach()
PHP Notice: in file /includes/functions.php on line 3824: Cannot modify header information - headers already sent by (output started at /includes/functions.php:3247)
PHP Notice: in file /includes/functions.php on line 3826: Cannot modify header information - headers already sent by (output started at /includes/functions.php:3247)
PHP Notice: in file /includes/functions.php on line 3827: Cannot modify header information - headers already sent by (output started at /includes/functions.php:3247)
PHP Notice: in file /includes/functions.php on line 3828: Cannot modify header information - headers already sent by (output started at /includes/functions.php:3247)

I have 3.4.5 installed, the rest of the features (including wowhead profiles) work fine.

Post by Lockslap

By the way, it is generally a security risk to have PHP to report notices. You should only have it report warnings and fatal errors.

By default PHP doesn't report notices, so there's something changing it. To change it back open phpArmory.class.php and after the "<?php" tag put:

error_reporting(E_ALL ^ E_NOTICE);

Then save it and reupload to your site. This will report all errors except notices.

Now that my script only supports PHP 5 I am going to eventually rewrite the armory and guild modules to use SimpleXML. Currently I am looking into using PEAR for handling the DB functions.

Post by Lockslap

The site may be flooky today, I am moving the site over to Wordpress to handle the layout and CMS shizzle.

You can get the files from http://wowhead-tooltips.com/files/ until I get the download plugin written.

Post by 339971

This post was from a user who has deleted their account.

Post by Lockslap

Notice: Trying to get property of non-object in /mounted-storage/home114b/xxx/xxx-xxx/xxx/xxx/xxx/wowhead/includes/wowhead_craft.php on line 90

Warning: Invalid argument supplied for foreach() in /mounted-storage/home114b/xxx/xxx-xxx/xxx/xxx/xxx/wowhead/includes/wowhead_craft.php on line 90
Failed to add Pattern: Spellslinger's Slippers to the cache. You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' '')' at line 1

INSERT INTO wowhead_craftable_spell VALUES (45103, , '')

So far this is the only one i have problems with

It could be a bug in the code, so if you could post the BBCode you used when it produced that error, I can take a look.

On a side now, I have setup a separate Twitter account for this script: @wowheadtooltips

Also, the downloads are now up on the script's new site: http://wowhead-tooltips.com/downloads

Post by Lockslap

Installation instructions have been moved over. http://wowhead-tooltips.com/install/
A FAQ was created as well. http://wowhead-tooltips.com/faq/

Post by 12875

This post was from a user who has deleted their account.

Post by Lockslap

Yeah when you overload the armory and it blocks you, then it can sometimes generate an error. I believe I have fixed it, I will roll out the code in a day or so.

Post by Lockslap

I am in the process of writing a Site Database plugin for the site to display all the sites that are currently using this script. If you wish to have your site added then send an e-mail to sites@wowhead-tooltips.com with the following format:Name: <Guild Name>
URL: <Your Guild's Website URL>
Region: <Your Guild's Region>
Realm: <Your Guild's Realm>
Faction: <Alliance or Horde>

So for example below would be my guild:Name: Twisted
URL: http://www.twistedguild.info
Region: US
Realm: Bleeding Hollow
Faction: Alliance

I am toying with the idea of making the script automatically add sites by reading the e-mails it sees. So if you wish to have your site added then shoot an email to sites@wowhead-tooltips.com with the above format and I will get you added.

The Site DB's home will eventually be: http://wowhead-tooltips.com/sitedb/

UPDATE
Okay now if you send an e-mail to sites@wowhead-tooltips.com with the correct format from above it will add your site to the database automatically and send you back an e-mail saying it succeeded. If it fails, then you won't get an e-mail back. So if you do it but don't receive a response then you formatted it improperly. I will change it at some point to tell you what you did wrong, but I'm lazy to leave me alone. I have to finish the front-end of the site db plugin, all the admin portions are done.

Post by Lockslap

Please add your guild's, or your site to the database, I would like to get a comprehensive list going of those using my script. Me love you long time. <3

Post by eqsanctum

Thank you for the quick fix and update.... however, I still haven't found what I'm looking for. I see the intended use of BBcodes, I see direct script installation instructions of the files and editing. I however see nothing on what to put in the posting section of the ACP to enable use of the BBcodes.

IE:
blah,
before you can use that you need in the posting section of the acp exactly where the bbcode is pointing to. Where are those instructions?

I mean I've read several pages back and understand the editing is yo make it look at the bbcodes automagically but if I wanted to just point it at the script can I do that? If I decided to how could I go about doing it?

Reason I'm asking is I use portal xl 5.0 premod which is fine with a direct wowhead js link but Id rather BBcode if I could.

Add Wowhead tooltips to your forums or CMS.

Post Reply