ocStore 3 рефакторинг

[borodatych 0]

Здравствуйте,

 

На базе ocStore 3 делаю интернет-магазин по продаже автозапчастей.

Части над которими работаю копирую в свое пространство.

Благо, система позволяет делать так: `/index.php?route=namespace/account/register`.

Раз уж я скопировал с свое пространство, то заодно рефакторю код.

...это что бы не было вопросов, а нафига ты это делаешь...опущу остальной ликбез и сразу к делу.

 

К примеру, public/catalog/controller/account/register.php 80-84:

if (isset($this->error['telephone'])) {
   $data['error_telephone'] = $this->error['telephone'];
} else {
   $data['error_telephone'] = '';
}

 

Почему когда я заменяю на:

$data['error_telephone'] = @$this->error['telephone'];

 

У меня уже на страницы идет ошибка по этому полю!

 

Пока писал, еще проверил, проблема в 83 строчке:

$data['error_telephone'] = '';

 

Вот так работает:

$data['error_telephone'] = FALSE;

А вот так, как и мой вариант отрабатывал, уже нет:

$data['error_telephone'] = NULL;

 

Подскажите, почему так происходит?

 

Edited September 16, 2019 by borodatych

[chukcha 2,161]

потому что вы показали код контроллера, а не код шаблона

[borodatych 0]

Не могу понять на какую часть мое текста вы ответили?!

Шаблон родной\дефолтный от ocStore, вот конкретная секция:

 

[chukcha 2,161]

Выводом ошибки занимается - шаблон

Я не могу понять зачем вам такие извращения

 

$data['error_telephone'] = @$this->error['telephone'];

 

Если п какой либо причине ві получаете вывод ошибки, то, скорей всего, вы каким-то образом  вы переменную превращайте в true

[borodatych 0]

Там же дальше еще есть, можно просто заменить пустые кавычки '' на NULL и будет ошибка, проверьте, если есть возможность
А если сделать FALSE, то не будет ошибки
Хоть это и глупо, я делал vardump в контроллере, переменная $data['error_telephone'] была ровно такая, как я ее определил если !isset по ней: '',NULL,FALSE

 

Если:

$data['error_telephone'] = NULL;

То как тут:

{% if error_telephone %}
<div class="text-danger">{{ error_telephone }}</div>
{% endif %} </div>

Оказалось, что error_telephone равно тексту из language файла?

 

 

[chukcha 2,161]

Не поверите...

Прежде чем вам ответить

Я взяо ваш код повторил (мало ли, может я что-то упустил)

У меня не получилось

Потому были наводящие вопросы



 

[borodatych 0]

Только что выкачал и развернул ocStore 3.0.2.0 локально
Пробовал на php 5.6 и 7.1 ошибка повторилась

 

В данном файле:

public\catalog\controller\account\register.php

 

В 83 строке заменил: $data['error_telephone'] = '';

На: $data['error_telephone'] = NULL;

 

Итог тот же:

 

...сейчас с OpenCart попробую

Edited September 16, 2019 by borodatych

[chukcha 2,161]

        if (isset($this->error['telephone'])) {
            $data['error_telephone'] = $this->error['telephone'];
        } else {
            $data['error_telephone'] = '';
        }

А теперь скажите, откуда берется  $this->error['telephone'];

И.. почему не отрабатывет
 

        if (isset($this->error['email'])) {
            $data['error_email'] = $this->error['email'];
        } else {
            $data['error_email'] = '';
        }

 

[borodatych 0]

17 минут назад, chukcha сказал:

А теперь скажите, откуда берется  $this->error['telephone'];

 

1 час назад, borodatych сказал:

Оказалось, что error_telephone равно тексту из language файла?

Тут должно быть так:
Оказалось, что error_telephone равно тексту из language файла!

 

Вы из-за знака препинания не правильно интерпретировали что я написал?

 

Дико извиняюсь, вы мои знания проверяете или реально не знаете?

Если я в validate методе верну сразу TRUE (между 224 и 225 прописать return TRUE), будет ровно так же!
Тогда вопрос, откуда она `нахрен` тогда берется??

 

Если вы не знаете ocStore 3, то давайте не будет тратить ни мое ни ваше время

Я вот точно ее не знаю, поэтому и обратился за помощью

 

17 минут назад, chukcha сказал:

И.. почему не отрабатывет

Если я 

$data['error_email'] = '';

Заменю на 

 

$data['error_email'] = NULL;

То тоже на голой форме увижу ошибку валидации

Акцентирую, я просто перешел на страницу, данные не отправлял

Edited September 16, 2019 by borodatych

[chukcha 2,161]

11 минут назад, borodatych сказал:

Если вы не знаете ocStore 3, то давайте не будет тратить ни мое ни ваше время

https://opencartforum.com/profile/675243-chukcha/content/?type=downloads_file

 

 

[borodatych 0]

Внушительно
Это вы меня значит решили погонять...
Повторюсь, я недавно с этой коробочкой, гонять меня бесполезно, все очевидное я вроде попробовал
Говорю же в валидации тупо ретурн тру и ито он от куда то берет этот текст и лэнгвич файла

Сейчас установил opencart 3.0.2.3 там в 73 строке все того же регистр контроллера $data['error_email'] сделал NULL и вот так же на пустом поле почты сработала ошибка валидация

Вот как есть он там с рендеренгом что-то кружит и NULL в каком то условие не отрабатывает и он переподтягивает значение
Но это, опять таки тыкая пальцем в небо, либо твиг что-то мутит, либо js-ом бегается
В common.js поискал по id-шкам, не нашел или проглядел...

Вы так же на голой системе проверяли мою писанину?
Что за среда?

Edited September 16, 2019 by borodatych

[spectre 1,013]

Вам надо пойти в system/config/catalog.php и изучить вот этот кусочек

 

 

перед view грузится language и если переменная не определена 

то

 

грубо говоря если переменная null она запишется данными из языкового файла перед выводом во view

[borodatych 0]

34 минуты назад, spectre сказал:

грубо говоря если переменная null она запишется данными из языкового файла перед выводом во view

От души, все по делу! Просветили

 

Подскажите как карму апнуть (если здесь такое есть)

[chukcha 2,161]

хм, ждя того чтобы оно заменилась у вас должна была быть языковая переменная $_['error_telephone']
Которая по приведенным вами фрагментами нигде не отображалась

 

[borodatych 0]

В 16.09.2019 в 17:51, chukcha сказал:

хм, ждя того чтобы оно заменилась у вас должна была быть языковая переменная $_['error_telephone']
Которая по приведенным вами фрагментами нигде не отображалась

Ну что да потому...
Почему вы не хотите по серьезнее отнестись?!
В текущем обсуждение все есть!!
 

 

В 16.09.2019 в 15:11, borodatych сказал:

В 16.09.2019 в 14:55, chukcha сказал:

А теперь скажите, откуда берется  $this->error['telephone'];

 

В 16.09.2019 в 13:56, borodatych сказал:

Оказалось, что error_telephone равно тексту из language файла?

Тут должно быть так:
Оказалось, что error_telephone равно тексту из language файла!

 

Второй раз вам цитирую!

Ладно, проблема решена, знания пополнены.
Всех благодарю за потраченное время!