[PHP][MySQL] Upload bannerów z wpisem do bazy Opcje

[wyrdhamster]

Mam zlepiony i ogarnięty skrypt w PHP do uploadu plików:

*Formularz:

[HTML] pobierz, plaintext 
<form enctype="multipart/form-data" action="bannersadd.php" method="POST">
<input type="hidden" name="MAX_FILE_SIZE" value="500000000000" /> <input
	name="plik" type="file" /> <input type="submit" value="Wyślij plik" />
</form>
[HTML] pobierz, plaintext 

*Skrypt:

[PHP] pobierz, plaintext 
<?php 
// podłączamy plik  connection.php 
require "connection.php"; 
// wywołujemy funkcję connection() 
connection();
 
$path = $_SERVER['DOCUMENT_ROOT'];
$sciezka = 'psw'; // można wywalić później
$katalog = $path.'/'.$sciezka.'/banners';
$plik_tmp = $_FILES['plik']['tmp_name']; 
$plik_nazwa = $_FILES['plik']['name']; 
$plik_rozmiar = $_FILES['plik']['size']; 
 
 
if(is_uploaded_file($plik_tmp)) { 
     move_uploaded_file($plik_tmp, $katalog);
    echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze 
    <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!"; 
}
 
if(isset($_POST['submit'])) {
$zapytanie = "INSERT INTO banners ( filename ) VALUES ('$plik_nazwa')";
mysql_query($zapytanie) or die(mysql_error());
print("<h1>Dane zostały dodane<br />");
}
?>
[PHP] pobierz, plaintext 

Od dwóch dni się z nim gryze i nie mam pojęcia co jest nie tak.

Formularz działa, pobiera plik, ale skrypt wrzucający na serwer już nie, i wywala takie o to komunikaty:

Warning: move_uploaded_file(/home/klient.dhosting.pl/adkl/psw/banners) [function.move-uploaded-file]: failed to open stream: Is a directory in /home/klient.dhosting.pl/adkl/psw/bannersadd.php on line 21

Warning: move_uploaded_file() [function.move-uploaded-file]: Unable to move '/home/klient.dhosting.pl/adkl/.tmp/phpNSNMEH' to '/home/klient.dhosting.pl/adkl/psw/banners' in /home/klient.dhosting.pl/adkl/psw/bannersadd.php on line 21

Plik: przód.jpg o rozmiarze 25726 bajtów został przesłany na serwer!

Czy ktoś może mi powiedzieć co tu może być nie tak?

Ten post edytował wyrdhamster 2.12.2009, 22:13:56

[bim2]

failed to open stream: Is a directory in

Na 100% jest taki katalog i nadane ma chmody poprawne?

[wyrdhamster]

Na 100% jest taki katalog i nadane ma chmody poprawne?

Istnieje on od 36 godzin, a chmoda sprawdzałem z n razy już teraz. Ale zapytam się żeby być pewnym na 1000% - powinno być 777? Może to być wina hostingu?

[ChrisB]

[PHP] pobierz, plaintext 
move_uploaded_file($plik_tmp, $katalog);
[PHP] pobierz, plaintext 

jako parametr przyjmuje:
1)nazwe pliku w tempie to jest ok
2)nazwe pliku gdzie ma przeniesc (teraz masz tam tylko katalog gdzie ma przeniesc i o to sie pieni)

plus pisanie że się powiodło bez sprawdzenia czy się powiodło jest lekko rzecz ujmując:D
ogolnie o to chodzi:

[PHP] pobierz, plaintext 
move_uploaded_file($plik_tmp, $katalog.'jakasnazwa.xxx');
[PHP] pobierz, plaintext 

Ten post edytował ChrisB 2.12.2009, 23:10:41

[darko]

No tak, a dlaczego nie podajesz nazwy pliku tylko sam folder jako destination dla funkcji move_uploaded_file

Linie 15-19 zamieniłbym na to:

[PHP] pobierz, plaintext 
if(is_uploaded_file($plik_tmp)) { 
     if(move_uploaded_file($plik_tmp, $katalog."/".str_replace(" ","_",$plik_nazwa))) {
       echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
    } else {
       echo "Błąd przesyłania pliku!";
    }
}
[PHP] pobierz, plaintext 

[wyrdhamster]

No tak, a dlaczego nie podajesz nazwy pliku tylko sam folder jako destination dla funkcji move_uploaded_file

Bo myślałem ze wystarczy wskazać tylko katalog docelowy...

Dziękuję, zaraz wypróbuje poprawki...

I działa. Przesyła pliki jak trzeba. Jeszcze raz dzięki darko.

Ale pojawił się inny problem - nie robi wpisu do bazy powiązanego z uploaddem. Struktura tabeli banners:

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `banners` (
  `id_banner` int(5) NOT NULL AUTO_INCREMENT,
  `filename` varchar(30) collate utf8_bin NOT NULL,
  PRIMARY KEY  (`id_banner`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_bin AUTO_INCREMENT=1 ;
[SQL] pobierz, plaintext 

Ten post edytował wyrdhamster 3.12.2009, 13:09:51

[darko]

[PHP] pobierz, plaintext 
$wpis_do_bazy = $katalog."/".str_replace(" ","_",$plik_nazwa);
if(is_uploaded_file($plik_tmp)) { 
     if(move_uploaded_file($plik_tmp, $wpis_do_bazy)) {
       echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
    } else {
       echo "Błąd przesyłania pliku!";
    }
}
[PHP] pobierz, plaintext 

i dalej...

[PHP] pobierz, plaintext 
$zapytanie = "INSERT INTO banners VALUES (null, '$wpis_do_bazy')";
[PHP] pobierz, plaintext 

Ten post edytował darko 3.12.2009, 14:23:29

[wyrdhamster]

Dziwne, ciągle nie robi nowego wpisu do bazy, choć linki przesyłają się aż miło... Ponieważ są jakieś porblemy także ze sciagniem newsów do wyświetlenia, może problemem jest źle napisania konfiguracja połączenia? Oto skrypt connection.php:

[PHP] pobierz, plaintext 
<?php 
/******************************************************
* connection.php 
* konfiguracja połączenia z bazą danych 
******************************************************/
 
function connection() { 
    // serwer 
    $mysql_server = "***"; 
    // admin 
    $mysql_admin = "***"; 
    // hasło 
    $mysql_pass = "***"; 
    // nazwa baza 
    $mysql_db = "***"; 
    // nawiązujemy połączenie z serwerem MySQL 
    @mysql_connect($mysql_server, $mysql_admin, $mysql_pass) 
    or die('Brak połączenia z serwerem MySQL.'); 
    // łączymy się z bazą danych 
    @mysql_select_db($mysql_db) 
    or die('Błąd wyboru bazy danych.'); 
} 
 
?>
[PHP] pobierz, plaintext 

Oczywisćie *** to dane prywatne, których nie ujawnię na forum.

[wyrdhamster]

Ten skrypt, wyświetlający newsy z bazy...

[PHP] pobierz, plaintext 
<?php
 include "connection.php";
 connection();
 $sqlcount = "SELECT COUNT(id_news) ile FROM news";
 $resultcount = mysql_query ($sqlcount,$conn) or die(mysql_error());
 $rowcount = mysql_fetch_array($resultcount);
 $count = $rowcount["ile"];
 $l2 = 10;
 if (isset($_GET["l1"])){
 	$l1 = $_GET["l1"];
 }
 else {
 	$l1 = 0;
 }
 $sql = "SELECT news_name, user_name, date, n.info info
         FROM news n JOIN users u ON n.user_id = u.id_user
         ORDER BY date DESC
         LIMIT ".$l1.", ".$l2;
 $result = mysql_query ($sql,$conn) or die(mysql_error());
 $cont = "";
 while ($row = mysql_fetch_array($result)) {
	 $cont .= "<div class='news'><h3>";
	 $cont .= $row["news_name"];
	 $cont .= "</h3><div><span class='user'>";
	 $cont .= $row["user_name"];
	 $cont .= "</span><span class='date'>";
	 $cont .= $row["date"];
	 $cont .= "</span></div><div class='info'>";
	 $cont .= $row["info"];
	 $cont .= "</div></div>";
 }
echo $cont;
if (($l1+10) < $count){
  echo "<a href='news.php?l1=".($l1+10)."'>zobacz wcześniejsze... </a>";
}
if ($l1 > $count){
  echo "<a href='news.php?l1=".($l1-10)."'>zobacz późniejsze... </a>";
}
 
?>
[PHP] pobierz, plaintext 

Generuje tylko takie ostrzeżenie:

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in /home/klient.dhosting.pl/adkl/psw/news.php on line 5

Może rzeczywiście coś jest nie tak z połączeniem z bazą, z connection.php?

[PHP] pobierz, plaintext 
<?php
function lacz_bd() {
   $wynik = new mysqli('localhost', 'root', '' , 'test2');
   if (!$wynik) {
      throw new Exception('Połączenie z serwerem bazy danych nie powiodło się');
   } else {
      return $wynik;
   }
}
 
?>
[PHP] pobierz, plaintext 

Zakładając że miało być w kodzie mysqli('<serwer>', '<user>', '<hasło>' , '<baza>'), to nadal nie chodzi to połaczenia. Teraz dowala jeszcze taki błąd:

Warning: mysqli::mysqli() [mysqli.mysqli]: (HY000/2003): Can't connect to MySQL server on 'air.dhosting.pl' (111) in /home/klient.dhosting.pl/adkl/psw/connection.php on line 26

(Taki duzy numer linii, bo wcześniejszą funkcje za komentowałem, i wkleiłem twoją, onkel. )

[bim2]

Czytaj błędy jakie ci wyskakują, a jak ich nie rozumiesz to naucz się angielskiego.

Can't connect to MySQL server on 'air.dhosting.pl' -> Nie można się połączyć z serwerem MySQL 'air.dhosting.pl'

Rozumiesz w czym tkwi problem już?

[wyrdhamster]

Problem był w tym że nie łaczyłem się z baza SQl tylko FTPem.

Kumpel jednak zapropował inny skrypt łączenia się z bazą.

[SQL] pobierz, plaintext 
<?php
	$conn = mysql_connect("<serwer_sql>", "<user>", "<hasło>")
		OR die(mysql_error());
	$db = mysql_select_db("<baza>")
		OR die(mysql_error());
	mysql_query("SET NAMES 'latin2'");	
?>
[SQL] pobierz, plaintext 

[darko]

Lepiej mysql_query("SET NAMES 'utf8'"); jak już.

Daj jeszcze print_r(mysql_connect($mysql_server, $mysql_admin, $mysql_pass)); powinno być resource id# jak nie ma, to nie jesteś w ogóle połączony

Ten post edytował darko 5.12.2009, 17:22:22

[wyrdhamster]

Okej, wprowadziłem poprawke odnośnie UTFa.

Mam teraz skrypt, niby nie bezpośrednio powiązny z tematem, ale jego uzupełninie - chodzi o skrypt wyświetlający obrazki z pomocą bazy danych i pozwalający dany z nich wywalić, zarówno z serwera jak i bazy. Tutaj jest kod:

[PHP] pobierz, plaintext 
 <?php 
require "conn.inc.php"; 
$path = str_replace('/dashboard/banners_delete.php','',$_SERVER['PHP_SELF']);
$sciezka = 'psw'; // można wywalić później
$katalog = "$path/banners"; 
 
$banners = mysql_query('SELECT * FROM banners', $conn) or die("Nie udało się pobrać danych z bazy") ; 
 
while($row = mysql_fetch_assoc($banners)){
	foreach($row as $key => $value) {
		$$key = $value;
	}
	$link="$katalog/$filename";
	echo "<div class='banner'><img src='$link' alt='$filename' ></br>";
echo '<form name="usun" id="usun" method="post" action="banners_delete.php"/>';
echo '<input type=hidden name=id value="$id" />';
echo '<input type=submit value="Usuń" onclick="return usun();"/>';
echo '</form></div>';
}
 
function usun()
{
	$numer=$id;
	 unlink("$katalog/$filename");
	 $sql = "DELETE FROM `banners` WHERE `id_banner`='$numer'";
	 return mysql_query($sql, $conn) or die("Nie udało się usunąć danych z bazy") ;
}
?>
[PHP] pobierz, plaintext 

Obrazki się wyświetlają, natomiast przycisk nie usuwa wpisu - wiem ze coś z nimi nie tak, ale nie jestem pewien co. Pewnie przesyłanie tego id do funkcji usun().

[darko]

Prawdopodobnie funkcja usun nie widzi zmiennych:

$numer, $id, $katalog oraz $filename

Nie wiem dokładnie gdzie definiujesz te zmienne w każdym razie powinieneś albo przekazać je do funkcji usuń albo uczynić globalnymi wewnątrz funkcji:

global $numer, $id, $katalog, $filename;