[RESOLVED] Problems with a date function

duckduckgoose

I'm sure there's a better way to do this, but I'm still learning so go easy....

I'm trying to write a function that will print out options for a select menu. For the current month, I want to add one month to it and make that option the default checked one. I tried to use the following, but it keeps defaulting to February, which means that the part where it should get the month isn't working. :/

I'm new to functions and I must be doing something wrong, but I can't figure it out. Help?


function print_months_d() {

    for ($i=1; $i<=12; $i++) {

    $param1=date('n');       

    if($i == $param1) {  // NOTE: I tried the opposite verbage as well (i.e., $param1 == $i
            $month_number = $i + 1; 
            $month = date('F',mktime(0,0,0,$month_number)); 
            echo "<option value=\"$month_number\" CHECKED>$month</option>";
    } else { 
            $month_number = $i; 
            $month = date('F',mktime(0,0,0,$month_number)); 
            echo "<option value=\"$month_number\">$month</option>";
    }

    }
}

tsinka

Hi,

replace checked with selected and it should work. checked doesn't work with drop down inputs.

Thomas

Houdini

Hi,

replace checked with selected and it should work. checked doesn't work with drop down inputs.

Thomas

also remove or comment this line

$month_number = $i + 1;

or it will select the next month.

duckduckgoose

Hi, thanks for your replies. I did change the CHECKED to SELECTED (ohhhhh yeahhhh, btw), but it's still defaulting to February. It should default to August. :/

Any other insights?

tsinka

Try this one:

function print_months_d() {
    $nextMonth = date('n')+1;
    $nextMonth = $nextMonth%12; // special case december

for ($i=1; $i<=12; $i++) {
    $selected = ($i == $nextMonth) ? " selected" : "";
    $month = date('F',mktime(0,0,0,$i));
    echo "<option value=\"$i\"$selected>$month</option>";
}
}

Thomas

Houdini

It will default to the next month the way your code is (but here where I am it is still July). It shows July (minus the $i=$i+1) as written well I did change it some here is what I have

if($i == $param1) {  // NOTE: I tried the opposite verbage as well (i.e., $param1 == $i (this can actually be either way if it is equal then it is).
                //$month_number = $i + 1;
                $month = date('F',mktime(0,0,0,$month_number));
                echo "<option value= '$month_number' selected>$month</option>";

just use single quotes instead of escapeing double quotes within double quoted strings.

duckduckgoose

Oh, well that works just lovely! Thank you so much.