Сергей Шлоков

Сергей Шлоков

С нами с 31 января 2013; Место в рейтинге пользователей: #5
01 апреля 2018, 14:05
+2
Это же не багфиксы. Эти предложения нужно для начала обсудить.
01 апреля 2018, 08:55
0
Вот ещё один из примеров (класс modRequest).


А если присмотреться повнимательнее:
1. Если метод «alias» — ищется id ресурса методом findResource().
2. Дальше (в скриншот не попало) методом getResource() загружается этот ресурс в $modx->resource.
3. А в этом методе getResource() опять забито условие
if ($method == 'alias') {
      $resourceId = $this->modx->findResource($identifier);
}
В общем, рефакторить и рефакторить.
31 марта 2018, 17:54
+2
Да весь этот функционал уже давно требует рефакторинга. Мало того, что нет никакой возможности добавить свои аттрибуты в методы regClient*(), так ещё и в качестве ключей хранить код как минимум не профессионально. Т.е. при регистрации сырого кода типа
<script>
   Тут яваскрипт код...
</script>
он прописывается в качестве ключа в массив
// $src - это сырой код
$this->loadedjscripts[$src]= true;
А потом он проверяется на существование
isset ($this->loadedjscripts[$src])
Я понимаю, что PHP всё стерпит. Тут всё-таки напрашивается хэш.

Вот этот код тоже выглядит немного по школьному
$this->sjscripts[count($this->sjscripts)]= $src;


П.С. Я даже в какой-то момент при исследовании кода MODX хотел целую статью написать про то, что не боги горшки обжигают. Но потом просто PR делал. Что стоит один только код получения id страницы ошибки
$errorPageId = $modx->getOption('error_page', null, 404);
и страницы для неавторизованных пользователей
$PageId = $modx->getOption('unauthorized_page', null, 401);
31 марта 2018, 16:20
+2
У Сергея Шлокова тоже
type="text/javascript"
Я так понимаю, что и
type="text/css"
тоже устарел. Завтра выпущу новую версию. Заодно в ней и поправлю это дело.
30 марта 2018, 06:56
0
Написать в техподдержку магазина.
29 марта 2018, 22:09
0
modx_users — таблица пользователей.
modx_membergroup_names — таблица групп.
modx_member_groups — pivot таблица (связь пользователей с группами).
29 марта 2018, 11:12
0
А в этом сообществе только один Василий должен помогать? А остальные должны предварять свой ответ целым абзацем про то, что никто не помогает?
Зайди в сообщество Laravel. А затем перечитай свой комментарий.

П.С. Кроме того, сейчас модно тусоваться в Слаке и Телеграме. Лично я ушёл из Телеграм-канала MODX, надоело ломать глаза глядя на полотенца кода.
20 марта 2018, 22:10
+12
Не нравится MODX?
composer require laravel/laravel
20 марта 2018, 13:01
+1
oneBooking при установке требует miniShop2. Именно для него и созданы модули оплаты. Но имейте ввиду, что oneBooking несовместим с минишопом выше версии 2.1.12. См. документацию.

Кроме того, проверьте, есть ли нужный вам модуль оплаты в магазине.
20 марта 2018, 07:37
0
Доступные модули для оплаты можно посмотреть тут.
03 марта 2018, 13:13
+4
В нашей стране диктатура. Могут и в Гулаг сослать. А вот у них свобода и демократия — поэтому так можно. :)

П.С. Я понимаю, конечно, что Николай имел ввиду коммерческую целесообразность за данную работу. Но это решать пользователям. Не нравится — не ешь. Не вижу предмета для спора.
03 марта 2018, 11:17
+3
Так их, Николай. Не останавливайся. :) Вот ещё один пакет fontAwesome, который просто прописывает в head ссылку. У него аж под 2 тысячи скачиваний. %)

В общем, работы в этом направлении завались. )
28 февраля 2018, 19:09
+1
Быстрое решение — в таблице modx_stat_online_users полю message выставить дефолтное значение null.
23 февраля 2018, 17:54
+1
'templates' => [ 
	["access": true, "id": 36],
	["access": true, "id": 37],
]
Т.е. массив массивов.