Isset on Get Fails

[unemployment]

Is this supposed to work?

 

$companies = fetch_companies(isset($_GET['industry']));

[premiso]

No. It would just pass true/flase to the function since isset returns true / false depending on if the variable has been set.

 

$industry = isset($_GET['industry'])?$_GET['industry']:null;
$companies = fetch_companies($industry);

 

Then just make sure to check for null in the fetch_companies and handle it accordingly.

[wildteen88]

EDIT: Beaten to it

 

isset only returns TRUE or FALSE. It does not return the value of $_GET['industry'];

 

You need to pass $_GET['industry'] to your function when it is set. You'd write the code like so

if(isset($_GET['industry']))
{
    $companies = fetch_companines($_GET['industry']);
}
else
{
    $companies = ''; // set companies to an empty string/value
}

[AyKay47]

depends on whether or not you have your custom function set to handle bool responses as an argument.... without seeing your custom function code, we can't know what's causing it not to work

 

Edit: triple response here guys...

[unemployment]

No. It would just pass true/flase to the function since isset returns true / false depending on if the variable has been set.

 

$industry = isset($_GET['industry'])?$_GET['industry']:null;
$companies = fetch_companies($industry);

 

Then just make sure to check for null in the fetch_companies and handle it accordingly.

 

Perfect solution.  Thanks guys.