[Resolved] Coding download page.

E-Beast

Hello folks,

I am new to the wonderful world of PHP and just recently started to learn this great lang.

I am trying to code a website as i learn from book "Beggining PHP, Apache, MySQL Web Development".

I have stumbled upon creating a download section of my site.

What i have in mind to code a download section for about 15 files. The files are binary and are in tar.gz format residing in directory not available from web. I would like to hide the filenames and use something like filename.php?dl=name for the link. I would like to also provide a display of the filesize and md5sum and perhaps a download count. Maybe even interact with mysql database.

What is the best way to accomplish this task? Any pointers or suggestions or even simple code snippets would be greatly appreciated!

Thanks,

Newbie to PHP

[I use PHP5 and Apache2 with MySQL 4.1.x on FreeBSD 5.2.1]

fean0r

Heres some code I have for selecting a hidden url and counting.

$items = mysql_query("SELECT * FROM ads WHERE name='$name'"); // Query
$myrow = mysql_fetch_array($items); // Result
$page = $myrow['url']; // Get Actual Download url
$hit = $myrow['hit']+1; // Get current download times + add 1
$sql = "UPDATE ads SET hit='$hit' WHERE ID='$ID'"; // Update hit count
if ($result = mysql_query($sql)) { header ("Location: $page"); } // Redirect to download
else { die(mysql_error()); } // Error Checking

E-Beast

Thanks a lot for the code, what should the mysql table look like? I guess i can use array() to define the files and links but how do i incorporate your code into the array()?

Again thanks for help.

fean0r

You can make the database as you please and edit the code around it.

ID
url
hits

Thats about all you need, why do you need to use array ?

E-Beast

To include the tar.gz files since i will not be storing them in the mysql db.

I have something like this:

<?php
$files = array("file1-1.0.0" => "../somepath/to/file1-1.0.0.tar.gz",
             "file2-2.2.1" => "../somepath/to/file-2.2.1.tar.gz");

$names = array("file1name",
"file2name");

foreach($files as $value => $key) {
echo "<br /><table width='75%' cellpadding='4' cellspacing='1' bgcolor='#000000'>";
echo "<tr>";
echo "<td bgcolor='#FFFFFF'><a href='dl.php?names=$key'>$value</a></td>";
echo "</tr>";
echo "</table>";
//header('Content-Description: File Transfer');
//header('Content-Type: application/force-download');
//header('Content-Length: ' . filesize($filename));
//header('Content-Disposition: attachment; key=' . basename($files));
readfile($files);
}
?>

Hope you get the idea of what i am trying to code.

That's what i've written so far. Thanks for helping.

Paul_Jr

You don't need an array.

This is basically how it goes. You have a page with all the links to the files. The URL of each link is something like count.php?id=2, where 2 is the ID of the file. On count.php, you retrieve the ID, update the hits for that row in the database, then redirect to the URL in that row -- the URL in the database is the actual URL of the file -- so the person can download the file.

E-Beast

Thanks a lot Paul Jr. it slowly starts to make sense.

Would you mind writing a sample code of your suggestion so i could get better idea of how it should like?

Your help is appreciated.

fean0r

Thats what the code I gave you does

E-Beast

OKie getting further with the code. But the hits are not increasing and when downloading in links browser on Linux i get the download page instead of the filename i am downloading when i choose "save as" (works properly in mozilla firefox).

index.php (where the files are listed):

<?php
// mysql conn info
include "db.inc.php";

// run query
$query = "select id,name,size,md5,hit from table order by id";

// get results
if($result = mysql_db_query($db_name, $query, $conn)) {

// loop through results
while($row = mysql_fetch_array($result)) {
echo "<br /><table width='75%' cellpadding='4' cellspacing='1' bgcolor='#000000'>";
echo "<tr>";
echo "<td bgcolor='#FFFFFF'><a href=\"dl.php?id=".$row["id"]."\">".$row["name"]."</a> Size: ".$row["size"]." MD5: ".$row["md5"]." Downloads: ".$row["hit"]."</td>";
echo "</tr>";
echo "</table>";
}
}
?>

dl.php code:

<?php
// mysql conn info
include "db.inc.php";

$id  = $_GET["id"];
// setup the query
$query = "select file, path from table where id = $id";

// get results
if($result = mysql_db_query($db_name, $query, $conn)) {

// if we have results
if($row = mysql_fetch_array($result)) {

// get filename and path
$file = $row["file"];
$path = $row["path"];
$hit  = $row['hit']+1;
$sql = "UPDATE table SET hit='$hit' WHERE id='$id'";
if ($result = mysql_query($sql)) {
header("Content-type: application/octet-stream");
header("Content-disposition: attachment; filename=$file");
header("Content-transfer-encoding: binary\n");
header("Content-length: " . filesize($path.$file) . "\n");
}
$fp=fopen($path.$file, "r");
fpassthru($fp);
}
}
// close db connection
mysql_close($conn);
?>

Now when i don't include

$id  = $_GET["id"];

dl.php is blank when hitting the download link but when the _GET is used it works.
Also noticed in the httpd log:

PHP Notice:  Undefined index:  hit in dl.php on line 18

Where am i going wrong in the code? what else can i include or modify to get it right?

Thanks again for your help!

E-Beast

I think i will mark this thread as resolved because the code is working. Unless there's a better way to implement this...

final code:

index.php

<?php
// mysql conn info
include "db.inc.php";

// run query
$query = "select id,name,size,md5,hit from table order by id";

// get results
if($result = mysql_db_query($db_name, $query, $conn)) {

// loop through results
while($row = mysql_fetch_array($result)) {
echo "<br /><table width='75%' cellpadding='4' cellspacing='1' bgcolor='#000000'>";
echo "<tr>";
echo "<td bgcolor='#FFFFFF'><a href=\"dl.php?id=".$row["id"]."\">".$row["name"]."</a><b> Size:</b> ".$row["size"]." | <b>MD5:</b> ".$row["md5"]." | <b>Downloads:</b> ".$row["hit"]."</td>";
echo "</tr>";
echo "</table>";
}
}
?>

dl.php

<?php
// mysql conn
include "db.inc.php";

$id  = $_GET["id"];
// setup the query
$query = "select file, path from table where id=$id";

// get results
if($result = mysql_db_query($db_name, $query, $conn)) {

// if we have results
if($row = mysql_fetch_array($result)) {

// get filename and path
$file = $row["file"];
$path = $row["path"];
$items = mysql_query("SELECT * FROM table WHERE name=name"); // Query
$row = mysql_fetch_array($items); // Result
$sql = "UPDATE table SET hit=hit+1 WHERE id=$id";
$result = mysql_query($sql);
header("Content-type: application/octet-stream");
header("Content-disposition: attachment; filename=$file");
header("Content-transfer-encoding: binary\n");
header("Content-length: " . filesize($path.$file) . "\n");
readfile($path.$file, "r");
}
}
// close db connection
mysql_close($conn);
?>

db.inc.php

<?php
$db_user = "username";
$db_pass = "password";
$db_host = "localhost";
$db_name = "database_name";

// connect to mysql db
$conn = mysql_connect($db_host, $db_user, $db_pass);
?>

To all who replied and provided code and help i thank you. Your help is appreciated.

Hope this helps someone and you are free to use or modify this code.

E-Beast

Forgot to add the mysql create table info.

Table setup in mysql:

CREATE TABLE `table` (
  `id` int(11) NOT NULL auto_increment,
  `name` text NOT NULL,
  `path` mediumtext NOT NULL,
  `file` text NOT NULL,
  `hit` int(11) default '0',
  `size` varchar(100) NOT NULL default '',
  `md5` varchar(100) NOT NULL default '',
  `date` datetime NOT NULL default '0000-00-00 00:00:00',
  PRIMARY KEY  (`id`));