Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www] Autorski sklep...
Forum PHP.pl > Inne > Oceny
ixpack
Przyznam, że boję się trochę - ale jak Zyx mi w jednym temacie napisał...
Cytat
Niektórym wręcz przydaje się zejście na ziemię i pokazanie im, że muszą się jeszcze podszkolić i według mnie pochwalenie czegoś, co ewidentnie jest źle zrobione, tylko wyrządziłoby im krzywdę.

Wszystko prócz większości pluginów jQuery, PHPmailer'a, fpdf'a jest napisane przeze mnie.

Adres strony: The Clothesline Online - wiem długaaa nazwa...

Stawiałem na szybkość, funkcjonalność bez zbędnych "pierdół". Wiem, że muszę dodać takie funkcjonalności jak: search i paginację :/
Kodu nie wstawiam ponieważ ciągle go przepisuję "kroczek po kroczku" na oop. W tym momencie jest to głównie php strukturalne + kilka klas wink.gif. Z resztą raczej mi umowa nie pozwala wink.gif, ale wstawię kawałki w przyszłości.

cron updateuje bazę "ze sklepu" do "sklepu online" co 5 minut w dni robocze... Gdy kończy się asortyment wywoływany jest skrypt tworzący od nowa menu - aby nie było "pustego linku" (choć muszę przerobić bo nie przewidziałem jednego "ale" - gdy coś jest, ale nie ma fotki...). Wpisy dla seo jak title jest pół dynamiczny wink.gif.
W sklepie są takie funkcjonalności jak: 10% off dla nowego klienta, x%off - miesięczna wygrana dla 1 usera, wirtualny kredyt, vat, shipping cost bazując na rodzajach i ilości produktów, akceptacja paypal i najważniejszych kart, front page jest łatwo edytowalny z poziomu panelu admina (użyłem ckeditor) - uorzystając z kilku tameplate'ów, produkty są łączone, kross kategoryzowane i inne...

Kod pisany był często "na wczoraj" więc na pewno są błędy/potwory spagetti - które mi umknęły.

Co proponujecie zmienić/dodać? Teraz oddelegowany jestem do innego projektu, ale niedługo będę miał za zadanie dokończyć "to", a skrypt będzie używany przy zleceniach "ecommerce", zanim to jednak nastąpi dodam search i paginację + tak zwane "wear with" dla strony produktu (polecane produkty z produktem x) + WASZE rekomendacje (o ile będą sensowne). Ogólnie kupa roboty dla 1 osoby, ale jak się sprawdzę to dostanę bolka do pomocy ;D.

Ps. jeżeli ktoś zechce założyć konto "vip" - przy wyborze rozmiaru była ładna tabelka... Ale ponieważ IE powodował błędy gdy radio miało "display:none", to to że tak powiem wywaliłem - jest brzydko wink.gif ale wiem jak to zrobić... Użyć text-indent... Jednak nie ma czasu hehe
r4xz
aktualnie nie mam czasu. mogę tylko powiedzieć że pierwsze wrażenie extra. jedynie w menu powiększyłbym czcionkę (dodatkowo w drugiej kolumnie wyrównanie do lewej proponuję). w inputach paddind 1-2 px chociaż (Fashion newsletter:).
toaspzoo
Przy scrollowaniu strony logo i jakiś panel z koszykiem u góry "position: fixed" mógłby się znaleźć
lwc
Kilka błędów ode mnie:

http://www.theclotheslineonline.co.uk/?pag...uct&ref[]a=

Efekt:

Kod
Warning: Illegal offset type in /home/tclo/www/index.php on line 583


oraz

Kod
Warning: urlencode() expects parameter 1 to be string, array given in /home/tclo/www/index.php on line 1570


http://www.theclotheslineonline.co.uk/?page=product&ref=

Efekt:
Kod
Warning: Invalid argument supplied for foreach() in /home/tclo/www/index.php on line 583


Dodatkowo XSS
ixpack
Cytat(r4xz @ 10.05.2011, 16:43:13 ) *
[...]Jedynie w menu powiększyłbym czcionkę (dodatkowo w drugiej kolumnie wyrównanie do lewej proponuję). w inputach paddind 1-2 px chociaż (Fashion newsletter:).

W panelu można ustawić wielkości czcionki wink.gif. Ustawienie wyglądu menu też jest - przewidziałem kilka wariantów, akturat ten co jest - wybrał klient wink.gif choć też jestem za wyrównaniem wszystkiego do lewej.
1-2px. Dobra myśl - jako nie designer ciężko mi zobaczyć takie rzeczy wink.gif. Przyda się.


Cytat(lwc @ 10.05.2011, 16:54:41 ) *
Kilka błędów ode mnie:...


Super - o to chodziło =]. Naprawiłem na prędce. Mam nadzieję, że ktoś znajdzie inne rzeczy - które na 100% mi umknęły :/. Lepiej tu jak później przez jakiegoś domorosłego hakera :/
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.