Help needed adding echo to single line of script

[nickharambee]

Hi,

I am pretty much a new newbie when it comes to PHP, and have a problem that I need to solve, for a website that has to go live tomorrow.

Basically I have been using a javascript upload script called 'uploadify' which had an option for the upload folder which was added to the script in the form:

'folder'    : '/songs/<?php echo $_SESSION["name"];?>',

I added the php echo to return the username from a PHP login, so it gets added to the path to the upload folder (each user has their own subfolder for uploading to).

With the new version of the uploadify script, the folder option has been moved to a separate PHP file where it is now in the form:

$targetFolder = '/songs/';

I need to find a way of adding the username variable to this line. I have tried using echo in various ways, and googled about, but it has stumped me, simple as it may be.

If anyone could let me know how I construct this line I'd be very grateful. Time is of the essence, as they say...

Thanks,

Nick

[dagon]

bit of a guess due to lack of information (code) but try this, if it fails we need to see more code

$targetFolder = '/songs/'.$_SESSION["name"];

may need a slash at the end

for reference i suggest you familiarise yourself with: strings

[nickharambee]

Thanks. I tried that and it doesn't work. I think it may be because the session var isn't available to the uploadify.php page.

Here is the code for the login.php:

<?
session_name("MyLogin");
session_start();

if($_GET['action'] == "login") {
$conn = mysql_connect("","",""); // your MySQL connection data
$db = mysql_select_db(""); //put your database name in here 
$name = $_POST['user'];
$q_user = mysql_query("SELECT * FROM USERS WHERE login='$name'");

if(mysql_num_rows($q_user) == 1) {

$query = mysql_query("SELECT * FROM USERS WHERE login='$name'");
$data = mysql_fetch_array($query);
if($_POST['pwd'] == $data['password']) { 
$_SESSION["name"] = $name;
header("Location: http://monthlymixup.com/index.php"); // success page. put the URL you want 
exit;
} else {
header("Location: login.php?login=failed&cause=".urlencode('Wrong Password'));
exit;
}
} else {
header("Location: login.php?login=failed&cause=".urlencode('Invalid User'));
exit;
}
}

// if the session is not registered
if(session_is_registered("name") == false) {
header("Location: login.php");
}

?>

and here is the code from the uploadify php:

$targetFolder = '/songs/' . $_SESSION['name'] . '/'; // Relative to the root

if (!empty($_FILES)) {
	$tempFile = $_FILES['Filedata']['tmp_name'];
	$targetPath = $_SERVER['DOCUMENT_ROOT'] . $targetFolder;
	$targetFile = rtrim($targetPath,'/') .'/'. $_FILES['Filedata']['name'];

// Validate the file type
$fileTypes = array('m4a','mp3','flac','ogg'); // File extensions
$fileParts = pathinfo($_FILES['Filedata']['name']);

if (in_array($fileParts['extension'],$fileTypes)) {
	move_uploaded_file($tempFile,$targetFile);
	echo '1';
} else {
	echo 'Invalid file type.';
}
}
?>

I tried it with and without the backslash at the end.

[bradgrafelman]

Do you ever call [man]session_start/man in the second script?

[nickharambee]

I have just added:

session_start()

print_r($_SESSION);

to the start of the uploadify.php page, and the code you suggested doesn't work, and there is no info displayed on the uploadify.php page when viewed in my browser.

When I add

print_r($_SESSION);

to my main index page, it returns

Array ( [name] => nick )

[bradgrafelman]

[man]print_r/man will always display something, even if it's an empty array. Thus if you're getting "no info" then you either aren't looking in the right place, your code wasn't executed, etc.

[nickharambee]

I now have the session variable available to the uploadify.php page, by adding

session_name("MyLogin");

above

session_start();

Now when I add

echo $targetFolder;

it returns the correct path: '/songs/nick', but the uploads are still going to the parent directory 'songs'. When I manually enter

$targetFolder = '/songs/nick';

all works fine. Which seems rather weird. Does anyone have any ideas as to what might be going on?

Thanks,

Nick