Issue #1080620 by m4olivei: Incorrect string value error when
[project/filefield.git] / filefield_widget.inc
index 98c971b..eda6d76 100644 (file)
@@ -1,46 +1,58 @@
 <?php
-// $Id$
+
 /**
  * @file
- * FileField: Defines a CCK file field type.
+ * This file contains CCK widget related functionality.
  *
  * Uses content.module to store the fid and field specific metadata,
- * and Drupal's {files} table to store the actual file data.
- *
- * This file contains CCK widget related functionality.
+ * and Drupal's files table to store the actual file data.
  */
 
 /**
- * @file
- *
- * FileField Widget Settings Hooks.
- * @todo: move description property to filefield_widget widget callbacks
- *        (filefield_widget_widget_{$op}).
+ * Implementation of CCK's hook_widget_settings($op == 'form').
  */
-
 function filefield_widget_settings_form($widget) {
   $form = array();
+
+  // Convert the extensions list to be a human-friendly comma-separated list.
+  $extensions = is_string($widget['file_extensions']) ? $widget['file_extensions'] : 'txt';
   $form['file_extensions'] = array(
     '#type' => 'textfield',
     '#title' => t('Permitted upload file extensions'),
-    '#default_value' => is_string($widget['file_extensions']) ? $widget['file_extensions'] : 'txt',
+    '#default_value' => $extensions,
     '#size' => 64,
+    '#maxlength' => 512,
     '#description' => t('Extensions a user can upload to this field. Separate extensions with a space and do not include the leading dot. Leaving this blank will allow users to upload a file with any extension.'),
+    '#element_validate' => array('_filefield_widget_settings_extensions_validate'),
+    '#pre_render' => array('_filefield_widget_settings_extensions_value'),
     '#weight' => 1,
   );
+
+  $form['progress_indicator'] = array(
+    '#type' => 'radios',
+    '#title' => t('Progress indicator'),
+    '#options' => array(
+      'bar' => t('Bar with progress meter'),
+      'throbber' => t('Throbber'),
+    ),
+    '#default_value' => empty($widget['progress_indicator']) ? 'bar' : $widget['progress_indicator'],
+    '#description' => t('Your server supports upload progress capabilities. The "throbber" display does not indicate progress but takes up less room on the form, you may want to use it if you\'ll only be uploading small files or if experiencing problems with the progress bar.'),
+    '#weight' => 5,
+    '#access' => filefield_progress_implementation(),
+  );
+
   $form['path_settings'] = array(
     '#type' => 'fieldset',
     '#title' => t('Path settings'),
-    '#collapsible' => true,
-    '#collapsed' => true,
+    '#collapsible' => TRUE,
+    '#collapsed' => TRUE,
     '#weight' => 6,
   );
-
   $form['path_settings']['file_path'] = array(
     '#type' => 'textfield',
     '#title' => t('File path'),
     '#default_value' => is_string($widget['file_path']) ? $widget['file_path'] : '',
-    '#description' => t('Optional subdirectory within the "%dir" directory where files will be stored. Do not include trailing slash.', array('%dir' => variable_get('file_directory_path', 'files'))),
+    '#description' => t('Optional subdirectory within the "%directory" directory where files will be stored. Do not include preceding or trailing slashes.', array('%directory' => variable_get('file_directory_path', 'files') . '/')),
     '#element_validate' => array('_filefield_widget_settings_file_path_validate'),
     '#suffix' => theme('token_help', 'user'),
   );
@@ -53,19 +65,15 @@ function filefield_widget_settings_form($widget) {
     '#collapsible' => TRUE,
     '#collapsed' => TRUE,
   );
-  // upload validator. @todo: consider replacing with global
-  // && node validate.
   $form['max_filesize']['max_filesize_per_file'] = array(
     '#type' => 'textfield',
     '#title' => t('Maximum upload size per file'),
     '#default_value' => is_string($widget['max_filesize_per_file'])
                         ? $widget['max_filesize_per_file']
                          : '',
-    '#description' => t('Specify the size limit that applies to each file separately. Enter a value like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes) in order to restrict the allowed file size. If you leave this this empty the file sizes will be limited only by PHP\'s maximum post and file upload sizes.'),
+    '#description' => t('Specify the size limit that applies to each file separately. Enter a value like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes) in order to restrict the allowed file size. If you leave this empty the file sizes will be limited only by PHP\'s maximum post and file upload sizes (current limit <strong>%limit</strong>).', array('%limit' => format_size(file_upload_max_size()))),
     '#element_validate' => array('_filefield_widget_settings_max_filesize_per_file_validate'),
   );
-
-  // node validate.
   $form['max_filesize']['max_filesize_per_node'] = array(
     '#type' => 'textfield',
     '#title' => t('Maximum upload size per node'),
@@ -75,27 +83,54 @@ function filefield_widget_settings_form($widget) {
     '#description' => t('Specify the total size limit for all files in field on a given node. Enter a value like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes) in order to restrict the total size of a node. Leave this empty if there should be no size restriction.'),
     '#element_validate' => array('_filefield_widget_settings_max_filesize_per_node_validate'),
   );
+
   return $form;
 }
 
+/**
+ * Implementation of CCK's hook_widget_settings($op == 'save').
+ */
 function filefield_widget_settings_save($widget) {
-  return array(
-    'file_extensions', 'file_path', 'max_filesize_per_file',
-    'max_filesize_per_node', 'file_widgets'
-  );
+  return array('file_extensions', 'file_path', 'progress_indicator', 'max_filesize_per_file', 'max_filesize_per_node');
+}
+
+/**
+ * A FAPI #pre_render() function to set a cosmetic default value for extensions.
+ */
+function _filefield_widget_settings_extensions_value($element) {
+  $element['#value'] = implode(', ', array_filter(explode(' ', str_replace(',', ' ', $element['#value']))));
+  return $element;
+}
+
+/**
+ * A FAPI #element_validate callback to strip commas from extension lists.
+ */
+function _filefield_widget_settings_extensions_validate($element, &$form_state) {
+  // Remove commas and leading dots from file extensions.
+  $value = str_replace(',', ' ', $element['#value']);
+  $value = str_replace(' .', ' ', $value);
+  $value = array_filter(explode(' ', $value));
+  $value = implode(' ', $value);
+  form_set_value($element, $value, $form_state);
 }
 
 function _filefield_widget_settings_file_path_validate($element, &$form_state) {
   // Strip slashes from the beginning and end of $widget['file_path']
   $form_state['values']['file_path'] = trim($form_state['values']['file_path'], '\\/');
+
+  // Do not allow the file path to be the same as the file_directory_path().
+  // This causes all sorts of problems with things like file_create_url().
+  if (strpos($form_state['values']['file_path'], file_directory_path()) === 0) {
+    form_error($element, t('The file path (@file_path) cannot start with the system files directory (@files_directory), as this may cause conflicts when building file URLs.', array('@file_path' => $form_state['values']['file_path'], '@files_directory' => file_directory_path())));
+  }
 }
 
 function _filefield_widget_settings_max_filesize_per_file_validate($element, &$form_state) {
   if (empty($form_state['values']['max_filesize_per_file'])) {
     return; // Empty means no size restrictions, so don't throw an error.
   }
-  else if (!is_numeric(parse_size($form_state['values']['max_filesize_per_file']))) {
-    form_error($element, t('The "Maximum file size for each file" option must contain a valid value. You can either leave the text field empty or enter a string like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes).'));
+  elseif (!is_numeric(parse_size($form_state['values']['max_filesize_per_file']))) {
+    form_error($element, t('The "@field" option must contain a valid value. You can either leave the text field empty or enter a string like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes).', array('@field' => t('Maximum upload size per file'))));
   }
 }
 
@@ -103,210 +138,351 @@ function _filefield_widget_settings_max_filesize_per_node_validate($element, &$f
   if (empty($form_state['values']['max_filesize_per_node'])) {
     return; // Empty means no size restrictions, so don't throw an error.
   }
-  else if (!is_numeric(parse_size($form_state['values']['max_filesize_per_node']))) {
-    form_error($element, t('The "Maximum file size per node" option must contain a valid value. You can either leave the text field empty or enter a string like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes).'));
+  elseif (!is_numeric(parse_size($form_state['values']['max_filesize_per_node']))) {
+    form_error($element, t('The "@field" option must contain a valid value. You can either leave the text field empty or enter a string like "512" (bytes), "80K" (kilobytes) or "50M" (megabytes).', array('@field' => t('Maximum upload size per node'))));
   }
 }
 
 /**
  * Determine the widget's files directory
  *
- * @param $field CCK field
- * @return files directory path.
+ * @param $field
+ *   A CCK field array.
+ * @param $account
+ *   The user account object to calculate the file path for.
+ * @return
+ *   The files directory path, with any tokens replaced.
  */
-function filefield_widget_file_path($field_instance) {
-  $dest = $field_instance['widget']['file_path'];
+function filefield_widget_file_path($field, $account = NULL) {
+  $account = isset($account) ? $account : $GLOBALS['user'];
+  $dest = $field['widget']['file_path'];
+  // Replace user level tokens.
+  // Node level tokens require a lot of complexity like temporary storage
+  // locations when values don't exist. See the filefield_paths module.
   if (module_exists('token')) {
-    global $user;
-    $dest = token_replace($dest, 'user', $user);
+    $dest = token_replace($dest, 'user', $account);
+  }
+  // Replace nasty characters in the path if possible.
+  if (module_exists('transliteration')) {
+    module_load_include('inc', 'transliteration');
+    $dest_array = array_filter(explode('/', $dest));
+    foreach ($dest_array as $key => $directory) {
+      $dest_array[$key] = transliteration_clean_filename($directory);
+    }
+    $dest = implode('/', $dest_array);
   }
 
   return file_directory_path() .'/'. $dest;
 }
 
+/**
+ * Given a FAPI element, save any files that may have been uploaded into it.
+ *
+ * This function should only be called during validate, submit, or
+ * value_callback functions.
+ *
+ * @param $element
+ *   The FAPI element whose values are being saved.
+ */
 function filefield_save_upload($element) {
-  $upload_name = $element['#field_name'] .'_'. $element['#delta'];
-  $field_instance = content_fields($element['#field_name'], $element['#type_name']);
+  $upload_name = implode('_', $element['#array_parents']);
+  $field = content_fields($element['#field_name'], $element['#type_name']);
 
   if (empty($_FILES['files']['name'][$upload_name])) {
     return 0;
   }
 
-  $dest = filefield_widget_file_path($field_instance);
+  $dest = filefield_widget_file_path($field);
   if (!field_file_check_directory($dest, FILE_CREATE_DIRECTORY)) {
-    watchdog('imagefield', 'The upload directory %directory for the file field %field (content type %type) could not be created or is not accessible. A newly uploaded file could not be saved in this directory as a consequence, and the upload was canceled.', array('%directory' => $dest, '%field' => $element['#field_name'], '%type' => $element['#type_name']));
+    watchdog('filefield', 'The upload directory %directory for the file field %field (content type %type) could not be created or is not accessible. A newly uploaded file could not be saved in this directory as a consequence, and the upload was canceled.', array('%directory' => $dest, '%field' => $element['#field_name'], '%type' => $element['#type_name']));
     form_set_error($upload_name, t('The file could not be uploaded.'));
     return 0;
   }
 
   if (!$file = field_file_save_upload($upload_name, $element['#upload_validators'], $dest)) {
-    watchdog('imagefield', 'The file upload failed. %upload', array('%upload' => $upload_name));
-    form_set_error($upload_name, t('The Image upload failed.'));
+    watchdog('filefield', 'The file upload failed. %upload', array('%upload' => $upload_name));
+    form_set_error($upload_name, t('The file in the @field field was unable to be uploaded.', array('@field' => $element['#title'])));
     return 0;
   }
   return $file['fid'];
 }
 
 /**
- * FileField widget element callbacks.
+ * The #value_callback for the filefield_widget type element.
  */
 function filefield_widget_value($element, $edit = FALSE) {
   if (!$edit) {
-    $file =  field_file_load($element['#default_value']['fid']);
+    $file = field_file_load($element['#default_value']['fid']);
     $item = $element['#default_value'];
   }
   else {
-    // uploads take priority over value of fid text field.
+    $item = $edit;
+    $field = content_fields($element['#field_name'], $element['#type_name']);
+
+    // Uploads take priority over value of fid text field.
     if ($fid = filefield_save_upload($element)) {
-      $edit['fid'] = $fid;
+      $item['fid'] = $fid;
+    }
+    // Check for #filefield_value_callback values.
+    // Because FAPI does not allow multiple #value_callback values like it does
+    // for #element_validate and #process, this fills the missing functionality
+    // to allow FileField to be extended purely through FAPI.
+    elseif (isset($element['#filefield_value_callback'])) {
+      foreach ($element['#filefield_value_callback'] as $callback) {
+        $callback($element, $item);
+      }
     }
 
-    // load file.
-    $file = field_file_load($edit['fid']);
-    $item = array(
-      'fid' => $edit['fid'],
-      'list' => !empty($edit['list']),
-      'data' => empty($edit['data']) ? array() : $edit['data'],
-    );
+    // Load file if the FID has changed so that it can be saved by CCK.
+    $file = isset($item['fid']) ? field_file_load($item['fid']) : NULL;
+
+    // If the file entry doesn't exist, don't save anything.
+    if (empty($file)) {
+      $item = array();
+    }
+
+    // Checkboxes loose their value when empty.
+    // If the list field is present make sure its unchecked value is saved.
+    if (!empty($field['list_field']) && empty($edit['list'])) {
+      $item['list'] = 0;
+    }
+  }
+  // Merge file and item data so it is available to all widgets.
+  if (isset($item['data']) && isset($file['data'])) {
+    $file['data'] = array_merge($item['data'], $file['data']);
   }
-  // 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;
 }
 
+/**
+ * An element #process callback for the filefield_widget field type.
+ *
+ * Expands the filefield_widget type to include the upload field, upload and
+ * remove buttons, and the description field.
+ */
 function filefield_widget_process($element, $edit, &$form_state, $form) {
+  static $settings_added;
 
   $item = $element['#value'];
   $field_name = $element['#field_name'];
   $delta = $element['#delta'];
+  $element['#theme'] = 'filefield_widget_item';
 
-  $field = content_fields($element['#field_name'], $element['#type_name']);
+  $field = $form['#field_info'][$field_name];
+
+  // The widget is being presented, so apply the JavaScript.
+  drupal_add_js(drupal_get_path('module', 'filefield') .'/filefield.js');
+  if (!isset($settings_added[$field_name]) && isset($element['#upload_validators']['filefield_validate_extensions'])) {
+    $settings_added[$field_name] = TRUE;
+    $settings = array(
+      'filefield' => array(
+        $field_name => $element['#upload_validators']['filefield_validate_extensions'][0],
+      ),
+    );
+    drupal_add_js($settings, 'setting');
+  }
+
+  // Title is not necessary for each individual field.
+  if ($field['multiple'] > 0) {
+    unset($element['#title']);
+  }
+
+  // Set up the buttons first since we need to check if they were clicked.
+  $element['filefield_upload'] = array(
+    '#type' => 'submit',
+    '#value' => t('Upload'),
+    '#submit' => array('node_form_submit_build_node'),
+    '#ahah' => array( // with JavaScript
+       'path' => 'filefield/ahah/'.   $element['#type_name'] .'/'. $element['#field_name'] .'/'. $element['#delta'],
+       'wrapper' => $element['#id'] .'-ahah-wrapper',
+       'method' => 'replace',
+       'effect' => 'fade',
+    ),
+    '#field_name' => $element['#field_name'],
+    '#delta' => $element['#delta'],
+    '#type_name' => $element['#type_name'],
+    '#upload_validators' => $element['#upload_validators'],
+    '#weight' => 100,
+    '#post' => $element['#post'],
+  );
+  $element['filefield_remove'] = array(
+    // With default CCK edit forms, $element['#parents'] is array($element['#field_name'], $element['#delta']).
+    // However, if some module (for example, flexifield) places our widget deeper in the tree, we want to
+    // use that information in constructing the button name.
+    '#name' => implode('_', $element['#parents']) .'_filefield_remove',
+    '#type' => 'submit',
+    '#value' => t('Remove'),
+    '#submit' => array('node_form_submit_build_node'),
+    '#ahah' => array( // with JavaScript
+      'path' => 'filefield/ahah/'.   $element['#type_name'] .'/'. $element['#field_name'] .'/'. $element['#delta'],
+      'wrapper' => $element['#id'] .'-ahah-wrapper',
+      'method' => 'replace',
+      'effect' => 'fade',
+    ),
+    '#field_name' => $element['#field_name'],
+    '#delta' => $element['#delta'],
+    '#weight' => 101,
+    '#post' => $element['#post'],
+  );
+
+  // Because the output of this field changes depending on the button clicked,
+  // we need to ask FAPI immediately if the remove button was clicked.
+  // It's not good that we call this private function, but
+  // $form_state['clicked_button'] is only available after this #process
+  // callback is finished.
+  if (_form_button_was_clicked($element['filefield_remove'])) {
+    // Delete the file if it is currently unused. Note that field_file_delete()
+    // does a reference check in addition to our basic status check.
+    if (isset($edit['fid'])) {
+      $removed_file = field_file_load($edit['fid']);
+      if ($removed_file['status'] == 0) {
+        field_file_delete($removed_file);
+      }
+    }
+    $item = array('fid' => 0, 'list' => $field['list_default'], 'data' => array('description' => ''));
+  }
+
+  // Set access on the buttons.
+  $element['filefield_upload']['#access'] = empty($item['fid']);
+  $element['filefield_remove']['#access'] = !empty($item['fid']);
+
+  // Add progress bar support to the upload if possible.
+  $progress_indicator = isset($field['widget']['progress_indicator']) ? $field['widget']['progress_indicator'] : 'bar';
+  if ($progress_indicator != 'throbber' && $implementation = filefield_progress_implementation()) {
+    $upload_progress_key = md5(mt_rand());
+
+    if ($implementation == 'uploadprogress') {
+      $element['UPLOAD_IDENTIFIER'] = array(
+        '#type' => 'hidden',
+        '#value' => $upload_progress_key,
+        '#attributes' => array('class' => 'filefield-progress'),
+      );
+    }
+    elseif ($implementation == 'apc') {
+      $element['APC_UPLOAD_PROGRESS'] = array(
+        '#type' => 'hidden',
+        '#value' => $upload_progress_key,
+        '#attributes' => array('class' => 'filefield-progress'),
+      );
+    }
 
-  // check remove buttons...
-  $remove_name = $element['#field_name'] .'_'. $element['#delta'] .'_remove_btn';
-  if (isset($form_state['clicked_button']) && $form_state['clicked_button']['#name'] == $remove_name) {
-    $item = _filefield_default_value($field);
+    // Add the upload progress callback.
+    $element['filefield_upload']['#ahah']['progress']['type'] = 'bar';
+    $element['filefield_upload']['#ahah']['progress']['path'] = 'filefield/progress/' . $upload_progress_key;
   }
 
-  // figute out our fid...
-  $element['fid'] = array('#type' => 'hidden', '#value' =>  $item['fid']);
+  // Set the FID.
+  $element['fid'] = array(
+    '#type' => 'hidden',
+    '#value' => $item['fid'],
+  );
 
   if ($item['fid'] != 0) {
-    $element['preview'] = array('#type' => 'markup', '#value' => theme($element['#type'] .'_preview', $item));
+    $element['preview'] = array(
+      '#type' => 'markup',
+      '#value' => theme('filefield_widget_preview', $item),
+    );
+  }
+
+  // Grant access to temporary files.
+  if ($item['fid'] && isset($item['status']) && $item['status'] == 0 && variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC) == FILE_DOWNLOADS_PRIVATE) {
+    $_SESSION['filefield_access'][] = $item['fid'];
   }
 
   // placeholder.. will be serialized into the data column. this is a place for widgets
   // to put additional data.
-  $element['data'] = array('#tree' => 'true');
+  $element['data'] = array(
+    '#tree' => 'true',
+    '#access' => !empty($item['fid']),
+  );
 
-  if ($field['show_description']) {
+  if (!empty($field['description_field'])) {
     $element['data']['description'] = array(
       '#type' => 'textfield',
       '#title' => t('Description'),
       '#value' => isset($item['data']['description']) ? $item['data']['description'] : '',
+      '#type' => variable_get('filefield_description_type', 'textfield'),
+      '#maxlength' => variable_get('filefield_description_length', 128),
     );
   }
 
-  if ($field['force_list_default']) {
+  if (!empty($field['list_field'])) {
     $element['list'] = array(
-      '#type' => 'hidden',
-      '#value' => $field['list_default'],
+      '#type' => empty($item['fid']) ? 'hidden' : 'checkbox',
+      '#title' => t('List'),
+      '#value' => isset($item['list']) && !empty($item['fid']) ? $item['list'] : $field['list_default'],
+      '#attributes' => array('class' => 'filefield-list'),
     );
   }
   else {
     $element['list'] = array(
-      '#type' => 'checkbox',
-      '#title' => t('List'),
-      '#value' => isset($item['list']) ? $item['list'] : $field['list_default'],
-      '#attributes' => array('class' => 'filefield-list'),
+      '#type' => 'hidden',
+      '#value' => '1',
     );
   }
 
-
-
   foreach ($element['#upload_validators'] as $callback => $arguments) {
     $help_func = $callback .'_help';
-    $desc[] = call_user_func_array($help_func, $arguments);
+    if (function_exists($help_func)) {
+      $desc[] = call_user_func_array($help_func, $arguments);
+    }
   }
+
   $element['upload'] = array(
-    '#name' => 'files['. $element['#field_name'] .'_'. $element['#delta'] .']',
+    '#name' => 'files[' . implode('_', $element['#array_parents']) . ']',
     '#type' => 'file',
-    '#title' => t('New Upload'),
     '#description' => implode('<br />', $desc),
-    '#attributes' => array(
-      'accept' => implode(',', array_filter(explode(' ', $field['widget']['file_extensions']))),
-    )
+    '#size' => 22,
+    '#access' => empty($item['fid']),
   );
 
-
-  if ($item['fid'] != 0) {
-    $element['upload']['#title'] = t('Replace');
-  }
-
-  $element['#prefix'] = '<div id="'. $element['#id'] .'-ahah-wrapper" class="filefield-ahah-wrapper">';
+  $element['#attributes']['id'] = $element['#id'] .'-ahah-wrapper';
+  $element['#prefix'] = '<div '. drupal_attributes($element['#attributes']) .'>';
   $element['#suffix'] = '</div>';
-  $element['upload_btn'] = array(
-    '#type' => 'submit',
-    '#value' => t('Upload'),
-    '#process' => array('form_expand_ahah'),
-    '#submit' => array('node_form_submit_build_node'),
-    '#ahah' => array( // with JavaScript
-       'path' => 'filefield/ahah/'.   $element['#type_name'] .'/'. $element['#field_name'] .'/'. $element['#delta'],
-       'wrapper' => $element['#id'] .'-ahah-wrapper',
-       'method' => 'replace',
-       'effect' => 'fade',
-    ),
-    '#field_name' => $element['#field_name'],
-    '#delta' => $element['#delta'],
-    '#type_name' => $element['#type_name'],
-    '#upload_validators' => $element['#upload_validators'],
-  );
-  if ($item['fid'] != 0) {
-    $element['remove_btn'] = array(
-      '#name' => $element['#field_name'] .'_'. $element['#delta'] .'_remove_btn',
-      '#type' => 'submit',
-      '#value' => t('Remove'),
-      '#process' => array('filefield_widget_process_remove_btn', 'form_expand_ahah'),
-      '#submit' => array('node_form_submit_build_node'),
-      '#ahah' => array( // with JavaScript
-        'path' => 'filefield/ahah/'.   $element['#type_name'] .'/'. $element['#field_name'] .'/'. $element['#delta'],
-        'wrapper' => $element['#id'] .'-wrapper',
-        'method' => 'replace',
-        'effect' => 'fade',
-      ),
-      '#field_name' => $element['#field_name'],
-      '#delta' => $element['#delta'],
-    );
-  }
-
 
   return $element;
 }
 
-function filefield_widget_validate($element, &$form_state) {
-
-}
-
-function _filefield_widget_validate($element, &$form_state) {
+/**
+ * An #element_validate callback for the filefield_widget field.
+ */
+function filefield_widget_validate(&$element, &$form_state) {
+  // If referencing an existing file, only allow if there are existing
+  // references. This prevents unmanaged files (outside of FileField) from
+  // being deleted if this node were to be deleted.
+  if (!empty($element['fid']['#value'])) {
+    $field = content_fields($element['#field_name'], $element['#type_name']);
+    if ($file = field_file_load($element['fid']['#value'])) {
+      $file = (object) $file;
+      if ($file->status == FILE_STATUS_PERMANENT) {
+        if (field_file_references($file) == 0) {
+          form_error($element, t('Referencing to the file used in the %field field is not allowed.', array('%field' => $element['#title'])));
+        }
+      }
+    }
+    else {
+      form_error($element, t('The file referenced by the %field field does not exist.', array('%field' => $element['#title'])));
+    }
+  }
 }
 
-
 /**
  * FormAPI theme function. Theme the output of an image field.
  */
 function theme_filefield_widget($element) {
+  $element['#id'] .= '-upload'; // Link the label to the upload field.
   return theme('form_element', $element, $element['#children']);
 }
 
 function theme_filefield_widget_preview($item) {
-  return '<div class="filefield-preview clear-block">'.
+  // Remove the current description so that we get the filename as the link.
+  if (isset($item['data']['description'])) {
+    unset($item['data']['description']);
+  }
+
+  return '<div class="filefield-file-info">'.
            '<div class="filename">'. theme('filefield_file', $item) .'</div>'.
            '<div class="filesize">'. format_size($item['filesize']) .'</div>'.
            '<div class="filemime">'. $item['filemime'] .'</div>'.
@@ -314,39 +490,81 @@ function theme_filefield_widget_preview($item) {
 }
 
 function theme_filefield_widget_item($element) {
-  return '<div class="filefield-row clear-block">'.
-              '<div class="filefield-filename clear-block">'. drupal_render($element['preview']) . '</div>'.
-              '<div class="fielfield-edit clear-block">'.
-                '<div class="filefield-list">'. drupal_render($element['list']) . '</div>' .
-                '<div class="filefield-description">'. drupal_render($element['description']) . '</div>' .
-                '<div class="filefield-stuff">'. drupal_render($element) .'</div>'.
-              '</div>'.
-         '</div>';
+  // Put the upload button directly after the upload field.
+  $element['upload']['#field_suffix'] = drupal_render($element['filefield_upload']);
+  $element['upload']['#theme'] = 'filefield_widget_file';
+
+  $output = '';
+  $output .= '<div class="filefield-element clear-block">';
+
+  if ($element['fid']['#value'] != 0) {
+    $output .= '<div class="widget-preview">';
+    $output .= drupal_render($element['preview']);
+    $output .= '</div>';
+  }
+
+  $output .= '<div class="widget-edit">';
+  $output .=  drupal_render($element);
+  $output .= '</div>';
+  $output .= '</div>';
+
+  return $output;
 }
 
 /**
- * #require validation for filetype fields.
+ * Custom theme function for FileField upload elements.
+ *
+ * This function allows us to put the "Upload" button immediately after the
+ * file upload field by respecting the #field_suffix property.
  */
+function theme_filefield_widget_file($element) {
+  $output = '';
+
+  $output .= '<div class="filefield-upload clear-block">';
+
+  if (isset($element['#field_prefix'])) {
+    $output .= $element['#field_prefix'];
+  }
+
+  _form_set_class($element, array('form-file'));
+  $output .= '<input type="file" name="'. $element['#name'] .'"'. ($element['#attributes'] ? ' '. drupal_attributes($element['#attributes']) : '') .' id="'. $element['#id'] .'" size="'. $element['#size'] ."\" />\n";
+
+  if (isset($element['#field_suffix'])) {
+    $output .= $element['#field_suffix'];
+  }
 
+  $output .= '</div>';
+
+  return theme('form_element', $element, $output);
+}
+
+/**
+ * Additional #validate handler for the node form.
+ *
+ * This function checks the #required properties on file fields and calculates
+ * node upload totals for all file fields. The #required property is not
+ * properly supported on file fields by Drupal core, so we do this manually.
+ */
 function filefield_node_form_validate($form, &$form_state) {
-  $type = content_types($form['type']['#value']);
-  foreach ($type['fields'] as $field_name => $field) {
-    if (!(in_array($field['module'], array('imagefield', 'filefield'))) || !$field['required']) continue;
+  foreach ($form['#field_info'] as $field_name => $field) {
+    if (!(in_array($field['module'], array('imagefield', 'filefield')))) continue;
     $empty = $field['module'] .'_content_is_empty';
-    $valid = false;
+    $valid = FALSE;
     $total_filesize = 0;
-    foreach($form_state['values'][$field_name] as $delta => $item) {
-      if ($empty($item, $field)) continue;
-      $valid = true;
-      $total_filesize += (int)$item['filesize'];
+    if (!empty($form_state['values'][$field_name])) {
+      foreach ($form_state['values'][$field_name] as $delta => $item) {
+        if ($empty($item, $field)) continue;
+        $valid = TRUE;
+        $total_filesize += (int)$item['filesize'];
+      }
     }
 
-    if (!$valid) {
+    if (!$valid && $field['required'] && filefield_edit_access($field['type_name'], $field_name)) {
       form_set_error($field_name, t('%title field is required.', array('%title' => $field['widget']['label'])));
     }
     $max_filesize = parse_size($field['widget']['max_filesize_per_node']);
     if ($max_filesize && $total_filesize > $max_filesize) {
-      form_set_error($field_name, t('Total filesize for %title, %tsize,  exceeds field settings of %msize.',
+      form_set_error($field_name, t('Total filesize for %title, %tsize, exceeds field settings of %msize.',
                                     array(
                                       '%title' => $field['widget']['label'],
                                       '%tsize' => format_size($total_filesize),
@@ -356,11 +574,3 @@ function filefield_node_form_validate($form, &$form_state) {
     }
   }
 }
-
-function filefield_node_form_submit($form, $form_state) {
-  // we ignore all but the save button here.
-  if ($form_state['values']['op'] != t('Save')) {
-    return;
-  }
-  // @todo:  try to delete removed files.
-}