wdrożenia PrestaShop Powiększenie zdjęć moduł

Zabezpieczenie sklepu.

  • 33 Odpowiedzi
  • 8735 Wyświetleń

0 użytkowników i 2 Gości przegląda ten wątek.

*

Offline design4VIP

  • user
  • Mieszkaniec
  • *****
  • 3,866
  • 131
    • design4VIP z innej strony...
  • PrestaShop: PS
  • Status: Programista
Odp: Zabezpieczenie sklepu.
« Odpowiedź #30 dnia: Październik 06, 2014, 08:21:56 pm »
kurde, to fajnie masz z zona, jezlei tak szybko dochodzi :D :D :D

powaga, to na prade  nardzo prosta robota... otwierasz plik glowny modulu  'nazwamodulu.php', szukasz czy tam checkbox czy radio i odczytujesz jego nazwe, nazwa zazwyczaj jest zmienna wysylana do szablonu ($zmienna), w samym szalbonie, jak pwialem wyzej z pliku authentication.tpl wrzucasz {$zmienna|var_dump} i na samej gorze storny (albo nieraz w okolicach gdzie zmienna zostala umeiszczona) pokazuje ci to co zwraca zmienna, czyli patrzysz spowrotem na na plik .php i odczytane radio i widzisz jakie 'value' powinna miec przy wlaczonym a jakie przy wylaczonym i sprawdzasz czy poprawnie zwraca 'dump'

no kurde, to mzoe skomplikowanie brzmi, ale prosciej nie potrafie tego opisac, a jak zaczniesz wg tego kombinowac, bedziesz zalowal, ze zona tak szybko nie dochodzi jak udalo ci sie to rozkminic :D 
design4VIP z innej strony...

*

Offline qwazar

  • Tubylec
  • ****
  • 166
  • 0
  • prestashop
  • PrestaShop: 1.6
  • Status: Właściciel
Odp: Zabezpieczenie sklepu.
« Odpowiedź #31 dnia: Październik 07, 2014, 11:03:09 am »
Dzięki deejay3, pokombinuję taj jak piszesz.
Ale napisałem też do autora modułu i bardzo szybko mi odpisał:


Cytuj
Hello,


Thanks for reporting me this issue.
You're right, even if the option was desactivated, the captcha was still visible on the create account form.
I correct this issue in the last version ( 0.2.2 ) .
You can find this version in attachment.


Regards,


 8)

*

Offline design4VIP

  • user
  • Mieszkaniec
  • *****
  • 3,866
  • 131
    • design4VIP z innej strony...
  • PrestaShop: PS
  • Status: Programista
Odp: Zabezpieczenie sklepu.
« Odpowiedź #32 dnia: Październik 07, 2014, 11:14:54 am »
no to gdzies masz w zalaczniku poprawke, tak jak pisze. a tak jak ja pisalem wczesniej, takie modulu czesto maja problemy z poprawnym wysylaniem watrosci radio czy checkbox, albo, to juz poza tematem, czest nawet jak sie wybierze to pozniej nie widac tego w modulach, tylko zaznaczany jest default... z tym juz sie spotkalem wielokrotnie. czysta nieumiejetnosc uzycia 'checked' ;)  ale co wymagac za darmo :D i tak dziwne, ze odpisal :D

patrzac w modul no ewidentnie jest blad, wlasnie z uzyciem 'checked':

plik eicaptcha.php:100-112

<div>
                     <label>'.$this->l('Yes').'</label>
                     <input type="radio" name="captcha_enable_account" value="1" ';
                     if ( Configuration::get('CAPTCHA_ENABLE_ACCOUNT') == 1 )
                        $this->_html .= 'checked="checked"';
                    $this->_html.='/>   
                     <br class="clear" />
                     <label>'.$this->l('No').'</label>
                     <input type="radio" name="captcha_enable_account" value="0" ';
                     if ( Configuration::get('CAPTCHA_ENABLE_ACCOUNT') == 0 )
                        $this->_html .= 'checked="checked"';
                    $this->_html.='/>
                     </div>
juz od zarania nie uzywa sie checked="checked" tylko samo "checked"

ale nigdzie nie widze, zeby gdzie dalej byl jakis warunek wysweitlania w rejestracji. trzeba by go wpisac albo w kontrolerze dla hook (to by bylo poprawnie) albo w samym .tpl (to poprawnie ale nie do konca ;D)
design4VIP z innej strony...

*

Offline qwazar

  • Tubylec
  • ****
  • 166
  • 0
  • prestashop
  • PrestaShop: 1.6
  • Status: Właściciel
Odp: Zabezpieczenie sklepu.
« Odpowiedź #33 dnia: Październik 07, 2014, 11:43:23 am »
Grunt że działa.
W załączniku nowy, poprawiony moduł.