Почему неправильно записывает в БД радиокнопки?

Question

[timers]

Собственно делаю рейтинг, ничего особенного обычные радио кнопки и обычная форма, но он почему-то записывает только value="5", выбираешь другие 1,2,3,4 все равно 5 записывает, в чем может быть причина?

<label>
	<input type="radio" checked="" name="otziv" class="otziv-ra" value="1">					
1
</label>
<label>
	<input type="radio" name="otziv" class="otziv-ra" value="2">					
2
</label>
<label>
	<input type="radio" name="otziv" class="otziv-ra" value="3">					
3
</label>
<label>
	<input type="radio" name="otziv" class="otziv-ra" value="4">					
4
</label>					
<label>
<input type="radio" name="otziv" class="otziv-ra" value="5">					
5
</label>

include_once ENGINE_DIR . '/classes/plugins.class.php';

if( !defined( "DATALIFEENGINE" ) ) die( "Hacking attempt!" );

if( ( $is_logged || $Katalog->Config['com_anonim'] == "yes" ) && isset( $_POST['addcomment'] ) )
	{
		$error = "";
		if( intval( $Katalog->Config['com_antispam'] ) > 0 && $_SESSION['katalog_add_com_time'] )
			{
				$limit_sec = intval( $proposal_config['com_antispam'] );
				$this_time = ( time () + ( $config['date_adjust'] * 60 ) );
				$period = $this_time - $_SESSION['katalog_add_com_time'];
				if( $period < $limit_sec ) 
					$error = "Сработала антиспам система, следующий отзыв вы сможете добавить через ".( $limit_sec - $period )." сек.";
				else
					@session_unregister( "katalog_add_com_time" );
			}
		
		$GroupComCaptcha = explode( ",", $Katalog->Config['com_groups_captcha'] );
		if( in_array( $member_id['user_group'], $GroupComCaptcha ) )
			{
				if( $_REQUEST['sec_code'] != $_SESSION['sec_code_session'] || !$_SESSION['sec_code_session'] ) $error = "Введён не верный код безопастности!";
			}
		
		require_once( ENGINE_DIR."/classes/parse.class.php" );
		$author = $member_id['name'];
		$author_id = $member_id['user_id'];
		$email = $member_id['email'];
		$answer_id = intval( $_POST['answer_id'] );
		
		$parse = new ParseFilter();
		$parse->safe_mode = true;
		$parse->allow_url = $user_group[ $member_id['user_group'] ]['allow_url'];
		$parse->allow_image = $user_group[ $member_id['user_group'] ]['allow_image'];
		$comment = $db->safesql( $parse->BB_Parse( $parse->process( $_POST['comments'] ), false ) );
		$otziv = $db->safesql( $parse->BB_Parse( $parse->process( $_POST['otziv'] ), false ) );
		$name_otziv = $db->safesql( $parse->BB_Parse( $parse->process( $_POST['name_otziv'] ), false ) );
		$tit_otziv = $db->safesql( $parse->BB_Parse( $parse->process( $_POST['tit_otziv'] ), false ) );
		
		$thisdate = date ( "Y-m-d H:i", ( time () + ( $config['date_adjust'] * 60 ) ) );
		$_IP = $_SERVER['REMOTE_ADDR'];
		
		if( !$is_logged )
			{
				$author_id = "";
				$email = trim( $_POST['email'] );
				$author = $db->safesql( htmlspecialchars( trim( CharsetConvert( $_POST['name'] ) ) ) );
				if( !$author || strlen( $author ) < 3 ) $error = "Имя пользователя, должно быть не менее 4-х символов.";
				if( !preg_match( "/^[\.A-z0-9_\-]+[@][A-z0-9_\-]+([.][A-z0-9_\-]+)+[A-z]{1,4}$/", $email ) ) $error = "Вы ввели неверный E-mail адрес.";
				
				if( empty( $error ) )
					{
						$db->query( "SELECT name FROM ".PREFIX."_users WHERE LOWER(name) = '".strtolower( $author )."' OR email = '{$email}'" );
						if( $db->num_rows() > 0 ) $error = "Имя или email уже используется зарегестрированным пользователем.";
					}
			}
		
		if( strlen( $comment ) < 10 ) $error = "Минимальное колличество символов: 10";
		if( strlen( $comment ) > 2000 ) $error = "Максимальное колличество символов: 2000";
		if( empty( $error ) )
			{
				$ok_add_com = true;
				$db->query( "INSERT INTO ".PREFIX."_katalog_comments (author,author_id,email,text,otziv,name_otziv,tit_otziv,date,katalog_id,del,ip,answer) VALUES('$author','$author_id','$email','$comment','$otziv','$name_otziv','$tit_otziv','$thisdate','$id','0','$_IP','$answer_id')" ); 
				$db->query( "UPDATE ".PREFIX."_katalog_post SET comm_num=comm_num+1 WHERE id='{$id}'" );
				
				if( in_array( "on_create", $Katalog->AllowSendEmail ) )
					{
						$rowUser = $db->super_query( "SELECT `name`,`email` FROM ".USERPREFIX."_users WHERE `user_id`='{$row['author_id']}'" );
						if( $rowUser['email'] )
							{
								$Katalog->SendEmail( "send_on_comment", "Оставлен отзыв об оффере", "", $rowUser['email'], array(
									"title"			=> stripslashes( $row['title'] ),
									"link"			=> "https://".$_SERVER['HTTP_HOST'].$Katalog->ReturnLinkPost( $row['id'], $row['alt_name'], $row['category'] ),
									"user"			=> stripslashes( $rowUser['name'] ),
								) );
							}
					}
				
				@session_register( "katalog_add_com_time" );
				$_SESSION['katalog_add_com_time'] = ( time () + ( $config['date_adjust'] * 60 ) );
				$_SESSION['sec_code_session'] = "";
				$comm_num++;
						
				$comment = "";
			}
	}

Comments

[Сергей]

А где весь код? Который записывает в БД, если JS используется, то и его тоже нужно приложить

[timers]

Сергей, добавил

[happyer]

Разве у кнопок не должны быть разные имена ?

[happyer]

у вас в итоге если какая-то кнопка нажата будет значение $_POST["otziv"] = значение последней нажатой кнопки

[timers]

happyer, т.е. в любом случае нужно делать для каждой кнопки типо name="otziv-1" ?

Просто я это сделать не смогу))

[alex-1917]

да ну на! имя у радиокнопок должно быть одинаковое для одной группы!
смотри в консольке, у тебя где-то глюк
и JS так и не выложен, без него толку 0.

otziv точно только 5 радиокнопок? мож еще где-то на странице есть? поменяй value c 5 на 6 -что покажет?

[timers]

alex-1917, да короче сделал через селект

Answer 1

[happyer]

Попробуйте каждой кнопке присвоить свое имя и сделать что-то по типу такого

<input type="radio" name="otziv-1" class="otziv-ra" value="1">
<input type="radio" name="otziv-2" class="otziv-ra" value="2">
<input type="radio" name="otziv-3" class="otziv-ra" value="3">

А на сервере разбить $_POST с помощью explode и пробежаться по массиву значений

Comments

[timers]

Ну это же геморой.

Причем я пробовал так:

<select name="otziv" id="otziv">
					<option value="1">Кредит наличными</option>
					<option value="2">Микрозаймы</option>
					<option value="3">Кредитные карты</option>
					<option value="4">Банки</option>
					<option value="5">Кредит для бизнеса</option>
</select>

Так все работает как нужно, но мне нужны именно кнопки.

[happyer]

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

[timers]

happyer, нет через js не пойдет, а как сделать name="otziv-1",name="otziv-2" тд., чтобы мне не делать 5 таблиц)

[happyer]

timers, не совсем понял о чем вы

[alex-1917]

это тупиковый путь. у радио кнопок из одной группы должнобыть только одно имя епти!!
азбука-букварь-вокзал

Answer 2

[feycot]

А что выводит $_POST? В форме есть что-то еще кроме чекбоксов? Сейчас проверил радиобокс и работает корректно.