Іван Клімчук

Іван Клімчук

С нами с 16 декабря 2012; Место в рейтинге пользователей: #12
07 апреля 2016, 17:09
+3
Я бы пометил эту фичу как deprecated и выпилил бы через релиз. Как промежуточный вариант — оставил бы возможность выставить настройку в empty при установке. Понятно, что обратную совместимость ломать не хочется и жалобы будут, да, но раз системно все переписывается, хотелось бы стройности во всем. Как фича — это да, удобно для новичков на старте, но не универсально.
Развивая мысль, можно было бы вынести оформление магазина в отдельный пакет, а в самом miniShop оставить голое ядро и учитывая, что с 2.4 версии есть возможность указывать зависимости, ставить пакет оформления по умолчанию, но при этом давать возможноть отказаться от него. В таком случае в пакете оформления можно было бы реализовать полценный демо-магазин (аля сборка) для желающих посмотреть, а как оно что работает.
Для конечно пользователя все остается так же — установил одной кнопкой, открыл сайт — о! магазин работает.
Для сайтоделов, вроде меня и других — при установке снял галку и голова не болит, делаешь что нужно и лишнего на сайте ничего нет.
07 апреля 2016, 15:27
+8
Я бы еще убрал такой момент, как вывод товаров категории через поле content по умолчанию. Когда делается сайт, шаблоны все равно переписываются, добавляется постраничная навигация и свои обертки и тд и тп, а в поле контент пишется текстовый блок описания самой категории (для сео-шмео). В итоге, если вовремя не убрать это описание и потом накопировать категорий, начинаются свистопляски на страница, когда вместо поля content выводятся товары в неправильной верстки. Сталкивался не один раз и было неудобно.
Имхо, можно сделать так, как это для товара сделано, где все оформление вынесено в чанк и для прототипирования можно его использовать или писать свой код.
Вот такой вот issue получился. В остальном все круто.
06 апреля 2016, 19:48
+1
Будет выглядеть как троллинг :)

Вообще ребят я понять могу, пробуют новую тему, начинают с введения и собираются смотреть на отклик. Ну у меня такое чувство, что просто придумали давайте делать и не провели никакого анализа. Ибо как заметили, введения хватает, а вот что-то более серьезного, нет. Да и уроков и ответов на вопросы на этом сайте уже пару тысяч. Я в своем плане тоже думал начать с основ, но после анализа отказался.
Так что наберитесь терпения, надеюсь очень скоро будет где учиться глубинам MODX. Я преодолел юридический барьер, разработка сервиса в самом разгаре, контент тоже готовиться )

А ребятам успеха желаю, все же знания — они всегда в цене. Найдутся и те, кому введение будет интересно.
06 апреля 2016, 17:45
0
Такс! Ждем анонс! :) У меня бреветов в это время нет, можно и зарулить. Хотя там Василий примерно в это время в Минск вроде собирался, возможно случится накладка. В любом случае ждем подробностей )
06 апреля 2016, 10:10
0
Без проблем, проверено на 30 тысячах.
03 апреля 2016, 18:51
+1
Вы не правы. Новичкам возможно и все равно (но толку с них мало), а вот профессиональных разработчиков вы от себя оттолкнули основательно. В вашем случае видится классический пример студии, в которой основная идеология — прибыль владельца студии, а как там себя рабы чувствуют в кабинетах, это уже вторично. Поверхностно вопрос может и звучит нелепо, подумаешь, неправильно название написали, но по опыту знаю, что за этим кроется.
03 апреля 2016, 17:52
+1
Стремают меня ребята, которые пишут ModX, да еще и на Evo… Сомнительная работа, имхо. Хоть Evo и развивается, думаю, если позвать толково разработчика, дешевле выйдет переписать все это, чем поддерживать в таком виде. Легаси-код до добра еще ни разу не доводил.
31 марта 2016, 09:46
0
События в MODX срабатывают и обрабатываются исключительно на бэкенде, т.е. на сервере в php коде. Все, что касается админки, это уже клиентский код и там они силы не имеют (только косвенно), поэтому чтобы работало такое обновление полей, нужно подключать к форме свой компонент на ExtJS, где и обновлять страницу или отдельные поля формы.
30 марта 2016, 13:45
+1
2.4.3 несовместима с php7. Баги по совместимости исправлены в 2.5.0-RC1
24 марта 2016, 23:11
+1
у меня такое было на mariadb, но проблема была не в БД непосредственно, а в кривых руках, который настраивали сервер. Если прописать localhost в качестве хоста базы, в том числе и через Gitify — не работает, а если 127.0.0.1 — все отлично.
23 марта 2016, 15:48
+3
А я уже понадеялся почитать про настоящую двухуровневую авторизация с смсками и токенами, а тут просто basic-авторизацию навесили. Эх…
22 марта 2016, 10:35
+2
Ну там и объем работы соответствующий. Или вы клиентам тоже за «Спасибо» сайты делаете? xD
21 марта 2016, 12:29
0
Тем не менее отлично работает :)
21 марта 2016, 10:51
0
Поддержу. Если нужно взломать сайт, то переименование без обеспечения нормального уровня безопасности ни разу не помогут. Это как на дверь повесить табличку с надписью «это окно» и наоборот.
16 марта 2016, 13:37
+1
Количество галочек на скорости практически не отобразится, так как таблица со значениями TV компактная и с проставленными индексами, плюс у вас же будет limit + сортировка по дате. Если сомневайтесь, по получите итоговый SQL, включив showLog=1 и прогоните через explain в phpmyadmin или еще где либо. Т.е. в главные новости будет попадать только N последних новостей, хотя отмечено может быть много.
Если мусор хочется почистить иногда, то можете создать скрипт, который будет выбирать новости старше N дней и убирать с них галку Главная новость и запускать такой скрипт в cron раз в сутки.
Парится по поводу того, что не попадет в тематический виджет, не стоит. Потому что в них тоже будут новости и количество их будет лимитировано. Допустим была главная новость в экономике, она больше не главная, но за это время добавилось 5 новостей в раздел Экономика, а виджет показывает только 5 штук. Следовательно она и так не попадет в виджет. Т.е. временные отрезки и там и там одинаковы, а учитывая, что главные новости добавляются реже, то к моменту потери актуальности они будут далеко в конце ленты. А если какую-то новость нужно таки снять с главной прямо сейчас, то это уже будет в ручном режиме происходить.
Вариант с пересохранением отживших новостей через плагин не самый подходходящий, так как вы не знаете точное количество главных новостей сегодня (их может вообще не быть).
16 марта 2016, 12:52
+2
Как-то переусложнено. Делаете TV «Главная новость» в виде чекбокса, в вызовах где есть виджеты используете TV фильтр tvFilters, которые исключает все заметки, где Главная новость == true.
16 марта 2016, 09:41
0
ms2 позволяет переопределить javascript файл, ответственный за обработку событий на сайте. Можете скопировать js-файл минишопа в свою директорию, в системных настройках указать путь к нужно файлу и там переписать чать, отвечающую за обработку чекбоксов.
13 марта 2016, 22:30
+2
Активно работаю последние 2 дня с modx сайтами, включая fenom (в повседневной жизни пишу на матером PHP немного другие проекты) и вот чего мне не хватает, так это модификаторов в fenom, которые дублируют встроенные MODX фильтры (особенно когда старый код переносишь на fenom). Можно писать сниппеты и тд, конечно, но это накладно и лениво. Если не сильно противорчечит парадигме, плюсую за это улучшение (надеюсь руки дойдут портировать самостоятельно в видео pull request).

В остальном спасибо, работать сильно удобнее.
02 марта 2016, 13:44
+1
Тогда уж лучше использовать класс DateTime, хоть нагляднее будет.
return (new DateTime($input))->format(DateTime::ISO8601);
02 марта 2016, 11:30
0
Можно попробовать обычным модификатором MODX сделать правильную дату.
[[+date:strtotime:date=`%Y-%m-%d\T%H:%M:%S%Z`]]