[www] Autorski sklep... Opcje

[ixpack]

Przyznam, że boję się trochę - ale jak Zyx mi w jednym temacie napisał...

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 (IMG:style_emoticons/default/wink.gif) . Z resztą raczej mi umowa nie pozwala (IMG:style_emoticons/default/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 (IMG:style_emoticons/default/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 (IMG:style_emoticons/default/wink.gif) ale wiem jak to zrobić... Użyć text-indent... Jednak nie ma czasu hehe

Ten post edytował ixpack 10.05.2011, 16:40:08

[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]

[...]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 (IMG:style_emoticons/default/wink.gif) . Ustawienie wyglądu menu też jest - przewidziałem kilka wariantów, akturat ten co jest - wybrał klient (IMG:style_emoticons/default/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 (IMG:style_emoticons/default/wink.gif) . Przyda się.

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 :/