Багрепорт

Новости

Портфолио MODX-клуба

Эксперименты и исследования

Blog by Fi1osof

Не очищается список адресатов при использовании modUser::sendEmail()

Возобновление общения на сайте MODX-Клуба

О новом логотипе замолвим слово..

Емейл-уведомления о новых комментариях в топиках

Добавлена подсветка кода PrettyPrint

Восстановление пароля

Фабрика сайтов - отзывы :)

По поводу производительности сайта MODX-Клуба

Создание топиков на сайте Клуба

modxSmarty - отключение modx-парсера и существенное ускорение

Интересный эксперимент :) 
 Но все же есть пара моментов. И вот основной:
 

$modx->smarty->display(«tpl/{$tpl}»);
 return ""; // отпрвить $ничего на вывод через modx-парсер. :)

Таким образом рушится механизм кеширования MODX, так как он не получает отрабатываемого контента. Ежели вы хотите именно отключить MODX-парсер, то есть более лаконичное решение — системная настройка parser_class. Она учитывается в методе MODx::getParser()
 

public function getParser() {
    return $this->getService('parser', $this->getOption('parser_class', null, 'modParser'), 
        $this->getOption('parser_class_path', null, ''));
}

То есть пишете парсер на замену modParser и все. А в нем, где надо, облегчаете методы по максимум, чтобы просто возвращали по минимуму и все.
 

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

Добрый день! Интересный эксперимент :) 
 Но все же есть пара моментов. И вот основной: $modx->smarty->display(«tpl/{$tpl}»);
 return ""; // отпрвить $ничего на вывод через modx-парсер. :) Таким образом рушится механизм кеширования MODX, так как он не получает отрабатываемого контента. Ежели вы хотите именно отключить MODX-парсер, то есть более лаконичное решение — системная настройка parser_class. Она учитывается в методе MODx::getParser() public function getParser() {
    return $this->getService('parser', $this->getOption('parser_class', null, 'modParser'), 
        $this->getOption('parser_class_path', null, ''));
} То есть пишете парсер на замену modParser и все. А в нем, где надо, облегчаете методы по максимум, чтобы просто возвращали по минимуму и все. Только вешайте его именно на внешний контекст, то есть в настройки контекста прописывайте, а не в системные, чтобы в mgr работал родной парсер, а во фронте ваш. Потому что системный парсер используется в том числе и для обработки системных настроек и т.п.

Добрый день! Интересный эксперимент :) 
 Но все же

В общем, я решил тоже поучаствовать в этом эксперименте, и написать свой парсер на замену. Облегчить его на 100% не получается в любом случае. Во-первых, MODX довольно активно использует его отдельные методы, так что как минимум пустые методы надо прописать. Во-вторых, в тех же шаблонах у нас в параметрах есть необходимые параметры, типа tpl или phptemplates.non-cached, и чтобы они учитывались в системе, пришлось один метод более живым оставить. Вот такой класс у меня получился:
 

<?php


class modWebParser {
    /**
     * A reference to the modX instance
     * @var modX $modx
     */
     
    public $modx= null; 
    
    /**
     * @param xPDO $modx A reference to the modX|xPDO instance
     */
    function __construct(xPDO &$modx) {
        $this->modx =& $modx;
    }
 
    public function setProcessingElement($arg = null) {
    } 
    
    public function processElementTags($parentTag, & $content, 
        $processUncacheable= false, $removeUnprocessed= false, $prefix= "[[", 
        $suffix= "]]", $tokens= array (), $depth= 0) {
        return $processed;
    } 
    
    public function collectElementTags($origContent, array &$matches, $prefix= '[[', $suffix= ']]') {
        return 0;
    }

    /**
     * Parses an element/tag property string or array definition.
     *
     * @param string $propSource A valid property string or array source to
     * parse.
     * @return array An associative array of property values parsed from
     * the property string or array definition.
     */
    public function parseProperties($propSource) {
        
        $properties= array ();
        
        if (!empty ($propSource)) {
            if (is_string($propSource)) {
                $properties = $this->parsePropertyString($propSource, true);
            } elseif (is_array($propSource)) {
                foreach ($propSource as $propName => $property) {
                    if (is_array($property) && array_key_exists('value', $property)) {
                        $properties[$propName]= $property['value'];
                    } else {
                        $properties[$propName]= $property;
                    }
                }
            }
        }
        return $properties;
    }
    
    public function isProcessingUncacheable() {
        $result = false;
        return $result;
    }
    
    public function isRemovingUnprocessed() {
        $result = false;
        return $result;
    }

    public function parsePropertyString($string, $valuesOnly = false) {
        $properties = array();
        return $properties;
    }
}

Закинул его в core/components/modxsite/model/modxsite/model/modwebparser.class.php
 

В настройках контекста web прописал:
 parser_class = modWebParser
 parser_class_path = {core_path}components/modxsite/model/modxsite/
 

В целом все работает, и API MODX-а получается использовать, к примеру {$modx->getOption('site_url')}
 

Попробуйте установить ShopModxBox и погонять с этим парсером и без него. Но здесь надо учитывать, что многое не будет работать в принципе. Лично мне кажется, что овчинка не будет стоить выделки. К примеру, мы не сможем использовать некешируемые блоки через [[!smarty?tpl=`....`]] и т.п. Конечно же мы можем просто отрабатывать шаблоны каждый раз, включив кеширование самого Smarty, но это гораздо более заморочено.

В общем, я решил тоже поучаствовать в этом эксперименте, и написать свой парсер на замену. Облегчить его на 100% не получается в любом случае. Во-первых, MODX довольно активно использует его отдельные методы, так что как минимум пустые методы надо прописать. Во-вторых, в тех же шаблонах у нас в параметрах есть необходимые параметры, типа tpl или phptemplates.non-cached, и чтобы они учитывались в системе, пришлось один метод более живым оставить. Вот такой класс у меня получился: <?php


class modWebParser {
    /**
     * A reference to the modX instance
     * @var modX $modx
     */
     
    public $modx= null; 
    
    /**
     * @param xPDO $modx A reference to the modX|xPDO instance
     */
    function __construct(xPDO &$modx) {
        $this->modx =& $modx;
    }
 
    public function setProcessingElement($arg = null) {
    } 
    
    public function processElementTags($parentTag, & $content, 
        $processUncacheable= false, $removeUnprocessed= false, $prefix= "[[", 
        $suffix= "]]", $tokens= array (), $depth= 0) {
        return $processed;
    } 
    
    public function collectElementTags($origContent, array &$matches, $prefix= '[[', $suffix= ']]') {
        return 0;
    }

    /**
     * Parses an element/tag property string or array definition.
     *
     * @param string $propSource A valid property string or array source to
     * parse.
     * @return array An associative array of property values parsed from
     * the property string or array definition.
     */
    public function parseProperties($propSource) {
        
        $properties= array ();
        
        if (!empty ($propSource)) {
            if (is_string($propSource)) {
                $properties = $this->parsePropertyString($propSource, true);
            } elseif (is_array($propSource)) {
                foreach ($propSource as $propName => $property) {
                    if (is_array($property) && array_key_exists('value', $property)) {
                        $properties[$propName]= $property['value'];
                    } else {
                        $properties[$propName]= $property;
                    }
                }
            }
        }
        return $properties;
    }
    
    public function isProcessingUncacheable() {
        $result = false;
        return $result;
    }
    
    public function isRemovingUnprocessed() {
        $result = false;
        return $result;
    }

    public function parsePropertyString($string, $valuesOnly = false) {
        $properties = array();
        return $properties;
    }
} Закинул его в core/components/modxsite/model/modxsite/model/modwebparser.class.php В настройках контекста web прописал:
 parser_class = modWebParser
 parser_class_path = {core_path}components/modxsite/model/modxsite/ В целом все работает, и API MODX-а получается использовать, к примеру {$modx->getOption('site_url')} Попробуйте установить ShopModxBox и погонять с этим парсером и без него. Но здесь надо учитывать, что многое не будет работать в принципе. Лично мне кажется, что овчинка не будет стоить выделки. К примеру, мы не сможем использовать некешируемые блоки через [[!smarty?tpl=`....`]] и т.п. Конечно же мы можем просто отрабатывать шаблоны каждый раз, включив кеширование самого Smarty, но это гораздо более заморочено.

В общем, я решил тоже поучаствовать в этом экспери

Отлично! Радует что есть реальный прогресс в деле Сообщества MODX клуба! Спасибо за сообщение данное, будем вновь чаще сюда заглядывать/читать/рассуждать/делать !!!)))

Отлично! Радует что есть реальный прогресс в деле 

Бывает :)

Но почему-то список адресатов не очищался... И с каждым новым письмом список получателей увеличивался. В итоге самые давние пользователи сайта получили наибольшее число писем. Еще раз извините! Я не специально.

Собственно ничего страшного!)) Конечно был некоторый шок/недоумение что это вдруг такой поток одного и того же письма))) :) С другой стороны даже настроение стало лучше, и всё будет хорошо!))) Мы всё понимаем! :)

Но почему-то список адресатов не очищался... И с каждым новым письмом список получателей увеличивался. В итоге самые давние пользователи сайта получили наибольшее число писем. Еще раз извините! Я не специально. Собственно ничего страшного!)) Конечно был некоторый шок/недоумение что это вдруг такой поток одного и того же письма))) :) С другой стороны даже настроение стало лучше, и всё будет хорошо!))) Мы всё понимаем! :)

Но почему-то список адресатов не очищался... И с к

Моя сделать тестовый коммент, чтобы видеть, что все работает и радоваться ентому факту. 
Когда написал что-нибудь и проголосовал - текст пропадает. Ну, это так, на всякий. Потом сделаете аджаксом.


Моя сделать тестовый коммент, чтобы видеть, что вс

Рад что уведомили на почту, но зачем же это делать 40(!) раз?

Рад что уведомили на почту, но зачем же это делать

И при отправке комментариев как видите есть глюки...

И при отправке комментариев как видите есть глюки.

Не надо так делать - https://yadi.sk/i/LVOdadZxaABqQ у меня по умолчанию алерты режутся, поэтому все дальнейшее выполнение кода прерывается, и отредактировать комменты нельзя, так что извините за множество разных.

Не надо так делать - https://yadi.sk/i/LVOdadZxaAB

Привет тестирую вас :)

Алишер, привет! :)

Сергей, привет! Спасибо за понимание! :)

Сегодня прикручу уведомления о новых комментариях, чтобы общение было еще более оперативным. Повеселее будет :)

Сегодня прикручу уведомления о новых комментариях,

Написал и пропадает текст - это комментарий написал? Можно чуть подробней о том, что делал и что произошло? Так как сейчас комментарии аджаксовые и худо-бедно с сервера приходит HTML с опубликованным комментом и он сразу подставляется в документ (правда без кнопки Ответить, но она и не нужна автору комментария).

Написал и пропадает текст - это комментарий написа

Извини за спам :( 
Отписался по этому поводу здесь.

Извини за спам :( 
Отписался по этому поводу здесь

Редактировать не работает но в целом уже неплохо!

Спасибо за фидбэк! Да, скоро уведомления будут переделаны на нормальные. Просто я по фронту практически не работаю вообще, и вставил пока обычные алерты, а Сергей уехал отдыхать :) Вот вернется и приведет фронт в порядок.

Спасибо за фидбэк! Да, скоро уведомления будут пер

Привет :)

Письмо об активации в спам пришло

Странно... Видимо гугль обиделся на нас сегодня за утрешнюю выходку :) А так вроде должно быть все ОК. И домен не новый, и rDNS прописан. Странно.

Странно... Видимо гугль обиделся на нас сегодня за

Написал комментарий -> не жмякая "опубликовать комментарий", нажал лайк темы (палец вверх который, там где  Fi1osof 
 2014-08-21 
 14 
   4 ) - текст пропал.

Написал комментарий -> не жмякая "опубликовать ком

Спасибо!
Но вопрос что там за шрифт? Надо же дописать CLUB, а это надо как минимум подобрать похожий шрифт. И я совсем не дезигнер... Фавиконку-то я сейчас вставлю, а лого допиливать надо.

Спасибо!
Но вопрос что там за шрифт? Надо же допис

Быстрофикс. Все-таки редактирование нужно. Да и предпросмотр не помешает 
Написал комментарий -> не жмякая "опубликовать комментарий", нажал лайк темы (палец вверх который, там где Fi1osof 2014-08-21 14 4 ) - текст пропал.

Быстрофикс. Все-таки редактирование нужно. Да и пр

Ну да страанно Subjekt прописан откуда тоже написано всё гуд. Я от логина сколько делал тест регистрации ниразу в спам не залетало

Ну да страанно Subjekt прописан откуда тоже написа

Все когда-нибудь в первый раз. Вполне возможно из-за сегодняшней массовой спам-рассылки гугл начал нас немного фильтровать. Со временем все восстановится.

Все когда-нибудь в первый раз. Вполне возможно из-

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

Ааа, понял. Да, после голосования страница обновля

Все будет. И редактирование, и удаление, и предпросмотр. Но чуть позже. Качественный фронт - это не я, а Сергей Прохоров. Вот вернется на следующей неделе и займется.
А сегодня я сделаю емейл-уведомления различные :)

Все будет. И редактирование, и удаление, и предпро

А сегодня я сделаю емейл-уведомления различные :) 

А то ни уведомления о новых комментах/ответах, ни о новых топиках, ничего. Надо поправлять.

А сегодня я сделаю емейл-уведомления различные :) А то ни уведомления о новых комментах/ответах, ни о новых топиках, ничего. Надо поправлять.

Согласен ато много время так теряешь проверять всё

Да нет, если зашел, то не долго - справа есть лента, да и страница всех комментариев есть. http://modxclub.ru/comments/
Но каждый раз заходить чтобы просто проверить писал кто-то что-то или нет - лениво.

Да нет, если зашел, то не долго - справа есть лент

Там же psd всё просто

и шрифты приложены в архиве

Шрифты не заметил)) Спасибо! Да, теперь можно лого запилить. Шрифты установил и все ОК стало. Спасибо!

Шрифты не заметил)) Спасибо! Да, теперь можно лого

Сделано :)
Спасибо! Так гораздо лучше))

с бквами меню контраст мне кажется а так лого крутяк)

с бквами меню контраст мне кажется а так лого крут

тот же шрифт на меню можно кинуть

ОК. Ваш комментарий обязательно будет учтен :) Но повторюсь: это чуть позже. Я фронтом не занимаюсь, только по мелочи.

ОК. Ваш комментарий обязательно будет учтен :) Но 

Перевел кеширование на memcached. Прирост производительности еще процентов 20. Среднее время рендеринга страницы - 0.3-0.5 сек. Вообще уже не плохо :)

Перевел кеширование на memcached. Прирост производ

Вот страница, на которой больше всего комментариев (127) http://modxclub.ru/blog/dokumentatsiya-dlya-spetsialistov/147.html
 Рендеринг 1.6 на холодную, и потом 0.6-1.1 сек.

Вот страница, на которой больше всего комментариев

О красиво! Да еще на менюху бы буквы шрифта подтянуть было бы здорово, только кажись в шрифте том нет кириллической поддержки..
 А так да стало веселей!

О красиво! Да еще на менюху бы буквы шрифта подтян

да еще забыл, в принципе для сочетаемости цветовых решений, текущую подсветку меню при наведении и также всё что выделено цветом так называемым зеленым из ui флэт дизайна (заголовки и прочие элементы), можно попробовать перекрасить в цвет тот же зеленый но только тот оттенок зеленого как на самой темной части зеленых элементов логотипа. 


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


Ну это так на досуге конечно, просто если уж до оформлять то по полной эксперимент проводить, пробовать. ))

да еще забыл, в принципе для сочетаемости цветовых решений, текущую подсветку меню при наведении и также всё что выделено цветом так называемым зеленым из ui флэт дизайна (заголовки и прочие элементы), можно попробовать перекрасить в цвет тот же зеленый но только тот оттенок зеленого как на самой темной части зеленых элементов логотипа. Ну т.е. попробовать уйти от расцветки ui флэт дизайна, к скажем так более индивидуальной и сочетаемой с корпоративным цветом логотипа)) Ну это так на досуге конечно, просто если уж до оформлять то по полной эксперимент проводить, пробовать. ))

да еще забыл, в принципе для сочетаемости цветовых

а про красный и синий забыли про цвет? :( тоже ведь та же подсветка значков к примеру ок и против голоса за сообщение или тот жезначок комментариев. Просто сливается все в едином "зеленом" порыве, как-то выделять было бы просто симпатичнее. ))))

а про красный и синий забыли про цвет? :( тоже вед

Сергей, пока у нас нет здесь дизайна вообще. Все оформление - чисто техническое. Толкьо когда будет докручен минимальный желаемый функционал и когда все будет отлажено, тогда только будет прорабатываться индивидуальный дизайн. Главное здесь -техническая реализация и информация.

Сергей, пока у нас нет здесь дизайна вообще. Все о

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

Да это понятно конечно)) я в русле такого минималь

Николай! кстати не работает "Редактировать" только "Ответить" норм, хочу добавить ничего не происходит, ну это так мелкий баг..


В тему редизайна, концепций, и прочих измышлений (всё-таки тема по-любому требует пристального взгляда так или иначе), вот из свежего про редизайн вКонтакте на хабре статья небольшая

Николай! кстати не работает "Редактировать" только "Ответить" норм, хочу добавить ничего не происходит, ну это так мелкий баг.. В тему редизайна, концепций, и прочих измышлений (всё-таки тема по-любому требует пристального взгляда так или иначе), вот из свежего про редизайн вКонтакте на хабре статья небольшая

Николай! кстати не работает "Редактировать" только

Блин, я думал у меня только кнопки Редактировать :) (на самом деле у меня еще и Скрыть и Удалить, но они тоже пока не работают). Забыл что авторам комментов эта кнопка тоже выводится. Скрыл пока. Появится позже, когда заработает.

Блин, я думал у меня только кнопки Редактировать :

Уже отчаялся повторить эту ошибку. Гнал по несколько емейлов на свои ящики, и все шло ОК, то есть без кучи копий в адресатах. Вроде тот же самый скрипт, а ошибки нет... И тут меня осенило! Я все письма гнал на разные ящики, но одного домена. А что если на разные? Добавил в список получателей емейл с другим доменом. И вот тут ошибка и проявилась)))
 Чуть позже более предметно ее изучу, разберусь в чем проблема и отпишусь.

Уже отчаялся повторить эту ошибку. Гнал по несколь

Все, разобрался. Оказалась целая история :) Это из разряда сложных логических ошибок.


Для начала кратко напомню задачу и суть проблемы: я решил всем пользователям разослать кратенькую новость. Для этого я сделал выборку всех активных пользователей и в цикле запустил отправку писем каждому через метод $user->sendEmail() (скрипт смотрите выше). Проблемы выразилась в том, что каждому новому письму помимо самого адресата добавлялся список всех предыдущих адресатов, то есть с каждым новым письмом количество адресатов увеличивалось на одного и некоторые пользователи получили десяток и больше писем. Благо я быстро заметил и понял проблему и рестартанул веб-сервер и сендмейл.


Сейчас вот полтора часа разбирался в сути проблемы и разобрался. Оказывается, все дело в моем личном емейле. Ну и плюс логическая ошибка в методе modUser::sendMail(). Ошибка заключалась в том, что если отправка письма выполняется с ошибкой, то метод возвращает false, не доходя до очистки списка получателей. Вот эти строчки:
 if ($this->xpdo->mail->send() == false) {
     return false;
 }
 $this->xpdo->mail->reset();
 return true;
 Как видите, в случае ошибки $this->xpdo->mail->reset() просто не может выполниться. А раз список не очищается, в нем присутствует и мой злополучный емейл. А это значит, что при следующей попытке отправить письмо (точнее письма) phpMailer опять вернет ошибку (хотя всем, кроме меня он письма отправит успешно), ну и так далее в цикле.
 Конечно же это бага, и пуллреквест я отправил https://github.com/modxcms/revolution/pull/11931/files


Так а в чем косяк моего емейла? В чем он виноват?
 А вот это вообще отдельная история :) Дело в том, что домен modxclub.ru привязан к серверу, на котором и размещается сайт modxclub.ru (что вполне логично). Так вот, при попытке отправки письма сервер определял, что он сам и является машиной-получателем, а не на гугл надо слать, где на самом деле почта и располагается. И он такой шлет запрос-отправку на локалхост, но шлет он для info@. А инфо (как и любой другой ящик) - это пользователь. И он такой спрашивает "а есть такой пользователь info?", на что система отвечает "нет, такого пользователя нет", и веб-сервер такой "а, ну тогда я не могу письмо доставить", и phpMailer возвращает false. Ну а все остальное я описал выше :)


Вот такая веселая бага)))

Все, разобрался. Оказалась целая история :) Это из разряда сложных логических ошибок. Для начала кратко напомню задачу и суть проблемы: я решил всем пользователям разослать кратенькую новость. Для этого я сделал выборку всех активных пользователей и в цикле запустил отправку писем каждому через метод $user->sendEmail() (скрипт смотрите выше). Проблемы выразилась в том, что каждому новому письму помимо самого адресата добавлялся список всех предыдущих адресатов, то есть с каждым новым письмом количество адресатов увеличивалось на одного и некоторые пользователи получили десяток и больше писем. Благо я быстро заметил и понял проблему и рестартанул веб-сервер и сендмейл. Сейчас вот полтора часа разбирался в сути проблемы и разобрался. Оказывается, все дело в моем личном емейле. Ну и плюс логическая ошибка в методе modUser::sendMail(). Ошибка заключалась в том, что если отправка письма выполняется с ошибкой, то метод возвращает false, не доходя до очистки списка получателей. Вот эти строчки:
 if ($this->xpdo->mail->send() == false) {
     return false;
 }
 $this->xpdo->mail->reset();
 return true;
 Как видите, в случае ошибки $this->xpdo->mail->reset() просто не может выполниться. А раз список не очищается, в нем присутствует и мой злополучный емейл. А это значит, что при следующей попытке отправить письмо (точнее письма) phpMailer опять вернет ошибку (хотя всем, кроме меня он письма отправит успешно), ну и так далее в цикле.
 Конечно же это бага, и пуллреквест я отправил https://github.com/modxcms/revolution/pull/11931/files Так а в чем косяк моего емейла? В чем он виноват?
 А вот это вообще отдельная история :) Дело в том, что домен modxclub.ru привязан к серверу, на котором и размещается сайт modxclub.ru (что вполне логично). Так вот, при попытке отправки письма сервер определял, что он сам и является машиной-получателем, а не на гугл надо слать, где на самом деле почта и располагается. И он такой шлет запрос-отправку на локалхост, но шлет он для info@. А инфо (как и любой другой ящик) - это пользователь. И он такой спрашивает "а есть такой пользователь info?", на что система отвечает "нет, такого пользователя нет", и веб-сервер такой "а, ну тогда я не могу письмо доставить", и phpMailer возвращает false. Ну а все остальное я описал выше :) Вот такая веселая бага)))

Все, разобрался. Оказалась целая история :) Это из

Мне кажется, что последние комментарии в правом верхнем блоке было бы лучше выводить с группировкой по публикации, как в ливстрит. Ну и ссылкой на комментарий сделать текст (как в блоке последних публикаций), а не решетку (:

Мне кажется, что последние комментарии в правом ве

Да, подкинули мне задачку на часик... Проблема MySQL в том, что нельзя выполнить сначала сортировку, а потом группировку. Можно только выполнить группировку, а в ней сортировка уже на воле Ктулху. В итоге приходится лепить подзапрос, в котором сортируешь или выборку максимальных делаешь. А такая задача в одном объекте xPDOQuery нереальна.
 Вот подзапросик добавил в процессор:
 

       // Получаем последние комментарии из этих топиков

        if($this->getProperty('one_comment_per_thread')){ 

            $ids = array();

            

            $sub_query = $this->modx->newQuery($this->classKey); 

            $sub_query->select(array(

                "max(id) as id",

                "thread_id",

            ));

            $sub_query->groupby('thread_id');

            

            $s = $sub_query->prepare();

            $s->execute();

            

            while($row = $s->fetch(PDO::FETCH_ASSOC)){

                $ids[] = $row['id'];

            }

            

            if($ids){

                $query->where(array(

                    "id:in" => $ids,

                ));

            }

        }

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


Да, подкинули мне задачку на часик... Проблема MySQL в том, что нельзя выполнить сначала сортировку, а потом группировку. Можно только выполнить группировку, а в ней сортировка уже на воле Ктулху. В итоге приходится лепить подзапрос, в котором сортируешь или выборку максимальных делаешь. А такая задача в одном объекте xPDOQuery нереальна.
 Вот подзапросик добавил в процессор: // Получаем последние комментарии из этих топиков

        if($this->getProperty('one_comment_per_thread')){ 

            $ids = array();

            

            $sub_query = $this->modx->newQuery($this->classKey); 

            $sub_query->select(array(

                "max(id) as id",

                "thread_id",

            ));

            $sub_query->groupby('thread_id');

            

            $s = $sub_query->prepare();

            $s->execute();

            

            while($row = $s->fetch(PDO::FETCH_ASSOC)){

                $ids[] = $row['id'];

            }

            

            if($ids){

                $query->where(array(

                    "id:in" => $ids,

                ));

            }

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

Да, подкинули мне задачку на часик... Проблема MyS

Для полноты решения нужно еще добавить название топика, но это в общем-то мелочь.
 Интересно другое - если усложнить задачу, например, не 1 последний комментарий на публикацию, а 5. Тогда только xPDO->exec()?

Для полноты решения нужно еще добавить название то

С xPDO->exec() я ошибся, для выборки он не подойдет.

С xPDO->exec() я ошибся, для выборки он не подойде

Не важно xPDO:exec() или что другое. Главное - это SQL правильный. Здесь сам принцип усложняется. Придется делать перебор всех последних веток и в цикле выдергивать по максимум 5 комментариев. Овчинка вообще не стоит выделки.

Не важно xPDO:exec() или что другое. Главное - это

Все, пуллреквест приняли, бага поправлена :)

Мертвая компания, которая умеет только оперативно выставить счет и все! Первый сайт сроки были нарушены, переделка первого на новый — уже 2! недели меня кормят завтраками и до сих пор ничего! Просят оставить контактный телефон, почту — и как в бездонную бочку — ни звонка ни ответа ни привета! Чтобы добиться ответа на вопрос — надо оооочень позаморачиваться в обращениях. Короче, денег заплатила за 2 сайта — в итоге без сайта вообще! Если не хотите нервничать — заплатите лучше чуть больше, но в нормальной компании! Всем удачи!

Мертвая компания, которая умеет только оперативно 

А я сейчас с ними сужусь. Они подали на меня в суд :)
 Но вот уже ждем решения суда и очень даже рассчитываем на положительное решение. Как результаты будут, я дополнительно еще отпишусь. Там есть еще масса интересных моментов с ними.

А я сейчас с ними сужусь. Они подали на меня в суд

Коля а есть где почитать про то как ты прикрутил подсветку и как у тебя устроено это в бекенде
 Спасибо

Коля а есть где почитать про то как ты прикрутил п

Загуглите, это же не наша библиотека. А вообще просто добавляются на страницу скрипты.
 

И оборачивается код в блок <pre class=«prettyprint»>

Загуглите, это же не наша библиотека. А вообще просто добавляются на страницу скрипты. <script src="/assets/components/modxsite/templates/v22/libs/prettify/prettify.js" type="text/javascript"></script>
        <link href="/assets/components/modxsite/templates/v22/libs/prettify/prettify.css" rel="stylesheet">
        <script type="text/javascript">
            $(function(){
                prettyPrint();    
            });
        </script> И оборачивается код в блок <pre class=«prettyprint»>

Загуглите, это же не наша библиотека. А вообще про

Спасибо
 А от парсера ты как экранируешь
 

Спасибо
 А от парсера ты как экранируешь [[*content]]

Спасибо
 А от парсера ты как экранируешь [[*conten

К претти это уже никакого отношения не имеет. Просто при сохранении надо скобки [ заменять на &#91;. Но для этого еще надо разрешить передачу MODX-тегов в POST. Подробное руководство не готов сейчас расписать, слишком много тонкостей там. Придется вам самостоятельно копать.

К претти это уже никакого отношения не имеет. Прос

Спасибо, пожалуйста только на ты )

Не за что.
 ОК, постараюсь.

«Повезло» поработать тут 3 месяца примерно. Уволился в первых числах июня. Долг по зарплате остался за май, собственно первые дни июня и даже немного за апрель. Сейчас 1 августа. То есть прошло грубо говоря 2 месяца. Долг не выплачен. Т.к. выплаты процентов на 90 неофициальные, дожидаюсь их под честное слово. Как-то надежды дождаться уже начинают реально таять.

«Повезло» поработать тут 3 месяца примерно. Уволил

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

) Судя по отзывам, ребята которые пишут сайты для 

ну а вообще сама идея конечно фикс ) взять бесплатную Cms, перекрасить и продавать…

ну а вообще сама идея конечно фикс ) взять бесплат

Кстати, мне тоже нравится эта идея. Почему блин нет??? Гениально же! Зачем изобретать велосипед? Когда можно взять проект и подстроить под себя. Многие так делают. И я считаю, что не надо в мире, где нужно все делать быстрее заморачиваться над тем, чтобы изобрести что-то эдакое, когда оно уже есть. Они же не выступили на рынке с этим продуктом, заявив, что они первые в этой области. Я с ними работала давно правда и только по seo. Ну неплохо. Может у меня сайт был не такой и конкурентный, но никто меня не бросил, всегда отвечали на письма, созвоны были, отчеты регулярненько. Сейчас не знаю, я год назад заказывала на несколько месяцев сразу.

Кстати, мне тоже нравится эта идея. Почему блин не

Почему нет? Хотя бы просто потому что существует понятие «Лицензий». Лицензия MODX подразумевает, что в админке не должны быть затерты их копирайты. Делая проект для конечного клиента компания должна говорить «Это делается на MODX», а не «это делается на FCMS». Вы разницу улавливаете? С вашим этим подходом, вас не смущает тот факт, что «бизнесменов», которые на своих пиратских ресурсах крутят чужие фильмы тянут в суды и т.п.? Вопрос: за что? Генеально же! Зачем изобретать велосипед (писать сценарии, нанимать актеров, снимать все это и т.п.), когда можно просто взять чужой фильм и крутить его?

Почему нет? Хотя бы просто потому что существует п

Эта фабрика сайтов шарага какая то! все что написано, подписываюсь под каждым словом! ничерта не доделано, постоянно отмораживаются, счета выставляют раньше чем успеваешь сделать запрос! с сегодняшнего дня вообще трубку не берут!!! бесконечный поток менеджеров которые ведут проект. компанию НЕ рекомендую!!!

Эта фабрика сайтов шарага какая то! все что написа

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

Александр, вы через форму обратной связи в контакт

Сорри, это вот они делают вам так сайт? joxi.ru/EA4NeMZfDPDbbm

Сорри, это вот они делают вам так сайт? joxi.ru/EA

Заказ.7 картинок,3 таблицы,2 тел,одна почта,5 строк -делают два дня .Домен и хостинг мой (нарушение сроков) Большой проект делать не рекомендую ЦЕНА 2999 проекта (на большее не тянутъъъъъъъъъъъъъъъ)

Заказ.7 картинок,3 таблицы,2 тел,одна почта,5 стро

 Заказывали создание сайта у fabricasaitov.ru (они же фабрика-сайтов.рф). Не буду пока писать про все проблемы, которые возникли с fabricasaitov.ru, а для начала опишу последнюю проблему.
В октябре на нашем сайте перестала функционировать Корзина (из-за того, что у fabricasaitov.ru что то там поломалось…). Сначала fabricasaitov.ru обещала починить сайт в течении недели. По прошествии недели ничего не было починено. Естественно началась кормёжка «завтраками», которая закончилась предложением заплатить деньги за ремонт, который они обещали сделать в течении 4-х дней после оплаты.
25 октября мы оплатили 43500р.
Сегодня 22 ноября и мы до сих пор не можем нормально принимать заказы через корзину (хотя прошел уже почти месяц, вместо обещанных 4-х дней). Т.е. задержка по сроку – в шесть раз больше обещанного!!!
Никаких рекомендаций мы не даём – каждый может сделать выводы сам. 

Заказывали создание сайта у fabricasaitov.ru (они 

43 500 за починку? Или все-таки 4 500? За 43,5 много кто средний магазин с нуля сделает.

43 500 за починку? Или все-таки 4 500? За 43,5 мно

43 500 р.  Мы знаем, что за 43 500 р. можно сделать нормальный магазин (у нас несколько сайтов на разных CMS). Просто он уже есть, а лучшее - враг хорошего. Если бы это был основной сайт - давно бы от них ушли. А так как ими созданный сайт - не основной - пока не уходим. Хотя, чувствую, скоро терпение лопнет... 

43 500 р.  Мы знаем, что за 43 500 р. можно сделат

Жесть... Сегодня на почту еще один человек написал, тоже по этой компании. Сайт-визитку ему сделать не могут.

Жесть... Сегодня на почту еще один человек написал

Всем здравствуйте! Рекомендую бесплатный сайт юридической помощи 9111.ru Опытные специалисты решат все ваши вопросы и это абсолютно бесплатно. Можно позвонить по горячей линии или написать на сайте о вашей проблеме. Желаю всем успехов!

Комментарий к топику Фабрика сайтов - отзывы :)

Терпение лопнуло! Таких мудаков как  fabricasaitov.ru (они же фабрика-сайтов.рф) я ещё не встречал. Как с вами связаться, чтобы вы перенесли сайт на modx ?

Терпение лопнуло! Таких мудаков как  fabricasaitov

Владимир, можете мне на почту написать. n.lanets@modxclub.ru

Владимир, можете мне на почту написать. n.lanets@m

Блин, фабрика еще одну засаду на сайте оставляет... Во всех формах заказов и обратной связи прописывает обратный емейл &emailFrom=`webmaster@[[domain]]`

В итоге настраиваешь на сайте правильную авторизованную отправку с авторизацией через сторонний smtp с указанием логина и пароля, например, noreplay@ или info@, а сайт со всех сторон пытается слать письма с webmaster@. И получается задача отдельная ходить по сайту и выискивать эти емейлы, или заводить почту для отправки webmaster@. Но webmaster@ выглядит еще менее презентабельно, чем noreplay@. Вопрос: почему не использовать системную настройку [[++emailsender]] или хотя бы не завести специальную системную настройку под это, чтобы можно было в одном месте подправить и все? Все не как у людей делается.

Блин, фабрика еще одну засаду на сайте оставляет... Во всех формах заказов и обратной связи прописывает обратный емейл &emailFrom=`webmaster@[[domain]]` В итоге настраиваешь на сайте правильную авторизованную отправку с авторизацией через сторонний smtp с указанием логина и пароля, например, noreplay@ или info@, а сайт со всех сторон пытается слать письма с webmaster@. И получается задача отдельная ходить по сайту и выискивать эти емейлы, или заводить почту для отправки webmaster@. Но webmaster@ выглядит еще менее презентабельно, чем noreplay@. Вопрос: почему не использовать системную настройку [[++emailsender]] или хотя бы не завести специальную системную настройку под это, чтобы можно было в одном месте подправить и все? Все не как у людей делается.

Блин, фабрика еще одну засаду на сайте оставляет..

Зря мы гнали на фабрику. Там, судя по всему, очень трудолюбивые программисты работают, не ленятся вот такие галереи "программировать"))))

<div class="goods-photo">
   [[*image:is=`userfiles/images/no-photo.png`:or:is=``:then=`
        <img src="[[phpthumbof? &input=`userfiles/images/no-photo.png` &options=`w=250&h=250&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]">
    `:else=`
        <a href="[[phpthumbof? &input=`[[*image]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]">
            <img src="[[phpthumbof? &input=`[[*image]]` &options=`w=250&h=250&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]">
        </a>
    `]]
	[[*image2:notempty=`<a href="[[phpthumbof? &input=`[[*image2]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image2]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image3:notempty=`<a href="[[phpthumbof? &input=`[[*image3]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image3]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image4:notempty=`<a href="[[phpthumbof? &input=`[[*image4]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image4]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image5:notempty=`<a href="[[phpthumbof? &input=`[[*image5]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image5]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image6:notempty=`<a href="[[phpthumbof? &input=`[[*image6]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image6]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image7:notempty=`<a href="[[phpthumbof? &input=`[[*image7]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image7]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image8:notempty=`<a href="[[phpthumbof? &input=`[[*image8]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image8]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image9:notempty=`<a href="[[phpthumbof? &input=`[[*image9]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image9]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image10]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image10]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image11]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image11]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image12]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image12]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image13]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image13]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image14]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image14]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image15]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image15]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image16]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image16]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image17]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image17]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image18]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image18]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image19]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image19]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
    [[*image10:notempty=`<a href="[[phpthumbof? &input=`[[*image20]]` &options=`&fltr=wmi|/assets/templates/default/images/water-big2.png|C|40|||`]]" class="zoom-gal" title="[[*pagetitle]]"><img src="[[phpthumbof? &input=`[[*image20]]` &options=`w=95&h=95&fltr[]=size`]]" alt="[[*pagetitle]]" title="[[*pagetitle]]"></a>`]]
</div>