Razvoj web-stranica - Rasprava

0.) Ovaj kod će ti stvoriti jednu ili više rezervacija, ovisno o tome imaš li više korisnika s istim e-mailom. Odakle ti varijabla e-mail? Ako je korisnik registriran i logiran, njegove login podatke čuvaš u sessionu, pa imaš id bez da ga vadiš iz baze po emailu, ako nije, onda nema smisla razdvajati usere i rezervacije jer ne možeš znati da isti korisnik nužno koristi istu e-mail adresu.

1.) mysqli_fetch_all po defaultu daje rezultat u kojem je array indexiran po redoslijedu kolona (dakle, nemaš 'id_u', već index 0), slanje konstante MYSQLI_ASSOC kao drugog parametra daje associjativni array u kojem imaš index 'id_u'.

2.) $conn varijablu nemaš, pa ti zato javlja notice

3.) ovo je nastavno na 2.) mysqli_query() očekuje da je prvi parametar link, što je u tvom kodu $mysqli, a ne $conn. Dodatno, ako miješaš malo objektno ($mysqli->query($query)) i proceduralno (mysqli_query($mysqli, $query)), kod je nepregledniji, a vjerojatnost greške veća nego ako koristiš uvijek isti način.

Oho.. pa ljepo je vidit da neki od vas imaju i ovih znanja. Mrblic me napokon pozitivno iznenadio...  😀

Nego, kakvi ste sa wordpress tehnikalijama ?

0.) Ok, ako je na bazi email unique, pa znaš da neće biti više korisnika, ali onda nema smisla raditi foreach petlju, već samo dohvat prvog retka. Međutim, kao što sam već i napisao, nema nikakavog razloga da ti id_usera ne bude session varijabla jer je upravo id primary key - jedinstvena identifikacija tog retka u bazi.

1.) Kad u PHP-u dodaješ nešto u array bez eksplicitnog navođenja key-a, onda se dodijeljuje prvi najniži slobodni numerički key počevši od nule.

Primjerice:

$array1 = array(1, 'mrblc@bug.hr');
$array2 = array('id_u' => 1, 'email' => 'mrblc@bug.hr');
var_dump($array1, $array2);

Daje output:

array(2) {
[0]=>
int(1)
[1]=>
string(12) "mrblc@bug.hr"
}
array(2) {
["id_u"]=>
int(1)
["email"]=>
string(12) "mrblc@bug.hr"
}

mysqli_fetch_all prima dodatni paramtera koji mu govori kako će puniti array s rezultatom za svaki redak. Defaultno je MYSQLI_NUM što rezultira prvim arrayem, a ako za drugi argument staviš MYSQLI_ASSOC dobiješ ovaj drugi oblik.

Isto tako, za dohvatiti redak po redak, možeš koristiti mysqli_fetch_row koji vraća jedan po jedan redak u prvom obliku (primjerice unutar while petlje ili samo jednom pozoveš ako očekuješ jedan rezultat kao u slučaju usera), odnosno mysqli_fetch_assoc koji vraća jedan po jedan redak u drugom obliku. Isto tako imaš i funkciju mysqli_fetch_array koja prima i drugi argument kao i funkcija mysqli_fetch_all, ali vraća redak po redak kao i ove _row i _assoc funkcije. Možeš računati kao da je interni kod PHP-a:

function mysqli_fetch_row ($result) {

   return mysqli_fetch_array ($result, MYSQLI_NUM );

}

function mysqli_fetch_assoc ($result) {

   return mysqli_fetch_array ($result, MYSQLI_ASSOC );

}

3.) Sve (ili barem one koje tebi trebaju) mysqli funkcije su dostupne u dva oblika:

$obj->ime_funkcije($argument1, ...);

mysqli_ime_funkcije($obj, $argument1, ...);

$obj je ono nad čim vršiš operaciju - query izvršavaš na bazi, a vađenje podataka nad setom rezultata. Objektu kažeš da sam nad sobom izvrši neku operaciju. Primjerice, setu rezultata kažeš "daj mi prvi redak". Set rezultata ima unutar sebe sve što mu treba i tebe ne zanima kako on to radi niti mu trebaš dati neke dodatne informacije o samom setu. To se zove objektno orijentirano programiranje - set je objekt koji je "zatvoreni" sustav - ima set podataka koji čine logičku cijelinu i metode za baratanje tim podacima (primjerice dohvat).

Drugi oblik je proceduralno. Znači, tebi je set rezultata varijabla i ti pozivaš vanjsku funkciju kojoj šalješ taj set rezultata da s njim nešto napravi, ali ta funkcija nije dio seta. S obzirom da čitav tvoj kod počiva na ovom načinu programiranja, ovaj oblik bi ti trebao biti "prirodniji". Nije greška ni u proceduralnom programiranju koristiti objekte i pozive metodama objekata. U konkretnom slučaju ionako to završi izvršavanjem istog koda. Problem je kad miješaš malo jedno i malo drugo, pa više ne znaš što ti je što, onda ti se lako dogodi greška.

Što se tiče datuma, problem je što i on mora biti u navodnicima. '2018-11-12' će u bazu poslati današnji datum, dok će 2018-11-12 pokušati spremiti broj 1995.

a skužaj, dok ja svatin o čemu se radi, evo napokon znan stavit vrijednost querya u varijablu, sad će sve bit sto puta lakše..

izbacia san for each petlju i fetch all, stavia row i bez ikakve petlje, datumi u navodnike, radi sve ko urica.. 

od prve godine faksa slušan taj pojam objektno orijentirano programiranje i evo tek sad znam šta je, a za proceduralno nisam ni zna dok nisam rad počea radit ..

fala puno na svemu 

Aj ti onda nekaj i mene nauči : Ako imam sadasnju wp instalaciju ( komplicirana, woocommerce..) na npr serveru A, i dodje do npr malware problema i sl, backup hostinga isto zarazen, ja imam backup star par dana ali nemogu ju samo tako vratiti jer je i db kompromitirana. 

Znači, sve u home dir hostinga izbrišem, kreiram novu db, indtaliram novi wp, i da li tada mogu vratiti moj backup ( uvesti tablice iz stare db d novu, i datoteke iz backupa naseliti u public html mapu ? Nešto mi govori da to nebu šljakalo.

Verzija b, prelazak na novi host, no tada imam server B gdje mi moj backup još manje vrijedi...?

Evo, ako ti se da, malo me nauči o svemu tome... 

Situacija nije hipotetska nego stvarna...nažalost... Razmišljamo u smjeru upravljanog wp hostinga tipa wpengine, siteground, kinsta, i sl....

Napravi novu varijablu, $floor = $flors[$i];

i onda koristi $floor umjesto $flors[$i] unutar SQL querija.

kad var_dumpam $floors[$i] u petlji, dobijem

string(12) "Ground Floor" string(9) "1st Floor" string(9) "2nd Floor"

šta je u redu, ali dalje me u queryu zeza.. kad da stvaim jednostruke zagrade? na jedan od ovih gore primjera ili?

proba sam sljedeće

('$floors[$i]')

'($floors[$i])'

($floors[$i])

'(".$floors[$i].")'

'".$floors[$i]."'

('".$floors[$i]."')

'{$floors[$i]}'

('{$floors[$i]}')

'({$floors[$i]})'

'".$floors[$i]."'  i '{$floors[$i]}' vi trebalo biti u redu, ne znam da li je nešto drugo problem s querijem. 

Prvo, ako još jednom staviš screenshot koda, prestati ću ti pomagati. Kraći kod ubaci kroz editor i formatiraj kao kod, a dulji stavi na pastebin ili sličan servis. Iz screenshota ne mogu napraviti c/p za testirati ili ti dati korigiranu verziju.

Drugo, ako ti query ne radi, prvo što napraviš je umjesto da ga daješ direktno u $mysqli->query, prvo ga spremi u varijablu i nju dump-aj. Dakle, umjesto: 

$mysqli->query("SELECT * FROM table WHERE type = '$var'");

stavi:

$sql = "SELECT * FROM table WHERE type = '$var'";

var_dump($sql);

$mysqli->query($sql);

Tako ćeš puno lakše uočiti grešku jer ćeš vidjeti što točno šaljem prema bazi, a možeš i ubaciti u phpmyadmin i dobiti neku suvisliju grešku.

Treće, opet kompliciraš sa stringovima. Ako ti svaka soba ima id, zašto iz frontenda ne vraćaš id-ove umjesto naziva? Ako koristiš (multi)select:

<option value="<?php echo $floor['id_roo']"; ?>"><?php echo $floor['type']"; ?></option>

post ti već vrati array id-ova.

Ne, krivo si shvatio. Ovo ide na kontrolu u kojoj korisnik bira - tamo korisniku prikazuješ tekstualni oblik, a value postavljaš da je id, tako da se na server ne šalje tekst, već id. Ako korisnik bira u selectu onda ide:

<select multiple name="id_roo[]">

<?php foreach($loors as $floor): ?>

   <option value="<?php echo $floor['id_roo']"; ?>"><?php echo $floor['type']"; ?></option>

<?php endforeach; ?>

</select>

Ako je checkbox:

<?php foreach($loors as $floor): ?>

   <input type="checkbox" name="id_roo[]" id="room_<?php echo $floor['id_roo']"; ?>" value="<?php echo $floor['id_roo']"; ?>">

   <label for="room_<?php echo $floor['id_roo']"; ?>"><?php echo $floor['type']"; ?></label>

<?php endforeach; ?>

P.S. Ovo: room_<?php echo $floor['id_roo']"; ?> služi da bi povezao checkbox i label, pa korisnik može kliknuti na label - svaki checkbox ima unique id, a svaki label ima id checkboxa u for attributu.

 Potkrala mu se greska, te navodne znakove treba izbaciti.

Dupli navodnik je greška u c/p.

U tvom kodu ne vidim checkbox, a na prikazu vidim. U svakom slučaju, vrijednost prikazana korisniku i vrijednost koja se šalje na server ne moraju biti iste, što značajno olakšava stvar.

U ovom drugom kodu, ti koristiš $rezz1 koji je zadni puta postavljen u zadnjoj iteraciji foreach petlje iz prvog koda, što znači da ponavljaš uvije rezultate zadnje rezervacije.

U dohvati možeš napraviti:

$rezz = mysqli_fetch_all($rez, MYSQLI_ASSOC);
foreach ($rezz as $index => $key) {
   $key1= $key['id_res'];
   $rez1 = $mysqli->query("SELECT rooms.floor FROM rooms
     INNER JOIN availability ON rooms.id_roo =availability.id_roo
     WHERE availability.id_res = '$key1';") or die($mysqli->error());
   $rezz1 = mysqli_fetch_all($rez1, MYSQLI_ASSOC);
  $rezz[$index]['rooms'] = $rezz1;
}

Pa onda u prikazu:

<?php foreach($rezz as $post) : ?>
   <li class="list-group-item">
     <?php echo 'ID: '.$post['id_res'].'<br> Dates: '.$post['indate'].' to '.$post['outdate'].'<br> Price: '.$post['tprice'].' € <br> Apartment/s: ';
     foreach ($post['rooms'] as $rez1){
       echo ' - '.$rez1['floor'];
     }
    echo '<br> Note: '.$post['note'].'<br> Date of reservation: '.$post['resdate']; ?>
   </li>
<?php endforeach; ?>

header("Location: error.php"); ti samo postavi header, međutim, ne spriječava daljnje izvršavanje koda. Ako dalje u kodu imaš taj blok koji ispisuje message, onda će ti se isti ispisati, unsetati, poslati browseru koji će to odbaciti jer redirect ima prioritet.

Nakon redirecta pozovi exit; kako se ne bi kod dalje izvršavao.

Nije da ti želim preporučiti to rješenje, već sam vidio da ga koristiš, pa sam zato napisao moguć uzrok problema s tim rješenjem.

Najbolje ti je uopće ne koristiti session, već prije samog html-a pokupiš post, obradiš što imaš, postaviš filtrirane vrijednosti za ponovni prikaz forme, error poruke i tek onda ideš na print html-a, ovisno o rezultatima obrade. Na taj način prilikom prikaza već znaš rezultat, sve se izvrši u istom pozivu, a korisniku možeš vratiti već unesene vrijednosti ako nešto mora popraviti (recimo, ako je datum zauzet, a ti mu vratiš error bez njegovih upisa, neće ponovno sve upisivati, već otići sa stranice).

napisa sam kod za promjenu lozinke, ali mi non stop vrača kako mi stara šifra iz baze i stara šifra koju upišem nisu jednake..

preko vardumpa sam provjeria da mi dobro uvati šifru iz baze, međutim kad upišem staru iz nekog razloga nije jednaka ovoj

kod

https://pastebin.com/cNappDnx