Использование ISSET с флажками

Это немного похоже на создание формы поиска – похоже, вы создаете форму для каждого отдельного результата? Аннотированный код и комментарии ниже. В своем ответе я старался придерживаться стиля кодирования.

 $amenity_array = array(); $id = get_query_var('site'); // this first section gets the selected meta properties for the queried property (if available) // [skipping for brevity ] global $imic_options; if (isset($imic_options['properties_amenities']) && count($imic_options['properties_amenities']) > 1) { // this loops over all of the globally defined properties foreach ($imic_options['properties_amenities'] as $properties_amenities) { // gets the name of the property $am_name = strtolower(str_replace(' ', '', $properties_amenities)); // and sets up to check the box if the property has the defined amenity // if this is for a search form, why check boxes based on a result? $check = ''; if (in_array($properties_amenities, $amenity_array)) { // note: this only really needs to be 'checked' $check = 'checked="checked"'; } 

Тем не менее, здесь все идет немного по-другому.

  if (isset($_GET['p_am'])) { $ams = $_GET['p_am']; echo '<div class="checkbox"><input type="checkbox" name="p_am" ' . $check . ' class="styled" value ="' . $properties_amenities . '"><label for="' . $am_name . '">' . $properties_amenities . '</label></div>'; } else { echo '<div class="checkbox"><input type="checkbox" name="p_am" ' . $check . ' class="styled" value ="' . $properties_amenities . '"><label for="' . $am_name . '">' . $properties_amenities . '</label></div>'; } 

Это говорит о том, что все флажки имеют одно и то же имя ( "p_am" ) вместо использования имени цикла для удобства ( $am_name ) или какой-либо комбинации форм, если вы хотите, чтобы все удобства "p_am[$am_name]" в одном и том же массиве входных данных поиска (например, "p_am[$am_name]" ).

Каждый флажок также изменил бы значение $check если он был $_GET массив $_GET .

  $ams = $_GET['p_am']; if (isset($ams[$am_name])) { $check = 'checked'; } 

Каждый флажок имел бы name="p_am['.$am_name.']" Как имя.

  echo '<div class="checkbox"><input type="checkbox" name="p_am[' . $am_name . ']" ' . $check . ' class="styled" value ="' . $properties_amenities . '"><label for="' . $am_name . '">' . $properties_amenities . '</label></div>'; 

Если вы хотите, чтобы у каждого из удобств были уникальные имена (смотря на исходный флажок, который вообще не называется p_am ), а не в массиве в PHP, вы просто используете имя цикла для удобства ( $am_name ) , вот так:

  if (isset($_GET[$am_name])) { $check = 'checked'; } 

Каждый флажок имел бы name="'.$am_name.'" Как имя.

UPDATE: после обновления OP, похоже, что каждый флажок должен иметь одно и то же имя, но не быть введенным ключом. Для этой ситуации ваши флажки должны быть вызваны p_am[] (на обеих страницах поиска исходная страница), и вам нужно использовать что-то вроде in_array() вместо isset() для проверки результата, например:

  if (in_array($properties_amenities, $_GET['p_am'])) { $check = 'checked'; } 

Дополнительное примечание: вы также используете $am_name для метки, не устанавливая при этом соответствующий атрибут id , а также не $am_name символы (например, круглые скобки), поэтому ассоциация ярлыков не будет работать ,

  echo '<div class="checkbox"><input type="checkbox" name="' . $am_name . '" ' . $check . ' class="styled" value ="' . $properties_amenities . '"><label for="' . $am_name . '">' . $properties_amenities . '</label></div>'; 

Для выбора начальной загрузки вы просто не проверяете, какие параметры выбраны:

 <select name="property_type[]" id="pt-multi" class="form-control multi-select2" multiple="multiple"> <?php $terms = get_terms( "property-type", array( 'hide_empty' => 0 ) ); $count = count($terms); if ( $count > 0 ){ echo "<option value='Any'>All</option>"; foreach ( $terms as $term ) { // if the option is 'checked', you need to add the 'selected' atttibute here echo "<option value='" . $term->slug . "'>" . $term->name . "</option>"; } } ?> </select> 

Таким образом, внутри цикла будет выглядеть примерно так:

  // for efficiency's sake should live outside the loop as a one-off, but here for illustrative purposes $types = $_GET['property_type']; $selected = isset($types[$term->slug]) ? 'selected' : ''; echo "<option value='" . $term->slug . "'" . $selected . ">" . $term->name . "</option>"; 

Если вы используете более старую версию начальной загрузки, вам может потребоваться выбрать выбранную строку 'selected="selected" или 'selected="selected" data-selected="true" , в зависимости от используемой вами версии. Для текущей версии указанная выше строка должна быть в порядке.

Если <select> не является массивом параметров (то есть не кратным), тогда [] следует удалить из имени и работать аналогично коду флажка:

 <select name="property_type" id="pt-multi" class="form-control multi-select2"> <?php $terms = get_terms( "property-type", array( 'hide_empty' => 0 ) ); $count = count($terms); if ( $count > 0 ){ echo "<option value='Any'>All</option>"; foreach ( $terms as $term ) { $type = $_GET['property_type']; $selected = isset($type) && $type == $term-slug ? 'selected' : ''; echo "<option value='" . $term->slug . "'" . $selected . ">" . $term->name . "</option>"; } } ?> </select> 

Исходный HTML для <select> фактически не работает так же – все, что он делает, это добавить дубликат <option> в начало списка опций, и поскольку ничего не выбрано, форма будет обрабатывать дубликат, созданный вверху как 'selected' для <select> .

ОБНОВЛЕНИЕ 2: причина, по которой функция поиска ломается с массивом входных данных флажка, заключается в том, что вы используете сравнение LIKE для вашего мета-запроса. Из документации по Meta Query WordPress :

Это может быть массив только при сравнении: «IN», «NOT IN», «BETWEEN» или «NOT BETWEEN».

В документации также приведены примеры того, как объединить их. Поскольку вы не указали, как вы на самом деле делаете поисковый запрос, это немного угадывает, но похоже, что для массива флажков ваш код должен быть чем-то похожим на:

 if (!empty($amenities)) { foreach ($amenities as $amenity) { array_push($meta_query, array( 'key' => 'imic_property_amenities', 'value' => $amenity, 'compare' => 'LIKE' )); } } 

Как и в примерах в документации, вам нужно убедиться, что существует relation => 'OR' или relation => 'AND' чтобы определить, как вы хотите, чтобы ваш поиск работал.

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

<input type="checkbox" value="condo" name="var_name[]">

[] в конце атрибута name должен сказать PHP, что он должен обрабатывать переменную как массив.

а затем в скрипте PHP вы можете просто проверить длину массива, используя somthing like:

$arr = $_POST['var_name']; echo count($arr);

основанный на комментарии leith, если он сгенерирован автоматически, то вам не следует беспокоиться о том, чтобы проверить флажки вручную.

вместо этого просто выберите выбранные параметры, например:

 <?php $selections = $_GET['property_type']; // or something similar ?> <select name="property_type[]" id="pt-multi" class="form-control multi-select2" multiple="multiple"> <?php $terms = get_terms( "property-type", array( 'hide_empty' => 0 ) ); $count = count($terms); if ( $count > 0 ){ echo "<option value='Any'>All</option>"; foreach ( $terms as $term ) { $selected = in_array($term->slug, $selections); echo "<option value='" . $term->slug . "' " . ($selected?' selected':'') . ">" . $term->name . "</option>"; } } ?> </select> 

см. этот скрипт https://jsfiddle.net/u1w158tp/

в нем вы только что selected option s, и она автоматически проверит checkbox es

У ваших флажков одинаковое имя. Попробуйте добавить другое имя или использовать «p_am []», чтобы рассматривать их как массив.