[PHP] Probleem met zoekfunctie werkend krijgen

[MarcoZ20]

Ik heb een werkende zoekfunctie voor de site waar ik mee bezig ben. Om te testen heb ik de resultaten weergegeven in een nieuw venster. Het is echter de bedoeling dat de resultaten gewoon in de website getoond worden.
Ik weet echter niet hoe ik dit moet doen, omdat de pagina's van de website opgebouwd worden volgens dit stukje script;

PHP:

<?
if (isset($_REQUEST["pid"]) && isset($inc_myrow))
{
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" valign=\"top\">";
    require("inc/menu.inc.php");
    echo "</TD><TD class=\"content\" valign=\"top\">".$content."</TD></TR></TABLE>";
} else
{
    include ("inc/default.inc.php");
}
?>

nu had ik geprobeerd om met het volgende script de zoekresultaten weer te geven, maar dat werkte niet.

PHP:

<?
if (isset($_REQUEST["pid"]) && isset($inc_myrow))
{
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" valign=\"top\">";
    require("inc/menu.inc.php");
    echo "</TD><TD class=\"content\" valign=\"top\">".$content."</TD></TR></TABLE>";
} elseif (isset($_REQUEST["searchresult"])) {
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" valign=\"top\">";
    require("inc/menu.inc.php");
    echo "</TD><TD class=\"content\" valign=\"top\">".searchresult()."</TD></TR></TABLE>";
} else
{
    include ("inc/default.inc.php");
}
?>

Hierbij kreeg ik alle inhoud van de database en het wordt verkeerd weergegeven op de site, maar dat is niethet belangrijskte nu.


Voor de volledigheid;

Zoekform;
<form style="display: inline;" method="post" action="/?searchresult=1">Zoeken: <input type="text" value="" name="zoekterm" /> <input type="image" src="library/arrow.gif" value="Submit" name="B1" /></form>


Zoekscript;

function searchresult()
{
$sql = "SELECT *
FROM cursussen
WHERE cursusnaam LIKE '%$_POST[zoekterm]%'
OR code LIKE '%$_POST[zoekterm]%'
OR examen LIKE '%$_POST[zoekterm]%'
ORDER BY examen ASC
";


$res = mysql_query($sql);


if (mysql_num_rows($res) >= 1)
{

while ($row = mysql_fetch_array($res))
{
echo "<p><font face=Verdana size=1>Cursus: $row[code]<br>";
echo "Examen: $row[examen]<br>";
echo "<a href=\"index.php?pid=3&cursusid=$row[cursusid]&ai=20\">$row[cursusnaam]</a></font></p>";

}

echo "<a href=\"index.php\" title=\"zoek opnieuw\"><font face=Verdana size=1>zoek opnieuw</font></a>";
}
//-- als er geen resultaat is gevonden, dus als het zoekwoord niet gevonden is:
else
{
echo "<p><font face=Verdana size=1>Er is niets gevonden op de zoekterm:<b> $_POST[zoekterm]</b></font></p>";

echo "<p><a href=\"index.php\" title=\"zoek opnieuw\"><font face=Verdana size=1>zoek opnieuw</font></a></p>";
}
}


Iemand een idee hoe ik het goed werkend kan krijgen:?

[ Voor 71% gewijzigd door MarcoZ20 op 22-03-2006 11:38 ]

[Verwijderd]

Resultaten worden weergegeven in een nieuw venster? Dat kan wanneer er een window.open() functie in je onsubmit / onclick of iets anders staat. Die kan ik alleen niet vinden in je geposte code.

Kijk daar even naar

[MarcoZ20]

Verwijderd schreef op woensdag 22 maart 2006 @ 10:54:
Resultaten worden weergegeven in een nieuw venster? Dat kan wanneer er een window.open() functie in je onsubmit / onclick of iets anders staat. Die kan ik alleen niet vinden in je geposte code.

Kijk daar even naar

nee nee, dat bedoel ik niet.

Om te testen heb ik de resultaten in een nieuw venster laten komen. Dat is nu niet meer nodig.
De resultaten moeten gewoon in het zelfde venster getoond worden.

[NetForce1]

In de code die je hier laat zien wordt nergens een nieuw venster geopend. Je zult waarschijnlijk toch echt ergens een window.open hebben staan.

[Verwijderd]

Wat is je probleem dan? Want als je het resultaat in een nieuw verster kan laten verschijnen kun je het toch ook op de pagina zelf laten verschijnen?

[MarcoZ20]

als ik het in dezelfde pagina zou kunnen laten verschijnen, dan had ik dit topic niet geopend

Probleem is dat de pagina's van de website via dit script opgebouwd worden;

PHP:

<? 
if (isset($_REQUEST["pid"]) && isset($inc_myrow)) 
{ 
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" valign=\"top\">"; 
    require("inc/menu.inc.php"); 
    echo "</TD><TD class=\"content\" valign=\"top\">".$content."</TD></TR></TABLE>"; 
} else 
{ 
    include ("inc/default.inc.php"); 
} 
?>

Door deze elseif erin te plakken, probeer ik de zoekresultaten op te roepen, alleen werkt dat dus niet;

elseif (isset($_REQUEST["searchresult"])) {
echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" valign=\"top\">";
require("inc/menu.inc.php");
echo "</TD><TD class=\"content\" valign=\"top\">".searchresult()."</TD></TR></TABLE>";
}

Het probleem zit dus hierin of in de form action. Krijg nu namelijk gewoon alle data uit de database bij een zoekopdracht.

[gorgi_19]

* gorgi_19 gaat voor de volledigheid weer eens een opmerking maken over SQL Injection; je hebt een gigantische beveiligingslek in je code en je moet ook niet vreemd staan kijken als iemand je database een keer leeg gaat halen.

[Verwijderd]

Dus als ik het goed begrijp is de situatie als volgt;
Je hebt een formulier op je pagina staan en daarmee laat je de gebruiker een zoekactie uitvoeren.
In je testomgeving heb je deze data laten posten (action="anderePagina.php") naar een nieuwe pagina en dat werkte prima, maar nu wil je het formulier naar dezelfde pagina laten posten - en daar gaat het mis.

Kun je dit even bevestigen?
En kun je misschien de hele pagina posten (niet in dit forum maar extern) want, met alle respect, je uitleg lijkt niet veel zoden aan den spreekwoordelijke dijk te zetten..

[ Voor 22% gewijzigd door Verwijderd op 22-03-2006 11:59 ]

[MarcoZ20]

gorgi_19 schreef op woensdag 22 maart 2006 @ 11:48:
* gorgi_19 gaat voor de volledigheid weer eens een opmerking maken over SQL Injection; je hebt een gigantische beveiligingslek in je code en je moet ook niet vreemd staan kijken als iemand je database een keer leeg gaat halen.

i know, maar dat komt later wel. Zit toch nog in de testomgeving, dus geen probleem nu nog. Maar thnx voor het opmerken

[MarcoZ20]

Verwijderd schreef op woensdag 22 maart 2006 @ 11:58:
Dus als ik het goed begrijp is de situatie als volgt;
Je hebt een formulier op je pagina staan en daarmee laat je de gebruiker een zoekactie uitvoeren.
In je testomgeving heb je deze data laten posten (action="anderePagina.php") naar een nieuwe pagina en dat werkte prima, maar nu wil je het formulier naar dezelfde pagina laten posten - en daar gaat het mis.

Kun je dit even bevestigen?
En kun je misschien de hele pagina posten want, met alle respect, je uitleg lijkt niet veel zoden aan den spreekwoordelijke dijk te zetten..

Dit klopt jah.

Code;

PHP:

<?php

require ("inc/config.inc.php");
require ("inc/functions.php");

$extrameta=null;

if (isset($_REQUEST["pid"]))
{
    if (isset($_REQUEST["ai"])) // activeitem ;)
    {
        $menu_query = "SELECT * FROM menu WHERE (parent_id='0' OR parent_id='".$_REQUEST["ai"]."') AND 

(location='left' OR location='both') ORDER BY rank,parent_id, text";
        $menu_result = mysql_query($menu_query) or die("Fout in query :(");
    }
    else
    {
        $menu_query = "SELECT * FROM menu WHERE parent_id='0' AND (location='left' OR location='both') 

ORDER BY rank,parent_id, text";
        $menu_result = mysql_query($menu_query) or die("Fout in query :(");
    }

    while ($menu_myrow = mysql_fetch_assoc($menu_result)) {
        $arr_menu[]=$menu_myrow;
    }

    $inc_query = "SELECT include,include_path FROM menu WHERE id = '".$_REQUEST["pid"]."'";
    $inc_result = mysql_query($inc_query) or die("Fout in query :(");
    $inc_myrow = mysql_fetch_assoc($inc_result);

    if ($inc_myrow["include"])
    {
        include ($inc_myrow["include_path"]);
    }
    else
    {
        $content_query = "SELECT * FROM content WHERE page_id = '".$_REQUEST["pid"]."'";
        $content_result = mysql_query($content_query) or die("Fout in query :(");
        $content_myrow = mysql_fetch_assoc($content_result);
        $content = "<div class=\"pagehead\">".$content_myrow["head"]."</div><br /><br /><div 

class=\"pagecontent\">".$content_myrow["text"]."</div>";
    }
}

$bottom_menu_query = "SELECT * FROM menu WHERE (location='bottom' OR location='both') ORDER BY rank";
$bottom_menu_result = mysql_query($bottom_menu_query) or die("Fout in query :(");

/*
Width = 776px breed zodat op een resolutie van 800x600 de pagina netjes zonder horizontal scrollbars erop past 

(incl. inactive vertical scrollbar).
Right-side = outerright.gif is 10px breed.
Table framespacing = 1px, wat een totaal van 4px maakt voor de tabel.
Frame 1,4,5,8 = 330px breed.
Frame 2,3,6,7,9: (((776-10)-4)-330) / 2 = 216px per frame.

|-----------------------|--------------|--------------|
|                       |              |              |
|                       |              |              |
|                       |              |              |
|     FRAME 1           |  FRAME 2     |  FRAME 3     |
|                       |              |              |
|                       |              |              |
|                       |              |              |
|-----------------------|--------------|--------------|
|     FRAME 4           |              |              |
|-----------------------|  FRAME 6     |              |
|     FRAME 5           |              |              |
|-----------------------|--------------|  FRAME 7     |
|                       |              |              |
|     FRAME 8           |  FRAME 9     |              |
|                       |              |              |
|-----------------------|--------------|--------------|

*/
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" 

"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<?=$extrameta?>
<title>* B.V.</title>
<link rel="stylesheet" type="text/css" href="library/style.css" />
<script language="Javascript" type="text/javascript">
function p()
{
    window.open('printfrm.php?<?=$_SERVER['QUERY_STRING'];?>', '', 

'toolbar=0,scrollbars=0,location=0,status=1,menubar=0,resizable=0,width=300,height=100,left=0,top=0');
}

var activelayer = 'id0';
var lastlayer = activelayer;

function showhide(divID) {
    hide();
    show(divID);
}

function show(NewDivID) {
    if (activelayer != NewDivID)
    {
        lastlayer = activelayer;
    }
    if (document.layers && document.layers[NewDivID] != null)
    {
        document.layers[NewDivID].visibility = 'visible';
    }
    else if (document.all)
    {
        document.all[NewDivID].style.visibility = 'visible';
    }
    else if (document.getElementById)
    {
        document.getElementById(''+NewDivID+'').style.visibility = 'visible';
    }
    activelayer = NewDivID;
//  alert ('Last Layer: '+lastlayer);
}

function hide() {
    if (document.layers && document.layers[activelayer] != null)
    {
        document.layers[activelayer].visibility = 'hidden';
    }
    else if (document.all)
    {
        document.all[activelayer].style.visibility = 'hidden'; 
    }
    else if (document.getElementById)
    {
        document.getElementById(''+activelayer+'').style.visibility = 'hidden';
    }
}
</script>
</head>

<body>

<table border="0" align="center" width="766" cellspacing="0" cellpadding="0" class="tblbase1">
    <tr>
<!--        <td width="10" class="page_start"><img src="library/spacer.gif" width="10" height="10" alt="" 

/></td>//-->
        <td valign="top">
            <table border="0" width="100%" cellspacing="0" cellpadding="0" class="tblbase2">
                <tr>
                    <td colspan="3" height="80" class="base1" valign="top">
                        <table border="0" cellspacing="0" cellpadding="0" width="100%">
                            <tr>
                                <td width="448"><img border="0" 

style="position:absolute; left: 50%; margin-left:-20px; top: 22px;" src="library/*_logo_prt1.gif" 

alt="" /><img border="0" style="position:absolute; left: 50%; margin-left:42px; top: 90px; " 

src="library/*_logo_prt2.gif" alt="" /></td>
                                <td width="328" align="right"></td>
                            </tr>
                        </table>
                    </td>
                </tr>
                <tr>
                    <td colspan="3" height="1" bgcolor="#ffffff"><img 

src="library/spacer.gif" height="1" width="1" alt="" /></td>
                </tr>
                <tr>
                    <td width="1" bgcolor="#ffffff"><img src="library/spacer.gif" 

height="1" width="1" alt="" /></td>
                    <td height="30" class="base2">
                        <table border="0" width="100%" cellspacing="0" cellpadding="0">
                            <tr>
                                <td style="padding-left: 10px;"><img border="0" 

src="library/base2_itprofs.gif" alt="" /></td>
                                <td style="padding-right: 3px;" align="right">
<form style="display: inline;" method="post" action="?searchresult">Zoeken: <input type="text" value="" 

name="zoekterm" />&nbsp;<input type="image" src="library/arrow.gif" value="Submit" name="B1" /></form></td>
                            </tr>
                        </table>
                    </td>
                    <td width="1" bgcolor="#ffffff"><img src="library/spacer.gif" 

height="1" width="1" alt="" /></td>
                </tr>
                <tr>
                    <td colspan="3" valign="top">
<?
if (isset($_REQUEST["pid"]) && isset($inc_myrow))
{
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" 

valign=\"top\">";
    require("inc/menu.inc.php");
    echo "</TD><TD class=\"content\" valign=\"top\">".$content."</TD></TR></TABLE>";
}
else if (isset($_POST["searchresult"]))
{
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" 

valign=\"top\">";
    require("inc/menu.inc.php");
    echo "</TD><TD class=\"content\" valign=\"top\">".$row."</TD></TR></TABLE>";
}
else
{
    include ("inc/default.inc.php");
}
?>
                    </td>
                </tr>
                <tr>
                    <td width="1" bgcolor="#ffffff"><img src="library/spacer.gif" 

height="1" width="1" alt="" /></td>
                    <td height="30" class="base3">&nbsp;</td>
                    <td width="1" bgcolor="#ffffff"><img src="library/spacer.gif" 

height="1" width="1" alt="" /></td>
                </tr>
                <tr>
                    <td colspan="3" height="25">
                        <table border="0" width="100%" cellspacing="1" cellpadding="0">
                            <tr>
<?
while ($bottom_menu_myrow = mysql_fetch_assoc($bottom_menu_result)) {
    echo "<td align=\"center\" class=\"bot_nav\" 

onclick=\"location.href='?pid=".$bottom_menu_myrow["id"]."';\" onmouseover=\"this.className='bot_nav_active';\" 

onmouseout=\"this.className='bot_nav';\">".$bottom_menu_myrow["text"]."</td>";
}
?>
                            </tr>
                        </table>
                    </td>
                </tr>
                <tr>
                    <td colspan="3" height="45" align="center" class="base4">
                        <table border="0" width="100%" cellspacing="0" cellpadding="2">
                            <tr>
                                <td align="center"><img border="0" 

src="library/logo_vue.jpg" width="73" height="43" alt="" /></td>
                                <td align="center"><img border="0" 

src="library/logo_ms_exam.jpg" width="83" height="44" alt="" /></td>
                                <td align="center"><img border="0" 

src="library/logo_linux.jpg" width="37" height="40" alt="" /></td>
                                <td align="center"><img border="0" 

src="library/logo_cisco.jpg" width="69" height="40" alt="" /></td>
                                <td align="center"><img border="0" 

src="library/logo_checkpoint.jpg" width="91" height="40" alt="" /></td>
                                <td align="center"><img border="0" 

src="library/logo_mysql.jpg" width="81" height="43" alt="" /></td>
                                <td align="center"><img border="0" 

src="library/logo_veritas.jpg" width="60" height="40" alt="" /></td>
                            </tr>
                        </table>
                    </td>
                </tr>
                <tr>
                    <td colspan="3" height="1" align="center" class="bg_blue"><img 

src="library/spacer.gif" height="1" width="1" alt="" /></td>
                </tr>
                <tr>
                    <td colspan="3" height="20" align="center" class="base5">&copy; 2005 

* - Alle rechten voorbehouden</td>
                </tr>
                <tr>
                    <td colspan="3" height="1" align="center" class="bg_blue"><img 

src="library/spacer.gif" height="1" width="1" alt="" /></td>
                </tr>
            </table>
        </td>
    </tr>
</table>

</body>
<!--<a href="getcatalogusKim.php">Kim</a>//-->
</html>
<?
// CLOSE DATABASE CONNECTION
mysql_close($connection);
?>

[Verwijderd]

Okee, er zijn dus twee condities in je script.

PHP:

// 1ste conditie
if(isset($_REQUEST["pid"]) && isset($inc_myrow)) {
}

// 2e conditie
if(isset($_POST["searchresult"])) {
}

Je zegt, ik zie mijn zoekresultaten niet.
Ik ga er vanuit dat je eerste conditie dus als `waar` (true) gezien wordt.
Het is dan logisch, omdat je een else-if constructie gebruikt, dat vervolgens je zoekresultaten níet getoond worden.
Wat kun je doen?

Wel draai de volgorde van de condities om:

PHP:

if(isset($_POST["searchresult"])) {
} else
if(isset($_REQUEST["pid"]) && isset($inc_myrow)) {
}

Of plaats de 2e conditiecontrole binnen de eerste
(waarbij ik zo snel even niet kon zien of dat uberhaubt gaat werken):

PHP:

if(isset($_REQUEST["pid"]) && isset($inc_myrow)) {

    if(isset($_POST["searchresult"])) {
        // toon zoekresultaten
    } else {
        // toon standaard data
    }

}

Daarnaast ben ik niet zo'n fan van de global REQUEST, en raadt ik je aan om gebruik te maken van POST en GET omdat je dan een beter overzicht over de gang van zaken houdt.
Verder zou ik een spuit-11 zijn als ik over veilig-/netheid begin...

[ Voor 17% gewijzigd door Verwijderd op 22-03-2006 12:37 ]

[MarcoZ20]

ok thnx, ga het proberen op de manier die je voorstelt.


Ik voeg alleen de zoekfunctie aan de site toe, de rest van het script is dus niet door mij gemaakt en ga ik verder ook niet aanpassen.

[MarcoZ20]

Nog een vraagje over het aanroepen van het script in het form. Volgens mij is het niet de juiste manier die ik gebruik. Kan iemand ermee helpen?

HTML:

<form style="display: inline;" method="post" action="?searchresult">Zoeken: <input type="text" value=""  

name="zoekterm" />&nbsp;<input type="image" src="library/arrow.gif" value="Submit" name="B1" /></form></td>

Het gaat om de action dus

[Verwijderd]

Het begint er steeds meer op te lijken dat je niet veel (niets?) van PHP weet, dus ik stel voor dat je eens een goed boek koopt en met die nieuwe kennis aan de slag gaat, want dit zijn wel heel erg basic vragen.. Succes.

[MarcoZ20]

WEet wel wat van php, kom alleen hier niet uit, doordat dit andere manier van opvragen is dan ik gewent ben...

[Swaptor]

Wat ik er van begrepen heb, is dat TS niet weet hoe hij de teruggekregen zoekdata in de originele pagina moet krijgen. (omdat deze wordt opgebouwd door externe includes).

Daarom moet ik me aansluiten bij ivy, en zeggen dat de opbouw zoals het nu is als volgt werkt:

ALS pid is gezet in de url EN $inc_myrow is ook aanwezig, DAN wordt het menu weergegeven (oa via een externe file), ANDERS geef alleen de default page weer (ook extern geladen)

In de herhaling: de code.

PHP:

<?
if (isset($_REQUEST["pid"]) && isset($inc_myrow))
{
    echo "<table border=\"0\" width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><TR><TD width=\"200\" valign=\"top\">";
    require("inc/menu.inc.php");
    echo "</TD><TD class=\"content\" valign=\"top\">".$content."</TD></TR></TABLE>";
} else
{
    include ("inc/default.inc.php");
}
?>