Fatal error: Call to undefined function: form_radios() in W:\www\modules\community\user_aggregator\user_aggregator.module on line 220

CommentFileSizeAuthor
#2 user_aggregator.module.txt16.3 KBFoodster

Comments

Prometheus6’s picture

Assigned: Unassigned » Prometheus6
Status: Active » Closed (won't fix)

Sorry...SO unsupported for anything but 4.6.

Foodster’s picture

Assigned: Prometheus6 » Unassigned
Status: Closed (won't fix) » Patch (to be ported)
StatusFileSize
new16.3 KB

I changed the form elements with the 4.7 form api, but i am stuck in the form() function, i think it need rewrite the work flow. If anyone know, please kindly help.

in the _user_aggregator_validation_form function line 85 and 86

print theme('page', form("

$table_header $output

" . form_submit('Submit')));
#return form("

$table_header $output

" . form_submit('Submit'));

i have attached the modification. (just remove the txt extension)

stborchert’s picture

Hi.
Just a short note: why don't you use theme('table') and the form api?

    while($pending_item = db_fetch_object($pending_items)) {
      $save_opt = "<input type=\"checkbox\" name=\"edit[save_user][$pending_item->uid]\"  value=\"$pending_item->uid\" />";
      $delete_opt = "<input type=\"checkbox\" name=\"edit[delete_user][$pending_item->uid]\"  value=\"$pending_item->uid\" />";
      $output .= "<tr>
        <td><a href=\"$pending_item->site_url\" target=\"feed_preview\">$pending_item->site_name</a></td>
        <td><a href=\"$pending_item->site_feed\" target=\"feed_preview\">$xml_icon</a></td>
        <td align=\"center\">$save_opt</td><td align=\"center\">$delete_opt</td>
      </tr>";
    }
print theme('page', form("<table>$table_header $output</table>" . form_submit('Submit')));

looks a bit weird and could be done with someting like this (not working, its thought as a hint!):

while($item = db_fetch_object($pending_items)) {
  $form['name'][$item->uid] = array('#value' => $item->site_name);
  $form['save_user'][$item->uid] = array('#type' => 'checkbox', '#default_value' => FALSE);
  $form['delete_user'][$item->uid] = array('#type' => 'checkbox', '#default_value' => FALSE);
}

and then in theme__user_aggregator_validation_form()

$header = array(
  array('data' => t('Feed name')),
  array('data' => t('Feed')),
  array('data' => t('Approve feed')),
  array('data' => t('Deny/Delete')));
);
$xml_icon  = theme('image', 'misc/xml.png', t('XML feed'), t('XML feed'));
$rows = array();
while($item = db_fetch_object($pending_items)) {
  $row = array();
  $row[] = form_render($form['name'][$item->uid]);
  $row[] = $xml_icon;
  $row[] = array('data' => form_render($form['save_user'][$key]), 'align' => 'center');
  $row[] = array('data' => form_render($form['delete_user'][$key]), 'align' => 'center');
  $rows[] = $row;
}
$output .= theme('table', $header, $rows, array('summary' => t('list of feeds'));
...

hth,

Stefan

Prometheus6’s picture

Status: Patch (to be ported) » Closed (fixed)