diff --git modules/image/image.field.inc modules/image/image.field.inc index 07cc1e0..ea258e2 100644 --- modules/image/image.field.inc +++ modules/image/image.field.inc @@ -49,7 +49,8 @@ function image_field_settings_form($field, $instance) { '#options' => $scheme_options, '#default_value' => $settings['uri_scheme'], '#description' => t('Select where the final files should be stored. Private file storage has significantly more overhead than public files, but allows restricted access to files within this field.'), - ); + ) + element_info('radios'); + array_unshift($form['uri_scheme']['#process'], 'image_field_settings_uri_scheme_process'); $form['default_image'] = array( '#title' => t('Default image'), @@ -555,3 +556,20 @@ function theme_image_formatter($variables) { return $output; } + +/** + * An element #process callback for the image field settings form. + * + * Makes sure that the correct upload location is set. + */ +function image_field_settings_uri_scheme_process($element, $form_state, &$complete_form) { + $array_parents = $element['#array_parents']; + // Remove the uri_scheme key and add default_image instead. + $array_parents[count($array_parents) - 1] = 'default_image'; + $default_image_element = &$complete_form; + foreach ($array_parents as $key) { + $default_image_element = &$default_image_element[$key]; + } + $default_image_element['#upload_location'] = $element['#value'] . '://default_image'; + return $element; +} diff --git modules/image/image.module modules/image/image.module index d2d081c..0fd5bd0 100644 --- modules/image/image.module +++ modules/image/image.module @@ -413,6 +413,47 @@ function image_image_style_delete($style) { } /** + * Implements hook_field_delete_field(). + */ +function image_field_delete_field($field) { + if ($field['type'] != 'image') { + return; + } + + // The value of a managed_file element can be an array if #extended == TRUE. + $fid = (is_array($field['settings']['default_image']) ? $field['settings']['default_image']['fid'] : $field['settings']['default_image']); + if ($fid && ($file = file_load($fid))) { + file_usage_delete($file, 'image', 'default_image', $field['id']); + } +} + +/** + * Implements hook_field_update_field(). + */ +function image_field_update_field($field, $prior_field, $has_data) { + if ($field['type'] != 'image') { + return; + } + + // The value of a managed_file element can be an array if #extended == TRUE. + $fid_new = (is_array($field['settings']['default_image']) ? $field['settings']['default_image']['fid'] : $field['settings']['default_image']); + $fid_old = (is_array($prior_field['settings']['default_image']) ? $prior_field['settings']['default_image']['fid'] : $prior_field['settings']['default_image']); + if ($fid_new != $fid_old) { + if ($fid_new && ($file_new = file_load($fid_new))) { + if ($file_new->status != FILE_STATUS_PERMANENT) { + $file_new->status = FILE_STATUS_PERMANENT; + file_save($file_new); + } + file_usage_add($file_new, 'image', 'default_image', $field['id']); + } + + if ($fid_old && ($file_old = file_load($fid_old))) { + file_usage_delete($file_old, 'image', 'default_image', $field['id']); + } + } +} + +/** * Clear cached versions of a specific file in all styles. * * @param $path diff --git modules/image/image.test modules/image/image.test index 00f79d8..b5b9fa6 100644 --- modules/image/image.test +++ modules/image/image.test @@ -796,7 +796,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase { // that would be used on the image field. $this->assertNoPattern('
', t('No image displayed when no image is attached and no default image specified.')); - // Add a default image to the imagefield instance. + // Add a default image to the public imagefield instance. $images = $this->drupalGetTestFiles('image'); $edit = array( 'files[field_settings_default_image]' => drupal_realpath($images[0]->uri), @@ -806,6 +806,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase { field_info_cache_clear(); $field = field_info_field($field_name); $image = file_load($field['settings']['default_image']); + $this->assertTrue($image->status == FILE_STATUS_PERMANENT, t('The default image status is permanent.')); $default_output = theme('image', array('path' => $image->uri)); $this->drupalGet('node/' . $node->nid); $this->assertRaw($default_output, t('Default image displayed when no user supplied image is present.')); @@ -831,6 +832,28 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase { field_info_cache_clear(); $field = field_info_field($field_name); $this->assertFalse($field['settings']['default_image'], t('Default image removed from field.')); + + // Create an image field that uses the private:// scheme and test that the + // default image works as expected. + // Create a private image field to test that other schemas work. + $private_field_name = strtolower($this->randomName()); + $this->createImageField($private_field_name, 'article', array('uri_scheme' => 'private')); + // Add a default image to the new field. + $edit = array( + 'files[field_settings_default_image]' => drupal_realpath($images[1]->uri), + ); + $this->drupalPost('admin/structure/types/manage/article/fields/' . $private_field_name, $edit, t('Save settings')); + $private_field = field_info_field($private_field_name); + $image = file_load($private_field['settings']['default_image']); + $this->assertEqual('private', file_uri_scheme($image->uri), t('Default image uses private:// scheme.')); + $this->assertTrue($image->status == FILE_STATUS_PERMANENT, t('The default image status is permanent.')); + + // Create a new node with no image attached an ensure that the default + // image is displayed when using the private:// scheme. + $node = $this->drupalCreateNode(array('type' => 'article')); + $default_output = theme('image', array('path' => $image->uri)); + $this->drupalGet('node/' . $node->nid); + $this->assertRaw($default_output, t('Default image displayed when no user supplied image is present using private:// scheme.')); } }