JohnCMS | Разработка следующей версии

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

AlkatraZ

Не проще и не понятнее ли было сразу написать так?
$load_cat . '/' . $fname

Лучше наверное не хардкодить слешами, как-то так:

$load_cat . DIRECTORY_SEPARATOR . $fname

# ДоХтор (26.10.2019 / 18:03)
AlkatraZ Лучше наверное не хардкодить слешами, как-то так:
$load_cat . DIRECTORY_SEPARATOR . $fname

А это смотря что формируется.
Если для файловой системы, тогда действительно лучше юзать константу.
Ну а если формируется URL, там может быть только /

# kantry (26.10.2019 / 10:26)
AlkatraZ, Странно как то роутер работает, пока не удалил в корне папку registration, он меня в неё регистрироваться отправлял

А ничего странного, тут роутер не виноват, до него просто дело не доходит.
Дело в том, что в .htaccess прописано правило, что если указанный файл, или папка (что для апача однохерственно) существует, то ссылка передается ему.
А если не существует, то открывается фронт-контроллер index.php в котором уже начинает действовать роутер.

Добавлено: 26.10.2019 / 18:46
AlkatraZ

Если для файловой системы, тогда действительно лучше юзать константу.

Судя по именам переменных $load_cat и $fname речь идёт как раз про ФС.

Добавлено: 26.10.2019 / 18:47
Ну да ты и в коде ведь можешь посмотреть.

# ДоХтор (26.10.2019 / 18:47)
Судя по именам переменных $load_cat и $fname речь идёт как раз про ФС.

Да там по идее дофига чего надо переделывать.
Но это уже выходит за рамки моих планов.

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

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

AlkatraZ, +++++

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

Все, работу по переносу функционала в модули и глобальную реструктуризацию завершил.
Сейчас переделываю все сылки, убираю любые упоминания в ссылках index.php
И уберу это из маршрутов.
Поэтому, к примеру /forum откроется нормально, а /forum/index.php не откроется и выдаст ошибку 404

Это делается для "очищения" ссылочной массы. А то для поисковиков /forum и /forum/index.php могут считаться как разные ссылки.
---
Потом по планам немного почищу код, уберу длинные вызовы классов в импорты (use), ну и выпилю лишние константы _IN_JOHNCMS, они сейчас в модулях не нужны, там все прикрыто общим .htaccess
Потом еще кое-что почищу и применю патч от style-ci чтоб там бейджик "позеленел"
---
Ну а после займусь наконец тем, что все так давно ждут
Буду запиливать шаблонизатор.

# 3KZO (26.10.2019 / 21:57)
AlkatraZ, единая точка входа была уже предложена? я просто пропустил, перенос исходного кода за паблик директорию, подключенный гулп и минификация ассетов и пр.? нужен чистый ооп, без логического, см

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

"нужен чистый ооп, без логического, смешанного вместе с процедурным стиль"
Ну так это ПОЛНОСТЬЮ придется переписывать двиг с нуля. Кто займется?
Да и какая разница, если все хорошо работает?
Чистый ООП будет в другом проекте.

"единая точка входа была уже предложена? "
Она уже внедрена. Тот код, что сейчас в ветке /develop в репозитории, имеет единую точку входа и маршрутизатор Fastroute

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

"подключенный гулп"
А чем Grunt, который у нас есть не устраивает? Для него модулей раза в 3 больше, чем для Gulp.
И встречный вопрос "для чего?" или просто спросил ради "модного слова"?
Grunt - это чисто внутреннее дело (запускатор задач гг) разработчиков.

"минификация ассетов"
А это зачем?
У нас и так все уже минифицировано, куда еще меньше?
Это нужно там, где стили (к примеру) собираются на-лету.