[PHP]obsluga formularza z tablica Opcje

[eminiasty]

Mam taki formularz, pole czy jak to nazwac:

[PHP] pobierz, plaintext 
<div class="col-xs-8">
          <label for="inputUsernameEmail">Zdjęcia</label>
		  <span class="pull-right">W przypadku większej ilości zdjęć używaj klawiszy SHIFT lub zaznaczenia myszkowego.</span>
                <div class="input-group">
                <span class="input-group-btn">
                    <span class="btn btn-black btn-file">
                        Browse&hellip; 
						<input type="file" name="file_news[]" multiple>
						<input name="max_file_size" type="hidden" value="1048576" />
                    </span>
                </span>
                <input type="text" class="form-control" readonly>
            </div>  
	</div>
[PHP] pobierz, plaintext 

Chce przesylac obrazki multiple(pojedynczo dziala!) no ok, czyli wydaje mi sie ze musze uzydc do tego tablicy, ok dopisuje [] do file_news i pieknie poki co..

Teraz trzeba to jakos obsluzyc:

[SQL] pobierz, plaintext 
IF(isset($_POST['add']) && ($_POST['add'])==1) {
 
#filtriowanie
   $file_size = $_FILES['file_news']['size'];//pojemnosc pliku
   $file_brand = $_FILES['file_news']['type']; // typ pliku
   $file_name = $_FILES['file_news']['name']; // nazwa pliku	
   $file_temp = $_FILES['file_news']['tmp_name']; // chwilowa nazwa pliku
 
   $title = filtrujDane($_POST['title']);
   $active = filtrujDane($_POST['active']);
   $categories = filtrujDane($_POST['categories']);
   $date_end = filtrujDane($_POST['date_end']);
   $content = filtrujDane($_POST['content']);
   $tag = filtrujDane($_POST['tag']);
#koniec filtrowania
 
#wycinamy rozszerzenie z pobieranego pliku
$x = explode(".", $file_name);
$roz= array_pop($x);
 
/* odbieramy dane z pola ukrytego i zaokrąglamy je do 3 miejsca po przecinku/dzielimy przez 1204*1024 by było w MB*/
$max_size=round(($_POST['max_file_size']/1048576),3)."MB";
 
//zaokrąglamy "round" do 2 miejsc po przecinku i przeliczamy rozmiar pliku na MB
$poj_MB=round(($file_size/1048576),2).'MB'; 
 
//kodujemy nasz plik metodą MD5 i dodajemy date i godziną oraz rozszerzenie pliku
$file_name_add = $file_name.$s;
$file_name_x=(md5((md5($file_name)).uniqid().uniqid())).".".$roz;
$folder="upload/news/";
 
IF ($file_size <= 0){
    header("location: admin.php?s=news&a=view&k=blad1");
  }
 
IF ($poj_MB > $max_size){
    header("location: admin.php?s=news&a=view&k=blad2");
  }
 
IF (file_exists($folder.$file_name_x)){
   header("location: admin.php?s=news&a=view&k=blad3");
  }
 
IF ( $roz != 'jpg' && $roz != 'jpeg' && $roz != 'JPG' && $roz != 'JPEG' && $roz != 'gif' && $roz != 'GIF' && $roz != 'png' && $roz != 'PNG' ){
   header("location: admin.php?s=news&a=view&k=blad4");
 }  
 
  else {
 
	  IF(!@move_uploaded_file($file_temp, $folder.$file_name_x)){
		 header("location: admin.php?s=news&a=view&k=blad5");
		}
	  else{
 
$today = date('Y-m-d H:m:s');
IF( $date_end < $today ){
header("location: admin.php?s=news&a=view&k=errdate");
}
else{ 
 
$query1="
INSERT INTO ".$prefix."news (`id`, `users_id`, `title`, `content`, `date_add`, `date_end`, `active`) 
VALUES('','".$user."','".$title."','".$content."', NOW(), '".$date_end."','".$active."')"; 
#var_dump($query1);
$result = mysqli_query($link,$query1) OR die(mysqli_error());
 
$news = mysqli_insert_id($link);
 
$query2="
INSERT INTO ".$prefix."news_categories (`news_id`, `categories_id`) 
VALUES('".$news."','".$categories."')"; 
$result = mysqli_query($link,$query2) OR die(mysqli_error());
 
$query5="
INSERT INTO ".$prefix."images (`id`, `name`, `name_code`, `brand`, `size`, `type`, `active`) 
VALUES('','".$file_name."','".$file_name_x."','".$file_brand."','".$file_size."','news','1')"; 
$result = mysqli_query($link,$query5) OR die(mysqli_error());
 
$imagesx = mysqli_insert_id($link);
 
$query6="
INSERT INTO ".$prefix."news_images (`news_id`, `images_id`) 
VALUES('".$news."','".$imagesx."')"; 
$result = mysqli_query($link,$query6) OR die(mysqli_error());
 
 
 
 
$work = 0;	
$tag=explode(',', $tag);
$tag=array_map('trim', $tag);
$tag=array_unique($tag);
foreach($tag AS $i => $k){
 $tag[$work] =$k;
$work=$work+1;
}
 
 
 
FOR($i=0;$i<$work;$i++){
$query3="
INSERT INTO ".$prefix."tags (`id`, `name`, `type`) 
VALUES('','".$tag[$i]."','news')"; 
$result = mysqli_query($link,$query3) OR die(mysqli_error());	
 
$tagx = mysqli_insert_id($link);
 
$query4="
INSERT INTO ".$prefix."news_tags (`news_id`, `tags_id`) 
VALUES('".$news."','".$tagx."')"; 
$result = mysqli_query($link,$query4) OR die(mysqli_error());
}
 
header("location: admin.php?s=news&a=wys&k=nowe");
			}
 
	}
  }
}
[SQL] pobierz, plaintext 

TO jest moj kod oblugi formularza... tak wiem zabijecie mnie dlugi i nie chcialo by sie analizowac ale prosze jednak o pomoc.

O co chodzi tak naprawde? Mam tabele news_images w bazie dodajac kilka obrazkow chce zeby zapisywaly sie tam na zasadzie id|id czyli do jednego newsa mozna doadac milion obrazkow(teoretycznie)

ale tu sa bledy po rozpoczaciu zmian,czyli przerobieniu na tablice w formularzu:

[PHP] pobierz, plaintext 
Warning: explode() expects parameter 2 to be string, array given in C:\xamp\htdocs\foodbook\inc\admin\news.php on line 80
 
Warning: array_pop() expects parameter 1 to be array, null given in C:\xamp\htdocs\foodbook\inc\admin\news.php on line 81
 
Fatal error: Unsupported operand types in C:\xamp\htdocs\foodbook\inc\admin\news.php on line 87
[PHP] pobierz, plaintext 

Nie zabardzo wiem jak sie ich pozbyc, probowalem zmienne przerabiac na tablice dopisujac [] ale to tylko pogarsza sprawe.

[Tomplus]

Dlaczego jak dodajesz tablicę, to nie robisz tego w kodzie PHP?

masz: $_FILES['file_news']['size']

a powinno być (przykład)

[PHP] pobierz, plaintext 
         $file_size =  $_FILES['file_news']['size'][0]; // brakuje u Ciebie pętli do obsługi tablicy wielowymiarowej _FILES, teraz przypisze do zmiennej tylko pierwszy obrazek.
[PHP] pobierz, plaintext 

Przez to masz błąd w funkcji explode() w 2 parametrze, bo nie przesyłasz string, tylko tablicę.

Z Array_POP jest to samo.

W sumie wszystko ci pisze w błędach. Jak coś sprawdzaj sobie funkcją var_dump co dodajesz do danej funkcji, a jaki tym zmiennej powinna być dodana.

Ten post edytował Tomplus 30.09.2015, 23:10:11

[eminiasty]

[PHP] pobierz, plaintext 
         $file_size =  $_FILES['file_news']['size'][0];
[PHP] pobierz, plaintext 

czyli powinno byc np.

[PHP] pobierz, plaintext 
         $file_size =  $_FILES['file_news']['size'][size];
[PHP] pobierz, plaintext 

czyli to co w wiersza to i w kolumnach?

[mar1aczi]

Tomplus już Ci podpowiadał jak sprawdzić co masz dokładnie w zmiennej.
Wstaw między linie 2 a 3 kodu, który pokazałeś

Kod

var_dump($_FILES);

i zobaczysz jak wygląda zawartość tej zmiennej.

Ma być dokładnie jak napisał Tomplus

[PHP] pobierz, plaintext 
 $file_size =  $_FILES['file_news']['size'][0];
[PHP] pobierz, plaintext 

Indeks "[0]" będzie się zmieniał w zależności od ilości plików jakie prześlesz.

Ten post edytował mar1aczi 1.10.2015, 08:30:28

[eminiasty]

[PHP] pobierz, plaintext 
$len = count($_FILES['file_news']['name']);
 
for($i = 0; $i < $len; $i++) {
   $file_size = $_FILES['file_news']['size'][$i];//pojemnosc pliku
   $file_brand = $_FILES['file_news']['type'][$i]; // typ pliku
   $file_name = $_FILES['file_news']['name'][$i]; // nazwa pliku	
   $file_temp = $_FILES['file_news']['tmp_name'][$i]; // chwilowa nazwa pliku
}
[PHP] pobierz, plaintext 

mam taka petle ale jakos nie bardzo ona dziala, cos w niej nie tak? zawsze i tak dodaje sie pierwszy plik chodz w zmiennej _FILES jest nascie plikow

[com]

a nie czasem ostatni? bo w pętli musisz mieć też dodawanie, a tak on wykona pętle i dodaje ost jej wynik

[com]

bo nie mogą, musi to być w jednej pętli, bo to nie to samo, zrób var_dump na te zawartość $roz i $x

[viking]

I rzuciło mi się w oczy że sprawdzasz pliki po rozszerzeniu. Nie jest to bezpieczne. Sprawdzaj typ mime.

[eminiasty]

tylko mime pliku chyba sie zmienia gdzies tak czytalem ?

Chodzi ci o cos takiego?

[PHP] pobierz, plaintext 
if ( $file_brand != 'image/jpeg' && $file_brand != 'image/png' && $file_brand != 'image/gif' && $file_brand != 'image/bmp'){ header("location: admin.php?s=news&a=view&k=blad4"); } 
[PHP] pobierz, plaintext 

takie cos pdf puscilo oO

Ten post edytował eminiasty 1.10.2015, 10:26:08

[com]

a zrobiłeś na tym var_dump? rozszerzenia tym bardziej można zmienić

[mar1aczi]

[PHP] pobierz, plaintext 
if ( $file_brand != 'image/jpeg' && $file_brand != 'image/png' && $file_brand != 'image/gif' && $file_brand != 'image/bmp'){ header("location: admin.php?s=news&a=view&k=blad4"); } 
[PHP] pobierz, plaintext 

takie cos pdf puscilo oO

Bo pdf po mime zapewne spełnił powyzszy warunek.
"!=" oznacza różne, a "&&" spójnik 'i' - w dużym uproszczeniu. Poczytaj o operatorach w php.
Wg. Twojego ifa $file_brand dla pdf'a był rożny dla wszystkich wykazanych typów.

[Tomplus]

Sprawdzaj warunki, najlepiej notując sobie ołówkiem w zeszycie czy to co wrzucasz jest logiczne lub nie.

np.
brand = 'pdf';

(pdf != jpg) ^ (pdf != png) ^ (pdf != gif) ^ (pdf != bmp) => T ^ T ^ T ^ T [ bo 1*1*1*1 = 1 ]=> TRUE

brand = 'jpg';
(jpg != jpg) ^ (jpg != png) ^ (jpg != gif) ^ (jpg != bmp) => F ^ T ^ T ^ T [ bo 0*1*1*1 = 0 ]=> FALSE

A co do puszczenia PDF, to sprawdź jaki faktyczną wartość masz MIME Type. Jak nie chcesz var_dumpować wartości, to wrzuć sobie testowo do URL np. "&brand={$file_brand}"

[eminiasty]

problem byl tego typu ze pdf przechodzil gdy na multiple byl obrazek +pdf , sam pdf nie przechodzi wiec teoretycznie ok

[PHP] pobierz, plaintext 
$categories = null;
$work = 0;		
foreach($_POST['categories'] as $kat){
#filtriowanie
   $kat=filtrujDane($kat);
#koniec filtrowania   
	$categories[] .=$kat;	
	$work=$work+1;
  }		
[PHP] pobierz, plaintext 

Chce do jednego ogloszenia moc dodawac x kategorii.
mam cos takiego, wszystko dziala, ale zakomentowalem headera przekierowujacego i ..

[PHP] pobierz, plaintext 
Notice: Undefined index: categories in 
[PHP] pobierz, plaintext 

wywala mi notice, czy musze deklarowac jakos posta jesli przechodzi on przez petle i pochodzi z POSTA? jesli tak w jaki sposob to zrobic na moim przykladzie? moze ten blad jest ze wzgledu na wersje php, uzywam xampa wiec chyba aktualna

czy da sie tego foreach zamienic na for'a? z takim wykorzystaniem?

[Tomplus]

Notice to tylko uwaga, a nie błąd.

Ale zadeklaruj $categories jako tablicę a nie wartość null.

$categories = array();

[Pyton_000]

Notice to tylko uwaga, a nie błąd.

Notice to też błąd weźcie się ludzie nauczcie tego w końcu. Równie dobrze jak Ci się w Aucie zapali Pomarańczowe Check Engine to też stwierdzisz że to tylko uwaga? Przecież auto jeździ...

[eminiasty]

no ok.. zadeklarowalem jako array

[PHP] pobierz, plaintext 
$categories = array();
$work = 0;		
foreach($_POST['categories'] as $kat){
#filtriowanie
   $kat=filtrujDane($_POST['categories']);
#koniec filtrowania   
	$categories[] .=$kat;	
	$work=$work+1;
  }	
[PHP] pobierz, plaintext 

blad w lini:

[PHP] pobierz, plaintext 
$kat=filtrujDane($_POST['categories']);
[PHP] pobierz, plaintext 

[SQL] pobierz, plaintext 
Warning: trim() expects parameter 1 TO be string, array given IN 
 
Warning: trim() expects parameter 1 TO be string, array given IN
string(0) ""
Warning: mysqli_error() expects exactly 1 parameter, 0 given IN 
[SQL] pobierz, plaintext 

Teraz tak funkcja z tej linijki ma sie tak:

[PHP] pobierz, plaintext 
function filtrujDane($zmienna){
$zmienna = mysql_real_escape_string(trim($zmienna));
return $zmienna;
}
[PHP] pobierz, plaintext 

ale ale:

[PHP] pobierz, plaintext 
 $kat=$_POST['categories'];
[PHP] pobierz, plaintext 

wyrzucajac funkcje i zostawiajac tylko przypisanie jest komunikatowa magia : ( na poziomie wyzej rozumiem ze blad z trimiem nie powinno sie go uzyc czy cos w ten desen)

[SQL] pobierz, plaintext 
Notice: Array TO string conversion IN
 
Notice: Array TO string conversion IN 
string(5) "Array"
Warning: mysqli_error() expects exactly 1 parameter, 0 given IN
[SQL] pobierz, plaintext 

Zapewne chodzi o konwersje do typu tekstowego ale czy ta tablica nie jest "JUZ" jako typ tekstowy? to po co i ew jak to przekonwertowac?

Czy nie musze filtorwac zmiennej _POST[castegories] w moze abstrakcyjny sposob ktory wymyslem?

[Tomplus]

Notice to też błąd weźcie się ludzie nauczcie tego w końcu. Równie dobrze jak Ci się w Aucie zapali Pomarańczowe Check Engine to też stwierdzisz że to tylko uwaga? Przecież auto jeździ...

Ja uważam że to ostrzeżenie. I tak jak kontrolka w samochodzie ostrzega, aby uniknąć błędu.

@eminiasty
Musisz kontrolować jakie typy zmiennych używasz, możesz sobie sprawdzać typ np.:

W twoim wypadku wysyłasz do funkcji filtrujDane() całą tablicę zamiast jego elementu czyli $_POST['categories'][$kat]

Zawsze możesz sprawdzić czym jest zmienna warunkami np:

[PHP] pobierz, plaintext 
if (is_string($var)) echo "Tekst";
if (is_array($var)) echo "Tablica";
if (is_object($var)) echo "Objekt";
if (is_integer($var)) echo "Liczba";
if (isset($var)) echo "typ logiczny";
[PHP] pobierz, plaintext 

Zwracają ci true lub false, a takich zapytań jest więcej, is_scalar, is_bool, is_...


Ten post edytował Tomplus 1.10.2015, 21:27:05

[eminiasty]

[PHP] pobierz, plaintext 
if(!empty($_POST['categories'])) {
$categories = array();
$work = 0;		
foreach($_POST['categories'] as $kat){  
	$kat = filtrujDane($_POST['categories'][$kat]);
	$categories[] .=$kat;	
	$work=$work+1;
	var_dump($_POST['categories'][$kat]) ;
}	
[PHP] pobierz, plaintext 

1. czy jest dobry warunek skoro w name formularza jest categories[] ?
2.

[PHP] pobierz, plaintext 
Notice: Undefined offset: 3
[PHP] pobierz, plaintext 

Nie mam pojecia co to pierwszy raz widze blad z jakims ofsetem
3. to:

[PHP] pobierz, plaintext 
var_dump($_POST['categories'][$kat]) ;
[PHP] pobierz, plaintext 

zwraca NULL?
zas :

[PHP] pobierz, plaintext 
var_dump($_POST['categories'][$work]) ;
[PHP] pobierz, plaintext 

bo wydaje mi sie ze tak powinno byc( w sensie zamiast kat -> work
6(id) null
drugi parametr zawsze wychodzi na null..?

[Tomplus]

Mój błąd, zapomniałem się, już późno jest.

stosujesz $kat jako wynik tablicy, a celem jest aby pobrać klucz tablicy, zmień sobie pętle na

[PHP] pobierz, plaintext 
foreach($_POST['categories'] as $key => $kat){ 
// a potem wartość $kat na $key
$kat = filtrujDane($_POST['categories'][$key]);
 
[PHP] pobierz, plaintext 

a najlepiej na:

[PHP] pobierz, plaintext 
foreach($_POST['categories'] as $key => $kat) { 
    $categories[$key] = filtrujDane($_POST['categories'][$key]);
} // takie proste a da to samo co chcesz, poniżej sprawdzenie tablic Input -> Output
var_dump($_POST['categories']) ;
var_dump($categories) ;
 
[PHP] pobierz, plaintext 

Ten post edytował Tomplus 1.10.2015, 21:50:23

[eminiasty]

Pytalem o tego ifa?

[PHP] pobierz, plaintext 
if(!empty($_POST['categories'])) {
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
var_dump("<br>".$categories);
[PHP] pobierz, plaintext 

Notice: Undefined variable: categories in

[PHP] pobierz, plaintext 
var_dump("<br>".$_POST['categories']);
[PHP] pobierz, plaintext 

Notice: Array to string conversion in

chce sprawdzic /pobrac wartosc tego name, a warunek z mojego ifa sie nie sprawdza kiedy w name jest tablica? if musi byc w petli z iteracja?

[PHP] pobierz, plaintext 
<select name="categories[]"  multiple="multiple" >
[PHP] pobierz, plaintext