Hi,

I currently have an API problem fixing a plugin. Using mysql_query() will result in errors on some boards so I wanted to change this using $bbdb->get_results(). This works, but mysql_fetch_array will error things up now. What do I need to change to make this work correctly?

Current code (without using $bbdb->get_results()):

$r = mysql_query("SELECT * FROM $bbdb->menu

WHERE set = 'inactive' ORDER BY order ASC");

while($rw = mysql_fetch_array($r))

[@null Member 17 years, 5 months ago]

Or use:

$bbdb->query ??

[@box87 Member 17 years, 5 months ago]

Stick with the ezSQL format… it’ll be much easier.

Something like this (using foreach instead of while):

$query = "SELECT * FROM $bbdb->menu

WHERE set = 'inactive' ORDER BY order ASC";

$r = $bbdb->get_results( $query );



foreach( $r as $rw ) {

// now mess with $rw

echo $rw->database_row . "<br>n";

}

Something that might be helpful would be to browse the source of the ezSQL class in /wp-includes/wp-db.php

The latest from Justin Vincent, the author

http://www.woyano.com/jv/ezsql

The version used in bbPress and WordPress

https://trac.wordpress.org/browser/trunk/wp-includes/wp-db.php

It might help you… you’d be able to see where mysql_query is begin used, and how to use the ezSQL format best.

I think the reason you’re running into problems with mysql_fetch_array is because it’s already used.

Not sure on that one though… I would have to check.

Also, what might cause an error: $bbdb->query will not return anything at all. $bbdb->get_results needs to be used for multi-dimensional array data, $bbdb->get_row or $bbdb->get_col need to be used for single array data, and $bbdb->get_var needs to be used for non-array/string data. $bbdb->query is used for inserting data, the others are used for retrieving it.

Someone correct me if I’m wrong, but I’m pretty positive that’s how it goes.

ezSQL is lovely. It certainly takes the pain out of MySQL data manipulation.

So I guess that makes the answer to your question “Neither.” $bbdb->get_results should do the trick.

And sorry for the screed.

[@null Member 17 years, 5 months ago]

-double post-

[@null Member 17 years, 5 months ago]

Hi,

Wheren’t you the guy that fixed (or claimed to have fixed) this particulair error I had with the bbMenu plugin?

Thinking… yeah you where it, you said you fixed it using $bbdb->get_results. So I replaced all mysql_query() with it, but this caused an error with the mysql_fetch_array. So what more did you change? Cause I can’t seem to fix it I can hardly program, so some help plz

Thx

[@box87 Member 17 years, 5 months ago]

Give me a day or so and I’ll work on it. I’ve got about three jobs going at the moment.

[@null Member 17 years, 5 months ago]

Okay okay, 1 day (just kidding)

[@maryjane Member 17 years, 4 months ago]

Im having the same issue with this plugin. I switched all my tags to $bbdb->get_results and it stoped the errors. But i never got this plugin to work i just got the errors gone. It never created the table. I manually created the table but i dont know how to populate it with the neccessary information needed. Also since it never created the table im assuming thats why i never saw a admin link to it in my admin control panel. Does anyone know how to manually populate a table? Or can you make a .sql file that manually creates and fill this table for me?

Thank you.

[@null Member 17 years, 4 months ago]

@ MaryJane download and use phpMyAdmin. It gives you acces to you database. You still need to know how to make/use SQL queries though

@Box, dude had any time to help me out with the bbMenu plugin thingy? I want to fix this issue before I start with the next version (using jquery and fixing alot of bugs)

Thx

[@maryjane Member 17 years, 4 months ago]

I dont need to download phpmyadmin. I already have it with my hosting. Thats what i used to make the table to bein with. Since i cant get this to work without a bunch of errors ive disabled it until a better solution is figured out.

[@box87 Member 17 years, 4 months ago]

@Null – Working on it…

Don’t lose hope

Sorry about the “day or so”…

[@box87 Member 17 years, 4 months ago]

The new changes have been made:

http://www.box.net/shared/cgrj1na0t7

I think that should cover the problems. I took the liberty of changing other things as well… hope you don’t mind. I removed some redundancies in the CSS, and changed some little code things.

There’s the modified bbmenu.php file, as well as the diff file to make it a little easier for you to see what I changed. If you cant make sense of the diff there are diff file viewers that color the text and make it more human-readable.

[@null Member 17 years, 4 months ago]

Testing this tomorrow after work…. nah after diner after work

Thx for the help mate!

_Null

ps. how to open a .diff?? (windows machine)

[@fel64 Member 17 years, 4 months ago]

You need something like TortoiseSVN that’ll let you apply patches to files.

[@null Member 17 years, 4 months ago]

I have that TortoiseSVN installed, still need to figure out how to let this update the trac aswell

[@null Member 17 years, 4 months ago]

Hmmm got this error:

Warning: Invalid argument supplied for foreach() in /home/sites/site116/web/bbpress/my-plugins/bbmenu.php on line 390

the foreach:

foreach( $r1 as $rw ) {

Null