капча

[savage4pro]

все версии 2.х, включая оригинальный опенкарт

 

при включенной капче (что стандартной, что рекапча от гугла) и попытке отправить сообщение с ошибками заполнения в языковых данных перезаписывается heading_title

решения 2:

1. на примере страницы контактов

catalog/controller/information/contact.php

после 

		$this->document->setTitle($this->language->get('heading_title'));		if (($this->request->server['REQUEST_METHOD'] == 'POST') && $this->validate()) {			...		}

еще раз загрузить языковые переменные и переустановить тайтл (при этом до блока с обработкой POST-данных и валидации строку $this->document->setTitle... можно убрать)

		$this->load->language('information/contact');		$this->document->setTitle($this->language->get('heading_title'));

2. вырезать к бесам в языковых файлах

catalog/language/*/captcha/*.php

// Heading$_['heading_title'] = '...';

[savage4pro]

читать так:

при этом

до блока с обработкой POST-данных и валидации

строку $this->document->setTitle... можно убрать

[Blade]

Удалил в языковых файлах 

 

// Heading

$_['heading_title'] = 'Капча';

 

В файле catalog/controller/information/contact.php , в блоке (строка 12-35)

 

public function index() {

$this->load->language('information/contact');

 

$this->document->setTitle($this->language->get('heading_title'));   --------------удалил

 

if (($this->request->server['REQUEST_METHOD'] == 'POST') && $this->validate()) {

$mail = new Mail();

$mail->protocol = $this->config->get('config_mail_protocol');

$mail->parameter = $this->config->get('config_mail_parameter');

$mail->smtp_hostname = $this->config->get('config_mail_smtp_hostname');

$mail->smtp_username = $this->config->get('config_mail_smtp_username');

$mail->smtp_password = html_entity_decode($this->config->get('config_mail_smtp_password'), ENT_QUOTES, 'UTF-8');

$mail->smtp_port = $this->config->get('config_mail_smtp_port');

$mail->smtp_timeout = $this->config->get('config_mail_smtp_timeout');

 

$mail->setTo($this->config->get('config_email'));

$mail->setFrom($this->request->post['email']);

$mail->setSender(html_entity_decode($this->request->post['name'], ENT_QUOTES, 'UTF-8'));

$mail->setSubject(html_entity_decode(sprintf($this->language->get('email_subject'), $this->request->post['name']), ENT_QUOTES, 'UTF-8'));

$mail->setText($this->request->post['enquiry']);

$mail->send();

 

$this->response->redirect($this->url->link('information/contact/success'));

 

}

 

$this->load->language('information/contact');

        $this->document->setTitle($this->language->get('heading_title'));      --------------- добавил

 

После всего этого слово "Капча" пропало, но появилось другое   Что не так делаю ?

[Blade]

Так как это возникает при ошибке, до ответа на мой вопрос выше, сделал так:

 

// Heading

$_['heading_title'] = 'Проверьте форму';

[savage4pro]

во-первых, варианты на то и варианты, что либо либо

а во-вторых - кэш

[Blade]

во-первых, варианты на то и варианты, что либо либо

а во-вторых - кэш

ничего не понял

как мне исправить что бы было правильно ? я написал что изменил, но после этого стало появляться "Captcha"

 

зы хотя не совсем понимаю - а что там должно появляться когда все правильно ?

[savage4pro]

если идти по первому решению, то языковые файлы менять не надо вовсе (а после изменения контроллера сбросить кэш модификаторов, если хотя бы один из них кэширует этот файл)
если идти по второму - то надо, притом во всех языках

catalog/language/*/captcha/*.php

когда все правильно, в крошках и в заголовке будет оригинальное название страницы, в данном случае, это "Связаться с нами"

[savage4pro]

Такое можно даже пулреквестом отправить на opencart, только вот валидацию убирать не надо.

отправил

https://github.com/opencart/opencart/pull/3639

[admin]

отправил

https://github.com/opencart/opencart/pull/3639

Даниель парень специфичный поэтому может отказать в принятии но потом исправит каким то только ведомым ему способом

[savage4pro]

да ну и бог с ним, решит и ладно

[Blade]

если идти по первому решению, то языковые файлы менять не надо вовсе (а после изменения контроллера сбросить кэш модификаторов, если хотя бы один из них кэширует этот файл)

если идти по второму - то надо, притом во всех языках

когда все правильно, в крошках и в заголовке будет оригинальное название страницы, в данном случае, это "Связаться с нами"

оффтоп конечно, то есть если в заголовке после ошибочного ввода указывается не "Связаться с нами" а "Проверьте форму" ничего страшного не происходит?)

или получается что происходит переход не на ту страницу ? 

Не понимаю если честно что не так.

[savage4pro]

нет, страница та же, просто заголовок страницы заменяется на заголовок из языкового файла капчи (и крошки на этой странице, мета тайтл не заменяется)

в некоторых темах заголовок капчи используется на странице с формой, если вы не будете использовать такие темы, то можно просто вырезать заголовок из языковых файлов

 

можно и забить, т.к. это ерунда по сути, мало кто из посетителей вообще внимание обратит на это

 

но если подходить педантично и от этой ошибки не спится, то, конечно, поправить надо

то есть по сути надо только одну строку добавить в каждый из файлов, что и куда, можно посмотреть здесь

 

https://github.com/savage4pro/opencart/commit/15b8f9041843c6959ddbc79361bad3246577db85

 

вероятно, когда-нибудь товарищ https://github.com/danielkerrпридумает, как это пофиксить красивей, и внесет исправления в ядро, но пока так

[savage4pro]

Merged #3639.