Troubles calling functions

[fadinha]

I have a troubles calling functions. I have four different functions in one page, and I'm using below code to switch them. If I use $POST, the whole page won't work (just the first one that lists the details from db to page). But If I use $GET it works with delMethod and EditMethod but not with do_EditMethod. I've been twiddling these around too much already.

Oh yeah, and the EditMethod calls a form and do_Edit.. should update the details into db.

	if (isset($_POST['action'])){
		 switch($_POST['action']) {
		       case 'EditMethod':
		                EditMethod();
		                break;
		       case 'do_EditMethod':
		 		do_EditMethod();
		 		break;
		    case 'delMethod':
		                delMethod();
		    	        break;
		 }}
else{
		 ListMethods();
		 }

I'd be happy if someone could help me with this.

[ucffool]

Can you add the following to the page (in your switch):

case default:
 echo "No match for POST 'action' = " . $_POST['action'];
  break;

Then run it through the 4 scenarios, so you can verify that the input you are getting is what you have in each of your cases?

[fadinha]

Still nothing happens....
This is bugging me a lot, because i got the same scripts working in another page, except in that page I'm just switching between two functions...

[ucffool]

what do you mean nothing happens?

[fadinha]

Well, the page doesn't switch between the functions, if I use $POST...
and if I use $GET, it switches between deleting and editing, but it doesn't call the do_Edit.. Or is the problem somewhere else???

Here's the whole code, and yeah now it's with $_GET:

		<?php

	if (isset($_GET['action'])){
	 switch($_GET['action']) {
	 	case 'EditMethod':
	       EditMethod();
	    	break;
	    case 'delMethod':
	       delMethod();
	    	break;
	    case 'Tallenna':
	 		do_EditMethod();
                            break;}
	 else{
	 	ListMethods();
	 }

	function ListMethods(){
		global $page ;

		$methods_per_page = "20";

		if (!isset($page) or $page=="") $page=1;
		$nexlimit = $page * $methods_per_page - $methods_per_page;

		$result = mysql_query("SELECT * FROM mcards ORDER BY MCID ASC limit $nexlimit,$methods_per_page");

		$result_all_rows = mysql_query("SELECT * FROM mcards");

		$totalrows = mysql_num_rows($result_all_rows);



		echo "<form method=\"post\" name=\"ListMCards\" action=\"methods.php\">
		<table border=\"1\" width=\"100%\" id=\"table1\">
		<tr>
		<td colspan=\"8\" align=\"center\">Menetelm&#228;kortteja yhteens&#228;: ($totalrows)</td>
		</tr>
		<tr>
		<td align=\"center\">ID</td>
		<td>Kortin nimi</td>
		<td>Viimeksi muokattu</td>
		<td align=\"center\">Options</td>
		</tr>";
		while ($row = mysql_fetch_array($result)){

			$MCID = $row['MCID'];
			$MCname = $row['MCname'];
			$MCcontent = $row['MCcontent'];
			$MCadded = $row['MCadded'];


			echo "<tr>
			<td align=\"center\">$MCID</td>
			<td>$MCname</td>
			<td>$MCadded</td>
			<td align=\"center\">[<a href=\"methods.php?action=EditMethod&MCID=$MCID\">Edit/View</a>]
                [ <a href=\"methods.php?action=delMethod&MCID=$MCID\">Delete</a> ]
			</td>
			</tr>";
		}//end while

		echo "</table></form>";
		// get pages:
		$totpages = ceil($totalrows/$methods_per_page);
		for ($i=1;$i<=$totpages;$i++) {
			if ($i==$page) {
				$pages .= " $i ";
			} else {
				$pages .= " <a href=\"methods.php?action=ListMethods&page=$i\">$i</a> ";
			}
		}
		echo "<br>[ <a href=\"add_mcard.php\">LIS&#196;&#196; UUSI MENETELM&#196;KORTTI</a> ]<br>";
		echo "<br><center>Pages: $pages <br>";
}
function delMethod(){

	  $MCID = $_GET['MCID'];
      $result = mysql_query("DELETE from mcards WHERE MCID='$MCID'");


      echo "<div align=\"center\" class=\"div\">Menetelm&#228;kortti poistettu. <br>Odota hetki...</div>";
      echo "<META HTTP-EQUIV=Refresh CONTENT=\"2; URL=methods.php\">";

	}
function EditMethod(){

     EditMethodForm();

}
function EditMethodForm(){

		//GET THE SELECTED MCARD ID
		$MCID = $_GET['MCID'];

		$result = mysql_query("SELECT * FROM mcards WHERE MCID='$MCID'");
		$row = mysql_fetch_array($result);

		//SET THE DETAILS TO VARIABLES
			$MCID = $row['MCID'];
			$MCname = $row['MCname'];
			$MCcontent = $row['MCcontent'];




		// PRINT THE FORM
echo"		<form method=\"post\" name=\"EditMethodCard\" action=\"methods.php\">
			<table><tr><td>Menetelm&#228;n nimi: </td>
			<td><input type=\"text\" name=\"MCname\" size=\"50\" value=\"$MCname\"></td></tr>
			<tr><td colspan=\"2\"><textarea name=\"MCcontent\" id=\"MCcontent\" rows=\"10\" cols=\"40\">
			$MCcontent
			</textarea><script language=\"javascript1.2\">make_wyzz('MCcontent');</script></td></tr>
			<tr><td><input type=\"hidden\" name=\"MCID\ value=\"$MCID\"></td></tr>
			<tr><td><input type=\"submit\" name=\"submit\" value=\"Tallenna\"></td>
			<td><input type=\"submit\" name=\"cancel\" value=\"Peruuta\"></td></tr></table></form>";


}
function do_EditMethod(){
				//SET THE DETAILS TO VARIABLES
				$MCID = $_GET['MCID'];
				$MCname = $_GET['MCname'];
				$MCcontent = $_GET['MCcontent'];



		//this function will check fields incase of javascript not working.
				if(isset($_GET['MCname']) && ($_GET['MCcontent']) ){

				$result = mysql_query("UPDATE mcards SET MCname='$MCname',MCcontent='$MCcontent',MCadded=NOW() WHERE MCID='$MCID'") or die ("Error: ". mysql_error());
					if (!$result){
						echo "<div align=\"center\" class=\"div\">Error: Tietoja ei voitu p&#228;ivitt&#228;&#228;.</div>";
          				echo "<META HTTP-EQUIV=Refresh CONTENT=\"3; URL=methods.php\">";
					}
					else{
						echo "<div align=\"center\" class=\"div\">Informaatio on tallennettu. <br>Odota hetki...</div>";
						echo "<META HTTP-EQUIV=Refresh CONTENT=\"4; URL=methods.php\">";
					}
		}
		else{
			 EditMethodForm();
			echo "<center><font class=\"error\">Error: Please fill all fields.</font></center>\n";
			}
	}

?>

And yeah, this is the biggest thing so far I've done with PHP, so plz be gentle with me :o

[ucffool]

Right off the bat, there is a } missing before the else. I used the following code to make sure that your switch is working (tested all possible GET strings you are expecting), which it is.
Secondly, your case matches are case sensitive, so if your query string is action=EditMethod it will work, but if it is action=editmethod it will not. I would strongly recommend always using lowercase in the situation that you are passing some sort of instructions through GET or POST.
I haven't looked past these two items because, after checking the above two noted items, it works fine through GET. Here is the code I used:

<?php 

    if (isset($_GET['action'])){
     switch($_GET['action']) {
         case 'EditMethod':
           echo 'EditMethod()';
            break;
        case 'delMethod':
           echo 'delMethod()';
            break;
        case 'Tallenna':
             echo 'do_EditMethod()';
                            break;}
     }else{
         echo 'ListMethods()';
     }
?>