Forum PHP.pl

Jasne że nie jest. Ale jest bardziej semantyczne i czytelne jak się zacznie robić powiązania między większą ilością tabel.

Hej Laxus,

Proponowałbym, żebyś zaczął używać join (i ładne graficzne wyjaśnienie).

[SQL] pobierz, plaintext 
SELECT
    k.id, k.nazwa, s.numer
FROM kontrahenci AS k
JOIN sale AS s ON k.salaID = s.id
[SQL] pobierz, plaintext 

Wydaje mi się, że po prostu CSSem, określając rozmiar procentowo. Chyba, że masz coś innego na myśli.
https://jsfiddle.net/bostaf/0k61kc21/

Po kolei Drapeta. Na razie nie zrobiłeś tego o czym kilka osób tutaj Ci napisało - nie ma w twoim kodzie żadnego ifa. Zrób dokładnie tak jak napisałem. Jeszcze raz podaję co i jak, i w nawiasach dodaję jak w języku programistycznym rozumieć słowa "sprawdź czy", "jeżeli", "jeśli nie", "istnieje". Na razie zrezygnuj z try/catch. Będzie prościej na początek. Później sobie dodasz.

1. Sprawdzenie (isset), czy id_cytatu jest (isset) przekazane GETem ($_GET['id']). (żadnego echa na razie)
1.1. Jeśli jest (if), to pobranie z bazy cytatu z tym id. Ale na razie żadnego echa - zapamiętaj w zmiennej, np. $cytat. (żadnego echa na razie)
1.2. Jeśli nie (else), to sprawdzenie czy istnieje (isset) ciastko z zapisanym id ulubionego cytatu ($_COOKIE['id__ulubionego_cytatu']). (żadnego echa na razie)
1.2.1. Jeśli istnieje, to przekierowanie (header i exit) na url z id tego cytatu pobranym z ciastka ($_COOKIE). (żadnego echa na razie)
1.2.2. Jeśli nie istnieje, to wylosowanie cytatu i zapamiętanie go w zmiennej $cytat. (żadnego echa na razie)
2. Wydrukowanie (echo / czysty HTML po zamknięciu bloku kodu php "?>") strony razem z cytatem $cytat ORAZ ikonką polubienia cytatu z odpowiednim javascriptem zapisującym ciastko.

Korci mnie żeby Ci napisać gotowca, ale nie
Chyba trzeba od początku zacząć...
header Inicjuje przekierowanie, czyli wysyła tzw. nagłówki co powoduje przeładowanie strony na podany url. Nagłówki muszą być wysyłane zanim cokolwiek (nawet spacja) zostanie wysłane do przeglądarki (echo, print, var_dump, itd), w przeciwnym razie wywali błąd. Czyli już tutaj widać że Twój skrypt nie zadziała - masz echo przed wywołaniem header. Ale to tylko jeden z kilku problemów.
Napisz to dokładnie w takiej kolejności, i pisz kod punkt po punkcie:

1. Sprawdzenie, czy id_cytatu jest przekazane GETem. (żadnego echa na razie)
1.1. Jeśli jest, to pobranie z bazy cytatu z tym id. Ale na razie żadnego echa - zapamiętaj w zmiennej, np. $cytat. (żadnego echa na razie)
1.2. Jeśli nie, to sprawdzenie czy istnieje ciastko z zapisanym id ulubionego cytatu. (żadnego echa na razie)
1.2.1. Jeśli istnieje, to przekierowanie (header z exitem) na url z id tego cytatu pobranym z ciastka ($_COOKIE). (żadnego echa na razie)
1.2.2. Jeśli nie istnieje, to wylosowanie cytatu i zapamiętanie go w zmiennej $cytat. (żadnego echa na razie)
2. Wydrukowanie (echo / czysty HTML po zamknięciu bloku kodu php "?>") strony razem z cytatem $cytat ORAZ ikonką polubienia cytatu z odpowiednim javascriptem zapisującym ciastko.

I tyle.
Odnośnie $_GET - jeśli url ma wyglądać tak: strona.pl?id_cytatu=32, to id cytatu będzie w $_GET['id_cytatu']. Czyli w p.1 sprawdzasz najpierw czy w ogóle $_GET['id_cytatu'] istnieje (isset).
Odnośnie try/catch - zrezygnuj z tego na razie, bo na tym etapie może Ci zakłócać zrozumienie reszty. Potem koniecznie z try/catch, ale na razie to wyrzuć.

Dokładnie. Id cytatu masz w $row['id'], czyli

[PHP] pobierz, plaintext 
$idCytatu = $row['id'];
[PHP] pobierz, plaintext 

Bez javascript to ta druga opcja o której pisał Pyton - redirect. Za pomocą header. (btw. wg mnie to powinna być pierwsza opcja):

To troszkę inaczej zrób.
1. Obok cytatu daj ikonkę "ulubiony". Kliknięcie tej ikonki niech powoduje zapisanie id cytatu ($row['id']) w ciastku - javascriptem.
2. Ten kod, który masz lekko zmodyfikuj - na początku daj "if ciastko istnieje" then "pobierz cytat z danym id" a jeśli nie, to losowy.

Zdecydowanie delegacja. Zignoruj to co napisałem wcześniej, nie doczytałem wcześniejszych i nie zauważyłem, że już weszliście na wyższy poziom

Ciekawa sprawa. Lukałem nawet w kodzie phpmyadmina (błąd jest rzucany tutaj), żeby sprawdzić co się tam dzieje. Wygląda jakby Twój 2-megowy plik w ogóle nie był uploadowany, mimo ze konfiguracja zezwala.
Jedyne co mi przychodzi do głowy, to że zmodyfikowałeś nie ten php.ini, który jest ładowany przy starcie php. Bo taki scenariusz jest możliwy - w systemie może być kilka ini.
Daj w linii poleceń:

albo

i sprawdź czy na pewno modyfikujesz ten ini który jest ładowany.

by_ikar - po prostu _jeszcze_ Ci się to nie przydarzyło

@szczrzcz Rozszerzając to co już napisał by_ikar - gedit i inne edytory tekstowe generują pliki backup z tyldą na końcu nazwy. To można konfigurować w opcjach.

Na pierwszy rzut oka i generalnie to nie - za dużo tam wszystkiego i nie za bardzo wiadomo jakiemu celowi ma to konkretne filtrowanie służyć. Filtrowanie/eskejpowanie/czyszczenie musi mieć konkretny cel. Jednym algorytmem nie da się targetować wszystkich celów. Np. filtrowanie pod kątem wyświetlania tekstu w htmlu będzie inne niż filtrowanie pod kątem przesyłania danych do bazy.

Ja zgaduję, że ty chcesz zabezpieczyć dane z formularzy które lecą do bazy. W takim przypadku samo mysqli_real_escape_string wystarczy. (później dojdziesz do jeszcze lepszej parametryzacji) No i może trim. Chodzi o to, że to co user napisze Ci w textarea nijak nie skrzywdzi bazy danych. Baza ma immunitet na hakowanie javascriptem i innymi sztuczkami. Bazę można uszkodzić jedynie błędnie skonstruowanymi zapytaniami, którym mysqli_real_escape_string (a docelowo parametryzacja) i pozamykane apostrofy/cudzysłowy całkowicie zapobiegają.
Tu zresztą są różne opcje - albo oczyszczać przed wprowadzeniem albo przy wyprowadzaniu/pobieraniu.

Filtr htmlspecialchars wykorzystasz podczas pobierania danych z bazy w celu wyświetlenia ich HTMLem.

Co do addslashes to ja nie wiem do czego to komu potrzebne (ale chętnie bym się dowiedział). Kiedyś, jak były kombinacje z magic_quotes i ext_mysql to się z tego korzystało, ale teraz...?

Inna uwaga odnośnie budowy Twojej klasy. Tam inkludujesz plik i połączenie z bazą - unikaj czegoś takiego już na początku. Myśl o tym, żeby klasa i funkcja była samodzielna, a wszelkie zależności pobierała z zewnątrz, za pomocą przeznaczonego do tego mechanizmu czyli konstruktora. Inkludowanie plików w klasie jest niemoralne, złe i wyklęte. Po trzykroć hańba

Nie wiem czy nie za krytycznie Cię oceniłem. Jeśli tak to wygląda to nie powinno bo nie miałem takiego zamiaru. Ogólnie mi się podoba Twój kierunek - eksperymentowanie to podstawa. Im więcej się popsuje tym więcej się nauczy

content_style

[PHP] pobierz, plaintext 
<script>
tinymce.init({
    selector:'textarea', 
    formats : {
        p : {selector : 'textarea', classes : 'full'},
    },
    content_style: "body {text-align: justify;}"
});
[PHP] pobierz, plaintext 

To nie ma związku z pierwotnym problemem. Zaznacz ten wątek jako rozwiązany i otwórz nowy.

Rzuciłem sobie to wszystko na

[PHP] pobierz, plaintext 
var_dump('<pre>', $xml);
[PHP] pobierz, plaintext 

i zobaczyłem, że pierwsze 6 elementów to nie są wpisy bloga tylko jego parametry (title, link, ...). Trzeba je jakoś pominąć i załatwione. Tak na szybko z property_exists:

[PHP] pobierz, plaintext 
$xml = simplexml_load_file("http://blog.piotrnalepa.pl/feed/");
 
foreach ($xml as $channel)
{
    echo $channel->title;
 
    foreach($channel as $item)
    {
		if (property_exists($item, 'link')) {
			echo $item->link;
			echo $item->title;
			echo $item->description;
		}
    }
}
[PHP] pobierz, plaintext 

Ale na pewno da się inaczej i może bardziej skutecznie/ergonomiczne.

Cool
Ale ja bym to jednym zapytaniem zrobił, mniej mielenia i większa czytelność:

[PHP] pobierz, plaintext 
$nick_czata = mysqli_query($conn,
    "select users.*, avatary.avatar
    from users
    left join avatary on avatary.id_user = users.id
    where id in (
		select
			concat(if(`od` = '".$moje_id_bazy_danych."', '', `od`), if(`do` = '".$moje_id_bazy_danych."', '', `do`))
		from friend
		where `potwierdzenie` = 1 and (`od` = '".$moje_id_bazy_danych."' or `do` = '".$moje_id_bazy_danych."')
    )
    and '".$czas_czat."' < act_time"
);
[PHP] pobierz, plaintext 

To może na zamotane wyglądać, ale jak przejdziesz na zapytania parametryczne to będzie czytelniejsze. Albo gdybyś chciał z tego zrobić procedurę albo funkcję w bazie.

Aha, i data type dla act_time proponowałbym DATETIME.

No racja, to w końcu przedszkole.
1. dataType nie zmieniaj. Zostaw tak jak w oryginalnym kodzie 'json'
2. Zmień $_GET na $_POST - dla wyjaśnienia, tam w parametrach $.ajax masz

[JAVASCRIPT] pobierz, plaintext 
type      : 'POST', //Method type
[JAVASCRIPT] pobierz, plaintext 

i to właśnie określa metodę, którą ajax będzie przesłany. Napisane jes "POST" czyli dane trzeba pobierać z tablicy $_POST a nie $_GET.

Zrób to co kayman napisał - F12 w przeglądarce otwiera okno "developer tools". Wybierz zakładkę "network" albo "sieć" i odśwież swoją stronę. Tam w zakładce "network" pokazują się wszystkie zasoby (pliki) z mnóstwem dodatkowych informacji ładowane z danego adresu. Za każdym razem jak wywołasz swojego ajaxa, na dole tamtej listy powinien pokazać się dodatkowy wiersz z nazwą pliku wołanego ajaxem - "technikajax.php". Jak go klikniesz, to po prawej stronie pojawi się okienko z dodatkowymi informacjami - nagłówkami, treścią itd. Wykorzystaj to, żeby sprawdzić co się dzieje. Na preview będziesz mógł zobaczyć błędy jakie wyskakują w "technikajax.php". Przeanalizuj i jak coś jest jeszcze nie jasne to pisz.
Ale pamiętaj, że mimo że to przedszkole, to my z fusów i zdawkowych "nie działa" też nie wywróżymy.

Siema. Tu masz gotowca: https://jsfiddle.net/bostaf/dh5wnywu/
Nie objaśniam bo nie chciałeś, ale jak zmienisz zdanie to napisz.

Tu jest błąd:

[JAVASCRIPT] pobierz, plaintext 
var Podsumowanie = mySum(cenaW, cenaM);
[JAVASCRIPT] pobierz, plaintext 

"mySum" spodziewa się trzech parametrów a dostała dwa, stąd NaN.

Przerób sobie "mySum" żeby miała domyślne wartości parametrów:

[JAVASCRIPT] pobierz, plaintext 
function mySum(c1 = 0, c2 = 0, c3 = 0)
[JAVASCRIPT] pobierz, plaintext 

albo pamiętaj, żeby wołając mySum podawać 3 parametry:

[JAVASCRIPT] pobierz, plaintext 
var Podsumowanie = mySum(cenaW, cenaM, 0);
[JAVASCRIPT] pobierz, plaintext 

Nigdy tego nie próbowałem, ale poczytałem sobie dokumentację (How to Use PHP instead of Twig for Templates,) i nie znalazłem tam ani słowa o tym, że jest możliwe ani że nie nie jest.

Na logikę powiedziałbym, ze nie - bo Twig nie zrozumie metod PhpEngine'a i vice versa.

Ale tutaj jest jasno napisane, że można używać wielu systemów szablonów jednocześnie. Z opisu to jest nawet sensowne bo metoda polega na zdefiniowaniu jako systemu szablonowego "pośrednika" (klasy DelegatingEngine) działającego jako system szablonowy, ale w rzeczywistości delegującego zadania do właściwego podsystemu.
Pogooglowałem trochę z ciekawości i nie znalazłem żadnego projektu, który by czegoś takiego używał. Ja też nie wyobrażam sobie praktycznego zastosowania.

Znalazłem. Ta pierwsza wersja była oczywiście OK, ale po przekierowaniu w linii 38 brakowało instrukcji exit albo die.

Niom, w pierwszej wersji tak miał. Teraz zmienił. W pierwszej wersji teoretycznie wszystko wyglądało OK, ale nie działało tak jak powinno.

A w profile.php masz session_start na początku?

(ps. bardzo ładnie sformatowany kod. Porządek to podstawa, tak trzymać)

Problem rozwiązany.
Dosyć długi kod został zdefiniowany jako osobna funkacja, która następnie została wywołana osobno dla oryginalnego dokumentu jak i tego zmienionego dynamicznie:

[JAVASCRIPT] pobierz, plaintext 
function mojaFunkcja () {
    // cały długi kod, który nie hulał po ajaxie
}
$(document).ready(function() {mojaFunkcja();});
$(document).ajaxStop(function() {mojaFunkcja();});
[JAVASCRIPT] pobierz, plaintext 

document.ready oddziałuje tylko na elementy utworzone przed jego wywołaniem, a zatem nie uwzględnia tych utworzonych dynamicznie (np ajaxem), a ajaxStop operuje tylko na elementach dodanych dynamicznie. Stąd konieczność dwóch wywołań całego kodu.

...i wisienkę na torcie:
$("#odptext").focus();