Forum PHP.pl

Witam,
Robię prosty panel admina jednak mam przy nim jako osoba początkująca sporo wątpliwości.
Otóż mam formularz z kalendarzem w którym można wybrać datę (potrzebne gdy dodaję newsa i chcę go oznaczyć datą) :

[HTML] pobierz, plaintext 
 
<ul >
	<li id="li_1" >
		<label class="description" for="element_1" name="el1">data </label>
		<span>
			<input id="element_1_1" name="element_1_1" class="element text" size="2" maxlength="2" value="" type="text"> /
			<label for="element_1_1">MM</label>
		</span>
		<span>
			<input id="element_1_2" name="element_1_2" class="element text" size="2" maxlength="2" value="" type="text"> /
			<label for="element_1_2">DD</label>
		</span>
		<span>
	 		<input id="element_1_3" name="element_1_3" class="element text" size="4" maxlength="4" value="" type="text">
			<label for="element_1_3">YYYY</label>
		</span>
 
		<span id="calendar_1">
			<img id="cal_img_1" class="datepicker" src="calendar.gif" alt="Pick a date.">	
		</span>
		<script type="text/javascript">
			Calendar.setup({
			inputField	 : "element_1_3",
			baseField    : "element_1",
			displayArea  : "calendar_1",
			button		 : "cal_img_1",
			ifFormat	 : "%B %e, %Y",
			onSelect	 : selectDate
			});
		</script>
[HTML] pobierz, plaintext 

W taki sposób mniej więcej definiuję to co wysyłam do bazy danych - z nagłówkiem i treścią jest ok, ale co zrobić z datą która ma yyyy dd i mm w oddzielnych polach oznaczonych różnym name:

[PHP] pobierz, plaintext 
$data = $_POST['tu jest problem'];
$naglowek = $_POST['element_3'];
$tresc = $_POST['element_1_1'];
[PHP] pobierz, plaintext 

czy da się to jakoś zgrupować? jak można to rozwiązać nie zmieniając możliwie wygodnej formy kalendarza?

Możesz albo przesłać sobie wszystkie trzy części daty i połączyć w jedną datę po stronie PHP, albo utworzyć sobie ukryte (hidden) pole o nazwie data i odpowiednio zmodyfikować funkcję selectDate, aby ustawiała wartość tego pola jako jedną datę. Miło by było, gdybyś podał z jakiego skryptu kalendarza korzystasz, a jeszcze lepiej jakiś odnośnik, bo może sam skrypt ma podobne możliwości.

korzystałam z generatora formularzy online - link
a jak mogę połączyć trzy części w jedną? w tabeli bazy danych mam jedno pole na datę - to mam najpierw metodą post potraktować kolejno:

[PHP] pobierz, plaintext 
$rok = $_POST['element_1_3'];
$dzien= $_POST['element_1_2'];
$miesiac= $_POST['element_1_1'];
[PHP] pobierz, plaintext 

a później zrobić coś takiego?:

$data=

tylko tu już nie wiem w jaki sposób ponieważ w bazie danych daty wyświetlane są w formacie date np. 2011-11-30 więc jak to powinno się poprawnie łączyć żeby działało?

[PHP] pobierz, plaintext 
 $rok = $_POST['element_1_3'];
    $dzien= $_POST['element_1_2'];
    $miesiac= $_POST['element_1_1'];
$data = $_POST[$rok . '-' . $miesiac . '-' . $dzien];
[PHP] pobierz, plaintext 

coś takiego wykombinowałam i data wyświetla się prawidłowo ale nijak jej wysłać nie można do tabeli - pojawia się:
Notice: Undefined index: 2012-02-4 in C:\xampp\htdocs\strona\admin\podmenu\dodajnewsa.php on line 29
wpisana data w komunikacie błędu wprawdzie wyświetla się ładnie ale nie wysyła się, rozumiem że to ma pewnie związek z jej niepoprawnym zapisem? czy może inny błąd?

Pierwsze trzy linijki są dobrze, natomiast czwarta linijka powinna wyglądać tak:

[PHP] pobierz, plaintext 
$data = $rok . '-' . $miesiac . '-' . $dzien;
[PHP] pobierz, plaintext 

i zmienną $data wykorzystujesz w zapytaniu.

Ten post edytował mortus 4.02.2012, 20:47:09

poprawiłam wg twojej instrukcji, w prawdzie notice zniknęło ale i tak pojawia się ""Błąd nie udało się dodać nowego rekordu" no i w bazie nie dodało się nic...
Połączenie z bazą na pewno działa ok bo inaczej wywaliłoby komunikat. w bazie są pola data, naglowek, tresc, idnewsa (z atrybutem auto increment). co jeszcze może być źle?

[PHP] pobierz, plaintext 
  // dodajemy rekord do bazy
    $ins = @mysql_query("INSERT INTO newsy SET data='$data', naglowek='$naglowek', tresc='$tresc'");
 
    if($ins) echo "Rekord został dodany poprawnie";
    else echo "Błąd nie udało się dodać nowego rekordu";
 
    mysql_close($connection);
[PHP] pobierz, plaintext 

Nigdy nie wyciszaj żadnych funkcji.

[PHP] pobierz, plaintext 
$ins = mysql_query("INSERT INTO newsy SET data='$data', naglowek='$naglowek', tresc='$tresc'") or die(mysql_error());
[PHP] pobierz, plaintext 

I napisz co wyskakuje.

Plus lektura:
Temat: Jak poprawnie zadac pytanie

@DOWN:
Trochę za wolno i nadal wyciszasz błąd.

Ten post edytował Evinek 4.02.2012, 21:10:59

Zamień linijkę 2 na:

[PHP] pobierz, plaintext 
$ins = @mysql_query("INSERT INTO newsy SET data='$data', naglowek='$naglowek', tresc='$tresc'") or die(mysql_error());
[PHP] pobierz, plaintext 

i napisz, jaki błąd się pojawia.

dobra, dzięki wam dodaje się rekord, mało tego nawet data dodaje się poprawnie ale w tresci co bym nie wpisała pojawia się "02" po wysłaniu do bazy

przepraszam, błąd wprost banalny z tym co pisałam teraz, dziękuję za pomoc już wszystko pięknie działa;)

witam,
Wszystko działa, tylko teraz jeszcze dopracować chciałabym to do końca (dodają się rekordy, data też za co wielkie dzięki za pomoc)
a mianowicie pojawiają mi się informacje:

Notice: Undefined index: element_1 in C:\xampp\htdocs\..\admin\form\form\dodaj_newsa.php on line 89

Notice: Undefined index: element_3 in C:\xampp\htdocs\..\admin\form\form\dodaj_newsa.php on line 90

Notice: Undefined index: element_2_3 in C:\xampp\htdocs\..\admin\form\form\dodaj_newsa.php on line 91

Notice: Undefined index: element_2_2 in C:\xampp\htdocs\..\admin\form\form\dodaj_newsa.php on line 92

Notice: Undefined index: element_2_1 in C:\xampp\htdocs\..\admin\form\form\dodaj_newsa.php on line 93

Co mogą oznaczać te komunikaty? jeśli trzeba to wkleję wszystko ale nie wiem w zasadzie co miałoby być nie tak skoro te "element" to po prostu nazwy pól formularza w html ... ? coś trzeba dodać żeby to usunąć?
mam tylko formularz który wcześniej wklejałam no i odwołania do niego w postaci zmiennych w php

No cóż, podczas bezpośredniego uruchomienia skryptu (zanim zatwierdzisz formularz) określone indeksy w tablicy $_POST nie istnieją. Trzeba zatem najpierw sprawdzić, czy formularz został zatwierdzony i czy odpowiednie elementy zostały przesłane:

[PHP] pobierz, plaintext 
$rok = isset($_POST['element_1_3']) && !empty($_POST['element_1_3']) ? $_POST['element_1_3'] : '0000';
$dzien = isset($_POST['element_1_2']) && !empty($_POST['element_1_2']) ? $_POST['element_1_2'] : '00';
// itd.
[PHP] pobierz, plaintext 

No więc notice się nie wyświetla i z tym jest ok, ale jest jeszcze o wiele goszy problem.
Otóż mam formularz a pod nim link do głównej strony panelu admina.
Gdy z panelu admina wejdę w "dodaj newsa", następnie nic nie wpiszę i nie naciskając "submit" kliknę w link do głównej strony panelu to i tak do bazy zostaje dodany pusty news z polem daty uzupełnionym zerami.
Czemu tak się dzieje i jakim sposobem mogę temu zapobiec?

może inaczej, jak sprawdzić czy kliknięto "submit" ?


jak zdefiniować warunek dla takiego if co by to miał sprawdzać?

to zdaje się że działa
Tyle że chciałabym jeszcze dodać coś takiego co zabezpieczy przed kolejnym dodaniem identycznego rekordu po odświerzeniu strony - czy można dodać coś takiego? jak?

Nie ma przed "tym" skutecznego zabezpieczenia. Możesz użyć przekierowania (funkcja header()) na tę samą stronę, ale jeśli później wciśniesz klawisz wstecz przeglądarki i zatwierdzisz wysyłanie danych, to odpowiedni rekord zostanie ponownie dodany. Co prawda możesz się przed tym ustrzec sprawdzając adres strony, z której "wchodzisz" na stronę przetwarzającą formularz (manual $_SERVER['HTTP_REFERER']), jednak nie jest to metoda, której można "zaufać" (szczegóły pod wskazanym adresem). Może dobrym rozwiązaniem okazałaby się CAPTCHA, jednak nie próbowałem.

tworzy osobny input (type=hidden), generujesz w nim losowy ciąg. po prawidłowym wysłaniu formularza zapisujesz ten kod do sesji.
przy następnym wywołaniu (odświeżeniu strony) sprawdzenia czy formularz został wysłany oraz czy ten 'hash' nie został już wykorzystany.

No tak. Chyba najprostsza metoda, a do głowy mi nie wpadła. Pozwolę sobie jednak nieco sprostować, bo owszem trzeba wygenerować losowy ciąg znaków, ale do tablicy sesji zapisujemy ten ciąg wcześniej, a dokładnie w momencie, w którym tworzymy formularz wraz z dodatkowym ukrytym polem na nasz hash. Po zatwierdzeniu/wysłaniu formularza będzie już za późno. Trzeba natomiast po zatwierdzeniu formularza i prawidłowym dodaniu rekordu do bazy danych ten hash z tablicy sesji usunąć, a sprawdzenie, czy hash został już wykorzystany będzie polegać na sprawdzeniu, czy odpowiednia zmienna w ogóle istnieje w tablicy sesji. Kod:

[PHP] index.php - pobierz, plaintext 
<?php
session_start();
// przed wysłaniem formularza
$hash = uniqid(); // tutaj generujemy nasz losowy ciąg znaków, dla przykładu posłużyłem się funkcją uniqid()
$_SESSION['hash'] = $hash;
?>
<form action="akcja.php" method="post">
 <input type="hidden" name="hash" value="<?php echo $hash; ?>" />
 <!-- inne pola formularza i przycisk submit -->
</form>
[PHP] index.php - pobierz, plaintext 

[PHP] akcja.php - pobierz, plaintext 
<?php
session_start();
// sprawdzamy, czy formularz został wysłany
// możemy do tego użyć choćby pola ukrytego, czy też przycisku submit
// lub też sprawdzić REQUEST_METHOD
if($_SERVER['REQUEST_METHOD'] == 'POST') {
 // teraz sprawdzamy, czy hash "był już wykorzystany"
 if(isset($_SESSION['hash'])) {
  // nie był, zatem dodajemy rekord do bazy danych i usuwamy hash z tablicy sesji
  unset($_SESSION['hash']);
  // przekierowanie
 } else {
  // hash był już wykorzystany, można zatem wyświetlić odpowiedni komunikat błędu
  // przekierowanie
 }
}
[PHP] akcja.php - pobierz, plaintext