Index: filefield/filefield.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/filefield/filefield.module,v
retrieving revision 1.167
diff -u -r1.167 filefield.module
--- filefield/filefield.module	10 Mar 2009 18:40:12 -0000	1.167
+++ filefield/filefield.module	10 Mar 2009 18:42:20 -0000
@@ -266,35 +266,12 @@
  * Implementation of CCK's hook_widget_settings().
  */
 function filefield_widget_settings($op, $widget) {
-  $return = array();
-  // load our widget settings callbacks..
-  $op =  str_replace(' ', '_', $op);
-  $function = 'filefield_widget_settings_'. $op;
-  if (function_exists($function)) {
-    $result = $function($widget);
-    if (isset($result) && is_array($result)) {
-      $return = $result;
-    }
-  }
-
-  // sometimes widget_settings is called with widget, sometimes with field.
-  // CCK needs to make up it's mind here or get with the new hook formats.
-  $widget_type = isset($widget['widget_type']) ? $widget['widget_type'] : $widget['type'];
-  $widget_module = isset($widget['widget_module']) ? $widget['widget_module'] : $widget['module'];
-
-  // dynamically load widgets file and callbacks for other fields and widgets utilizing
-  // filefield's hook_widget_settings implementation.
-  module_load_include('inc', $widget_module, $widget_module .'_widget');
-
-  $function = $widget_type .'_widget_settings_'. $op;
-  if (function_exists($function)) {
-    $result = $function($widget);
-    if (isset($result) && is_array($result)) {
-      $return = array_merge($return, $result);
-    }
+  switch ($op) {
+    case 'form':
+      return filefield_widget_settings_form($widget);
+    case 'save':
+      return filefield_widget_settings_save($widget);
   }
-
-  return $return;
 }
 
 /**
@@ -393,18 +370,10 @@
   return array(
     'default' => array(
       'label' => t('Generic files'),
-      'suitability callback' => TRUE,
-      'field types' => array('filefield','image'),
+      'field types' => array('filefield'),
       'multiple values' => CONTENT_HANDLE_CORE,
       'description' => t('Displays all kinds of files with an icon and a linked file description.'),
     ),
-    'filefield_dynamic' => array(
-      'label' => t('Dynamic file formatters'),
-      'suitability callback' => TRUE,
-      'field types' => array('file'),
-      'multiple values' => CONTENT_HANDLE_CORE,
-      'description' => t('(experimental) An extensible formatter for filefield.'),
-    ),
   );
 }
 
@@ -545,35 +514,6 @@
 }
 
 /**
- * set the default values for imagefield.
- * This seems to work for all but add a new item on unlimited values which doesn't
- * get assigned a proper default.
- */
-function filefield_default_value(&$form, &$form_state, $field, $delta) {
-  $items = array();
-  $field_name = $field['field_name'];
-
-  switch ($field['multiple']) {
-    case 0:
-      $max = 1;
-      break;
-    case 1:
-      $max =  isset($form_state['item_count'][$field_name]) ? $form_state['item_count'][$field_name] : 1;
-      break;
-    default:
-      $max = $field['multiple'];
-      break;
-  }
-
-  for ($delta = 0; $delta < $max; $delta++) {
-    $items[$delta] = array('fid' => 0, 'list' => $field['list_default'], 'data' => array('description' => ''));
-  }
-  return $items;
-}
-
-
-
-/**
  * Implementation of hook_file_references().
  */
 function filefield_file_references($file) {
Index: filefield/filefield_widget.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/filefield/filefield_widget.inc,v
retrieving revision 1.58
diff -u -r1.58 filefield_widget.inc
--- filefield/filefield_widget.inc	10 Mar 2009 18:40:12 -0000	1.58
+++ filefield/filefield_widget.inc	10 Mar 2009 18:42:21 -0000
@@ -168,12 +168,6 @@
   // merge file and item data so it is available to all widgets.
   $item = array_merge($item, $file);
 
-  // if this widget is another type and leaning on filefield to do the dirty work....
-  // pass it back home.
-  $function = $element['#type'] .'_widget_value';
-  if (function_exists($function)) {
-    $item = array_merge($item, $function($element, $edit));
-  }
   return $item;
 }
 
Index: imagefield/imagefield_widget.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/imagefield/imagefield_widget.inc,v
retrieving revision 1.23
diff -u -r1.23 imagefield_widget.inc
--- imagefield/imagefield_widget.inc	10 Mar 2009 18:40:12 -0000	1.23
+++ imagefield/imagefield_widget.inc	10 Mar 2009 18:42:21 -0000
@@ -18,8 +18,9 @@
  * return values are merged with and will override filefields' widget
  * settings callback's return values.
  */
-function imagefield_widget_widget_settings_form($widget) {
-  $form = array();
+function imagefield_widget_settings_form($widget) {
+  $form = module_invoke('filefield', 'widget_settings', 'form', $widget);
+
   $form['max_resolution'] = array(
     '#type' => 'textfield',
     '#title' => t('Maximum resolution for Images'),
@@ -46,7 +47,7 @@
     '#default_value' => !empty($widget['file_extensions']) ? $widget['file_extensions'] : 'jpg jpeg png gif',
     '#size' => 64,
     '#maxlength' => 64,
-    '#description' => t('Extensions a user can upload to this field. Separate extensions with a space and do not include the leading dot.'),
+    '#description' => t('Extensions a user can upload to this field. Separate extensions with a space and do not include the leading dot. Only jpg, png, and gif images are supported with this widget.'),
     '#weight' => 2,
   );
 
@@ -63,7 +64,6 @@
     '#default_value' =>  !empty($widget['custom_alt']) ? $widget['custom_alt'] : 0,
     '#description' => t('Enable user input alternate text for images.'),
   );
-
   $form['alt_settings']['alt'] = array(
     '#type' => 'textfield',
     '#title' => t('Default ALT text'),
@@ -95,39 +95,52 @@
   return $form;
 }
 
-function imagefield_widget_widget_settings_save($widget) {
-  //@todo: rename custom_alt and custom_title to alt_custom and title_custom to be OCD.
-  return array('max_resolution', 'min_resolution', 'alt',  'custom_alt', 'title', 'custom_title');
+/**
+ * Validate callback for the CCK widget form.
+ */
+function imagefield_widget_settings_validate($widget) {
+  // Check that only web images are specified in the callback.
+  $extensions = array_filter(explode(' ', $widget['file_extensions']));
+  $web_extensions = array('jpg', 'jpeg', 'gif', 'png');
+  if (count(array_diff($extensions, $web_extensions))) {
+    form_set_error('file_extensions', t('Only web-standard images (jpg, gif, and png) are supported through the image widget. If needing to upload other types of images, change the widget to use a standard file upload.'));
+  }
 }
 
 /**
- * @} End defgroup "Filefield widget settings callbacks."
+ * Save callback for the CCK widget form.
  */
+function imagefield_widget_settings_save($widget) {
+  // TODO: Rename custom_alt and custom_title to alt_custom and title_custom.
+  $filefield_settings = module_invoke('filefield', 'widget_settings', 'save', $widget);
+  return $filefield_settings + array('max_resolution', 'min_resolution', 'alt',  'custom_alt', 'title', 'custom_title');
+}
 
 /**
- * @defgroup "FileField widget element callbacks.
- * @{
- *
- * The call backs are called by form elements that leverage the
- * filefield_widget_value and filefield_widget_process callbacks.
- * They will be called after the filefield callbacks and their
- * return values will be merged with the filefield callback's.
+ * Element #value_callback function.
  */
-function imagefield_widget_widget_value($element, $edit = FALSE) {
-  // handle additional
+function imagefield_widget_value($element, $edit = FALSE) {
+  $item = filefield_widget_value($element, $edit);
   if ($edit) {
-      return array(
-        'alt' => isset($edit['alt']) ? $edit['alt'] : '',
-        'title' => isset($edit['title']) ? $edit['title'] : '',
-      );
+    $item['alt'] = isset($edit['alt']) ? $edit['alt'] : '';
+    $item['title'] = isset($edit['title']) ? $edit['title'] : '';
   }
-  return array('alt' => '', 'title' => '');
+  else {
+    $item['alt'] = '';
+    $item['title'] = '';
+  }
+  return $item;
 }
 
-function imagefield_widget_widget_process($element, $edit, &$form_state, $form) {
+/**
+ * Element #process callback function.
+ */
+function imagefield_widget_process($element, $edit, &$form_state, $form) {
   $file = $element['#value'];
   $field = content_fields($element['#field_name'], $element['#type_name']);
 
+  $element['#theme'] = 'imagefield_widget_item';
+
   $element['data']['alt'] = array(
     '#title' => t('Alternate Text'),
     '#type' => 'value',
@@ -150,13 +163,6 @@
   return $element;
 }
 
-function imagefield_widget_widget_validate($element) {
-}
-
-/**
- * @} End defgroup "FileField widget element callbacks."
- */
-
 /**
  * FormAPI theme function. Theme the output of an image field.
  */
Index: imagefield/image_field.inc
===================================================================
RCS file: imagefield/image_field.inc
diff -N imagefield/image_field.inc
--- imagefield/image_field.inc	4 Mar 2009 05:28:23 -0000	1.2
+++ /dev/null	1 Jan 1970 00:00:00 -0000
@@ -1,119 +0,0 @@
-<?php
-// $Id: image_field.inc,v 1.2 2009/03/04 05:28:23 quicksketch Exp $
-
-/**
- * @file
- * Field Hooks and Callbacks for ImageField.
- */
-
-/**
- * Implementation of hook_field_settings().
- */
-
-function imagefield_image_field_settings_form($field) {
-  $form = array();
-  $form['default'] = array(
-    '#type' => 'fieldset',
-    '#title' => t('Default image'),
-    '#element_validate' => array('_imagefield_field_settings_default_validate'),
-  );
-      // Present a thumbnail of the current default image.
-  $form['default']['use_default_image'] = array(
-    '#type' => 'checkbox',
-    '#title' => t('Use default image'),
-    '#default_value' =>  $field['use_default_image'],
-    '#description' => t('Check here if you want to use a image as default.'),
-  );
-  if (!empty($field['default_image'])) {
-    $form['default']['default_image_thumbnail'] = array(
-      '#type' => 'markup',
-      '#value' => theme('imagefield_image', $field['default_image'], '', '', array('width' => '150'), false),
-    );
-  }
-  $form['default']['default_image_upload'] = array(
-    '#type'  => 'file',
-    '#title' => t('Upload image'),
-    '#description' => t('Choose a image that will be used as default.'),
-  );
-
-  // We set this value on 'validate' so we can get cck to add it
-  // as a standard field setting.
-  $form['default_image'] = array(
-    '#type' => 'value',
-    '#value' => $field['default_image'],
-  );
-  return $form;
-}
-
-function imagefield_image_field_settings_save($field) {
-  return array('default_image', 'use_default_image');
-}
-
-function imagefield_image_field_settings_database_columns($field) {
-  return array(
-    'title' => array('type' => 'varchar', 'length' => 255, 'not null' => true, 'default' => '', 'sortable' => true),
-    'alt' => array('type' => 'varchar', 'length' => 255, 'not null' => true, 'default' => '', 'sortable' => true),
-  );
-}
-
-function imagefield_image_field_settings_filters($field) {
-  return array(
-    'not null' => array(
-      'operator' => array('=' => t('Has Image')),
-      'list' => 'views_handler_operator_yesno',
-      'list-type' => 'select',
-      'handler' => 'imagefield_views_handler_filter_is_not_null',
-    ),
- );
-}
-
-/**
- * Element specific validation for imagefield default value.
- *
- * Validated in a separate function from imagefield_field() to get access
- * to the $form_state variable.
- */
-function _imagefield_field_settings_default_validate($element, &$form_state) {
-  // Verify the destination exists
-  $dst = file_directory_path() .'/imagefield_default_images';
-  if (!field_file_check_directory($dst, FILE_CREATE_DIRECTORY)) {
-    form_set_error('default_image', t("The default image could not be uploaded. The destination(%d) does not exist or is not writable by the webserver.", array('%d' => dirname($dst))));
-    return;
-  }
-
-  $validators = array(
-    'file_validate_is_image' => array(),
-  );
-
-  // We save the upload here because we can't know the correct path until the file is saved.
-  if (!$file = file_save_upload('default_image_upload', $validators, $dst)) {
-    // no upload to save we hope... or file_save_upload reported an error on its own.
-    return;
-  }
-
-  // set new value.
-  $form_state['values']['default_image'] = (array)$file;
-
-  // remove old image & clean up database.
-  if (file_delete(file_create_path($field['default_image']['filepath']))) {
-    db_query('DELETE FROM {files} WHERE fid=%d', $field['default_image']['fid']);
-  }
-}
-
-/**
- * Custom filter for imagefield NOT null.
- */
-function imagefield_views_handler_filter_is_not_null($op, $filter, $filterinfo, &$query) {
-  if ($op == 'handler') {
-    $query->ensure_table($filterinfo['table']);
-    if ($filter['value']) {
-      $qs = '%s.%s > 0';
-      $query->add_where($qs, $filterinfo['table'], $filterinfo['field']);
-    }
-    else {
-      $qs = '%s.%s = 0 OR %s.%s IS null';
-      $query->add_where($qs, $filterinfo['table'], $filterinfo['field'], $filterinfo['table'], $filterinfo['field']);
-    }
-  }
-
-}
Index: imagefield/imagefield.install
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/imagefield/imagefield.install,v
retrieving revision 1.21
diff -u -r1.21 imagefield.install
--- imagefield/imagefield.install	4 Mar 2009 05:28:24 -0000	1.21
+++ imagefield/imagefield.install	10 Mar 2009 18:42:21 -0000
@@ -25,107 +25,101 @@
 }
 
 /**
- * Data is now stored in per-field tables.
+ * Implementation of hook_update_last_removed().
  */
-function imagefield_update_1() {
-  $ret = array();
-
-  include_once(drupal_get_path('module', 'content') .'/content.module');
-  include_once(drupal_get_path('module', 'content') .'/content_admin.inc');
-
+function imagefield_update_last_removed() {
+  return 3;
+}
 
-  $fields = content_fields();
+/**
+ * Upgrade to CCK 2 and Drupal 6.
+ */
+function imagefield_update_6001() {
+  // This update was moved into 6004 so that it can be run again for users
+  // who were not properly updated.
+  return array();
+}
 
-  foreach ($fields as $field) {
-    switch ($field['type']) {
-      case 'file':
-        $columns = array(
-          'list' => array('type' => 'int', 'not null' => TRUE, 'default' => '0'),
-        );
-        content_alter_db_field(array(), array(), $field, $columns);
-        break;
-    }
-  }
-  db_query('DELETE FROM {cache}');
-  return $ret;
+/**
+ * Migrate fields to the new structure.
+ */
+function imagefield_update_6002() {
+  // This update was moved to 6004 so that it can be run again for users
+  // who were not properly updated.
+  return array();
 }
 
 /**
- * Schema change to enable alt and title tags.
+ * Convert image field type to filefield.
  */
-function imagefield_update_2() {
+function imagefield_update_6003() {
   $ret = array();
 
-  include_once(drupal_get_path('module', 'content') .'/content.module');
-  include_once(drupal_get_path('module', 'content') .'/content_admin.inc');
-
-  $fields = content_fields();
-
-  foreach ($fields as $field) {
-    switch ($field['type']) {
-      case 'image':
-        $oldcolumns = array(
-          'fid' => array('type' => 'int', 'not null' => TRUE, 'default' => '0'),
-        );
-        $newcolumns = array(
-          'fid' => array('type' => 'int', 'not null' => TRUE, 'default' => '0'),
-          'title' => array('type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => "''", 'sortable' => TRUE),
-          'alt' => array('type' => 'varchar', 'length' => 255, 'not null' => TRUE, 'default' => "''", 'sortable' => TRUE),
-        );
-        content_alter_db_field($field, $oldcolumns, $field, $newcolumns);
-        break;
-    }
-    drupal_set_message('altered: <br /><pre>'. print_r($field, true) .'</pre>');
+  if (drupal_get_installed_schema_version('filefield', TRUE) < 6001) {
+    $ret['#abort'] = array('success' => FALSE, 'query' => t('FileField must be updated to Drupal 6 before ImageField can be updated.'));
+    return $ret;
   }
 
-  db_query('DELETE FROM {cache}');
+  $ret[] = update_sql("UPDATE {" . content_field_tablename() . "} SET type = 'filefield', module = 'filefield', active = 1 WHERE module = 'imagefield' OR type = 'image'");
+  $ret[] = update_sql("UPDATE {" . content_instance_tablename() . "} SET widget_type = 'imagefield_widget', widget_active = 1 WHERE widget_type = 'image' OR widget_type = 'imagefield_widget'");
+  content_clear_type_cache();
+
   return $ret;
 }
 
 /**
- * Change default formatter key from 'default' to 'imagefield_default'.
+ * Migrate fields to the new structure.
  */
-function imagefield_update_3() {
+function imagefield_update_6004(&$context) {
+  module_load_install('content');
+  module_load_include('inc', 'imagefield', 'imagefield_file');
+  module_load_include('inc', 'content', 'includes/content.admin');
+  module_load_include('inc', 'content', 'includes/content.crud');
+
   $ret = array();
-  include_once(drupal_get_path('module', 'content') .'/content.module');
-  include_once(drupal_get_path('module', 'content') .'/content_admin.inc');
-  $result = db_query("SELECT field_name, type_name, display_settings FROM {node_field_instance} WHERE widget_type = 'image'");
-  while ($row = db_fetch_array($result)) {
-    $needs_update = FALSE;
-    $display_settings = unserialize($row['display_settings']);
-    foreach (_content_admin_display_contexts() as $context) {
-      if ($display_settings[$context]['format'] == 'default') {
-        $display_settings[$context]['format'] = 'imagefield_default';
-        $needs_update = TRUE;
+
+  if (!isset($context['sandbox']['progress'])) {
+    // Get the latest cache values and schema.
+    content_clear_type_cache(TRUE, TRUE);
+
+    // Grab the list of fields to update.
+    $context['sandbox']['fields'] = array();
+    foreach (content_types_install() as $type_name => $fields) {
+      foreach ($fields as $field) {
+        if ($field['type'] == 'filefield' && $field['widget']['type'] == 'imagefield_widget') {
+          // We only process a given field once.
+          $context['sandbox']['fields'][$field['field_name']] = $field;
+        }
       }
     }
-    if ($needs_update) {
-      db_query("UPDATE {node_field_instance} SET display_settings = '%s' WHERE type_name = '%s' AND field_name = '%s'", serialize($display_settings), $row['type_name'], $row['field_name']);
-    }
-  }
-  content_clear_type_cache();
-  return $ret;
-}
 
-/**
- * Upgrade to CCK 2 and Drupal 6.
- */
-function imagefield_update_6001() {
-  // Make sure CCK's been updated first.
-  if ($abort = content_check_update('imagefield')) {
-    return $abort;
-  }
+    if (empty($context['sandbox']['fields'])) {
+      return $ret;
+    }
 
-  module_load_include('inc', 'content', 'includes/content.admin');
-  module_load_include('inc', 'content', 'includes/content.crud');
+    // Add/update the database fields.
+    foreach ($context['sandbox']['fields'] as $field) {
+      $db_info = content_database_info($field);
+
+      // Convert the default value for the FID field to NULL.
+      db_change_field($ret, $db_info['table'], $field['field_name'] . '_fid', $field['field_name'] . '_fid', array('type' => 'int'));
+      $ret[] = update_sql("UPDATE " . $db_info['table'] . " SET " . $field['field_name'] . "_fid = NULL WHERE " . $field['field_name'] . "_fid = 0");
+
+      // Add the "data" and "list" columns to the field if not there already.
+      if (!db_column_exists($db_info['table'], $field['field_name'] . '_list')) {
+        db_add_field($ret, $db_info['table'], $field['field_name'] . '_list', array('type' => 'int', 'size' => 'tiny'));
+      }
+      if (!db_column_exists($db_info['table'], $field['field_name'] . '_data')) {
+        db_add_field($ret, $db_info['table'], $field['field_name'] . '_data', array('type' => 'text'));
+      }
 
-  $ret[] = update_sql("UPDATE {". content_instance_tablename() ."} SET widget_type = 'imagefield_widget' WHERE widget_type = 'image'");
-  content_associate_fields('imagefield');
+      // Set the default state of the global settings.
+      $field['list_field'] = '0';
+      $field['list_default'] = '1';
+      $field['description_field'] = '0';
 
-  foreach (content_types_install() as $type_name => $fields) {
-    foreach ($fields as $field) {
-      // Skip non imagefields.
-      if ($field['type'] != 'image') continue;
+      // Set default state of instance settings.
+      $field['widget']['file_path'] = $field['widget']['image_path'];
 
       // Map 'max_number_images' parameter to CCK 'multiple'.
       if (!empty($field['widget']['multiple']) && isset($field['widget']['max_number_images'])) {
@@ -136,80 +130,34 @@
           $field['multiple'] = 0; // 0 means "Not Multiple" in CCK.
         }
         else {
-          $field['multiple'] == $field['widget']['max_number_images'];
+          $field['multiple'] = $field['widget']['max_number_images'];
         }
       }
-      unset($field['widget']['max_number_images']);
-
-      // Rename image_path to file_path to help convergence with filefield.module.
-      $field['widget']['file_path'] = $field['widget']['image_path'];
 
       // Update format names.
-      $display_settings = array('teaser', 'full', '4');
-      foreach ($display_settings as $context) {
-        switch($field['display_settings'][$context]['format']) {
+      $display_settings = array('teaser', 'full');
+      foreach ($display_settings as $display_context) {
+        switch ($field['display_settings'][$display_context]['format']) {
           case 'imagefield_nodelink':
-            $field['display_settings'][$context]['format'] = 'image_nodelink';
+            $field['display_settings'][$display_context]['format'] = 'image_nodelink';
             break;
           case 'imagefield_imagelink':
-            $field['display_settings'][$context]['format'] = 'image_imagelink';
+            $field['display_settings'][$display_context]['format'] = 'image_imagelink';
             break;
           case 'imagefield_path':
-            $field['display_settings'][$context]['format'] = 'path_plain';
+            $field['display_settings'][$display_context]['format'] = 'path_plain';
             break;
           case 'imagefield_url':
-            $field['display_settings'][$context]['format'] = 'url_plain';
+            $field['display_settings'][$display_context]['format'] = 'url_plain';
             break;
           case 'imagefield_default':
-            $field['display_settings'][$context]['format'] = 'image_plain';
+            $field['display_settings'][$display_context]['format'] = 'image_plain';
             break;
         }
       }
 
-      // Set list options inherited from file field to behave consistently with old imagefield.
-      $field['list_default'] = 1;
-      $field['force_list_default'] = 1;
-
       content_field_instance_update($field);
     }
-  }
-
-  content_clear_type_cache(TRUE);
-
-  return $ret;
-}
-
-/**
- * Migrate fields to the new structure.
- */
-function imagefield_update_6002(&$context) {
-  include_once './'. drupal_get_path('module', 'content') .'/content.install';
-  include_once './'. drupal_get_path('module', 'imagefield') .'/imagefield_file.inc';
-
-  $ret = array();
-
-  if (!isset($context['sandbox']['progress'])) {
-    if ($abort = content_check_update('imagefield')) {
-      return $abort;
-    }
-
-    // Get the latest cache values and schema.
-    content_clear_type_cache(TRUE, TRUE);
-
-    // Grab the list of fields to update.
-    $context['sandbox']['fields'] = array();
-    foreach (content_types_install() as $type_name => $fields) {
-      foreach ($fields as $field) {
-        if ($field['type'] == 'image') {
-          // We only process a given field once.
-          $context['sandbox']['fields'][$field['field_name']] = $field;
-        }
-      }
-    }
-
-    if (empty($context['sandbox']['fields'])) {
-      return $ret;
-    }
 
     $context['sandbox']['progress'] = 0;
     $context['sandbox']['total'] = count($context['sandbox']['fields']);
@@ -223,12 +171,14 @@
   $col_alt = $field['field_name'] .'_alt';
   $col_title = $field['field_name'] .'_title';
   $col_data = $field['field_name'] .'_data';
+  $col_list = $field['field_name'] .'_list';
 
   $limit = 10;
   $result = db_query_range("SELECT * FROM {". $table ."} WHERE vid > %d ORDER BY vid ASC", $context['sandbox']['current_node'], 0, $limit);
   $has_processed = FALSE;
-  while ($row = db_fetch_array($result)) {
 
+  // Loop through each ImageField row and convert its alt and title columns.
+  while ($row = db_fetch_array($result)) {
     // Try to unserialize the data column.
     if (!empty($row[$col_data])) {
       $data = unserialize($row[$col_data]);
@@ -237,12 +187,25 @@
       $data = array();
     }
 
-    // Copy move the values from the columns into the array...
-    $data['alt'] = $row[$col_alt];
-    $data['title'] = $row[$col_title];
+    // Copy the values into the data array.
+    if (isset($row[$col_alt])) {
+      $data['alt'] = $row[$col_alt];
+    }
+    if (isset($row[$col_title])) {
+      $data['title'] = $row[$col_title];
+    }
+    $list = isset($row[$col_list]) ? $row[$col_list] : 1;
+
+    // Depending on if this is multivalue or not, update based on delta.
+    if ($field['multiple'] > 0) {
+      $query = "UPDATE {". $table ."} SET $col_data = '%s', $col_list = %d WHERE vid = %d AND delta = %d";
+    }
+    else {
+      $query = "UPDATE {". $table ."} SET $col_data = '%s', $col_list = %d WHERE vid = %d";
+    }
 
-    // ...serialize it and store it back to the db.
-    db_query("UPDATE {". $table ."} SET $col_data = '%s' WHERE vid = %d", serialize($data), $row['vid']);
+    // Serialize it and store it back to the db.
+    db_query($query, serialize($data), $list, $row['vid'], $row['delta']);
 
     // Create the thumbnail for that image.
     $file = db_fetch_object(db_query("SELECT * FROM {files} WHERE fid = %d", $row[$col_fid]));
Index: imagefield/imagefield.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/imagefield/imagefield.module,v
retrieving revision 1.76
diff -u -r1.76 imagefield.module
--- imagefield/imagefield.module	9 Mar 2009 02:47:20 -0000	1.76
+++ imagefield/imagefield.module	10 Mar 2009 18:42:21 -0000
@@ -14,6 +14,7 @@
 function imagefield_init() {
   // field_file hooks and callbacks.
   module_load_include('inc', 'imagefield', 'imagefield_file');
+  module_load_include('inc', 'imagefield', 'imagefield_widget');
   drupal_add_css(drupal_get_path('module', 'imagefield') .'/imagefield.css');
 }
 
@@ -90,13 +91,11 @@
   $elements['imagefield_widget'] =  array(
     // Indicate to FormAPI that this element needs processing and is not simply a render element.
     '#input' => TRUE,
-    // Specify the return structure of the element (experimental, unused in releases).
-    '#returns' => array('array' => array('fid' => 'int', 'title' => 'string', 'alt' => 'string')),
-    // Delegate element processing to Filefield.
-    '#process' => array('filefield_widget_process', 'imagefield_widget_widget_process'),
+    // Delegate element processing to FileField, then call ImageField.
+    '#process' => array('filefield_widget_process', 'imagefield_widget_process'),
     // See imagefield_widget[#process] documentation.
-    '#value_callback' => 'filefield_widget_value',
-    // Delegate to Filefield...
+    '#value_callback' => 'imagefield_widget_value',
+    // Delegate element validation to FileField, then call ImageField.
     '#element_validate' => array('filefield_widget_validate', 'imagefield_widget_validate'),
     '#description' => t('Changes made to the attachments are not permanent until you save this post.'),
   );
@@ -119,52 +118,6 @@
 }
 
 /**
- * Implementation of CCK's hook_field_info().
- *
- * This is the only field callback that is not delegated to FileField.
- * and is required to make sure CCK sets $field['module'] to imagefield
- * so filefield functions will know how to construct function names and
- * which includes to load.
- */
-function imagefield_field_info() {
-  return array(
-    'image' => array(
-      'label' => t('Image'),
-      'description' => t('Store an image file and optionally text for alt and title tags.'),
-    ),
-  );
-}
-
-/**
- * Implementation of CCK's hook_field_settings.
- *
- * Delegated to filefield_field_settings, which will include
- * imagefield_field.inc and call imagefield_image_field_settings_{$op}.
- * $module_$field_type_$hook_$op.
- */
-function imagefield_field_settings($op, $field) {
-  return filefield_field_settings($op, $field);
-}
-
-/**
- * Implementation of CCK's hook_field().
- *
- * @see: function imagefield_field_settings().
- */
-function imagefield_field($op, $node, $field, &$items, $teaser, $page) {
-  return filefield_field($op, $node, $field, $items, $teaser, $page);
-}
-
-/**
- * implementation of CCK's hook_content_is_empty
- *
- * Delegated to filefield.module.
- */
-function imagefield_content_is_empty($item, $field) {
-  return filefield_content_is_empty($item, $field);
-}
-
-/**
  * Implementation of CCK's hook_widget_info().
  */
 function imagefield_widget_info() {
@@ -172,13 +125,9 @@
   return array(
     'imagefield_widget' => array(
       'label' => t('Image'),
-      'field types' => array('image', 'filefield'),
+      'field types' => array('filefield'),
       'multiple values' => CONTENT_HANDLE_CORE,
       'callbacks' => array('default value' => CONTENT_CALLBACK_CUSTOM),
-      // Callback for dynamic Filefield widgets to determine
-      // if this widget is appropriate for a file type.
-      'suitability callback' => 'imagefield_handles_file',
-      // Description to use on forms to describe this widget.
       'description' => t('An edit widget for image files, including a preview of the image.'),
     ),
   );
@@ -186,11 +135,16 @@
 
 /**
  * Implementation of CCK's hook_widget_settings().
- *
- * Delegated to filefield.
  */
 function imagefield_widget_settings($op, $widget) {
-  return filefield_widget_settings($op, $widget);
+  switch ($op) {
+    case 'form':
+      return imagefield_widget_settings_form($widget);
+    case 'validate':
+      return imagefield_widget_settings_validate($widget);
+    case 'save':
+      return imagefield_widget_settings_save($widget);
+  }
 }
 
 /**
@@ -204,32 +158,27 @@
   if (empty($items[$delta])) {
     $items[$delta] = array('alt' => '', 'title' => '');
   }
-  $element = filefield_widget($form, $form_state, $field, $items, $delta);
 
-  $element['#upload_validators'] += imagefield_widget_upload_validators($field);
+  // Start with the FileField widget as a basic start.
+  $element = module_invoke('filefield', 'widget', $form, $form_state, $field, $items, $delta);
 
-  return $element;
-}
+  // Ensure that only web images are supported.
+  $web_extensions = array('jpg', 'jpeg', 'gif', 'png');
+  $extensions = array_filter(explode(' ', $element['#upload_validators']['filefield_validate_extensions'][0]));
+  if (empty($extensions)) {
+    $extensions = $web_extensions;
+  }
+  $element['#upload_validators']['filefield_validate_extensions'][0] = implode(' ', array_intersect($extensions, $web_extensions));
 
-/**
- * Get the upload validators for an image field.
- *
- * @param $field
- *   CCK Field
- * @return
- *   An array suitable for passing to file_save_upload() or the FileField
- *   element's '#upload_validators' property.
- */
-function imagefield_widget_upload_validators($field) {
-  $validators = array();
-  $validators['filefield_validate_is_image'] = array();
+  // Add validators for resolutions.
   if (!empty($field['widget']['max_resolution']) || !empty($field['widget']['min_resolution'])) {
-    $validators['filefield_validate_image_resolution'] = array(
+    $element['#upload_validators']['filefield_validate_image_resolution'] = array(
       $field['widget']['max_resolution'],
       $field['widget']['min_resolution'],
     );
   }
-  return $validators;
+
+  return $element;
 }
 
 /**
@@ -240,36 +189,31 @@
   $formatters = array(
     'image_plain' => array(
       'label' => t('Image'),
-      'field types' => array('image', 'filefield'),
-      'suitability callback' => 'imagefield_handles_file',
+      'field types' => array('filefield'),
       'css' => array($module_path .'/imagefield.css'),
       'description' => t('Displays image files in their original size.'),
     ),
     'image_nodelink' => array(
       'label' => t('Image linked to node'),
-      'field types' => array('image', 'filefield'),
-      'suitability callback' => 'imagefield_handles_file',
+      'field types' => array('filefield'),
       'css' => array($module_path .'/imagefield.css'),
       'description' => t('Displays image files in their original size.'),
     ),
     'image_imagelink' => array(
       'label' => t('Image linked to file'),
-      'field types' => array('image', 'filefield'),
-      'suitability callback' => 'imagefield_handles_file',
+      'field types' => array('filefield'),
       'css' => array($module_path .'/imagefield.css'),
       'description' => t('Displays image files in their original size.'),
     ),
     'path_plain' => array(
       'label' => t('Path to file'),
-      'field types' => array('image', 'filefield'),
-      'suitability callback' => 'imagefield_handles_file',
+      'field types' => array('filefield'),
       'css' => array($module_path .'/imagefield.css'),
       'description' => t('Displays image files in their original size.'),
     ),
     'url_plain' => array(
       'label' => t('URL to file'),
-      'field types' => array('image', 'filefield'),
-      'suitability callback' => 'imagefield_handles_file',
+      'field types' => array('filefield'),
       'css' => array($module_path .'/imagefield.css'),
       'description' => t('Displays image files in their original size.'),
     ),
@@ -318,10 +262,7 @@
 }
 
 function theme_imagefield_widget_item($element) {
-  return '<div class="imagefield-row clear-block">'.
-            '<div class="imagefield-preview">'. drupal_render($element['preview']) . '</div>' .
-            '<div class="imagefield-edit">' . drupal_render($element) . '</div>' .
-         '</div>';
+  return theme('filefield_widget_item', $element);
 }
 
 function theme_imagefield_admin_thumbnail($item = null) {
@@ -334,32 +275,3 @@
 /**
  * @} End defgroup "Theme Callbacks".
  */
-
-/**
- * A few miscellaneous functions in need of a proper home.
- */
-// CCK's default value callback... doesn't seem to work. I need to figure out what is going on in D6.
-function imagefield_default_value(&$form, &$form_state, $field, $delta) {
-  $items = filefield_default_value($form, $form_state, $field, $delta);
-  foreach($items as $delta => $item) {
-    $items[$delta]['data']['title'] = $field['widget']['title'];
-    $items[$delta]['data']['alt'] = $field['widget']['alt'];
-  }
-  return $items;
-}
-
-// Scale ImageField uploads.
-function _imagefield_scale_image($file, $resolution = 0) {
-  $info = image_get_info($file['filepath']);
-  if ($info) {
-    list($width, $height) = explode('x', $resolution);
-    if ($width && $height) {
-      $result = image_scale($file['filepath'], $file['filepath'], $width, $height);
-      if ($result) {
-        $file['filesize'] = filesize($file['filepath']);
-        drupal_set_message(t('The image %filename was resized to fit within the maximum allowed resolution of %resolution pixels', array('%resolution' => $resolution, '%filename' => $file['filename'])));
-      }
-    }
-  }
-  return $file;
-}
