5 ответ(ов) в теме
dan9
не в сети 7 лет
На сайте с 08.01.2014
Участник
Сообщения 17
1
10:30

Согласитесь не самый лучший вариант, хотя самій простой.

if($moderation==1){
wp_redirect('/?p='.$id_post);  exit;	
}else{
wp_redirect( get_permalink($id_post) );  exit;		
}

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

Я пока не вникал как именно плагин работает с ролями. Но хотелось бы, проверку прав пользователя на основе *_user_roles. Так безопаснее и удобнее. Ну зачем давать возможность загружать файлs тем, кому это не нужно.

У меня вывод миниатюры прописан в single.php и в index.php, то есть плагин выводить ещё и:

if(has_post_thumbnail($post->ID)){	
	   $large_image_url = wp_get_attachment_image_src( get_post_thumbnail_id($post->ID), 'large');
	   $large_image = '<table class="slide-thumb-recall" style="height:420px;"><tr><td style="text-align:center;vertical-align:middle;"><a rel="lightbox" class="largeImglink" href="'.$large_image_url[0].'"><img id="largeImg" src="'.$large_image_url[0].'"/></a></td></tr></table>

Я так понял в настройках поста можно указівать вівод слайдера, который независимо от его значения всё равно отображается.

0
Андрей CS
не в сети 6 минут
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 15001
2
13:20

а вы не вникайте в код отдельного дополнения, а просто возьмите и напишите свое дополнение для публикации пользователями. Уверен, что у вас это может получиться лучше.
А устраивать дискуссии "правильно/неправильно" непродуктивно, ибо только отнимает время.

0
dan9
не в сети 7 лет
На сайте с 08.01.2014
Участник
Сообщения 17
3
13:51

Я предполагал что ответ будет именно такой. Имеете право. Недочетов в плагине очень много, этого я думаю даже вы не будете отрицать. Всё это огромный труд и все это понимают, по этому не стоит на конструктивные, предложения реагировать так как будто вас в чем то обвиняют. Более чем возможно найдутся потенциальные клиенты, которым будет нужно именно так как написал я. Ведь это раздел "Идей".

И ещё:

if($get_fields){
					
					foreach((array)$get_fields as $custom_field){				
						$slug = str_replace('-','_',$custom_field['slug']);
						if($p) $postmeta = get_post_meta($p,$slug,1);
						$public_fields .= '<tr>
						<th><label>'.$custom_field['title'].':</label></th>';
						if($custom_field['type']=='text')
							$public_fields .= '<td><input type="text" name="'.$slug.'" class="regular-text" id="'.$slug.'" maxlength="50" value="'.$postmeta.'" /><br/></td>';
						if($custom_field['type']=='textarea')
							$public_fields .= '<td><textarea name="'.$slug.'" class="regular-text" id="'.$slug.'" rows="5" cols="50">'.$postmeta.'</textarea></td>';
						if($custom_field['type']=='select'){
							$fields = explode('#',$custom_field['field_select']);
							$count_field = count($fields);
							for($a=0;$a<$count_field;$a++){
								$field_select .='<option value="'.$fields[$a].'" '.selected($postmeta,$fields[$a],false).'>'.$fields[$a].'</option>';
							}
							$public_fields .= '<td><select name="'.$slug.'" class="regular-text" id="'.$slug.'">
							'.$field_select.'
							</select></td>';
						}
						if($custom_field['type']=='checkbox'){
							$chek = explode('#',$custom_field['field_select']);
							$count_field = count($chek);
							$public_fields .='<td>';							
							for($a=0;$a<$count_field;$a++){
								$slug_chek = $slug.'_'.$a;								
								$public_fields .='<input type="checkbox" name="'.$slug_chek.'" class="regular-text" value="'.$chek[$a].'" '.checked(get_post_meta($p,$slug_chek,1),$chek[$a],false).'> '.$chek[$a].'<br/>';
							}
							$public_fields .='</td>';
						}
						if($custom_field['type']=='radio'){
							$radio = explode('#',$custom_field['field_select']);
							$count_field = count($radio);
							$public_fields .='<td>';
							for($a=0;$a<$count_field;$a++){
								$public_fields .='<input type="radio" name="'.$slug.'" '.checked($postmeta,$radio[$a],false).' class="regular-text" value="'.$radio[$a].'"> '.$radio[$a].'<br/>';
							}
							$public_fields .='</td>';
						}
						$public_fields .= '</tr>';				
					}
				}

Мелочь: По умолчанию это не валидно. У тех кто юзает в своей верстке таблицы перекосит дизайн. С условием конечно, что форма будет иметь дополнительные поля.

0
Андрей CS
не в сети 6 минут
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 15001
4
14:03

а я никого не обвинял. Просто вижу, что человек может что то сам создать, вот и посоветовал делом заняться. А эти вырванные из контекста строки кода на самом деле никому не понятны и не нужны, следует это понимать. Когда вы предоставите цельный рабочий продукт это будет совсем другое дело, можно будет его поставить, попробовать, покритиковать.
По поводу последнего замечания, достаточно было отметить, что забыты теги table.

0
dan9
не в сети 7 лет
На сайте с 08.01.2014
Участник
Сообщения 17
5
14:07

Андрей Plechev сказал(а)

а я никого не обвинял. Просто вижу, что человек может что то сам создать, вот и посоветовал делом заняться. А эти вырванные из контекста строки кода на самом деле никому не понятны и не нужны, следует это понимать. Когда вы предоставите цельный рабочий продукт это будет совсем другое дело, можно будет его поставить, попробовать, покритиковать.
По поводу последнего замечания, достаточно было отметить, что забыты теги table.

Спасибо, учту. Буду писать по проще. Просто, наверное мне, было бы понятней если бы привели чать кода и сказали проблему.

0
Тема закрыта. Публикация новых сообщений запрещена.