[Resolved] Help please?

bluebellmorning

I've got the PHP to extract the gigs from the selected year, and then if there's a setlist or whatever it makes the box on the right blue and links the 'S' to open a popup window. The only problem is with the R for 'reviews'. That comes from a different table - tmt_press. In tmt_press, it has press type (which is 'live' for the live reviews) and then the date of the gig is in the same format as the dates in tmt_gigography table.

I have two problems:

a) it doesn't give a grey 'R' box when there is no reviews.
b) if there is more than one review, it shows two boxes.

Can anybody help me fix the problem?

	$requetepress = "SELECT * FROM tmt_press WHERE type='live' AND date='$tmt_press->date'";
	$resultpress = mysql_query ($requetepress,$db);

while ($tmt_press = mysql_fetch_object($resultpress))

if($tmt_gigography->date =="")
{
echo "<span class=\"gigoinfono\">R</span> ";
}
else
{
echo "<a href=\"linky\" class=\"gigoinfo\">R</a> ";
}

paulnaj

I'm confused ...

// $tmt_press must already be defined before next line
$requetepress = "SELECT * FROM tmt_press WHERE type='live' AND date='$tmt_press->date'"; 
$resultpress = mysql_query ($requetepress, $db); 

// $tmt_press is now going to contain the $resultpress objects
while ($tmt_press = mysql_fetch_object($resultpress)){
    if($tmt_gigography->date ==""){ 
      echo "<span class=\"gigoinfono\">R</span> "; 
    } else { 
      echo "<a href=\"linky\" class=\"gigoinfo\">R</a> ";
    }
}

parawizard

instead of drawing R everytime a review is found make a IF statement that will detect if there is reviews or NOT. then if there is u output 1 R that is a link to a review and if theres no reviews output the R with no link

do you understand?

bluebellmorning

That seems to work better, but how can I get it to see if there is a review that matches the date?

paulnaj

Ok. I understand what you're trying to do, in principle, but i'm still confused about two of the variables you're using.

On the first line ...

$requetepress = "SELECT * FROM tmt_press WHERE type='live' AND date='$tmt_press->date'";

... you use $tmt_press->date for the SQL. So that must already have been defined. What is it? Because, later on, in the while(...) clause ...

while ($tmt_press = mysql_fetch_object($resultpress)){

... you store the result of the query in it.

Then, while cycling through 0, 1, or more possible reviews, you don't once refer to this table record ($tmt_press) at all.

But, instead, for each review that does exist, you ask whether the date in a different object ($tmt_gigography) is equal to "" before deciding what to print ...

if($tmt_gigography->date ==""){

... .

bluebellmorning

I only showed you part of the code...

 $db = mysql_connect("localhost","themagictreehouse_co_uk","pass");
 mysql_select_db("themagictreehouse_co_uk_1",$db);

 $requete = "SELECT * FROM tmt_gigography WHERE year='$year' ORDER BY date";
 $result = mysql_query ($requete,$db);


while ($tmt_gigography =mysql_fetch_object($result))

And then there's stuff that shows the venues, setlists etc... followed by the code for the reviews bit.

paulnaj

Ok ... but my question ... What is $tmt_press->date ?

On the first line ...

$requetepress = "SELECT * FROM tmt_press WHERE type='live' AND date='$tmt_press->date'";

... you use $tmt_press->date for the SQL. So that must already have been defined. What is it? Because, later on, in the while(...) clause ...

while ($tmt_press = mysql_fetch_object($resultpress)){

... you store the result of the query in it.

bluebellmorning

tmt_press->date should be tmt_gigography->date. I think I changed it accidently!

paulnaj

I just noticed your earlier post ...

That seems to work better, but how can I get it to see if there is a review that matches the date?

How's it working now?

bluebellmorning

Well, it puts a grey box there all the time... but I can't get it to see if there is a review.

I need to to see if there is at least 1 review for the given date.

paulnaj

What about this?

$db = mysql_connect("xxxx","xxxx","xxxx"); 
mysql_select_db("xxxx",$db); 

$requete = "SELECT * FROM tmt_gigography WHERE year='$year' ORDER BY date"; 
$result = mysql_query($requete,$db); 
while ($tmt_gigography = mysql_fetch_object($result)){
  /*  

  ...
  Do venues, setlists etc
  ...
  */
  $requetepress = "SELECT * FROM tmt_press WHERE type='live' AND date='".$tmt_gigography->date."'";  

  $resultpress = mysql_query ($requetepress, $db);  

  if(mysql_num_rows($resultpress) == 0){  

    echo "<span class=\"gigoinfono\">R</span> ";  

  } else {  

    echo "<a href=\"linky\" class=\"gigoinfo\">R</a> "; 
  } 
}

bluebellmorning

That's excellent, thank you.

paulnaj

Fantastic ... got there!

Remember to mark the thread [resolved] ... there's a link near the bottom of the page.