Искать  
 
   
 
передать class в форму комментария (решение)
Отправлено: 30 Октябрь 2011 09:24 P.M.   [ Игнорировать ]
Студент
Avatar
RankRank
Всего сообщений:  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 Возможно это только у меня такая кривая сборка ?

Профиль
 
 
Отправлено: 31 Октябрь 2011 01:08 P.M.   [ Игнорировать ]   [ # 1 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

Думаю, вряд-ли стоит править файлы модуля комментов из-за

банального отсутствия возможности изменить передаваемый по

умолчанию id=“comment_form” на нечто иное... Тем более, что

она, форма комментариев, как правило, всегда одна на странице wink

Профиль
 
 
Отправлено: 31 Октябрь 2011 06:41 P.M.   [ Игнорировать ]   [ # 2 ]
Студент
Avatar
RankRank
Всего сообщений:  113
Зарегистрирован  2010-08-30

да, возможно вы правы, но я сам не верстаю, нужно либо верстальщика “заганять” под свои рамки, либо самому учится, лучше поправить пару строк, и потом передавать id, class(не зависеть от версталщика).

Заменить id формы - на дефолт(commentform), это верно для Id, для классов приходилось править файлы, как мне кажется тоже не очень хорошо.

p.s дефолту у меня - “commentform

Профиль
 
 
Отправлено: 31 Октябрь 2011 08:27 P.M.   [ Игнорировать ]   [ # 3 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

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

различного уровня подготовки: от самого низкого - до выше некуда,

дефолтного значения id вполне хватает!

Нашему-же брату, как всегда, недостает пятого колеса в телеге =)

Профиль
 
 
Отправлено: 31 Октябрь 2011 10:33 P.M.   [ Игнорировать ]   [ # 4 ]
Студент
Avatar
RankRank
Всего сообщений:  113
Зарегистрирован  2010-08-30

Да, так хватает, что они добавили этот параметр, который не работает ? (EE 1.x)
http://expressionengine.com/user_guide/modules/comment/index.html#form-class - 2.3.1 - почему же им дальше не хватало этого , дефолтного, видать у них этот брат тоже есть - которому к возу 5 колесо нужно ?

Профиль
 
 
Отправлено: 31 Октябрь 2011 11:26 P.M.   [ Игнорировать ]   [ # 5 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

Ну так переходите на двойку.

Коммерческий продукт, ребята!

Любой каприз, как говорится...

Профиль
 
 
Отправлено: 31 Октябрь 2011 11:44 P.M.   [ Игнорировать ]   [ # 6 ]
Студент
Avatar
RankRank
Всего сообщений:  113
Зарегистрирован  2010-08-30

я думаю это не каприз, а удобство, вы не находите ?

Да, перешел бы, только вот у меня бесплатная (core), а там онли лицензия ? =)

Профиль
 
 
Отправлено: 01 Ноябрь 2011 12:01 A.M.   [ Игнорировать ]   [ # 7 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

я думаю это не каприз, а удобство, вы не находите ?

Нет, не нахожу. Самый что ни на есть каприз американской домохозяйки,

пользователя EE, за счет которых и достигнуто расширение круга клиентов =)

Ведь им же, наверное, не в домек, что вместо:

.comment_form {} 

в CSS можно использовать:

#comment_form {} 

а еще можно так:

.my_class #comment_form {} 

и еще много-много по-разному wink

А вообще, если что-то новое появляется в свежей версии (релизе), это вовсе

не означает, что в более раньних версиях это не работало...

Профиль
 
 
Отправлено: 01 Ноябрь 2011 02:19 A.M.   [ Игнорировать ]   [ # 8 ]
Студент
Avatar
RankRank
Всего сообщений:  113
Зарегистрирован  2010-08-30
Afgan - 01 Ноябрь 2011 12:01 A.M.

я думаю это не каприз, а удобство, вы не находите ?

Нет, не нахожу. Самый что ни на есть каприз американской домохозяйки,

пользователя EE, за счет которых и достигнуто расширение круга клиентов =)

Ведь им же, наверное, не в домек, что вместо:

.comment_form {} 

в CSS можно использовать:

#comment_form {} 

а еще можно так:

.my_class #comment_form {} 

и еще много-много по-разному wink

А вообще, если что-то новое появляется в свежей версии (релизе), это вовсе

не означает, что в более раньних версиях это не работало...

Эх. сколько разных комбинация, а ведь можно просто так - form_id=“my_form_id” или form_class=“my_form_class”.

Профиль
 
 
Отправлено: 01 Ноябрь 2011 11:27 A.M.   [ Игнорировать ]   [ # 9 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

Это:

а ведь можно просто так  - form_id=“my_form_id” или form_class=“my_form_class”

Всего лишь добавит одну из многочисленных

возможных комбинаций, не более того.

Но не в этом дело...

Суть в следующем: если вас не устраивает квалификация 

верстальщика - не стоит под него переписывать и без того

дорогостоящий скрипт wink

Мало того - многие, начинающие, взирая на это, так или

иначе, вполне могут воспринять это в буквальном смысле...

Профиль
 
 
Отправлено: 01 Ноябрь 2011 04:37 P.M.   [ Игнорировать ]   [ # 10 ]
Студент
Avatar
RankRank
Всего сообщений:  113
Зарегистрирован  2010-08-30

Извините меня уважаемый afgan, Вы написали чистую воду.

Причем здесь квалификация верстальщика, он ведь не знает под что верстает(ему сказали макет - html), будь-то жумла, Вп, друпал, здесь удобство человека который натягивает всю это шкуру на ЦМС, и который не знает тонкостей с css(которые вы описали, в этом случаи я.), ему нужно все сделать по аналогии и передать класс.ид, который верстальщик назвал например не - commentform.

К чему вы ведете, к тому, что навязать мне, что удобно для меня ?

Да, кстати, я не под верстальщика переписываю скрипт - а исключительно для своего удобства али нет ?

Профиль
 
 
Отправлено: 01 Ноябрь 2011 04:53 P.M.   [ Игнорировать ]   [ # 11 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

Да, кстати, я не под верстальщика переписываю скрипт - а исключительно для своего удобства али нет ?

Удачи!

Только про себя, если можно.

Не следует других вводить в заблуждение...

Профиль
 
 
Отправлено: 01 Ноябрь 2011 05:13 P.M.   [ Игнорировать ]   [ # 12 ]
Студент
Avatar
RankRank
Всего сообщений:  113
Зарегистрирован  2010-08-30

Извините, может я где-то резко,грубо ответил, но это не со зла.
этот хак действительно мне понадобился и надеюсь еще кому-то понадобится.

Профиль
 
 
Отправлено: 01 Ноябрь 2011 06:00 P.M.   [ Игнорировать ]   [ # 13 ]
Member
Avatar
RankRankRank
Всего сообщений:  224
Зарегистрирован  2010-11-20

Можно даже исправить (в верхнем сообщении):

...проблема при передачи параметров class и id(css) в форму комментариев...

На что-либо более подходящее...

Профиль