How do I assign radio bttn. value inside

plaihonen

I'm trying to create a page where web surfer can choose a file by using radio buttons and then have this file e-mailed to him/her.

In order to prevent copious lines of code (if, elseif, etc...) I'm trying to assign into $filename variable a name of the real file that the script will use. The filename will be selected with radiobuttons and value of the selection should be applied into the path that points to the file.

At the moment "$fp = fopen($filename, "r");"
results only
(tones/.txt,"r") thus not very good at finding any files.

My newbie code:

<input type="radio" name="tonefile" value="dsandstorm">Sandstorm<br>
<input type="radio" name="tonefile" value="dfeelthebeat">Feel the Beat

<?
$filename = "tones/$tonefile.txt";
$fp = fopen($filename, "r");
$file_contents = fread($fp, filesize($filename));
fclose($fp);

//split with carriage return
$line = explode("\n", $file_contents);

//echo "$line[0]<br>";
//echo "$line[1]<br>";
//echo "$line[2]<br>";
?>

Script continues with e-mailing the contents of the file

Thank you for your help....

Petri

narch31

If you know the associated file name for each radio button, just attach the file name to the button.
<input type=radio name="filename" value="yourfile.txt">

then use

$fp = fopen($filename, r)

I think this might help 🙂

plaihonen

Thank you very much. I ripped off one extra line from my code. Seems like I was trying to make it more complicate, than necessary.

Now I also finally grasp the value variables in form elements.

Petri

[deleted]

Security note!

Your method is nice, but it allows clients to fake your form, and put another filename in place of the your filenames.

This means they can get your script to send them any file they like as long as they know the location. On unix all the important files have predifened locations.

A better solution would be to put all the filenames into an array in the script, and only print the number in the form.

in your script:
$aFiles = array("foo.txt", "bar.txt");

in your form:
input name="filenum" type="radio" value="0"
input name="filenum" type="radio" value="1"

in your script when the form is submitted:
$sFilename = $aFiles[$filenum];

This way only YOU control which files are available.

plaihonen

Thank You for additional advice. I will keep that in mind. I tried to ise include_path() statement, but for some reason that very statement gave me errors.

Anyway, this reading files, etc is just a mid-step in my overall project.

I've already started learning database-PHP interactions.

Thanks,
Petri