$op and functions

Client Joined: Oct 07, 2003 Posts: 735 Location: Ohio

I am updating my MILPACS module to Release Candidate 3 (RC3).

I have the upgrade working except for that last part.

At the end of the upgrade I want to be able to allow the user to import any vWar data into MILPACS.

At the end of my upgradeRC2-RC3.php page I have the following.

Code:

echo "<hr><b>Operation Complete!</b><hr>\n";


echo "<hr><b>REMOVE THIS SCRIPT WHEN FINISHED!</b><hr>\n";





form action is removed because of NukeSentinal.


<form>


$table = 'vwar';


if(mysql_num_rows($db->sql_query("SHOW TABLES LIKE '".$table."'"))==1) {


echo "vWar tables have been detected <br><br>";


echo "Do you want to import your vWar data into MILPACS?<br><br>";


echo "<select name=\"action\">";


echo "<option value=\"import_keep_vWar\">Import but keep your vWar tables</option>\n";


echo "<option value=\"import_delete_vWar\">Import and delete your vWar tables</option>\n";


echo "</select>&nbsp;&nbsp;<input type=\"submit\" value=\"Continue\" /><br><br>\n";


echo "<input type=\"hidden\" name=\"op\" value=\"ImportvWar\"/>";


echo "</form>";


}





CloseTable();


include("footer.php"); 








function ImportvWar($action) {





if ($action == 'import_keep_vWar') {





   echo"You have chosen to import and keep you vWar tables";





}else{





   echo"You have chosen to import and delete you vWar tables";


}


}

For now I'm just trying to get it to display the correct echo statment inside the function.

What happens is that it instead displays the default.php page.

The milpacs_install.php page has this which directs traffic.

Code:

switch($op) {


    default:include("milpacs_installer/default.php" );break;


    case "remove":include("milpacs_installer/remove.php");break;


   case "install":include("milpacs_installer/install.php");break;


   case "upgradeRC1-RC2":include("milpacs_installer/upgradeRC1-RC2.php");break;


   case "upgradeRC2-RC3":include("milpacs_installer/upgradeRC2-RC3.php");break;   


  }

I would rather not create another php page just for the import feature, yet I can't get it to work at the end of upgradeRC2-RC3.php

Posted: Mon Nov 19, 2007 12:43 am

We really do need to see the parameters of the FORM action

My guess is that is it submitting to your milpacs_install.php script with no $op parameters, thus it never goes to to the upgradeRC2-RC3.php script

Posted: Mon Nov 19, 2007 6:25 am

Code:

echo "<hr><b>Operation Complete!</b><hr>\n";


echo "<hr><b>REMOVE THIS SCRIPT WHEN FINISHED!</b><hr>\n";





echo "<form action='".$_SERVER['PHP_SELF']."' method='post'>\n";


$table = 'vwar';


if(mysql_num_rows($db->sql_query("SHOW TABLES LIKE '".$table."'"))==1) {


echo "vWar tables have been detected <br><br>";


echo "Do you want to import your vWar data into MILPACS?<br><br>";


echo "<select name=\"action\">";


echo "<option value=\"import_keep_vWar\">Import but keep your vWar tables</option>\n";


echo "<option value=\"import_delete_vWar\">Import and delete your vWar tables</option>\n";


echo "</select>&nbsp;&nbsp;<input type=\"submit\" value=\"Continue\" /><br><br>\n";


echo "<input type=\"hidden\" name=\"op\" value=\"ImportvWar\"/>";


echo "</form>";


}





CloseTable();


include("footer.php"); 








function ImportvWar($action) {





if ($action == 'import_keep_vWar') {





   echo"You have chosen to import and keep you vWar tables";





}else{





   echo"You have chosen to import and delete you vWar tables";


}


}


?>

I just noticed I do not have any ImportvWar in my case statement. I need this option to exist in two different places. If they are doing a full install and if they are upgrading from RC2 to RC3.

Code:

switch($op) { 


    default:include("milpacs_installer/default.php" );break; 


    case "remove":include("milpacs_installer/remove.php");break; 


   case "install":include("milpacs_installer/install.php");break; 


   case "upgradeRC1-RC2":include("milpacs_installer/upgradeRC1-RC2.php");break; 


   case "upgradeRC2-RC3":include("milpacs_installer/upgradeRC2-RC3.php");break;    


  }

Posted: Mon Nov 19, 2007 6:26 am

Wow it worked this time. I have never before been able to include the form action of the code without NukeSentinal choking on it.

Posted: Mon Nov 19, 2007 7:03 am

Raven's been working on something with Bob on NukeSentinel... Wink