"Parse Error" when using $_GET variable (PHP)

Question

I'm trying to create a webpage that shows me the contents of an SQL-Database. This works, but now i'm creating a search option that allows for filtering.

I can't seem to get the superglobal $_GET to work though, and I always get his error:

Parse error: syntax error, unexpected '$_GET' (T_VARIABLE) HTML/PHP code:

<header>
    <div id="divSearchBar">
        <form name="SearchBar" method="get" action="index.php">
            <input type="text" name="txtSearchBar"/>
            <input type="submit" value="Search"/>
            <input type="submit" name="sbmAddItem" value="Add Item"/>
        </form>
    </div>
</header>
<main>
        <?php
            include_once "Repository.php";

            $r = new Repository();
            $tmp = "";
            if (isset $_GET["txtSearchBar"])
            {
                $tmp = $_GET["txtSearchBar"];   
            }

            $myItems = $r->GetItems(tmp);
            for($i=0;$i<count($myItems);$i++)
            {
                print '<div id="divGen"><p id="pGen">' . $myItems[$i]->Item . '</p><p id="pGen2">' . $myItems[$i]->Category . "</p></div>";
            }
        ?>
</main>

And if ever in doubt, check the manual: http://php.net/isset — Qirel, Jun 02 '17 at 12:56
`isset()` need parantheses http://php.net/manual/en/function.isset.php — Mohammad, Jun 02 '17 at 12:56
And you can shorten it to one line using a ternary operator, `$tmp = isset($_GET["txtSearchBar"]) ? $_GET["txtSearchBar"] : '';` — Qirel, Jun 02 '17 at 12:57

score 2 · Accepted Answer · answered Jun 02 '17 at 12:55

2

You need to use parentheses with the isset function

if (isset($_GET["txtSearchBar"])) // <-- Here
{
    $tmp = $_GET["txtSearchBar"]; 
}

answered Jun 02 '17 at 12:55

Jerodev

29,019
11
72
94

B. Desai · Answer 2 · 2017-06-02T13:00:09.423

1

change your code to

if (isset($_GET["txtSearchBar"]))
{
      $tmp = $_GET["txtSearchBar"];   
}

edited Jun 02 '17 at 13:00

answered Jun 02 '17 at 12:55

B. Desai

16,092
5
22
43

score 1 · Answer 3 · answered Jun 02 '17 at 12:55

Use isset() function in round brackets

<?php
            include_once "Repository.php";

            $r = new Repository();
            $tmp = "";
            if (isset ($_GET["txtSearchBar"]))
            {
                $tmp = $_GET["txtSearchBar"];   
            }

            $myItems = $r->GetItems(tmp);
            for($i=0;$i<count($myItems);$i++)
            {
                print '<div id="divGen"><p id="pGen">' . $myItems[$i]->Item . '</p><p id="pGen2">' . $myItems[$i]->Category . "</p></div>";
            }
        ?>

"Parse Error" when using $_GET variable (PHP)

3 Answers3