Re: newbie needs help

I do apologize for the non-list reply, I will make sure and keep that in there on the next emails. I checked this morning and there was only one image, so the second mymonth code did not execute, but I replaced the code with what Richard gave me and it works also. And about the {} brackets, I am having to teach myself PHP code while trying to update and maintain a preprogrammed website so most of the code I see and I am using is chopped up code from the old programmer (which he did not keep a very tidy code). I figure out what a line of code does and then try to manipulate the existing code into what I need, so my syntax comes from the old programmer and what he had. On another note I want to say THANK YOU again, your response to my question blew my mind on how quickly you guys responded, I have posted questions and sent emails on some other programming sites and either takes a few days to hear back or I never get the answer I need. I will only be asking questions about PHP here from now on. You have a loyal user forever. Ben Daniel Brown wrote:

    Good catch on the non-list reply, Richard.  I didn't even notice that.

On 4/30/07, *Richard Davey* < rich@corephp.co.uk <mailto:rich@corephp.co.uk>> wrote:

    Ben Clapp wrote:

Thank you again for the help, it does work now but with an

    issue, here

is the code that i have for it right now:

    You should always reply to the php mailing list, so other people can
    benefit from the answers we give.

With this it works, but i am sure that when mymonth == 5 (may 1st, tomorrow) I will have two of the same pictures.

    You are right, you will. Because you're running the mothers day check
    twice, once with the 'new' code I gave you, and once with your old
    code.

If I take out this bit of code from the above code: then NOTHING shows up. Am I not getting something right or is there something elese that is getting in the way or is something not being completed?

    You've got missing { } around your if blocks. You also could make the
    code a lot more tidy / easy to read. Try the following (it replaces
    entirely the code you emailed me, swap all of it for this)

    <?php
         $mymonth = date('m');

         // novenmber is pancreatic cancer month
         if ($mymonth == 11)
         {
    ?>
    <tr><td align="center" bgcolor="#ffffff"><a
    href="http://pancan.com/Patient/pancreatic.html" target="_blank"><img
    src="../images/ads/pancanNov_banner.jpg" width="777" height="182"
    border="0"></a></td>
    </tr>
    <?php
         }

         $start_date = strtotime('13 April '.date("Y"));
         $end_date = strtotime('13 May '.date("Y"));
         $current = time();

         if ($current >= $start_date && $current <= $end_date)
         {
    ?>
    <tr><td align="center" bgcolor="#ffffff"><a
    href="../images/passportad/mothersdayBanner.jpg" target="_blank"><img
    src="../images/passportad/mothersdayBanner.jpg" width="771"
    height="112"
    border="0" /></a></td>
    </tr>
    <?php
         }
    ?>

    Cheers,

    Rich
    --
    Zend Certified Engineer
    http://www.corephp.co.uk

    "Never trust a computer you can't throw out of a window"

-- Daniel P. Brown [office] (570-) 587-7080 Ext. 272 [mobile] (570-) 766-8107