licznik pobrań pliku

[amii]

Przed odwołaniem do id dodajesz i uruchamiasz przez index.php

$id = mysql_real_escape_string($_GET['id']);

[pafeu]

przepraszam za takie pytanie

prościej by mi było jakbyś podał linijkę, w którą to wstawić

czy możesz podać nr linii?

albo wklej kawałek kodu linia przed i linia po

[pafeu]

no dobra wstawiłem i teraz mam teki błąd

ID ma wartosc 3
Blad w UPDATEYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '=licznik+1 WHERE id=3' at line 1

[amii]

Skąd ty ten skrypt brałeś, strasznie dużo w nim byków..

mysql_query("UPDATE kliki SETlicznik=licznik+1 WHERE id=$id",$baza) or die('Blad w UPDATE'. mysql_error());

zamień na:

mysql_query("UPDATE kliki SET licznik=licznik+1 WHERE id=$id",$baza) or die('Blad w UPDATE'. mysql_error());

[pafeu]

teraz jest

ID ma wartosc 3
Plik ma wartosc plik1.zip

Warning: Cannot modify header information - headers already sent by (output started at C:\xampp\htdocs\www\om4\licznik\pobierz.php:1) in C:\xampp\htdocs\www\om4\licznik\pobierz.php on line 25

a linia 25 to

		header("location: pliki/$plik"); exit;

[amii]

Wykasuj te echo //testowka i daj:

if(($plik<>"")and(file_exists("pliki/$plik")))

{

ob_start();

header("location:pliki/$plik");exit;

ob_end_flush();

}

[pafeu]

a teraz jest

Warning: Cannot modify header information - headers already sent by (output started at C:\xampp\htdocs\www\om4\licznik\pobierz.php:1) in C:\xampp\htdocs\www\om4\licznik\pobierz.php on line 26

linia 26 to

header("location:pliki/$plik");exit;

<?php
ob_start();
$baza=mysql_connect("localhost","root","222222")or die('Nie moge polaczyc'.mysql_error());
//error_reporting(E_ALL);
//ini_set('display_errors',TRUE);
//ini_set('display_startup_errors',TRUE);

$id = mysql_real_escape_string($_GET['id']); 
//echo 'ID ma wartosc '.$id.'<br>';  //testowka
if($id)
{
mysql_select_db("baza1",$baza) or die('Nie moge znalezc bazy'. mysql_error());
mysql_query("UPDATE kliki SET licznik=licznik+1 WHERE id=$id",$baza) or die ('Blad w UPDATE'. mysql_error());
$wynik=mysql_query("SELECT nazwa FROM kliki WHERE id=$id",$baza) or die('Blad w SELECT'. mysql_error());
mysql_close($baza);
}

if($wynik)
{
$w=mysql_fetch_row($wynik);
$plik=$w[0];
//echo 'Plik ma wartosc '. $plik . '<br>';  //testowka
	if(($plik<>"")and(file_exists("pliki/$plik")))
	{
	ob_start();
	header("location:pliki/$plik");exit;
	ob_end_flush();
	} 	else
		{
		header("location:index.php");exit;
		}
}

ob_end_flush();
?>

[amii]

Wywal to co poprzednio mówiłem bo to nic nie daje i powiedz mi jakiego edytora używasz i jak zapisujesz plik. Najprawdopodobniej masz problem z tzw. BOM

Zrób tak stwórz w ogóle nowy plik przekopiuj gotowy kod i zapisz jako saved as UTF no BOM czy coś podobnego. Oczywiście plik pod notepad++ lub gedit

[pafeu]

zainstalowałem sobie notatnik++

przekonwertowałem na utf-8 bez bom

i teraz jest tak https://www.om4.pl/licznik/index.php

czyli nic się nie dzieje

[amii]

Nie nic się nie dzieje tylko przekierowuje na index.php czyli wykonuje

else

{

header("location:index.php");exit;

}

Włącz testówki i wyświetlanie błędów

Geez, nie wiem kto pisał ten skrypt ale to tragedia

if (($plik<>"")and(file_exists("pliki/$plik")))

popraw na

if (($plik<>"")and(file_exists("pliki/".$plik)))

Najlepiej zostaw samo to w zupełności wystarczy :

if (file_exists("pliki/".$plik))

[pafeu]

wstawiłem i nic - nawet nie testuje - oczywiście pracuję na pierwotnym kodzie

<?php
if ($id) {
$baza = mysql_connect("localhost", "root", "222222");
error_reporting(E_ALL);
ini_set('display_errors', TRUE);
ini_set('display_startup_errors', TRUE);
$id = mysql_real_escape_string($_GET['id']); 
echo 'ID ma wartosc '. $id . '<br>';  //testowka
mysql_select_db("baza1",$baza);
mysql_query("UPDATE kliki SET licznik=licznik+1 WHERE id=$id", $baza);
$wynik = mysql_query("SELECT nazwa FROM kliki WHERE id=$id", $baza);
mysql_close($baza);
}
if ($wynik) {
$w = mysql_fetch_row($wynik);
$plik= $w[0];
echo 'Plik ma wartosc '. $plik . '<br>';  //testowka
if (($plik<>"")and(file_exists("pliki/.$plik"))) { 
header("location: pliki/.$plik"); exit; 
}
}
header("location: index.php"); exit;
?>

[amii]

Spójrz raz jeszcze co wstawiłeś

if (($plik<>"")and(file_exists("pliki/$plik")))

a co ja mówiłem żeby wstawić

if (file_exists("pliki/".$plik))

[pafeu]

wybacz ale mam już mętlik w głowie

już poprawiłem i nadal nic

proszę Cię o cierpliwość i pomoc aż do ukończenia tej męki

[amii]

to zakomentuj //header("location:index.php");exit;

włącz testówki

[pafeu]

w odpowiedzi dostałem

if (file_exists("pliki/".$plik))

a testy nie działają