Problem displaying what I want to

Radon3k

I have a set of links and whenever a link is clicked I want only the information from that link to be displayed. Example:

Link 1
Link 2

If link one is clicked then I want only the text, "Link 1" to be displayed and if link two is clicked I want only the text "Link 2" to be displayed. Nothing else. Here is the code that I have, however it doesn't work.

if($threadid) {
displayThread($threadid);
}

Also, the following does not work:

if($_GET['threadid']) {
displayThread($_GET['threadid']);
}

// Or

if($_POST['threadid']) {
displayThread($_POST['threadid']);
}

Any help? Thanks! 🙂

hermand

Whats the code for the display thread function?

Radon3k

function displayThread($threadid) {
	$getthreads = mysql_query("SELECT * FROM topics");
	if(!$getthreads) {
		echo("Unable to obtain thread information. Please try again later.");
		exit();
	}

while($threads = mysql_fetch_array($getthreads)) {
	$threadid = $threads['ID'];
	$threadtitle = $threads['title'];
	$threadmessage = $threads['message'];

	echo("<b>$threadtitle</b>");
	echo("<br>");
	echo("<b>Posted by:</b> $author");
	echo("<br><br>");
	echo($threadmessage);
	echo("<br><br>");
}
}
?>

hermand

Redefing the same variable for a different use isn't wise, and also, shouldn't the MySQL query be WHERE id='$threadid' or something?

Radon3k

I actually didn't even realize that I had redefined a variable. I did change that and fixed up a few other things. When a link is clicked, it does display the proper information, however the rest of the threads are still shown. I merely want "Link 2" displayed and nothing else. Thanks for the help thus far! 🙂

hermand

I dont quite understand how youre trying to go about doing what your doing. I think I'll have to leave it until I get back from college (4:00 BST)

Radon3k

Anybody else? I can't seem to get this to work and I've been working on it for a while. When the link is clicked the only thing that should be displayed should be that topic. That's it. Not the list of topics and that topic, just that one topic. Thanks. 🙂

crowezr

Don't you need a way to select the specific thread from the topics table? The way it reads now, you will select all records in the topics table, not a specific one. Maybe I'm mis-understanding your problem.

devinemke

change:

$getthreads = mysql_query("SELECT * FROM topics");

to:

$getthreads = mysql_query("SELECT * FROM topics WHERE threadid = $threadid");

...or whatever the name of your primary key field is

Radon3k

echo("<a href=\"$_SERVER[PHP_SELF]?threadid=$id\">$title</a>" . "<b> By:</b> " . $author . "on " . date("l, F j, Y") . "<br><br>");

// That's the link code that is embedded in the displayTopics function. Further on down in the actual HTML part of the page I have this:

if($_GET['threadid']) {
	displayThread($_GET['threadid']);
}

That's the code I have. It does specify which thread to display depending on which link was clicked.

devinemke

...yes, but in your SQL query inside your displayThread() function you are not specifying WHICH thread to pull out, you are pulling all the threads.

again, you need to change your query:

$getthreads = mysql_query("SELECT * FROM topics WHERE threadid = $threadid");

Radon3k

My apologies, I should have updated the above code. The code I have in my file (which I'll post below) does reflect the WHERE clause.

<?php
function displayThread($threadid) {
	$getthreads = mysql_query("SELECT * FROM topics WHERE ID = $threadid");
	if(!$getthreads) {
		echo("Unable to obtain thread information. Please try again later.");
		exit();
	}

while($threads = mysql_fetch_array($getthreads)) {
	$threadtitle = $threads['title'];
	$threadmessage = $threads['message'];

	echo("<b>$threadtitle</b>");
	echo("<br>");
	echo("<b>Posted by:</b> $author");
	echo("<br><br>");
	echo($threadmessage);
	echo("<br><br>");
}
}
?>

<html>
<head>
<title>Project Rain</title>
</head>

<?php

displayTopics();

if($_GET['threadid']) {
	displayThread($_GET['threadid']);
}
?>

Radon3k

Anybody have any ideas?

Radon3k

Not sure which page this is on by now but I'm still stuck. Please help if possible. Thanks. 🙂

Daveyz1983

I am not sure about your database structure, in any case, give this a try

change

$getthreads = mysql_query("SELECT * FROM topics WHERE ID = $threadid");

$getthreads = mysql_query("SELECT * FROM topics WHERE ID = '$threadid' ");

Radon3k

Nope, that didn't work. 🙁

Daveyz1983

So your trouble now is displaying ALL THE RESULTS in the database? or blank?

Because from what I see, after you put the WHERE clause, its either going to display an error, blank, but don't think it will display everything.

Can you put a

echo mysql_error();

after your mysql_query? So you can see whether there is anything wrong with your sql statement.

Morover I notice there isn't any mysql_connect or pconnect inside the function, did you connect to the database at all in the script? Please double check, mysql_error helps.

Radon3k

Sorry, there is no error. It displays the thread I want and only that thread. That problem is resolved. However, now I want it to display nothing but that thread. As it is, it shows that thread and the topics -- which is what I don't want. I want only the thread. Make sense?

Daveyz1983

Its a human error then , because your code is

<html>
<head>
<title>Project Rain</title>
</head>

<?php

displayTopics();

if($GET['threadid']) {
displayThread($GET['threadid']);
}
?>

take away the displayTOpics(); and ur topics will be gone.

Radon3k

That makes sense and it also makes me feel stupid 😉. I can fix that easily, however if I want to to link to another page (and I fixed the link part from the PHPSELF to the proper page) how do I display that thread on another page? Thanks. 🙂