simpla Новая Simpla b5-php-8.2.1-2-admin

[sergeevizh]

На счет проверки переменной везде в шаблоне функцией isset, чтобы не вывалились WARNING.

Проще в api/Design.php добавить & ~E_WARNING в

$this->smarty->error_reporting = E_ALL & ~E_WARNING & ~E_NOTICE;

Изменено 8 февраля, 2023 пользователем sergeevizh

[admin]

13 минут назад, sergeevizh сказал:

На счет проверки переменной везде в шаблоне функцией isset, чтобы не вывалились WARNING.

Проще в api/Design.php добавить & ~E_WARNING в

$this->smarty->error_reporting = E_ALL & ~E_WARNING & ~E_NOTICE;

Это как-то туповато будет) Дело не в том,  чтобы убрать ошибки, их можно и вовсе не выводить, а в том, что isset увеличивает скорость работы скрипта, безопасность, для того и вводятся в новых версиях php обязательные проверки на значение, чтобы зря код не выполнять.

Кроме этого, вы будете со временем получать fatal error и потом будете долго разгребать весь этот мусор среди массы ошибок.

Есть требования к коду и его надо соблюдать !!

[sergeevizh]

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

Это как-то туповато будет) Дело не в том,  чтобы убрать ошибки, их можно и вовсе не выводить, а в том, что isset увеличивает скорость работы скрипта, безопасность, для того и вводятся в новых версиях php обязательные проверки на значение, чтобы зря код не выполнять.

Кроме этого, вы будете со временем получать fatal error и потом будете долго разгребать весь этот мусор среди массы ошибок.

Есть требования к коду и его надо соблюдать !!

Чтобы соблюдать требования к коду и безопасность, нужно переписать код вывода, а не маскировать проверками на null

[admin]

6 минут назад, sergeevizh сказал:

Чтобы соблюдать требования к коду и безопасность, нужно переписать код вывода, а не маскировать проверками на null

Код вывода чего? Какие есть варианты? Вы предлагаете переписать функции или обращение к функциям?

Например? Код вывода товара какой может быть? Или переписать смарти? 

[sergeevizh]

44 минуты назад, admin сказал:

Код вывода чего? Какие есть варианты? Вы предлагаете переписать функции или обращение к функциям?

Например? Код вывода товара какой может быть? Или переписать смарти? 

Про Smarty я вам ничего не писал, ведь ошибка выходит из-за PHP8.

На счет вывода, я имел ввиду актуализировать код части view движка.

В остальном вы правы и пока выход в isset() или $this->smarty->error_reporting = E_ALL & ~E_WARNING & ~E_NOTICE;

[admin]

5 минут назад, sergeevizh сказал:

Про Smarty я вам ничего не писал, ведь ошибка выходит из-за PHP8.

На счет вывода, я имел ввиду актуализировать код части view движка.

В остальном вы правы и пока выход в isset() или $this->smarty->error_reporting = E_ALL & ~E_WARNING & ~E_NOTICE;

Smarty тоже написан на php и тоже дает ошибки, потому пришлось ставить Smarty4  адаптировать его под код симплы, т.к. там сейчас стоит не чистый 4 смарти.

Что такое актуализировать код части view движка? 

[admin]

В 2/7/2023 в 9:58 PM, sergeevizh сказал:

Ну и раз разнесли фронт и бек.

То и модели бека лучше перенести в папку /simpla/, контроллеры тоже вынести в отдельную папку и получить дерево:

/simpla/

/simpla/ajax/

/simpla/cml/

/simpla/controller/

/simpla/design/

/simpla/files/

/simpla/model/

/simpla/pclzip/

/simpla/rest/

/simpla/update/

/simpla/.htaccess

/simpla/.passwd

/simpla/.index.php

Папку /api/ разнести на модели фронта и настройки 

Папку /view/ переименовать в controller, так как view это design

Отказаться от генерации капчи средствами PHP

Smarty/PHPMailer/MobileDetect/PCLZip через Composer

Ну в общем, привести все к нормальному MVC 😃 и тогда будет пушка

Кстати, если вы хотите обновить фронт, а бэк оставить старым, т.к. там много чего уже сделано, фильтра, модули и т.д.

У меня сейчас есть такая задача.

То гораздо удобнее и проще будет просто все это слить в папку admin и переписать название классов в файлах и все.

[sergeevizh]

24 минуты назад, admin сказал:

Кстати, если вы хотите обновить фронт, а бэк оставить старым, т.к. там много чего уже сделано, фильтра, модули и т.д.

У меня сейчас есть такая задача.

То гораздо удобнее и проще будет просто все это слить в папку admin и переписать название классов в файлах и все.

Я об этом и написал, что и сделал сам, папку с models(api) админки слил в папку simpla. Сделал структуру controllers отдельно, models отдельно, view(design) отдельно, ajax отдельно и входная точка index.php.
Дальше backend на поддомен. И получаем, frontend сам по себе, backend сам по себе.

Изменено 9 февраля, 2023 пользователем sergeevizh

[admin]

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

Я об этом и написал, что и сделал сам, папку с models(api) админки слил в папку simpla. Сделал структуру controllers отдельно, models отдельно, view(design) отдельно, ajax отдельно и входная точка index.php.
Дальше backend на поддомен. И получаем, frontend сам по себе, backend сам по себе.

Этого всего можно и не делать для того, чтобы бэкенд подключить к отдельному домену. У вас в любом случае index находится в папке simpla, куда вы и обращаетесь. Привяжите к нему поддомен, а все остальное подключайте как хотите и где хотите.

[Alex]

У админа все как всегда хорошо, а у него даже Perl не стоит :)))) Сергееч дело говорит, а вы сразу в кусты!

На главной:

Warning: Undefined array key "lang" in C:\Apache24\htdocs\simpla.loc\public_html\view\IndexView.php on line 64
Warning: Undefined array key "googtrans" in C:\Apache24\htdocs\simpla.loc\public_html\view\IndexView.php on line 127

[admin]

3 часа назад, Alex сказал:

У админа все как всегда хорошо, а у него даже Perl не стоит :)))) Сергееч дело говорит, а вы сразу в кусты!

На главной:

Warning: Undefined array key "lang" in C:\Apache24\htdocs\simpla.loc\public_html\view\IndexView.php on line 64
Warning: Undefined array key "googtrans" in C:\Apache24\htdocs\simpla.loc\public_html\view\IndexView.php on line 127

У вас по какой-то причине не срабатывает скрипт создания lang куки, потому так. Думаю, что таким профи разобраться в этом не будет проблем !!)

[Alex]

Да не только у меня, у сергееча тоже самое. На первой странице посмотрите.

[admin]

6 минут назад, Alex сказал:

Да не только у меня, у сергееча тоже самое. На первой странице посмотрите.

Вам показывает не ошибку, а предпреждение, что еще нет $_COOKIE["googtrans"], пустое значение, т.к. кука еще не установлена в браузере. 

Кука устанавливается скриптом, потом это предупреждение пропадает, т.е. значение $_COOKIE["googtrans"] уже не пустое.

Если вам принципиально, чтобы не светилось это предупреждение, тогда вам надо добавить проверку на пустое значение.

К примеру, если у вас сейчас на 127 позиции Warning: Undefined array key "googtrans", то вам : 

if ($_COOKIE["googtrans"] != NULL && $this->settings->gtranslate != 1) {
            unset($_COOKIE['googtrans']);
        }

надо заменить на:

if (isset($_COOKIE["googtrans"]) && $_COOKIE["googtrans"] != NULL && $this->settings->gtranslate != 1) {
            unset($_COOKIE['googtrans']);
        }

Аналогично с Warning: Undefined array key "lang".

 

[Alex]

А сделать это сразу по умолчанию в скрипте нельзя? Меня удивляет ваш подход, за вами нужно дорабатывать.

[admin]

1 минуту назад, Alex сказал:

А сделать это сразу по умолчанию в скрипте нельзя? Меня удивляет ваш подход, за вами нужно дорабатывать.

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

Либо покупаешь платный скрипт, где люди за зарплату пишут код, а не за возможное спасибо.

Мне некоторые люди просто кидают бабки за то, что я уже сделал и это им очень помогло с магазином.

Некоторым из них я доделываю что-то или помогаю доделать, т.к. на симпле это сделать очень легко.

Потому есть и такие люди, которые понимают, что нормальный труд должен быть оплачен.

Но это уже на совести каждого человека индивидуально.

Вам модернизировали скрипт, чего не сделал сам разработчик, у которого он стоил 400$.

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

А здесь условия диктует тот, кто делает конструктор для дальней интеграции скрипта согласно умению самого разработчика.

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

А если есть умение, то дерзайте)