passing a parameter to a function

**Jerry Stuckle** · Sep 2 '08, 10:55 PM

Re: passing a parameter to a function

JRough wrote:

I'm getting the error message that the parameter passed to the
function is not a valid resource. The parameter is $result and it is
from a query in a switch statement. What do I have to do to get it to
accept it?
tia,
>
function MakeXclHeader($ result){
$fields = mysql_num_field s($result);
for ($i = 0; $i < $fields; $i++){
>
$header .= mysql_field_nam e($result, $i) ."\t";
}
$header .= "\n";
>
return 1;
}
----------------------------
switch ($_SESSION["LMS_USER_D ESC"]){
case 'internal':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
case 'owner':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease_owner($ id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
default:
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
}
>

You didn't show the functions you're calling to get $result - but your
query failed.

Always check the result of a query - never assume it worked. If the
result is false, the query failed. You then need to find out why your
query failed.

--
=============== ===
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
jstucklex@attgl obal.net
=============== ===

**JRough** · Sep 3 '08, 12:45 AM

Re: passing a parameter to a function

On Sep 2, 3:47 pm, Jerry Stuckle <jstuck...@attg lobal.netwrote:

JRough wrote:

I'm getting the error message that the parameter passed to the
function is not a valid resource. The parameter is $result and it is
from a query in a switch statement. What do I have to do to get it to
accept it?
tia,

>

function MakeXclHeader($ result){
$fields = mysql_num_field s($result);
for ($i = 0; $i < $fields; $i++){

>

$header .= mysql_field_nam e($result, $i) ."\t";
}
$header .= "\n";

>

return 1;
}
----------------------------
switch ($_SESSION["LMS_USER_D ESC"]){
case 'internal':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
case 'owner':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease_owner($ id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
default:
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
}

>
You didn't show the functions you're calling to get $result - but your
query failed.
>
Always check the result of a query - never assume it worked. If the
result is false, the query failed. You then need to find out why your
query failed.
>
--
=============== ===
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
jstuck...@attgl obal.net
=============== ===

I ran the query and it does work but it has to have a $id. Which I
think is at the beginning of the
page in the validate login step. The $order_by and $days parameters
are already set. I wonder if the query is
not picking up the $id variable?

Here is my function call. I don't know if it is right or not. I put
the print_r to debug it.
$xcl_header= MakeXclHeader($ header);
print_r($xcl_he ader);
}

thanks,
Validate_login( "idle_cars.php? id=".$id);

**Jerry Stuckle** · Sep 3 '08, 01:15 AM

Re: passing a parameter to a function

JRough wrote:

On Sep 2, 3:47 pm, Jerry Stuckle <jstuck...@attg lobal.netwrote:

>JRough wrote:

>>I'm getting the error message that the parameter passed to the
>>function is not a valid resource. The parameter is $result and it is
>>from a query in a switch statement. What do I have to do to get it to
>>accept it?
>>tia,
>>function MakeXclHeader($ result){
>>$fields = mysql_num_field s($result);
>>for ($i = 0; $i < $fields; $i++){
>> $header .= mysql_field_nam e($result, $i) ."\t";
>> }
>> $header .= "\n";
>>return 1;
>>}
>>----------------------------
>>switch ($_SESSION["LMS_USER_D ESC"]){
>> case 'internal':
>> $headers = GetHeaders($fil e_name);
>> $result = SELECT_idle_day s_lease($id,
>>$days,CLM_ord er_by($order_by ));
>> $lease_row = false;
>> break;
>> case 'owner':
>> $headers = GetHeaders($fil e_name);
>> $result = SELECT_idle_day s_lease_owner($ id,
>>$days,CLM_ord er_by($order_by ));
>> $lease_row = false;
>> break;
>> default:
>> $headers = GetHeaders($fil e_name);
>> $result = SELECT_idle_day s_lease($id,
>>$days,CLM_ord er_by($order_by ));
>> $lease_row = false;
>> break;
>> }

>You didn't show the functions you're calling to get $result - but your
>query failed.
>>
>Always check the result of a query - never assume it worked. If the
>result is false, the query failed. You then need to find out why your
>query failed.
>>
>--
>============== ====
>Remove the "x" from my email address
>Jerry Stuckle
>JDS Computer Training Corp.
>jstuck...@attg lobal.net
>============== ====

>
I ran the query and it does work but it has to have a $id. Which I
think is at the beginning of the
page in the validate login step. The $order_by and $days parameters
are already set. I wonder if the query is
not picking up the $id variable?
>
>
Here is my function call. I don't know if it is right or not. I put
the print_r to debug it.
$xcl_header= MakeXclHeader($ header);
print_r($xcl_he ader);
}
>
thanks,
Validate_login( "idle_cars.php? id=".$id);
>

Your query is failing. That much is obvious.

As I said above - ALWAYS CHECK THE RESULT OF A QUERY - NEVER ASSUME IT
WORKED. IF THE RESULT IS false, THE QUERY FAILED. YOU NEED THEN TO
FIND OUT WHY YOUR QUERY FAILED.

--
=============== ===
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
jstucklex@attgl obal.net
=============== ===

**FutureShock** · Sep 3 '08, 03:15 AM

Re: passing a parameter to a function

JRough wrote:

On Sep 2, 3:47 pm, Jerry Stuckle <jstuck...@attg lobal.netwrote:

>JRough wrote:

>>I'm getting the error message that the parameter passed to the
>>function is not a valid resource. The parameter is $result and it is
>>from a query in a switch statement. What do I have to do to get it to
>>accept it?
>>tia,
>>function MakeXclHeader($ result){
>>$fields = mysql_num_field s($result);
>>for ($i = 0; $i < $fields; $i++){
>> $header .= mysql_field_nam e($result, $i) ."\t";
>> }
>> $header .= "\n";
>>return 1;
>>}
>>----------------------------
>>switch ($_SESSION["LMS_USER_D ESC"]){
>> case 'internal':
>> $headers = GetHeaders($fil e_name);
>> $result = SELECT_idle_day s_lease($id,
>>$days,CLM_ord er_by($order_by ));
>> $lease_row = false;
>> break;
>> case 'owner':
>> $headers = GetHeaders($fil e_name);
>> $result = SELECT_idle_day s_lease_owner($ id,
>>$days,CLM_ord er_by($order_by ));
>> $lease_row = false;
>> break;
>> default:
>> $headers = GetHeaders($fil e_name);
>> $result = SELECT_idle_day s_lease($id,
>>$days,CLM_ord er_by($order_by ));
>> $lease_row = false;
>> break;
>> }

>You didn't show the functions you're calling to get $result - but your
>query failed.
>>
>Always check the result of a query - never assume it worked. If the
>result is false, the query failed. You then need to find out why your
>query failed.
>>
>--
>============== ====
>Remove the "x" from my email address
>Jerry Stuckle
>JDS Computer Training Corp.
>jstuck...@attg lobal.net
>============== ====

>
I ran the query and it does work but it has to have a $id. Which I
think is at the beginning of the
page in the validate login step. The $order_by and $days parameters
are already set. I wonder if the query is
not picking up the $id variable?
>
>
Here is my function call. I don't know if it is right or not. I put
the print_r to debug it.
$xcl_header= MakeXclHeader($ header);
print_r($xcl_he ader);
}
>
thanks,
Validate_login( "idle_cars.php? id=".$id);

Your error message is screaming at you that you code to obtain the
$result is bad before it even gets used by the function, not the
function that uses $result.

You need to show your code used to obtain the $result if you want help,
or we are just throwing mud at the wall.

You are trying to find out why your bulb burned out and all the time
your light switch is off.

Scotty

**JRough** · Sep 3 '08, 04:35 PM

Re: passing a parameter to a function

On Sep 2, 8:13 pm, FutureShock <futuresh...@at t.netwrote:

JRough wrote:

On Sep 2, 3:47 pm, Jerry Stuckle <jstuck...@attg lobal.netwrote:

JRough wrote:
>I'm getting the error message that the parameter passed to the
>function is not a valid resource. The parameter is $result and it is
>from a query in a switch statement. What do I have to do to get itto
>accept it?
>tia,
>function MakeXclHeader($ result){
>$fields = mysql_num_field s($result);
>for ($i = 0; $i < $fields; $i++){
> $header .= mysql_field_nam e($result, $i) ."\t";
> }
> $header .= "\n";
>return 1;
>}
>----------------------------
>switch ($_SESSION["LMS_USER_D ESC"]){
> case 'internal':
> $headers = GetHeaders($fil e_name);
> $result = SELECT_idle_day s_lease($id,
>$days,CLM_orde r_by($order_by) );
> $lease_row = false;
> break;
> case 'owner':
> $headers = GetHeaders($fil e_name);
> $result = SELECT_idle_day s_lease_owner($ id,
>$days,CLM_orde r_by($order_by) );
> $lease_row = false;
> break;
> default:
> $headers = GetHeaders($fil e_name);
> $result = SELECT_idle_day s_lease($id,
>$days,CLM_orde r_by($order_by) );
> $lease_row = false;
> break;
> }
You didn't show the functions you're calling to get $result - but your
query failed.

>

Always check the result of a query - never assume it worked. If the
result is false, the query failed. You then need to find out why your
query failed.

>

--
=============== ===
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
jstuck...@attgl obal.net
=============== ===

>

I ran the query and it does work but it has to have a $id. Which I
think is at the beginning of the
page in the validate login step. The $order_by and $days parameters
are already set. I wonder if the query is
not picking up the $id variable?

>

Here is my function call. I don't know if it is right or not. I put
the print_r to debug it.
$xcl_header= MakeXclHeader($ header);
print_r($xcl_he ader);
}

>

thanks,
Validate_login( "idle_cars.php? id=".$id);

>
Your error message is screaming at you that you code to obtain the
$result is bad before it even gets used by the function, not the
function that uses $result.
>
You need to show your code used to obtain the $result if you want help,
or we are just throwing mud at the wall.
>
You are trying to find out why your bulb burned out and all the time
your light switch is off.
>
Scotty

I don't understand why $result doesn't work because the page works
without the function?
Validate_login( "idle_cars.php? id=".$id);

$days = 3;
$TPL_carnumbers = "<table>";

if(empty($order _by)){
$order_by = 'sighting_date_ asc';
}

# GET CARS FOR GIVEN PARAMETER?
if(empty($id)){
$MSG_carlist = "IDLE CARS - NO MOVEMENT IN 3 DAYS";
$file_name = "idle_cars.php? ";
switch ($_SESSION["LMS_USER_D ESC"]){
case 'internal':
$headers = GetLeaseHeaders ($file_name);
$result = SELECT_idle_day s($days,CLM_ord er_by($order_by ));
$lease_row = true;
break;
case 'owner':
$headers = GetLeaseHeaders ($file_name);
$result = SELECT_idle_day s_owner($days,C LM_order_by($or der_by));
$lease_row = true;
break;
case 'customer':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_customer($day s,CLM_order_by( $order_by));
$lease_row = false;
break;
default:
$headers = GetLeaseHeaders ($file_name);
$result = SELECT_idle_day s($days,CLM_ord er_by($order_by ));
$lease_row = true;
break;
}
}else{
$MSG_carlist = "IDLE CARS - NO MOVEMENT IN 3 DAYS
".GetLeaseCompN ame($id);
$file_name = "idle_cars.php? id=".$id."&";
switch ($_SESSION["LMS_USER_D ESC"]){
case 'internal':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
case 'owner':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease_owner($ id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
default:
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
}
}
$TPL_carnumbers .= $headers;

if ($_POST['assign']!='Open in Excel'){

if(mysql_numrow s($result)==0){
$TPL_carnumbers .= GetNoCarsMsg($t h);
}else{
while ($row = mysql_fetch_ass oc($result)){
$TPL_carnumbers .=MakeSighting( $lease_row,$row );
}
}

$TPL_carnumbers .="</table>";

include "header.php ";
include $template_path. "template_carli st.html";
include "footer.php ";

}else{

$xcl_header= MakeXclHeader($ header);
print_r($xcl_he ader);
}
include "footer.php ";

**Jerry Stuckle** · Sep 3 '08, 06:05 PM

Re: passing a parameter to a function

JRough wrote:

On Sep 2, 8:13 pm, FutureShock <futuresh...@at t.netwrote:
>
I don't understand why $result doesn't work because the page works
without the function?
Validate_login( "idle_cars.php? id=".$id);
>

(lots of stuff trimmed)

All you're doing is setting $result to a string. You are not making a
mysql_query call with the string, so you will not have a mysql result in
the variable.

--
=============== ===
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
jstucklex@attgl obal.net
=============== ===

**FutureShock** · Sep 3 '08, 08:35 PM

Re: passing a parameter to a function

>

I don't understand why $result doesn't work because the page works
without the function?
Validate_login( "idle_cars.php? id=".$id);
>
$days = 3;
$TPL_carnumbers = "<table>";
>
if(empty($order _by)){
$order_by = 'sighting_date_ asc';
}
>
# GET CARS FOR GIVEN PARAMETER?
if(empty($id)){
$MSG_carlist = "IDLE CARS - NO MOVEMENT IN 3 DAYS";
$file_name = "idle_cars.php? ";
switch ($_SESSION["LMS_USER_D ESC"]){
case 'internal':
$headers = GetLeaseHeaders ($file_name);
$result = SELECT_idle_day s($days,CLM_ord er_by($order_by ));
$lease_row = true;
break;
case 'owner':
$headers = GetLeaseHeaders ($file_name);
$result = SELECT_idle_day s_owner($days,C LM_order_by($or der_by));
$lease_row = true;
break;
case 'customer':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_customer($day s,CLM_order_by( $order_by));
$lease_row = false;
break;
default:
$headers = GetLeaseHeaders ($file_name);
$result = SELECT_idle_day s($days,CLM_ord er_by($order_by ));
$lease_row = true;
break;
}
}else{
$MSG_carlist = "IDLE CARS - NO MOVEMENT IN 3 DAYS
".GetLeaseCompN ame($id);
$file_name = "idle_cars.php? id=".$id."&";
switch ($_SESSION["LMS_USER_D ESC"]){
case 'internal':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
case 'owner':
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease_owner($ id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
default:
$headers = GetHeaders($fil e_name);
$result = SELECT_idle_day s_lease($id,
$days,CLM_order _by($order_by)) ;
$lease_row = false;
break;
}
}
$TPL_carnumbers .= $headers;
>
if ($_POST['assign']!='Open in Excel'){
>
>
if(mysql_numrow s($result)==0){
$TPL_carnumbers .= GetNoCarsMsg($t h);
}else{
while ($row = mysql_fetch_ass oc($result)){
$TPL_carnumbers .=MakeSighting( $lease_row,$row );
}
}
>
$TPL_carnumbers .="</table>";
>
include "header.php ";
include $template_path. "template_carli st.html";
include "footer.php ";
>
}else{
>
>
$xcl_header= MakeXclHeader($ header);
print_r($xcl_he ader);
}
include "footer.php ";

*SIGH*

Of course it will work in this code, you are not expecting $result to be
anything, you are in this case setting it. '$result = xxxxx'.

In the function you where using $result as a parameter to
mysql_num_field s($result) in which case it needed to be a MySQL resource.

The best way to debug any code is to blackbox test it.

Write a 'SIMPLE' output function to test the input variables. ALWAYS
confirm your data is correct BEFORE trying to use it.

If you had error checking on your SQL code you may have caught the problem:

$query = "SELECT * FROM $table";
if(!$result = mysql_query($qu ery)) {
echo "Error on query: ".$query";
}

If $result gets past this, then most often it is a RESOURCE.

But hey man, if your sql query is that top secret that you do not want
to post it for help, maybe contact the CIA, they know all your secrets
anyhow, we can't help you any more.

Scotty

**Curtis** · Sep 4 '08, 09:55 AM

Re: passing a parameter to a function

JRough wrote:

On Sep 2, 8:13 pm, FutureShock <futuresh...@at t.netwrote:

>JRough wrote:

>>On Sep 2, 3:47 pm, Jerry Stuckle <jstuck...@attg lobal.netwrote:
>>>JRough wrote:
>>>>I'm getting the error message that the parameter passed to the
>>>>function is not a valid resource. The parameter is $result and it is
>>>>from a query in a switch statement. What do I have to do to get it to
>>>>accept it?
>>>>tia,
>>>>function MakeXclHeader($ result){
>>>>$fields = mysql_num_field s($result);
>>>>for ($i = 0; $i < $fields; $i++){
>>>> $header .= mysql_field_nam e($result, $i) ."\t";
>>>> }
>>>> $header .= "\n";
>>>>return 1;
>>>>}
>>>>----------------------------
>>>>switch ($_SESSION["LMS_USER_D ESC"]){
>>>> case 'internal':
>>>> $headers = GetHeaders($fil e_name);
>>>> $result = SELECT_idle_day s_lease($id,
>>>>$days,CLM_o rder_by($order_ by));
>>>> $lease_row = false;
>>>> break;
>>>> case 'owner':
>>>> $headers = GetHeaders($fil e_name);
>>>> $result = SELECT_idle_day s_lease_owner($ id,
>>>>$days,CLM_o rder_by($order_ by));
>>>> $lease_row = false;
>>>> break;
>>>> default:
>>>> $headers = GetHeaders($fil e_name);
>>>> $result = SELECT_idle_day s_lease($id,
>>>>$days,CLM_o rder_by($order_ by));
>>>> $lease_row = false;
>>>> break;
>>>> }
>>>You didn't show the functions you're calling to get $result - but your
>>>query failed.
>>>Always check the result of a query - never assume it worked. If the
>>>result is false, the query failed. You then need to find out why your
>>>query failed.
>>>--
>>>============ ======
>>>Remove the "x" from my email address
>>>Jerry Stuckle
>>>JDS Computer Training Corp.
>>>jstuck...@at tglobal.net
>>>============ ======
>>I ran the query and it does work but it has to have a $id. Which I
>>think is at the beginning of the
>>page in the validate login step. The $order_by and $days parameters
>>are already set. I wonder if the query is
>>not picking up the $id variable?
>>Here is my function call. I don't know if it is right or not. I put
>>the print_r to debug it.
>>$xcl_header = MakeXclHeader($ header);
>>print_r($xcl_ header);
>>}
>>thanks,
>>Validate_logi n("idle_cars.ph p?id=".$id);

>Your error message is screaming at you that you code to obtain the
>$result is bad before it even gets used by the function, not the
>function that uses $result.
>>
>You need to show your code used to obtain the $result if you want help,
>or we are just throwing mud at the wall.
>>
>You are trying to find out why your bulb burned out and all the time
>your light switch is off.

I don't understand why $result doesn't work because the page works
without the function?
>

<code snipped>

I'm fresh out of magic crystal balls that let me see into the source
code of people's APIs when not posted. Also, try trimming down the
code posted to only what's relevant.

However, your query functions, like "SELECT_idle_da ys", and its ilk,
are not returning valid query resources (a deduction made evident by
the error you reported having). If you want to fix it, find out what's
wrong with your queries, or what's wrong with your API in general.

--
Curtis

**FutureShock** · Sep 4 '08, 01:15 PM

Re: passing a parameter to a function

Curtis wrote:
<snip>

>
<code snipped>
>
I'm fresh out of magic crystal balls that let me see into the source
code of people's APIs when not posted. Also, try trimming down the code
posted to only what's relevant.
>
However, your query functions, like "SELECT_idle_da ys", and its ilk, are
not returning valid query resources (a deduction made evident by the
error you reported having). If you want to fix it, find out what's wrong
with your queries, or what's wrong with your API in general.
>
--
Curtis

Curtis, I think you need Top-Secret or better clearance to view that
code. I need to go into the Crystal Ball repair business.

Scotty

passing a parameter to a function

passing a parameter to a function

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment

Comment