Drupal 删除/更新数据库中的错误行

Question

我有一组可以使用表单编辑和删除的项目。现在，所有项目都以字段集的形式显示在列表中。使用此方法创建字段集：

 $venues = db_query('SELECT vid, name, address, postcode, city FROM venues v WHERE v.uid = :uid', array(':uid' => $user->uid));
 if($venues->rowCount() != 0) {
   foreach($venues as $venue) {
     $page['venue_editing_form'][] = drupal_get_form('venue_editor_form', $venue->vid, $venue->name, $venue->address, $venue->postcode, $venue->city); 
   }
 }

然后用这个函数创建表单：

function venue_editor_form($form, &$form_state, $vid, $vname, $vadd, $vpostc, $vcity) {
$form['#prefix'] = '<div class="vedit">';
$form['#suffix'] = '</div>';
$form['edit_fieldset_'.$vname] = array(
  '#type' => 'fieldset',
  '#title' => t($vname),
  '#collapsible' => TRUE,
  '#collapsed' => TRUE,
  '#attributes' => array('class' => array('venue-edit')),
);
$form['edit_fieldset_'.$vname]['venue_name'] = array(
  '#type' => 'textfield',
  '#title' => t('Venue name:'),
  '#description' => t('Enter the venue name.'),
  '#default_value' => $vname,
  '#attributes' => array('onfocus' => 'this.value=""')
);
$form['edit_fieldset_'.$vname]['address'] = array(
  '#type' => 'textfield',
  '#title' => t('Address:'),
  '#description' => t('The address of this venue.'),
  '#default_value' => $vadd,
  '#attributes' => array('onfocus' => 'this.value=""')
);
$form['edit_fieldset_'.$vname]['postcode'] = array(
  '#type' => 'textfield',
  '#title' => t('Postcode'),
  '#description' => t('Enter the postcode for this event, so planB can position it on the map'),
  '#default_value' => $vpostc,
  '#attributes' => array('onfocus' => 'this.value=""', 'onblur' => 'getLatLong(this)')
);
$form['edit_fieldset_'.$vname]['city'] = array(
  '#type' => 'textfield',
  '#title' => t('City'),
  '#description' => t('The city this venue is in.'),
  '#default_value' => $vcity,
  '#attributes' => array('onfocus' => 'this.value=""')
);
$form['edit_fieldset_'.$vname]['edit_submit'] = array(
  '#type' => 'submit',
  '#value' => t('Submit changes'),
  '#validate' => array('venue_creation_form_validate')
);
$form['edit_fieldset_'.$vname]['delete_venue'] = array(
  '#type' => 'submit',
  '#value' => t('Delete venue'),
  '#submit' => array('venue_form_delete')
);
$form['edit_fieldset_'.$vname]['venueid'] = array(
  '#type' => 'value',
  '#value' => $vid
);
return $form;

}

还有一个提交处理程序：

function venue_form_delete($form, &$form_state) {
  $deleted = db_delete('venues')
    ->condition('vid', $form['#venueid'], '=')
    ->execute();
    drupal_set_message('Venue deleted successfully.');
}

现在，问题是无论我按下哪个场地的删除按钮，Drupal 总是删除列表中的第一个。出于某种原因，$form['#venueid'] 始终设置为提交处理程序列表中的第一个地点，但是如果我在生成表单时打印一条消息，则每个表单都有不同的 ID，应该是。

以前有人遇到过这个问题吗？

Answer 1

几周前，我的一位同事遇到了这个问题。您遇到此问题的原因是因为您尝试在具有不同数据的页面上多次加载相同的表单。看起来这应该可行，但它不是因为所有表单最终都具有相同的令牌/ ID。当您提交每个表单时，系统不知道您真正点击了哪个表单，因此无法采取适当的操作。

有两种方法可以解决这个问题：

1) 重做您的表单，以便全部使用 1 个表单而不是多个表单来处理

2) 使用hook_forms(...) 创建一个“基本表单”，然后为每个场地生成映射到“基本表单”的唯一表单。这将确保每个表单都获得一个唯一的 ID，而无需为每个表单编写表单。

我没有 drupal 7 示例，但这里有一个页面显示如何使用 hook_forms http://www.computerminds.co.uk/drupal-6-multiple-instances-same-form-one-page 解决您的 drupal 6 问题。您应该能够应用相同的技术

我没有测试过这段代码，但这是基本思想：

修改您的 get_form 以使用唯一 ID

foreach($venues as $venue) {
     $page['venue_editing_form'][] = drupal_get_form('venue_editor_form_'. $venue->vid, $venue->vid, $venue->name, $venue->address, $venue->postcode, $venue->city); 
   }

实现 hook_forms

function MODULENAME_forms($form_id) {
  $forms = array();
  if (preg_match('/venue_editor_form_/', $form_id) {
    $forms[$form_id] = array(
      'callback' => 'venue_editor_form',
    );
  }
  return $forms;
}

然后我认为您必须修改表单的提交参数以确保它们都被路由到同一个提交处理程序。虽然看起来您确实为每个按钮添加了不同的提交，但您可能不必在那里做任何事情

$form['#submit'] => array('venue_editing_form_submit');

您可能还必须对验证函数执行相同的操作。

应该这样做。

Answer 2

我猜问题是您尝试通过表单处理传递场地ID值而不使用显式值字段：

$form['#venueid'] = $vid;

这只是为“组成”表单属性（前导“#”）分配一个值，该属性通常不存在。尝试使用明确的value field type 代替：

$form['venueid'] = array(
    '#type' => 'value',
    '#value' => $node->vid
);

这样，vid 应该在您的提交函数中可用，就像任何其他表单值一样，即在$form_state['values']：

function venue_form_delete($form, &$form_state) {
  $deleted = db_delete('venues')
    ->condition('vid', $form_state['values']['venueid'], '=')
    ->execute();
    drupal_set_message('Venue deleted successfully.');
}

请注意删除了“#”字符 - 前导“#”表示表单（和表单元素）属性，Drupal 将在表单处理期间尝试解释这些属性，而“#venueid”不是已定义的属性。