передать class в форму комментария (решение) |
|
|
|
Студент
Всего сообщений: 113
Зарегистрирован 2010-08-30
|
Версия ExpressionEngine v 1.6.9(думаю и выше, ниже), хак который позволяет передать в форму комментариев class и id(css), до этого приходилось править вручную в файлах класс и айди.
Открываем - system/modules/comment/mod.comment
ищем ~ 1768 строка
$data = array( 'hidden_fields' => $hidden_fields, 'action' => $RET, 'id' => 'commentform' );
Как видно вписано id - commentform, что не всегда подходит. в документации (form_id=“news_comment_form”) не работает пример(хз в других версиях).
Заменяем
$data = array( 'hidden_fields' => $hidden_fields, 'action' => $RET, 'class' => $TMPL->fetch_param('form_class'), 'id' => $TMPL->fetch_param('form_id') );
Далее открываем - /system/core/core.functions.php
ищем ~ 511 строка
$form = '<form '.$data['id'].$data['name'].'method="post" action="'.$data['action'].'" '.$data['onsubmit'].' '.$data['enctype'].">\n";
Заменяем
$form_class = ''; if($data['class'] !== '') $form_class = ' class="'.$data['class'].'" '; $form = '<form '.$data['id'].$data['name'].$form_class.' method="post" action="'.$data['action'].'" '.$data['onsubmit'].' '.$data['enctype'].">\n";
Параметры, которые передавать
form_id - id формы
form_class - класс формы
P.S Возможно это только у меня такая кривая сборка ?
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
Думаю, вряд-ли стоит править файлы модуля комментов из-за
банального отсутствия возможности изменить передаваемый по
умолчанию id=“comment_form” на нечто иное... Тем более, что
она, форма комментариев, как правило, всегда одна на странице
|
|
|
|
|
|
Студент
Всего сообщений: 113
Зарегистрирован 2010-08-30
|
да, возможно вы правы, но я сам не верстаю, нужно либо верстальщика “заганять” под свои рамки, либо самому учится, лучше поправить пару строк, и потом передавать id, class(не зависеть от версталщика).
Заменить id формы - на дефолт(commentform), это верно для Id, для классов приходилось править файлы, как мне кажется тоже не очень хорошо.
p.s дефолту у меня - “commentform”
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
И тем не менее, тысячам.., десяткам тысяч вырстальщиков по всему миру
различного уровня подготовки: от самого низкого - до выше некуда,
дефолтного значения id вполне хватает!
Нашему-же брату, как всегда, недостает пятого колеса в телеге =)
|
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
Ну так переходите на двойку.
Коммерческий продукт, ребята!
Любой каприз, как говорится...
|
|
|
|
|
|
Студент
Всего сообщений: 113
Зарегистрирован 2010-08-30
|
я думаю это не каприз, а удобство, вы не находите ?
Да, перешел бы, только вот у меня бесплатная (core), а там онли лицензия ? =)
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
я думаю это не каприз, а удобство, вы не находите ?
Нет, не нахожу. Самый что ни на есть каприз американской домохозяйки,
пользователя EE, за счет которых и достигнуто расширение круга клиентов =)
Ведь им же, наверное, не в домек, что вместо:
.comment_form {}
в CSS можно использовать:
#comment_form {}
а еще можно так:
.my_class #comment_form {}
и еще много-много по-разному
А вообще, если что-то новое появляется в свежей версии (релизе), это вовсе
не означает, что в более раньних версиях это не работало...
|
|
|
|
|
|
Студент
Всего сообщений: 113
Зарегистрирован 2010-08-30
|
Afgan - 01 Ноябрь 2011 12:01 A.M. я думаю это не каприз, а удобство, вы не находите ?
Нет, не нахожу. Самый что ни на есть каприз американской домохозяйки,
пользователя EE, за счет которых и достигнуто расширение круга клиентов =)
Ведь им же, наверное, не в домек, что вместо:
.comment_form {}
в CSS можно использовать:
#comment_form {}
а еще можно так:
.my_class #comment_form {}
и еще много-много по-разному
А вообще, если что-то новое появляется в свежей версии (релизе), это вовсе
не означает, что в более раньних версиях это не работало...
Эх. сколько разных комбинация, а ведь можно просто так - form_id=“my_form_id” или form_class=“my_form_class”.
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
Это:
а ведь можно просто так - form_id=“my_form_id” или form_class=“my_form_class”
Всего лишь добавит одну из многочисленных
возможных комбинаций, не более того.
Но не в этом дело...
Суть в следующем: если вас не устраивает квалификация
верстальщика - не стоит под него переписывать и без того
дорогостоящий скрипт
Мало того - многие, начинающие, взирая на это, так или
иначе, вполне могут воспринять это в буквальном смысле...
|
|
|
|
|
|
Студент
Всего сообщений: 113
Зарегистрирован 2010-08-30
|
Извините меня уважаемый afgan, Вы написали чистую воду.
Причем здесь квалификация верстальщика, он ведь не знает под что верстает(ему сказали макет - html), будь-то жумла, Вп, друпал, здесь удобство человека который натягивает всю это шкуру на ЦМС, и который не знает тонкостей с css(которые вы описали, в этом случаи я.), ему нужно все сделать по аналогии и передать класс.ид, который верстальщик назвал например не - commentform.
К чему вы ведете, к тому, что навязать мне, что удобно для меня ?
Да, кстати, я не под верстальщика переписываю скрипт - а исключительно для своего удобства али нет ?
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
Да, кстати, я не под верстальщика переписываю скрипт - а исключительно для своего удобства али нет ?
Удачи!
Только про себя, если можно.
Не следует других вводить в заблуждение...
|
|
|
|
|
|
Студент
Всего сообщений: 113
Зарегистрирован 2010-08-30
|
Извините, может я где-то резко,грубо ответил, но это не со зла.
этот хак действительно мне понадобился и надеюсь еще кому-то понадобится.
|
|
|
|
|
|
Member
Всего сообщений: 224
Зарегистрирован 2010-11-20
|
Можно даже исправить (в верхнем сообщении):
...проблема при передачи параметров class и id(css) в форму комментариев...
На что-либо более подходящее...
|
|
|
|