Change of td color with php if else statement

Question

I want to change color of my td as per the date received from the sql query. Please help with following -

I want to change the color of td to green if the fetched date is less than today's date.

<?php
    if($mydate > date('d-M-Y'))
    {
        echo "<td style="background-color:green;">";
    }
    else
    {
       echo "<td>";
    }
?>

aldin27 has corrected one part (you did have an error there), but you might want to check your if statement to check you'd actually create the coloured td if it's true. — mfisher91, Sep 11 '15 at 09:37
you want the fetched date to be less than today but you have the greater than sign `>` in your code — CodeGodie, Sep 11 '15 at 09:37
@CodeGodie your correct. Maybe he's too confused about greater than and less than. lol — aldrin27, Sep 11 '15 at 09:38
is `$mydate` coming from a database? is it a string or an actual date object? show the code where you declare this — CodeGodie, Sep 11 '15 at 09:40
while($data=oci_fetch_array($array)) { $idno=$data[0]; $caseno=$data[1]; $pname=$data[2]; $rname=$data[3]; $purpose=$data[4]; $mydate=$data[5]; ?> — Ramuk Tima, Sep 11 '15 at 09:48
so `$mydate` is coming from your database. How is it stored in your database? is it of type DATE, DATETIME, .. ? — CodeGodie, Sep 11 '15 at 09:50
also can you `var_dump($data)` inside your `while` loop to see what you have — CodeGodie, Sep 11 '15 at 09:51
if its DATETIME then it is probably stored like this: `2015-09-11 05:57:05` not like this `11-sep-2015` — CodeGodie, Sep 11 '15 at 09:57

GiamPy · Answer 1 · 2015-09-11T09:55:34.793

2

You cannot check dates in string formats.

One way you could do that is by formatting dates in UNIX timestamp (so that you have an integer) allowing you to check properly if a date is actually newer or older than another one.

$mydate = (new DateTime($mydate))->format('U');
$date   = (new DateTime(date('Y-m-d')))->format('U');

if($mydate > $date)
{
    echo "<td style='background-color: green;'>";
}
else
{
    echo "<td>";
}

edited Sep 11 '15 at 09:55

answered Sep 11 '15 at 09:39

GiamPy

3,543
3
30
51

Do not downvote without stating the reason, please. – GiamPy Sep 11 '15 at 09:41
explain why this would answer the question – CodeGodie Sep 11 '15 at 09:41
thanks. looks better – CodeGodie Sep 11 '15 at 09:44
I can do this in JavaScript, is it not the same in PHP? `if ("2015-09-11" > "2015-09-10") ...` – mplungjan Sep 11 '15 at 09:44
PHP does not work that way. @mplungjan – Sougata Bose Sep 11 '15 at 09:46
@mplungjan: You must convert the date from string to a Date() object to do that. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date – GiamPy Sep 11 '15 at 09:46
1

@GiamPy No you do not need to do that if they are both string and both in YYYYMMDD format since the comparison will work on the string values of the digits too – mplungjan Sep 11 '15 at 09:48
@mplungjan Yes if the format is like that then no need to convert. PHP will treat them as integer. – Sougata Bose Sep 11 '15 at 09:49
http://stackoverflow.com/a/2114029/295783 – mplungjan Sep 11 '15 at 09:50
I'd rather have them in an integer or DateTime format although, but that's just my personal preference. – GiamPy Sep 11 '15 at 09:53
should be `if($mydate < $date)` since he said _"if the fetched date is less than today's date"_ – CodeGodie Sep 11 '15 at 09:59
@CodeGodie: I have just used what he's given us in the question. – GiamPy Sep 11 '15 at 09:59

score 1 · Answer 2 · edited Mar 31 '22 at 16:01

1

First, convert your both time to a single format and compare

try following

I have used div instead of td for ease of use

<?php
    $mydate="08-sep-2015";
    //$mydate="11-sep-2015";
    $today     = date('d-M-Y');
    $todayTime = strtotime($today);
    $compTime  = strtotime($mydate);
    if($compTime < $todayTime) {
      echo "<div style='background-color:green;'>fgd</div>";
    }
    else{
      echo "<div>sdfsdf</div>";
    }
?>

edited Mar 31 '22 at 16:01

xxx

1,153
1
11
23

answered Sep 11 '15 at 09:43

Sarath

608
4
12

@RamukTima you need to show us the code to see how you come up with `$mydate` – CodeGodie Sep 11 '15 at 09:47
@CodeGodie: [He mentioned it.](http://stackoverflow.com/questions/32520166/change-of-td-color-with-php-if-else-statement#comment52898083_32520166) – GiamPy Sep 11 '15 at 09:48
while($data=oci_fetch_array($array)) { $idno=$data[0]; $caseno=$data[1]; $pname=$data[2]; $rname=$data[3]; $purpose=$data[4]; $mydate=$data[5]; ?> – Ramuk Tima Sep 11 '15 at 09:50
@GiamPy he did mention it, but i think he is wrong since this is coming from the database, it would probably be stored as `2015-09-11` – CodeGodie Sep 11 '15 at 09:54
Updated the code please check it – Sarath Sep 11 '15 at 09:54

score 0 · Accepted Answer · answered Sep 11 '15 at 09:52

0

$mydate = "2015-10-01";
if(strtotime($mydate) > strtotime(date('Y-m-d')))
{
    echo "<td style='background-color:green;'>testing</td>";
}
else
{
   echo "<td>testing</td>";
}

answered Sep 11 '15 at 09:52

Shailesh Katarmal

2,757
1
12
15

thanks for help. It worked well. – Ramuk Tima Sep 11 '15 at 09:58

score -1 · Answer 4 · edited Mar 31 '22 at 18:23

-1

<!DOCTYPE html>
<html>

<head>

</head>

<body>

  <table>

    <tr>

      <?php
        $mydate = "2015-10-01";
        if(strtotime($mydate) > strtotime(date('Y-m-d')))
        {
            echo "<td  bgcolor='#FF0000'>testing</td>";
        }
        else
        {
           echo "<td>NOt Green</td>";
        }
        ?>

    </tr>
  </table>

</body>

</html>

edited Mar 31 '22 at 18:23

xxx

1,153
1
11
23

answered Sep 11 '15 at 10:06

Subodh

1
3

You are wrong. http://jsfiddle.net/mplungjan/Lqh4m7bx/ – mplungjan Sep 11 '15 at 11:14
yes .Your right.Instead of should i use can not.Sorry for the type.My point that you should use bgcolor instead of background-color:.Any way thanx for your info. – Subodh Sep 11 '15 at 15:01
I meant: you are wrong. The style attribute contains css- it does exactly the same as bgcolor but using 21st century values. I did not refer to spelling mistake – mplungjan Sep 11 '15 at 15:05
The HTML bgcolor attribute was deprecated back in 1997 (in W3C HTML 4.0); you should use the CSS background-color style instead. – ro͢binmckenzie Sep 14 '15 at 07:42

Change of td color with php if else statement

4 Answers4