Form not passing variables?

becky

Hi all! This is the first time I'm posted here, but I've been learning lots and lots just as a lurker, so thanks for that. 🙂

I'm working off some calendar functions I found in the snippet library to make a calendar for my blog site. When someone clicks "My calendar" calendar.php will show a calendar of the current month with linkable dates and such (haven't gotten that far yet 🙂. At the bottom of the calendar there will be two drop-down boxes for the user to select a different calendar. Then calendar.php will use those variables to create a new calendar. The problem is that the form drop-down boxes aren't passing the proper variables to back to the script. The first time oe selects a new month, no variables are passed at all; subsequent times, variables are passed but they seem to be randomly selected (ie they don't match what the user selected).

The page is <www.dalqe.com/calendar.php> and the code is below. Thanks for any advice you can give.

<?
include("header.php");
?>

<tr>

<td valign=top>
<div class=genericmiddlehead style="margin-bottom: 3px">
Contact Me
</div>
<div class=genericmiddlesubhead>
Via Email
</div>
<div class=genericmiddlecontent style="margin-bottom: 3px">
Click <a href="mailto:becky@dalqe.com">here</a> to send me an email. Your message will automatically be forwarded to my regular @.com inbox.
</div>

<div class=genericmiddlesubhead>
Via SMS
</div>
<div class=genericmiddlecontent style="margin-bottom: 3px">
SMS stands for "short message service." To send a text message to my cell phone, fill out the following form. SMS is totally free and guaranteed to make my day!
<P>
<FORM method="post" action="sendsms.php">
Your name:<br>
<INPUT name="name" type="text" size="35"><br><br>
Message:<BR>
<INPUT type="radio" name="msg" value="Wanna hang out?"> Wanna hang out?<BR>
<INPUT type="radio" name="msg" value="Just saying hi!"> Just saying hi!<BR>
<INPUT type="radio" name="msg" value="I think you're cute 🙂"> I think you're cute 🙂<BR>
<INPUT type="radio" name="msg" value="Very cool website."> Very cool website.<BR>
<INPUT type="radio" name="msg" value="custom"> Custom message:<BR><input type=text name="own" size="35"><BR><BR>

			If you want a reply, where should I send it?<BR>
			<input type="radio" name="back" value="1"> Call me @ <input type=text name=call size="30"><BR>
			<input type="radio" name="back" value="2"> Email me @ <input type=text name=em size="30"><BR>
			<input type="radio" name="back" value="3"> SMS me @ <input type=text name=sms size="30"><BR>
			<input type="radio" name="back" value="none"> No reply needed!<BR><BR>

			<INPUT type="submit" value="Send SMS">&nbsp&nbsp<INPUT type=reset value="Reset Form">


			</FORM>

</div>
</td>

</td>

</tr>

</table>

<?
include("footer.php");
?>

becky

I apologize, I copied the wrong code into the last post. The proper one is:

<?
include("header.php");
include("calendarfunctions.php");
?>

	<tr>

<td valign="top">
<?

if (mod == "newcal")

{ 
	$year = $y;
	$month = $m;

}

else

{

	$year = date("Y");
	$month = date("m");

}

gen_month($year,$month);

?>
<br><br>
<div align=center>

	<form method=post action="calendar.php?mod=newcal&m=<? echo $tempmonth; ?>&y=<? echo $tempyear; ?>">

	<select name=tempmonth size=1>
	<option value=01 selected>January</option>
	<option value=02>February</option>
	<option value=03>March</option>
	<option value=04>April</option>
	<option value=05>May</option>
	<option value=06>June</option>
	<option value=07>July</option>
	<option value=08>August</option>
	<option value=09>September</option>
	<option value=10>October</option>
	<option value=11>November</option>
	<option value=12>December</option>
	</select>

	<select name=tempyear size=1>
	<option value=2002 selected>2002</option>
	<option value=2003>2003</option>
	<option value=2004>2004</option>
	</select>

	<input type=submit value=Go>

</FORM>

</div>
</td>

</table>

<?
include("footer.php");
?>

Anon

Becky -

if (mod == "newcal")

change to:

if($mod == "newcal")

Anon

I think I found something...

Your code is :

	<form method=post action="calendar.php?mod=newcal&m=&y=">

	<select name=tempmonth size=1>
	<option value=01 selected>January</option>
	<option value=02>February</option>
	<option value=03>March</option>
	<option value=04>April</option>
	<option value=05>May</option>
	<option value=06>June</option>
	<option value=07>July</option>
	<option value=08>August</option>
	<option value=09>September</option>
	<option value=10>October</option>
	<option value=11>November</option>
	<option value=12>December</option>
	</select>

	<select name=tempyear size=1>
	<option value=2002 selected>2002</option>
	<option value=2003>2003</option>
	<option value=2004>2004</option>
	</select>

	<input type=submit value=Go>

</FORM>

I think the problem comes from that : « action="calendar.php?mod=newcal&m=&y=" »

Your code should be :

	<form method=post action="calendar.php">

<input name="mod" value="newcal" type="hidden">
<input name="m" value="" type="hidden">
<input name="y" value="" type="hidden">
<select name=tempmonth size=1>
<option value=01 selected>January</option>
<option value=02>February</option>
<option value=03>March</option>
<option value=04>April</option>
<option value=05>May</option>
<option value=06>June</option>
<option value=07>July</option>
<option value=08>August</option>
<option value=09>September</option>
<option value=10>October</option>
<option value=11>November</option>
<option value=12>December</option>
</select>

	<select name=tempyear size=1>
	<option value=2002 selected>2002</option>
	<option value=2003>2003</option>
	<option value=2004>2004</option>
	</select>

	<input type=submit value=Go>

</FORM>

You should correct a part of your script because when I look to the source, there are a lot of notices !

Example : <br />
<b>Warning</b>: mysql_query(): supplied argument is not a valid MySQL-Link resource in <b>/home/becky/dalqe-www/archivesidebar.php</b> on line <b>64</b><br />
<br />
<b>Warning</b>: mysql_num_rows(): supplied argument is not a valid MySQL result resource in <b>/home/becky/dalqe-www/archivesidebar.php</b> on line <b>65</b><br />
()</a><br><a class=entriesbylinks href="index.php?what=books
">books
<br />

Anon

Becky, here's what you need:

<?
include("header.php");
include("calendarfunctions.php");
?>

    <td width=20% valign=top> 
         <div class=sidebar> 
         <? 
                   include("leftsidebar.php"); 
          ?> 
          </div> 
     </td> 
     <td width=15> 
     </td> 
     <td valign="top"> 
                 <? 
                      if ($mod == "newcal") { 
                            $year = $y; 
                            $month = $m; 
                     } else { 
                           $year = date("Y"); 
                           $month = date("m"); 
                     } 
                     gen_month($year,$month); 
                  ?> 
                 <br><br> 
                 <div align=center> 

                 <form method=post action="calendar.php"> 
                          <input type="hidden" name="mod" value="newcal">

                          <select name="m" size=1> 
                                <option value=01 selected>January</option> 
                                <option value=02>February</option> 
                                <option value=03>March</option> 
                                <option value=04>April</option> 
                                <option value=05>May</option> 
                                <option value=06>June</option> 
                                <option value=07>July</option> 
                                <option value=08>August</option> 
                                <option value=09>September</option> 
                                <option value=10>October</option> 
                                <option value=11>November</option> 
                                <option value=12>December</option> 
                         </select> 

                        <select name="y" size=1> 
                               <option value=2002 selected>2002</option> 
                               <option value=2003>2003</option> 
                               <option value=2004>2004</option> 
                         </select> 

                         <input type=submit value=Go> 

                 </FORM> 

                 </div> 
                 </td> 

                 <td width=15> 
                 </td> 

                 <td width=20% valign=top> 
                 <div class=sidebar> 
                 <!-- 
                 <? 
                           include("archivesidebar.php"); 
                 ?> 
                 --> 
                 </div> 
     </td>

</tr>

</table>

<?
include("footer.php");
?>

becky

HL and Bobby -

So far, no dice. When I make the changes to the form that HL suggested, I get a calendar for December 1999. Bobby, you're right that I missed the "$" in front of mod, but it doesn't seem to be fixing the behavior of the script.

Something else I tried was using
if (isset($HTTP_POST_VARS['tempmonth']) etc, but it always returned false (that is, the calendar for the current month was display).

Any other ideas?

(The bit that really confuses me is why when after the second reload of the script, when variables are actually passed, they're not the ones corresponding to the user input. Try it out and you'll see).

becky

That would seem to make sense, Bobby, but it's just displaying a December calendar repeatedly.

I guess the other possibility is that there's something wrong with the functions. However, earlier I had it set up where I was defining

$show = $tempyear.$tempmonth;

and then doing some string splitting to define $year and $month.

If you typed in

calendar.php?show=200209

it worked just fine. The problem seems to lie in getting the variables from the form in the first place.

becky

Oi! You were right - I didn't notice that you had changed the names in the select tags, which was just me being dumb.

Isn't it annoying how the right answers always seem so obvious once you have them?

Thanks again 🙂