[HTML][PHP] Upload system - co tu jest nie tak Opcje

[Kam1k]

Wycinek kodu, co tu jest nie tak. Chciałbym aby podczas przesyłania tego samego pliku pokazywało że TAKI PLIK JUŻ ISTNIEJE, a pokazuje mi komunikat SUKCESYWNY :/

[PHP] pobierz, plaintext 
<?php
   if(isset($_FILES['image'])){
	  $errors= array();
	  $target_dir = "images/";
	  $target_file = $target_dir . basename($_FILES["image"]["name"]);
	  $imageFileType = pathinfo($target_file,PATHINFO_EXTENSION);
 
	  $checking = 1;
 
	  // Dodatki
	  $pU_url = 'xxx/upload/images/';
	  $pU_bbcode_1 = 'xxx/upload/images/';
	  $pU_bbcode_2 = 'xxx';
 
	  // Glowne zmienne
      $file_name = $_FILES['image']['name'];
      $file_size = $_FILES['image']['size'];
      $file_tmp = $_FILES['image']['tmp_name'];
      $file_type = $_FILES['image']['type'];
 
      $file_ext=strtolower(end(explode('.',$_FILES['image']['name'])));
 
	  //Rozszerzenia dostepne
      $expensions= array("jpeg","jpg","png", "gif", "bmp"); 
 
 
		// Sprawdzanie = Czy plik istnieje?
		if (file_exists($target_file)) {
			$errors[]= "<p><font color=red>Zmien nazwe pliku, poniewaz jest juz taki w bazie</font></p>";
			$checking = 0;
	    }
 
	   // Sprawdzenie = Czy plik ma odpowiedni rozmiar?
		if($file_size > 52428800) {
			$errors[]='<p><font color=red>Maksymalny rozmiar pliku to 50MB, w ciagu 24h mozesz przeslac 100MB!</font></p>';
			$checking = 0;
		}
 
		// Sprawdzenie = Czy plik ma odpowiednie rozszerzenie?
		if(in_array($file_ext,$expensions)=== false){
			$errors[]="<p><font color=red>Rozszerzenie pliku nie jest poprawne, obslugujemy tylko formaty zawarte w regulaminie</font></p>";
			$checking = 0;
		}
 
		if ($checking == 0) {
			echo implode('<br>', $errors); // Pierw sprawdza czy nie ma bledow
		} else {	// Jeśli CHECKING = 1 - wszystko jest OK - pokaze komunikat sukcesywny
			if(move_uploaded_file($_FILES["image"]["tmp_name"], $target_file)) {
				echo "<p><font color='green'>Brawo! Twoj plik zostal pomyslnie przeslany, ponizej wypisano szczegoly<br>Uploaduj wiecej jesli tylko chcesz!</font></p>";
			}	
		}	
	}
?>
<br>
			<form action = "" method = "POST" enctype = "multipart/form-data">
				 <input type = "file" name = "image" />
				 <input type = "submit" title = "Przeslij plik!"/>
					<hr>
				 <ul>
					<li>Twoj plik:<br>
						<?php echo $_FILES['image']['name'];  ?>
					<li>Rozmiar:<br>
						<?php echo $_FILES['image']['size'];  ?>
					<li>Typ:<br>
						<?php echo $_FILES['image']['type'] ?>
					<li>Link:<br>
						<?php echo $pU_url.$_FILES['image']['name'] ?>
					<li>BBCode IMG:<br>
						<?php echo $pU_bbcode_1.$_FILES['image']['name'].$pU_bbcode_2 ?>
				 </ul>
 
			</form>
[PHP] pobierz, plaintext 

Ten post edytował Kam1k 9.06.2017, 09:45:36

[Pyton_000]

Bo masz sprawdzać czy $target_file instnieje a nie $_FILE

[Kam1k]

Nawet po poprawce

[HTML] pobierz, plaintext 
Notice: Undefined index: url in xxxxxupload\index.php on line 48
 
Notice: Undefined index: bbcode in  xxxxxupload\index.php on line 49
 
Warning: Illegal string offset 'image' in  xxxxx\upload\index.php on line 67
 
Warning: Illegal string offset 'name' in  xxxxx\upload\index.php on line 67
 
Brawo! Twoj plik zostal pomyslnie przeslany, ponizej wypisano szczegoly
Uploaduj wiecej jesli tylko chcesz!
[HTML] pobierz, plaintext 

Ten post edytował Kam1k 8.06.2017, 11:24:36

[nospor]

Od kiedy pola FILES maja URL czy BBCODE??

Przenosze. I nie uzywaj bbcode MANUAL do wklejania komunikatow bledow:/

[Kam1k]

Od kiedy pola FILES maja URL czy BBCODE??

Przenosze. I nie uzywaj bbcode MANUAL do wklejania komunikatow bledow:/

Wiem wiem, mój bład z tym manual ale od razu poprawiłem.
Z tym url i bbcode to robiłem dla testu bo chciałem coś sprawdzić, więc na to nie ma co patrzeć.

Chciałbym tylko aby zwracało komunikat jeśli plik już jest w folderze images

[nospor]

więc na to nie ma co patrzeć.

To moze nie marnuj juz wiecej naszego czasu i pokaz poprawny wg ciebie kod, dokladnie bledy jakie zwraca, ze wskazaniem w kodzie, ktore to linie.

[Kam1k]

To moze nie marnuj juz wiecej naszego czasu i pokaz poprawny wg ciebie kod, dokladnie bledy jakie zwraca, ze wskazaniem w kodzie, ktore to linie.

Powtórzę po raz kolejny

Kod

Warning: Illegal string offset 'image' in C:\xampp\htdocs\phpBB3\upload\index.php on line 66

Warning: Illegal string offset 'name' in C:\xampp\htdocs\phpBB3\upload\index.php on line 66

Brawo! Twoj plik zostal pomyslnie przeslany, ponizej wypisano szczegoly
Uploaduj wiecej jesli tylko chcesz!

Linia 66, jednak że jest to wycinek całości to tutaj jest to w Lini 35, a cały kod jest w temacie, jednak dla ułatwienia:

Kod

     // Sprawdzanie pliku
     if (file_exists($target_file['image']['name'])) {
        echo "<p><font color=\'red\'>Zmien nazwe pliku, poniewaz jest juz taki w bazie</font></p>";
        $checking = 1;
     }

I powtórzę po raz kolejny. Chodzi mi tylko i wyłącznie o to, gdy wrzucamy plik na upload i jest już taki w folderze /images/ to zamiast pokazywać SUKCES, ma pokazywać, że JEST JUŻ TAKI W BAZIE. Ewentualnie aby zmieniał nazwę kodując, ale taki wymagający nie jestem.

Ten post edytował Kam1k 8.06.2017, 12:12:12

[Pyton_000]

nie `$target_file['image']['name']` a `$target_file`

[Kam1k]

nie `$target_file['image']['name']` a `$target_file`

Dzięki wielkie, pomogło. Jednak komunikaty są błędne, pokazuje:

Kod

Zmien nazwe pliku, poniewaz jest juz taki w bazie

Brawo! Twoj plik zostal pomyslnie przeslany, ponizej wypisano szczegoly
Uploaduj wiecej jesli tylko chcesz!

Pierwsze zdanie - jak najbardziej!
Jednak zwraca także ten komunikat o sukcesie :/

Ogólnie cały skrypt tak samo nie przestrzega limitów i reguł, że ma obsługiwać tylko te formaty

Kod

      $expensions= array("jpeg","jpg","png", "gif", "bmp");

A cokolwiek bym wrzucił to dodaje, pewnie także omija filtr limitu 50MB

//edit
Poprawiłem cały kod w temacie

Ten post edytował Kam1k 8.06.2017, 12:52:37

[nospor]

Co z tego ze robisz IFy na rozszerzenia i rozmiar jak potem nic z tym nie robisz. UPLOAD robisz bezwarunkowo.
To samo z takim samym plikiem. Najpierw robisz UPLOAD, wyswietlasz ze ok, a dopiero potem sprawdzasz czy plik istnieje ... Gdzie logika? Tak, plik bedzie istnial, bo dopiero co go tam wrzuciles

[Kam1k]

Spróbowałem jeszcze w ten sposób, jednak zwraca mi tylko Zmien nazwe pliku, poniewaz jest juz taki w bazie

[PHP] pobierz, plaintext 
	if(move_uploaded_file($_FILES["image"]["tmp_name"], $target_file)) {
 
		// Limity
		if(in_array($file_ext,$expensions)=== false){
			$errors[]="<p><font color=red>Rozszerzenie pliku nie jest poprawne, obslugujemy tylko formaty zawarte w regulaminie</font></p>";
		}
 
		if($file_size > 52428800) {
			$errors[]='<p><font color=red>Maksymalny rozmiar pliku to 50MB, w ciagu 24h mozesz przeslac 100MB!</font></p>';
		}
 
		// Sprawdzanie pliku
		if (file_exists($target_file)) {
			echo "<p><font color=red>Zmien nazwe pliku, poniewaz jest juz taki w bazie</font></p>";
		}
 
	} else {
		echo "<p><font color='green'>Brawo! Twoj plik zostal pomyslnie przeslany, ponizej wypisano szczegoly<br>Uploaduj wiecej jesli tylko chcesz!</font></p>";
	}	
[PHP] pobierz, plaintext 

[nospor]

Pozwol ze zacytuje ciebie
I powtórzę po raz kolejny.

Najpierw masz sprawdzac, czy mozna robic upload, dopiero potem masz robic upload a nie na odwrot.

[Kam1k]

Pozwol ze zacytuje ciebie
I powtórzę po raz kolejny.

Najpierw masz sprawdzac, czy mozna robic upload, dopiero potem masz robic upload a nie na odwrot.

Wiem o co chodzi, widzę to, tylko problem w tym, że nie wiem jak to napisać, zamienić, ale rozumiem że

[PHP] pobierz, plaintext 
if(move_uploaded_file($_FILES["image"]["tmp_name"], $target_file)) {
[PHP] pobierz, plaintext 

Musi być na samym końcu. No ale dobra, zacznę od początku, człowiek na błędach się uczy.

Chyba mi się udało, dodałem checking = 0 aby sprawdzał zrobiłem coś takiego:

[PHP] pobierz, plaintext 
		// Sprawdzanie = Czy plik istnieje?
		if (file_exists($target_file)) {
			echo "<p><font color=red>Zmien nazwe pliku, poniewaz jest juz taki w bazie</font></p>";
			$checking = 0;
	    }
 
	   // Sprawdzenie = Czy plik ma odpowiedni rozmiar?
		if($file_size > 52428800) {
			$errors[]='<p><font color=red>Maksymalny rozmiar pliku to 50MB, w ciagu 24h mozesz przeslac 100MB!</font></p>';
			$checking = 0;
		}
 
		// Sprawdzenie = Czy plik ma odpowiednie rozszerzenie
		if(in_array($file_ext,$expensions)=== false){
			$errors[]="<p><font color=red>Rozszerzenie pliku nie jest poprawne, obslugujemy tylko formaty zawarte w regulaminie</font></p>";
			$checking = 0;
		}
 
		if ($checking == 0) {
			echo " ";
		} else {	// Jeśli sprawdzenie = OK = plik zostanie przeslany na serwer 
			if(move_uploaded_file($_FILES["image"]["tmp_name"], $target_file)) {
				echo "<p><font color='green'>Brawo! Twoj plik zostal pomyslnie przeslany, ponizej wypisano szczegoly<br>Uploaduj wiecej jesli tylko chcesz!</font></p>";
			}	
		}	
[PHP] pobierz, plaintext 

Zwraca mi komunikat gdy sukcesywny oraz negatywny jednak jest ostatni problem przy przesyłaniu pliku z innym rozszerzeniem - wtedy nie pokazuje nic (IMG:style_emoticons/default/sad.gif)

Ten post edytował Kam1k 8.06.2017, 14:59:13

[nospor]

No a co ma ci niby zwracac skoro dla bledow dales
echo " ";

[Kam1k]

No a co ma ci niby zwracac skoro dla bledow dales
echo " ";

Nie do końca rozumiem jak chcesz mnie nakierować, zamiast echo mam dać $errors[]=" "; ?
Skoro tak.. spróbuje, zobaczę i dam znać

[Pyton_000]

Ehh..

Skoro dodajesz błędy do tablicy `$errors[]` to wypadałoby wyświetlić je np.

`echo implode('<br>', $errors);`

[Kam1k]

DZIĘKI POMOGŁO!

Została ostatnia rzecz

Jak wchodzę na ten swój upload to pokazuje mi błędy, a jeszcze nic nie jest że tak powiem robione. Pytanie czemu?
Jednak już teraz cały skrypt działa prawidłowo

[HTML] pobierz, plaintext 
 
    Twoj plik:
 
    Notice: Undefined index: image in C:\xampp\htdocs\phpBB3\upload\index.php on line 88
    Rozmiar:
 
    Notice: Undefined index: image in C:\xampp\htdocs\phpBB3\upload\index.php on line 90
    Typ:
 
    Notice: Undefined index: image in C:\xampp\htdocs\phpBB3\upload\index.php on line 92
    Link:
 
    Notice: Undefined index: image in C:\xampp\htdocs\phpBB3\upload\index.php on line 94
 
    Notice: Undefined variable: pU_url in C:\xampp\htdocs\phpBB3\upload\index.php on line 94
    BBCode IMG:
 
    Notice: Undefined index: image in C:\xampp\htdocs\phpBB3\upload\index.php on line 96
 
    Notice: Undefined variable: pU_bbcode_1 in C:\xampp\htdocs\phpBB3\upload\index.php on line 96
 
    Notice: Undefined variable: pU_bbcode_2 in C:\xampp\htdocs\phpBB3\upload\index.php on line 96
[HTML] pobierz, plaintext 

Te błędy mam na stronie głównej jak tylko wejdę :/

Ten post edytował Kam1k 9.06.2017, 07:56:09

[Pyton_000]

Przecież masz wyjaśnione...

[Kam1k]

Przecież masz wyjaśnione...

Zamieniłem wszystko, jedyne echo które zostało to już na HTML'u

[HTML] pobierz, plaintext 
					<li>Twoj plik:<br>
						 <?php echo $_FILES['image']['name'];  ?>
					<li>Rozmiar:<br>
						<?php echo $_FILES['image']['size'];  ?>
					<li>Typ:<br>
						<?php echo $_FILES['image']['type'] ?>
					<li>Link:<br>
						<?php echo $pU_url.$_FILES['image']['name'] ?>
					<li>BBCode IMG:<br>
						<?php echo $pU_bbcode_1.$_FILES['image']['name'].$pU_bbcode_2 ?>
[HTML] pobierz, plaintext 

//Zaktualizowałem cały kod w temacie na początku

[Pyton_000]

Przecież zanim wyślesz plik to nie masz nic w $_FILES.