Download Counter - PHP

Question

How to make download counter with php? the counter function was worked properly but download function can't get the file from database

the problem is here " print "window.location='../d_doc/$data[file]'"; "

please help me.

this is my download.php

    <?php
require_once 'config.php';

if(isset($_POST['gg']) ){
$download = $_POST['gg'];
$sql2="SELECT file from download";
$res2 = mysql_query($sql2);
$data = mysql_fetch_array($res2);
$sql = "UPDATE download
SET  ukuran_file = ukuran_file+1
WHERE id = '$download'";
$res = mysql_query($sql);
if($res){
echo'Edit Data Success!';
print "<script>window.location='../d_doc/$data[file]'</script>";
}

else{
echo'Edit Data Failed!<br/>';
echo mysql_error();
}
}

echo '<br/>';





?>

... download function can't get the file from database ????? — Joe Kdw, Mar 11 '15 at 06:51
@JeanGkol yeah.., I think there is something wrong with the placement — Henrikus Anthony, Mar 11 '15 at 06:54
@JeanGkol it was like the path can't read the variable what should I do then? — Henrikus Anthony, Mar 11 '15 at 07:03
$data[file] refers to file path which user could get the file — Henrikus Anthony, Mar 11 '15 at 07:16

score 0 · Answer 1 · edited May 23 '17 at 12:28

0

If I understand your problem correctly, instead of 'print ... ' you can try redirecting the user to the download loacation as explained here : https://stackoverflow.com/a/768472

edited May 23 '17 at 12:28

Community

1
1

answered Mar 11 '15 at 06:50

Sachin

33
4

score 0 · Answer 2 · answered Mar 11 '15 at 07:55

You have mess codes. try this:

<?php
require_once 'config.php';

if(isset($_POST['gg']) ){
    $file2download = $_POST['gg'];
    //start to fetch from $file2download
    $sql2="SELECT * FROM download WHERE file='$file2download'";
    $res2 = mysql_query($sql2);
    $data = mysql_fetch_array($res2);
    if ($data>0){
        $sql = "UPDATE download SET ukuran_file = ukuran_file+1 WHERE id = '$file2download'";
        $res = mysql_query($sql);
        if($res){
        echo "Edit Data Success!";
        echo "<meta http-equiv='refresh' content='0; url=../d_doc/"WHAT-IS-THIS">";
        }
    }
else{
    echo'Edit Data Failed!<br/>';
    echo mysql_error();
    }
}
echo '<br/>';
?>

Next time you should change to PDO or mysqli for better security and easy coding

I'd recommend using `header("Location: ...")` instead of the `` tag for the refresh. — Kirk Beard, Mar 11 '15 at 13:06
yes, I agree that. but the path is not clear yet since the variable is unknown (for me). — don magug, Mar 11 '15 at 17:43

score -3 · Answer 3 · answered Mar 11 '15 at 07:00

-3

if you want this with html file then you can use like this

<a href="path/to/file" download>Click here to download</a>

answered Mar 11 '15 at 07:00

Manoj Dhiman

5,096
6
29
68

I don't know.., that's not me – Henrikus Anthony Mar 11 '15 at 07:06
i don't know why people takes 1 sec to down-vote an answer . at least i am trying to solve the issue . if have guts then solve the problem instead down-voting the answers – Manoj Dhiman Mar 11 '15 at 07:09

Download Counter - PHP

3 Answers3