cotonti.com : Я возмущен https://www.cotonti.com Останні повідомлення в темі Cotonti en Tue, 24 Feb 2026 00:14:42 -0000 esclkm Сб, 28 Бер 2009 16:22:43 -0000 Trustmaster
С одной стороны, oc поступил очень некрасиво и спонтанно, откатив изменения без каких-либо предварительных обсуждений, за что я сделаю ему выговор и назначу исправительные работы.

С другой стороны, в одном он всё-таки прав: надо учиться использовать jQuery (особенно селекторы и события), иначе, как сейчас модно выражаться, нас ждёт большой рефакторинг. Поэтому поступим так: во избежание переписывания кода (а переписывать и так придется довольно много) мы пока внедрение AJAX в админке заморозим, и раз уж аякс для удобства полезен, то будем делать его правильно, чтобы потом не пришлось опять всё переписывать. Уточяю, что надо исправить:
[list=1]
  • Убрать аргументы $onclick и $object из sed_pagination() и sed_pagination_pn(). Да, именно убрать. Обработчик нужно составлять и вешать на это событие при помощи jquery при загрузке документа ($(document).ready), выбрав ссылки паджинации селектором. Возможно, придется при выводе паджинации снабдить её некоторыми аргументами или переменными, содержащими параметры паджинации, но в обработчиках их быть не должно.
  • Далеко не всегда требуется id, чтобы получить искомый элемент. Если структура позволяет это сделать однозначно селектором, то стоит обойтись без id.
  • Второй момент гораздо более болезненный и заключается в "неправильном использовании" самого аякса. Правило звучит так: обновляйте только те данные, которые изменились. В идеале клиент должен чётко понимать, что он меняет в данный момент, сервер должен возвращать изменившиеся данные (данные, а не отображение) в формате JSON или XML, изменения в отображении документа производятся клиентом (например, заменяется содержимое div'а или добавляется новый input). Компромиссное решение - подгружать часть шаблона, которая подвергается изменению. Но никак не всю страницу целиком (даже если она лишена шапки/подвала).
    • ]]>     Сб, 28 Бер 2009 14:26:21 -0000     esclkm
    oc порой предлагает слишком здравые идеи - но иногда у него крыша едет на ровном месте.
    Пример: он сделал ВКЛ комменты и рейтинги как одну из опций любой страницы. зачем непонятно.
    Рейтинги - то как он их переделал после траста- пальцы надо оторвать
    он регулярно увлекается - сделать так чтобы скрипт работал когда в браузере скрипты работают. иначе error. В общем странный подход. я его не понимаю.

    AJAx в админке:
    я считаю, что она (технология) нужна. 1. пажинация 2. банные листы. 3. ЧПУ. 4. Полл 5. категории страниц 6. полностью переработать интерфейс админки- сейчас он реально неудобный (да привычный, но неудобный).]]>     Сб, 28 Бер 2009 13:10:02 -0000     Dayver Сб, 28 Бер 2009 05:26:54 -0000 dervan
    Суть в том, что этот поступок oc'а нелья оставлять без последствий. Вы посмотрите, что он в тикете понаписал:
    Priority: major
    Type: defect

    Это - демонстративное хамство и неуважение к чужой работе. Если он и дальше будет продолжать в том же духе, с ним тяжело будет работать в команде.]]>     Сб, 28 Бер 2009 05:06:31 -0000     Dayver # Ratibor : Проголосовал за, но есть небольшое но...
    Ни на паджинации, ни тем более глобально аякс в админке нафиг не нужен,
    он нужет там всего в нескольких местах,
    таких как добавления пунктов в опросах,
    добавление новых правил для ЧПУ и т.п.
    Но все должно работать даже при отключенной яве.

    # Sergeich : Да уж, рейтинги испоганили основательно. Зачем там эту кнопку прицепили, которая постоянно ломает всю вёрстку я так и не понял. И удалять работу других руководствуясь исключительно своими интересами - это абсолютно неправильно.

    По поводу аякса в админке, согласен с Ратибором - всё хорошо в меру. Если там всё на уровне шаблонов подрубается, то предлагаю попросту сделать паралельно отдельную админку с аяксом и без. причём аяксовую на первых порах делать отдельно, привести её в состояние "круть" и предложить заменить в основной ветке :).

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

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

    согласен в той реализации в которой аякс был прикручен в рабочей версии ущербен и все же есть недовольные ..... но фишка в том что я не зря подчеркнул слово "рабочая" ..... а подчеркнул потому что к конечной версии все это претерпит много изменений и улучшений ...... как вариант возможно будет и отключатор или еще что либо для тех кому аякс не нужен ...... для тех кому нужен будет оптимизирован тот аякс что был сейчас ....... вариат отключености скриптов уже предусмотрен ..... но удалять это помоему хамство (пытаюсь сдерживатся в выражениях)]]>     Сб, 28 Бер 2009 05:06:27 -0000     Sergeich
    По поводу аякса в админке, согласен с Ратибором - всё хорошо в меру. Если там всё на уровне шаблонов подрубается, то предлагаю попросту сделать паралельно отдельную админку с аяксом и без. причём аяксовую на первых порах делать отдельно, привести её в состояние "круть" и предложить заменить в основной ветке :).

    Да, и ещё. Кто прилично владеет английским напишите в голосовании то, что Dayver в первом посте тут написал, а именно то, как подключается и работает аякс в админке.]]>     Сб, 28 Бер 2009 04:51:25 -0000     Ratibor Ни на паджинации, ни тем более глобально аякс в админке нафиг не нужен,
    он нужет там всего в нескольких местах,
    таких как добавления пунктов в опросах,
    добавление новых правил для ЧПУ и т.п.

    Но все должно работать даже при отключенной яве.

    # dervan : Если называть вещи своими именами, то действия oc'а - просто хамство. Считаю, он должен публично извиниться за свой поступок.
    И еще напинать, чтоб переделывал рейтинги.]]>     Сб, 28 Бер 2009 04:31:52 -0000     dervan oc'а - просто хамство. Считаю, он должен публично извиниться за свой поступок.]]> Сб, 28 Бер 2009 04:23:14 -0000 Dayver ос'а допустим он против того что в админке будет аякс но все же не ему это решать .... после того как он создал тикет о том что он хочет выкосить аякс я в свою очередь создал топик-опрос цель которого заключается в том что бы выяснить мнение общественности ..... не прошло и суток, проголосовало всего лишь 15 человек (кстати во время написания этого сообщения голосов "за аякс" было больше) .... но его же это не волнует ....... он взял да и удалил мою работу ..... нафига же я парился? ради того что кому то одному не понравится это и он возьмет и удалит все что ты делаешь? манал я это все ......

    Допустим я в чем то не прав ..... но давайте расмотрим его мнение ......
    как я понял из его сообщения он не полностью против аякса .....
    он хочет что бы он был "правильным" (ну да он же гуру этого дела наверное) ....
    он так же не хочет что бы аякс был в ядре .... точнее он согласен что это будет в ядре но таким образом что можно будет отключить аякс ........

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

    еще судя с его сообщения он не против если аякс был бы применен для удаления каких либо данных либо их просмотре но никак не для добавления чего либо (к примеру формы добавления правила бана айпишника) ...

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

    Все это демонстрирует непонятное отношеэние этого ос'а к трудам других людей. Пускай он против, пускай ему не нравится не сам аякс, а метод которым он применен .... ну так пускай исправит .... или напишет конкретный код который он считает "правильным" .... но нет проще ж все поудалять нафиг(не спрашивая никого).

    Кстати хотелось бы вспомнить труды ос'а - когда он в одних из первых версий кота сделал плуг голосований с применением аякса то он хакнул ядро без всяких смущений ....... а то как он переделал нормальную Трастовую рейтинговалку в что-то что на голову не налазит и работает через раз я уже молчу. В свое время к этой самой рейтинговалке он написал такую заметку
    oc:
    ...Also coding not finished, please test and tell what you want..
    тоесть непроверенный код ему можно заливать в рабочую копию, а остальным, судя по его реакции, нельзя!

    ИТОГ: Жду вашего мнения(так же голоса в топике), Trustmaster твоего мнение по вопросу дальнейших действий к тикету #258 в частности и к ревизии 665 в целом.]]>     Сб, 28 Бер 2009 03:01:56 -0000