Warning function mysql_query() and mysql_fetch_array() using TCPDF

Question

I'm trying to print data from mysql but get these error :

Warning: mysql_query() expects parameter 1 to be string, resource given in ..

Warning: mysql_fetch_array() expects parameter 1 to be resource, null given in ..

TCPDF ERROR: Some data has already been output, can't send PDF file

I have learn from these following links but still get warning:

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in
mysqli_fetch_array()/mysqli_fetch_assoc()/mysqli_fetch_row() expects parameter 1 to be resource or mysqli_result, boolean given

This is the code:

    $con=mysql_connect('localhost','root','','bkd_rev');
    $sql = 'select * from tbl';                       
    $result = mysql_query($con,$sql);

    if($result === FALSE) {
        die(mysql_error());
    }

    while($row = mysql_fetch_array($result))
    {
                    $id       = $row['id'];
                    $nam      = $row['name'];                       
        $tbl    .= '<tr> 
                <td>'.$id.'</td><td>'.$nam.'</td><td>
                </tr>';
    }

put this `$con=mysql_connect('localhost','root','bkd_rev');` instead of `$con=mysql_connect('localhost','root','','bkd_rev');`... what is the **blank value** for?? — Nishant Solanki, Mar 06 '14 at 13:14
@NishantSolanki that would be where his password is, that he wisely deleted I'm thinking. — Digital Chris, Mar 06 '14 at 13:19
@DigitalChris is it is his database password.. than what `bkd_rev` stands for?? — Nishant Solanki, Mar 06 '14 at 13:24
@NishantSolanki now that you mention it, you're right. His problem is that he's using mysql_connect as if it were [mysqli_connect](http://us2.php.net/manual/en/mysqli.construct.php) which he should really be using anyway. The 4th parameter is the database name. — Digital Chris, Mar 06 '14 at 13:39
@DigitalChris ohh yes.. `mysql_connect` function doesnt provide this... he is mixing mysql and mysqli.... — Nishant Solanki, Mar 06 '14 at 13:41
hm I learn from your comments, and that was my silly fault. I changed to `mysqli_*` and it works, thanks for inspiring me guys.. — andrian, Mar 06 '14 at 14:16

Fabio · Accepted Answer · 2014-03-06T13:21:36.143

3

The right syntax for mysql_query is the opposite of your. From documentation

mixed mysql_query ( string $query [, resource $link_identifier = NULL ] )

So you need to change to

$result = mysql_query($sql,$con);

Since connection link is not required if you only use a database connection you migh not use it

$result = mysql_query($sql);

You might need to select your database after connection

bool mysql_select_db ( string $database_name [, resource $link_identifier = NULL ] )

As side note i would advise you to switch to either PDO or mysqli since mysql_* api are deprecated and soon will be no longer mantained

edited Mar 06 '14 at 13:21

answered Mar 06 '14 at 13:16

Fabio

23,183
12
55
64

how about this? https://ideone.com/UK1npN but Noticed and Warning appears for the tcpdf file itself. – andrian Mar 06 '14 at 13:58
@adrian what should i see there? This seems to be another error not related to your actual question. If the answer was correct you should mark this answer as correct. If you have a new question after this problem was solved you should ask a new question with your new scenario. – Fabio Mar 06 '14 at 14:19
sorry I'm being crazy with these code. but your advice with `mysqli_*` was really helpful, thanks it works now. – andrian Mar 06 '14 at 14:25

score 2 · Answer 2 · answered Mar 06 '14 at 13:21

2

STOP USING mysql_ METHODS THEY ARE DEPRECATED AND VERY UNSAFE.

Instead use mysqli_, you should learn how to use this rather than trying to continue to learn mysql_ methods.

See Here: http://uk1.php.net/mysqli

answered Mar 06 '14 at 13:21

RaggaMuffin-420

1,762
1
10
14

i'm PosSible · Answer 3 · 2014-04-15T07:33:46.980

0

problem in parameter sequeance in mysql_query().

$con=mysqli_connect('localhost','root','','bkd_rev');
$sql = 'select * from tbl';                       
$result = mysql_query($sql);

if you want to pass connection object then,

$result = mysql_query($sql,$con);

edited Apr 15 '14 at 07:33

answered Mar 06 '14 at 13:17

i'm PosSible

1,373
2
11
30

Nishant Solanki · Answer 4 · 2014-03-06T13:43:03.067

try this...

$con=mysql_connect('localhost','root','');
mysql_select_db('bkd_rev',$con);
$sql = 'select * from tbl';                       
$result = mysql_query($sql);

if(!$result) {
    die(mysql_error());
}

while($row = mysql_fetch_array($result))
{
                $id       = $row['id'];
                $nam      = $row['name'];                       
    $tbl    .= '<tr> 
            <td>'.$id.'</td><td>'.$nam.'</td><td>
            </tr>';
}

score 0 · Answer 5 · answered Mar 06 '14 at 13:17

0

the first parameter should be the query it suppose t

$result = mysql_query($sql,$con);

answered Mar 06 '14 at 13:17

moh kaw

43
5

Warning function mysql_query() and mysql_fetch_array() using TCPDF

5 Answers5