Форумы / National / Russian / Я возмущен

Мы тут как работаем? Что хочу то и делаю?

Dayver
#1 28.03.2009 03:01
Собсно возмутили меня действия ос'а допустим он против того что в админке будет аякс но все же не ему это решать .... после того как он создал тикет о том что он хочет выкосить аякс я в свою очередь создал топик-опрос цель которого заключается в том что бы выяснить мнение общественности ..... не прошло и суток, проголосовало всего лишь 15 человек (кстати во время написания этого сообщения голосов "за аякс" было больше) .... но его же это не волнует ....... он взял да и удалил мою работу ..... нафига же я парился? ради того что кому то одному не понравится это и он возьмет и удалит все что ты делаешь? манал я это все ......

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

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

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

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

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

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

ИТОГ: Жду вашего мнения(так же голоса в топике), Trustmaster твоего мнение по вопросу дальнейших действий к тикету #258 в частности и к ревизии 665 в целом.
Pavlo Tkachenko aka Dayver
Отредактировано: Dayver (28.03.2009 05:15, 15 лет назад)
dervan
#2 28.03.2009 04:23
Если называть вещи своими именами, то действия oc'а - просто хамство. Считаю, он должен публично извиниться за свой поступок.
Ratibor
#3 28.03.2009 04:31
Проголосовал за, но есть небольшое но...
Ни на паджинации, ни тем более глобально аякс в админке нафиг не нужен,
он нужет там всего в нескольких местах,
таких как добавления пунктов в опросах,
добавление новых правил для ЧПУ и т.п.

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

# dervan : Если называть вещи своими именами, то действия oc'а - просто хамство. Считаю, он должен публично извиниться за свой поступок.
И еще напинать, чтоб переделывал рейтинги.
Не задавай глупых вопросов, не услышишь вранья.
Sergeich
#4 28.03.2009 04:51
Да уж, рейтинги испоганили основательно. Зачем там эту кнопку прицепили, которая постоянно ломает всю вёрстку я так и не понял. И удалять работу других руководствуясь исключительно своими интересами - это абсолютно неправильно.

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

Да, и ещё. Кто прилично владеет английским напишите в голосовании то, что Dayver в первом посте тут написал, а именно то, как подключается и работает аякс в админке.
Отредактировано: Sergeich (28.03.2009 04:54, 15 лет назад)
Dayver
#5 28.03.2009 05:06
# Ratibor : Проголосовал за, но есть небольшое но...
Ни на паджинации, ни тем более глобально аякс в админке нафиг не нужен,
он нужет там всего в нескольких местах,
таких как добавления пунктов в опросах,
добавление новых правил для ЧПУ и т.п.
Но все должно работать даже при отключенной яве.

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

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

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

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

согласен в той реализации в которой аякс был прикручен в рабочей версии ущербен и все же есть недовольные ..... но фишка в том что я не зря подчеркнул слово "рабочая" ..... а подчеркнул потому что к конечной версии все это претерпит много изменений и улучшений ...... как вариант возможно будет и отключатор или еще что либо для тех кому аякс не нужен ...... для тех кому нужен будет оптимизирован тот аякс что был сейчас ....... вариат отключености скриптов уже предусмотрен ..... но удалять это помоему хамство (пытаюсь сдерживатся в выражениях)
Pavlo Tkachenko aka Dayver
dervan
#6 28.03.2009 05:06
Мне непонятно, зачем в админке ajax - я с этим не разбирался и обосованного мнения на эту тему высказать не могу. Но не в этом сейчас суть.

Суть в том, что этот поступок oc'а нелья оставлять без последствий. Вы посмотрите, что он в тикете понаписал:
Priority: major
Type: defect

Это - демонстративное хамство и неуважение к чужой работе. Если он и дальше будет продолжать в том же духе, с ним тяжело будет работать в команде.
Dayver
#7 28.03.2009 05:26
Да еще забыл сказать .... да в том виде в котором был прикручен аякс именно к пагинации имеет большой недочет - отркутить аякс сложновато(прикрутка к другим элементам этого минуса лишена - смотрите модуль бан-лист) .... но поскольку для исправления этого нужно править функции пагинации то я это отложил на потом ...
Pavlo Tkachenko aka Dayver
esclkm
#8 28.03.2009 13:10
во многом согласен с dayver. Он прав по поводу поведения OC. Не понимаю я его позиции - ему можно. другим нельзя. Тем более что dayver ко всему подходит очень аккуратно, так как понимает что ajax мощный и нужный инструмент но надо думать так чтобы с отключенными скриптами пользователь работал комфортно. Возможно не везде он рационально исвльзует скрипт (допустим в плане пагинации надо еще почесать кепку, как пагинацию лучше сделать. но в тоже время - dayver все всегда поправляет и доводит до ума.

oc порой предлагает слишком здравые идеи - но иногда у него крыша едет на ровном месте.
Пример: он сделал ВКЛ комменты и рейтинги как одну из опций любой страницы. зачем непонятно.
Рейтинги - то как он их переделал после траста- пальцы надо оторвать
он регулярно увлекается - сделать так чтобы скрипт работал когда в браузере скрипты работают. иначе error. В общем странный подход. я его не понимаю.

AJAx в админке:
я считаю, что она (технология) нужна. 1. пажинация 2. банные листы. 3. ЧПУ. 4. Полл 5. категории страниц 6. полностью переработать интерфейс админки- сейчас он реально неудобный (да привычный, но неудобный).
littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты
Отредактировано: esclkm (28.03.2009 22:29, 15 лет назад)
Trustmaster
#9 28.03.2009 14:26
У меня двоякое впечатление.

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

С другой стороны, в одном он всё-таки прав: надо учиться использовать jQuery (особенно селекторы и события), иначе, как сейчас модно выражаться, нас ждёт большой рефакторинг. Поэтому поступим так: во избежание переписывания кода (а переписывать и так придется довольно много) мы пока внедрение AJAX в админке заморозим, и раз уж аякс для удобства полезен, то будем делать его правильно, чтобы потом не пришлось опять всё переписывать. Уточяю, что надо исправить:
[list=1]
  • Убрать аргументы $onclick и $object из sed_pagination() и sed_pagination_pn(). Да, именно убрать. Обработчик нужно составлять и вешать на это событие при помощи jquery при загрузке документа ($(document).ready), выбрав ссылки паджинации селектором. Возможно, придется при выводе паджинации снабдить её некоторыми аргументами или переменными, содержащими параметры паджинации, но в обработчиках их быть не должно.
  • Далеко не всегда требуется id, чтобы получить искомый элемент. Если структура позволяет это сделать однозначно селектором, то стоит обойтись без id.
  • Второй момент гораздо более болезненный и заключается в "неправильном использовании" самого аякса. Правило звучит так: обновляйте только те данные, которые изменились. В идеале клиент должен чётко понимать, что он меняет в данный момент, сервер должен возвращать изменившиеся данные (данные, а не отображение) в формате JSON или XML, изменения в отображении документа производятся клиентом (например, заменяется содержимое div'а или добавляется новый input). Компромиссное решение - подгружать часть шаблона, которая подвергается изменению. Но никак не всю страницу целиком (даже если она лишена шапки/подвала).
  • May the Source be with you!
    esclkm
    #10 28.03.2009 16:22
    согласен - что надо учиться использовать, а еще больше согласен про загрузку части, вместо всей страницы. очень важно...
    littledev.ru - мой маленький зарождающийся блог о котонти.
    снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты