Add Wowhead tooltips to your forums or CMS.

Post by 171689

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

Post by Talonos

tried your fix and no luck with it for me

still open to ideas thou

Post by Lockslap

the newest version of the script supports php4 and php5 using different XML parsers depending on the version you're using.

most likely you guys are downloading the old version from my former guild's website. in order to get the restored PHP4 support, you must download the newest version (2.0.1 BETA) from http://wowheadtooltips.googlecode.com/files/wowhead-2.0.1beta.zip.

i may be picking back up support for the script...i haven't completely decided yet.

Post by stressball

I hope you're back to check it out, craCkpot. How's the raiding going?
It seems like I have smiliar problems to others here, but I can't use any of the fixes they discuss here.

If I install the script according to your instructions, I get this error;
PHP Notice: in file /wowhead/parse.php on line 61: mysql_num_rows(): supplied argument is not a valid MySQL result resource

Also, this one here shows up;
General Error
SQL ERROR

Incorrect integer value: '' for column 'rank' at row 1

If I try to edit the parse.php file, as said earlier in the thread (on page 9 by mastertarget?), I get the item links to show, but I get some other errors;
PHP Notice: in file /wowhead/parse.php on line 61: mysql_num_rows(): supplied argument is not a valid MySQL result resource
PHP Notice: in file /includes/functions.php on line 3526: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2956)
PHP Notice: in file /includes/functions.php on line 3528: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2956)
PHP Notice: in file /includes/functions.php on line 3529: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2956)
PHP Notice: in file /includes/functions.php on line 3530: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2956)

Any help would be much appreciated! :D

Post by Lockslap

The reason for your error is that you are using MySQLi for your database connection. Download the most recent version (2.0.1 BETA), where the script will automatically determine whether you're using MySQL or MySQLi for its DB connection. It also cleans up the MySQL code a bit in order to make them compatible with either version.

Hope this helps.

And the raiding is going quite well, we're progressing into the later stages of Hyjal/BT.

Post by stressball

Thanks for the input, craCkpot, I'll look into it. Allthough, I am very sure I am using the 2.0.1 BETA allready, but I'll go through the installation process one more time. Hope it helps. Thanks again!

Post by stressball

Post by Lockslap

Nope. After a clean install, I still get the general error from my previous post.

Is there any information I can provide or something I can do to make this work?

Here's some info;
PHP Version 5.2.6

Database server: MySQL(i) 5.0.32-Debian_7etch5-log
Board version: 3.0.1

mysql
MySQL Support enabled
Active Persistent Links 0
Active Links 0
Client API version 4.1.11

MysqlI Support enabled
Client API library version 4.1.11
Client API header version 4.1.11

Okay so, the first error you posted about was a bone-headed mistake on my part. Basically, the variable $dbms is stored in phpBB's config file and stores the type of DB you use (for our purposes its 'mysql' or 'mysqli'). I was using this variable inside of parse.php to determine which function to use in order to get the results of our num_rows query. Well, the problem arises because $dbms would always return empty because I neglected to set it as a global variable. Oops?

I am fairly certain that I have fixed the issue related to the MySQL(i) queries by determining the value of rank and then setting it appropriately. I do not have MySQLi on any of my webservers, so I cannot test it. But install the version that I will link below and let me know how it works. (To do the update you actually only need to replace parse.php as it is the only change in this minor update).

Grab it from here.

Let me know how it works.

Post by leridder

Erreur générale
SQL ERROR

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 ' 'item' )' at line 16

Une erreur SQL est survenue lors du chargement de la page. Veuillez contacter l’administrateur du forum si ce problème persiste.

Hi, i was having the same issue.
The new parse.php solves the previous issue but ... a new one appears ;)

Thx for the great work :)

(my config.php is set to FR )

Post by stressball

I reuploaded the parse.php file, and that rid the first error.

Now I only get a new general error;
General Error
SQL ERROR

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 ' 'item' )' at line 16

It doesn't say (or it's at least not clear to me) what file it is, else I could have checked out line 16 myself.
If you have any ideas, I'd love to hear them! :D

Thanks again!

Post by Lockslap

Okay, I am fairly certain that I have the error narrowed down to a problem concerning the fix I issued last night with the MySQL query after determining the rank of a spell (whether set or not). So a quick fix for this is to use different queries if the rank is set or not. Therefore, I am proud to announce the release of version 2.0.3beta. As with 2.0.2, you only have to update parse.php, everything else remains the same.

Download: http://wowheadtooltips.googlecode.com/files/wowhead-2.0.3beta.zip

Let me know how it goes.

Post by stressball

I appreciate the effort, craCkpot! :)

It's just that...
SQL ERROR

Field 'rank' doesn't have a default value

I feel it's SOOOO close... ;)

Post by Lockslap

Okay the latest fix should take care of it.

Download: http://wowheadtooltips.googlecode.com/files/wowhead-2.0.4beta.zip

Again you only have to replace parse.php, blah blah blah.

Post by 172509

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

Post by Lockslap

Okay, for the next round of fixes, it seems that MySQLi is FAR more strict when it comes to its queries. So, I decided to rewrite the way the script builds queries to MySQL, and hopefully in the process fix the errors that arose.

Download: http://wowheadtooltips.googlecode.com/files/wowhead-2.0.5beta.zip

Once again, you only need to replace parse.php, nothing else has been changed.

Once the immediate bugs are worked out, I will release the script and take it out of BETA.

Post by 172509

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

Post by stressball

I downloaded the 2.0.5 beta and yes, I got the NULL not allowed error (concerning the rank field). I removed the 'NOT NULL' part from the .sql file, rerun create_table.php and now it works.

Let me know if you want me to test a final set of files before releasing, craCkpot.

edit: When I try to link spell Backstab (rank 3) as 2590, I get SQL ERROR

Column 'quality' cannot be null also.

Post by 169528

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

Post by Lockslap

I tryd every Beta, but i always have the problem of "the White Screen". That issnt a style problem, perhaps a php 4 problem. Current Version is PHP Version 4.3.10.

Under http://www.rkettenbeil.de/testforum you can try zu open a new Topic without Registration in the Forum "Bewerbungsraum". Mouseover and BBcodes are correkt, only when anyone will post a white Site will open.

I have tryed only in file "functions_content.php" the part "require_once($phpbb_root_path . '/wowhead/parse.php');" to include. At this moment the Problem has come and so i think it could be a problem in file parse.php.
I tested things, and it seems to be working properly. I will update the table_scheme.sql file to remove the NOT NULL. You will have to clear your cache and then it should work properly.

Sorry for all of the bugs, I don't have the ability to test any of the changes that I make.

UPDATE Okay I have adjusted table_scheme.sql to remove the NOT NULL clause. In order to make it work you must overwrite (or upload) table_scheme.php and then run create_table.php. This will clear your cache and setup the new database table for you. After that you should be good to go (::crosses fingers::).

Download: http://wowheadtooltips.googlecode.com/files/wowhead-2.0.6beta.zip

Post by 169528

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

Add Wowhead tooltips to your forums or CMS.

Post Reply