';
$output .= t('original') . ' (' . l(t('view actual size'), $original_url) . ')';
$output .= '
'; // End preview-image.
@@ -807,7 +807,7 @@ function theme_image_style_preview($variables) {
$output .= '
';
$output .= check_plain($style['name']) . ' (' . l(t('view actual size'), file_create_url($preview_file) . '?' . time()) . ')';
$output .= '
'; // End preview-image.
diff --git a/core/modules/image/image.field.inc b/core/modules/image/image.field.inc
index c3ac1d561f0f9a8d3c386dcb61b217d4784e83f6..2e9918932fa0f780ea7aebc3d838fd31aa45efbb 100644
--- a/core/modules/image/image.field.inc
+++ b/core/modules/image/image.field.inc
@@ -360,7 +360,7 @@ function image_field_widget_process($element, &$form_state, $form) {
if ($element['#file'] && $widget_settings['preview_image_style']) {
$variables = array(
'style_name' => $widget_settings['preview_image_style'],
- 'path' => $element['#file']->uri,
+ 'uri' => $element['#file']->uri,
);
// Determine image dimensions.
@@ -569,33 +569,24 @@ function image_field_formatter_view($entity_type, $entity, $field, $instance, $l
* An associative array containing:
* - item: An array of image data.
* - image_style: An optional image style.
- * - path: An array containing the link 'path' and link 'options'.
+ * - path: An optional array containing the link 'path' and link 'options'.
*
* @ingroup themeable
*/
function theme_image_formatter($variables) {
$item = $variables['item'];
- $image = array(
- 'path' => $item['uri'],
- 'alt' => $item['alt'],
- );
-
- if (isset($item['width']) && isset($item['height'])) {
- $image['width'] = $item['width'];
- $image['height'] = $item['height'];
- }
// Do not output an empty 'title' attribute.
- if (drupal_strlen($item['title']) > 0) {
- $image['title'] = $item['title'];
+ if (drupal_strlen($item['title']) == 0) {
+ unset($item['title']);
}
if ($variables['image_style']) {
- $image['style_name'] = $variables['image_style'];
- $output = theme('image_style', $image);
+ $item['style_name'] = $variables['image_style'];
+ $output = theme('image_style', $item);
}
else {
- $output = theme('image', $image);
+ $output = theme('image', $item);
}
if (!empty($variables['path']['path'])) {
diff --git a/core/modules/image/image.module b/core/modules/image/image.module
index 54d98ebaeff94108e37dc122cf26362878c7af0d..5cdd9c52bd34a8bfe2bfdca4b2d2b08e91df749e 100644
--- a/core/modules/image/image.module
+++ b/core/modules/image/image.module
@@ -876,15 +876,15 @@ function image_style_flush($style) {
*
* @param $style_name
* The name of the style to be used with this image.
- * @param $path
+ * @param $uri
* The path to the image.
* @return
* The absolute URL where a style image can be downloaded, suitable for use
* in an
tag. Requesting the URL will cause the image to be created.
* @see image_style_deliver()
*/
-function image_style_url($style_name, $path) {
- $uri = image_style_path($style_name, $path);
+function image_style_url($style_name, $uri) {
+ $uri = image_style_path($style_name, $uri);
// If not using clean URLs, the image derivative callback is only available
// with the query string. If the file does not exist, use url() to ensure
@@ -1162,7 +1162,7 @@ function image_effect_apply($image, $effect) {
* @param $variables
* An associative array containing:
* - style_name: The name of the style to be used to alter the original image.
- * - path: The path of the image file relative to the Drupal files directory.
+ * - uri: The path of the image file relative to the Drupal files directory.
* This function does not work with images outside the files directory nor
* with remotely hosted images.
* - width: The width of the source image (if known).
@@ -1187,7 +1187,7 @@ function theme_image_style($variables) {
$variables['height'] = $dimensions['height'];
// Determine the url for the styled image.
- $variables['path'] = image_style_url($variables['style_name'], $variables['path']);
+ $variables['uri'] = image_style_url($variables['style_name'], $variables['uri']);
return theme('image', $variables);
}
diff --git a/core/modules/image/image.test b/core/modules/image/image.test
index a29b4f3a1d7fbacd3b3f6956e47675fb36de941b..70f69e72719028fafc2b875290a976f45373754c 100644
--- a/core/modules/image/image.test
+++ b/core/modules/image/image.test
@@ -666,7 +666,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
// Test that the default formatter is being used.
$image_uri = $node->{$field_name}[LANGUAGE_NONE][0]['uri'];
$image_info = array(
- 'path' => $image_uri,
+ 'uri' => $image_uri,
'width' => 40,
'height' => 20,
);
@@ -713,7 +713,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
// Ensure the derivative image is generated so we do not have to deal with
// image style callback paths.
$this->drupalGet(image_style_url('thumbnail', $image_uri));
- $image_info['path'] = image_style_path('thumbnail', $image_uri);
+ $image_info['uri'] = image_style_path('thumbnail', $image_uri);
$image_info['width'] = 100;
$image_info['height'] = 50;
$default_output = theme('image', $image_info);
@@ -764,7 +764,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
// style.
$node = node_load($nid, NULL, TRUE);
$image_info = array(
- 'path' => image_style_url('medium', $node->{$field_name}[LANGUAGE_NONE][0]['uri']),
+ 'uri' => image_style_url('medium', $node->{$field_name}[LANGUAGE_NONE][0]['uri']),
'width' => 220,
'height' => 110,
);
@@ -773,7 +773,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
// Add alt/title fields to the image and verify that they are displayed.
$image_info = array(
- 'path' => $node->{$field_name}[LANGUAGE_NONE][0]['uri'],
+ 'uri' => $node->{$field_name}[LANGUAGE_NONE][0]['uri'],
'alt' => $this->randomName(),
'title' => $this->randomName(),
'width' => 40,
@@ -815,7 +815,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
$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));
+ $default_output = theme('image', array('uri' => $image->uri));
$this->drupalGet('node/' . $node->nid);
$this->assertRaw($default_output, t('Default image displayed when no user supplied image is present.'));
@@ -824,7 +824,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
$nid = $this->uploadNodeImage($images[1], $field_name, 'article');
$node = node_load($nid, NULL, TRUE);
$image_info = array(
- 'path' => $node->{$field_name}[LANGUAGE_NONE][0]['uri'],
+ 'uri' => $node->{$field_name}[LANGUAGE_NONE][0]['uri'],
'width' => 40,
'height' => 20,
);
@@ -858,7 +858,7 @@ class ImageFieldDisplayTestCase extends ImageFieldTestCase {
// Create a new node with no image attached and ensure that default private
// image is displayed.
$node = $this->drupalCreateNode(array('type' => 'article'));
- $default_output = theme('image', array('path' => $image->uri));
+ $default_output = theme('image', array('uri' => $image->uri));
$this->drupalGet('node/' . $node->nid);
$this->assertRaw($default_output, t('Default private image displayed when no user supplied image is present.'));
}
@@ -945,7 +945,7 @@ class ImageDimensionsUnitTest extends DrupalWebTestCase {
$variables = array(
'style_name' => 'test',
- 'path' => $original_uri,
+ 'uri' => $original_uri,
'width' => 40,
'height' => 20,
);
diff --git a/core/modules/simpletest/simpletest.pages.inc b/core/modules/simpletest/simpletest.pages.inc
index e65c4d4b727ff0b178027c25475e54195eb4ed59..d1891596ab0bf84764ef4f8aa3bceb704225328b 100644
--- a/core/modules/simpletest/simpletest.pages.inc
+++ b/core/modules/simpletest/simpletest.pages.inc
@@ -82,8 +82,8 @@ function theme_simpletest_test_table($variables) {
// Define the images used to expand/collapse the test groups.
$js = array(
'images' => array(
- theme('image', array('path' => 'core/misc/menu-collapsed.png', 'width' => 7, 'height' => 7, 'alt' => t('Expand'), 'title' => t('Expand'))) . '
(' . t('Expand') . ')',
- theme('image', array('path' => 'core/misc/menu-expanded.png', 'width' => 7, 'height' => 7, 'alt' => t('Collapse'), 'title' => t('Collapse'))) . '
(' . t('Collapse') . ')',
+ theme('image', array('uri' => 'core/misc/menu-collapsed.png', 'width' => 7, 'height' => 7, 'alt' => t('Expand'), 'title' => t('Expand'))) . '
(' . t('Expand') . ')',
+ theme('image', array('uri' => 'core/misc/menu-expanded.png', 'width' => 7, 'height' => 7, 'alt' => t('Collapse'), 'title' => t('Collapse'))) . '
(' . t('Collapse') . ')',
),
);
@@ -414,10 +414,10 @@ function simpletest_result_status_image($status) {
if (!isset($map)) {
$map = array(
- 'pass' => theme('image', array('path' => 'core/misc/watchdog-ok.png', 'width' => 18, 'height' => 18, 'alt' => t('Pass'))),
- 'fail' => theme('image', array('path' => 'core/misc/watchdog-error.png', 'width' => 18, 'height' => 18, 'alt' => t('Fail'))),
- 'exception' => theme('image', array('path' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('Exception'))),
- 'debug' => theme('image', array('path' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('Debug'))),
+ 'pass' => theme('image', array('uri' => 'core/misc/watchdog-ok.png', 'width' => 18, 'height' => 18, 'alt' => t('Pass'))),
+ 'fail' => theme('image', array('uri' => 'core/misc/watchdog-error.png', 'width' => 18, 'height' => 18, 'alt' => t('Fail'))),
+ 'exception' => theme('image', array('uri' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('Exception'))),
+ 'debug' => theme('image', array('uri' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('Debug'))),
);
}
if (isset($map[$status])) {
diff --git a/core/modules/system/system.admin.inc b/core/modules/system/system.admin.inc
index aa2f98fb7601aba85993c308d8cf3db1cb126b33..97a4673dd69ec2cbf53bb3beb05b637b58606217 100644
--- a/core/modules/system/system.admin.inc
+++ b/core/modules/system/system.admin.inc
@@ -154,7 +154,7 @@ function system_themes_page() {
foreach (array_reverse($theme_keys) as $theme_key) {
if (isset($themes[$theme_key]) && file_exists($themes[$theme_key]->info['screenshot'])) {
$theme->screenshot = array(
- 'path' => $themes[$theme_key]->info['screenshot'],
+ 'uri' => $themes[$theme_key]->info['screenshot'],
'alt' => t('Screenshot for !theme theme', array('!theme' => $theme->info['name'])),
'title' => t('Screenshot for !theme theme', array('!theme' => $theme->info['name'])),
'attributes' => array('class' => array('screenshot')),
@@ -1003,7 +1003,7 @@ function _system_modules_build_row($info, $extra) {
}
else {
$form['enable'] = array(
- '#markup' => theme('image', array('path' => 'core/misc/watchdog-error.png', 'alt' => $status_short, 'title' => $status_short)),
+ '#markup' => theme('image', array('uri' => 'core/misc/watchdog-error.png', 'alt' => $status_short, 'title' => $status_short)),
);
$form['description']['#markup'] .= theme('system_modules_incompatible', array('message' => $status_long));
}
diff --git a/core/modules/update/update.report.inc b/core/modules/update/update.report.inc
index 02150e9b71782c8fbeb7f1ecf1d81529958e02af..7703503a770c63e85386871392a3bd500cdd66f9 100644
--- a/core/modules/update/update.report.inc
+++ b/core/modules/update/update.report.inc
@@ -57,25 +57,25 @@ function theme_update_report($variables) {
switch ($project['status']) {
case UPDATE_CURRENT:
$class = 'ok';
- $icon = theme('image', array('path' => 'core/misc/watchdog-ok.png', 'width' => 18, 'height' => 18, 'alt' => t('ok'), 'title' => t('ok')));
+ $icon = theme('image', array('uri' => 'core/misc/watchdog-ok.png', 'width' => 18, 'height' => 18, 'alt' => t('ok'), 'title' => t('ok')));
break;
case UPDATE_UNKNOWN:
case UPDATE_FETCH_PENDING:
case UPDATE_NOT_FETCHED:
$class = 'unknown';
- $icon = theme('image', array('path' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('warning'), 'title' => t('warning')));
+ $icon = theme('image', array('uri' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('warning'), 'title' => t('warning')));
break;
case UPDATE_NOT_SECURE:
case UPDATE_REVOKED:
case UPDATE_NOT_SUPPORTED:
$class = 'error';
- $icon = theme('image', array('path' => 'core/misc/watchdog-error.png', 'width' => 18, 'height' => 18, 'alt' => t('error'), 'title' => t('error')));
+ $icon = theme('image', array('uri' => 'core/misc/watchdog-error.png', 'width' => 18, 'height' => 18, 'alt' => t('error'), 'title' => t('error')));
break;
case UPDATE_NOT_CHECKED:
case UPDATE_NOT_CURRENT:
default:
$class = 'warning';
- $icon = theme('image', array('path' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('warning'), 'title' => t('warning')));
+ $icon = theme('image', array('uri' => 'core/misc/watchdog-warning.png', 'width' => 18, 'height' => 18, 'alt' => t('warning'), 'title' => t('warning')));
break;
}
diff --git a/core/modules/user/user.module b/core/modules/user/user.module
index 703459d21ff9f1ab923aa879996d5be48f9244bb..6f1bc3c45a6f98ce3b1360c190726fd8ca824212 100644
--- a/core/modules/user/user.module
+++ b/core/modules/user/user.module
@@ -1411,10 +1411,10 @@ function template_preprocess_user_picture(&$variables) {
// If the image does not have a valid Drupal scheme (for eg. HTTP),
// don't load image styles.
if (module_exists('image') && file_valid_uri($filepath) && $style = variable_get('user_picture_style', '')) {
- $variables['user_picture'] = theme('image_style', array('style_name' => $style, 'path' => $filepath, 'alt' => $alt, 'title' => $alt));
+ $variables['user_picture'] = theme('image_style', array('style_name' => $style, 'uri' => $filepath, 'alt' => $alt, 'title' => $alt));
}
else {
- $variables['user_picture'] = theme('image', array('path' => $filepath, 'alt' => $alt, 'title' => $alt));
+ $variables['user_picture'] = theme('image', array('uri' => $filepath, 'alt' => $alt, 'title' => $alt));
}
if (!empty($account->uid) && user_access('access user profiles')) {
$attributes = array('attributes' => array('title' => t('View user profile.')), 'html' => TRUE);
diff --git a/core/modules/user/user.test b/core/modules/user/user.test
index 15c427b6e8987f0c69b799830a01c8ca7dbb84c0..66c4903ecf4b2fef0fd3fd0e56742a154acaf6d4 100644
--- a/core/modules/user/user.test
+++ b/core/modules/user/user.test
@@ -1077,7 +1077,7 @@ class UserPictureTestCase extends DrupalWebTestCase {
// Get the user picture image via xpath.
$elements = $this->xpath('//div[@class="user-picture"]/img');
$this->assertEqual(count($elements), 1, t("There is exactly one user picture on the user's profile page"));
- $this->assertEqual($pic_path, (string) $elements[0]['src'], t("User picture source is correct."));
+ $this->assertEqual($pic_path, (string) $elements[0]['src'], t("User picture source is correct: " . $pic_path . " " . print_r($elements, TRUE)));
}
function saveUserPicture($image) {
diff --git a/core/themes/seven/template.php b/core/themes/seven/template.php
index b1073b7f19480f818d0805eae811bdbe23b28845..9804d4de266a42608ea07f851b85b8ab8fcd9776 100644
--- a/core/themes/seven/template.php
+++ b/core/themes/seven/template.php
@@ -88,10 +88,10 @@ function seven_tablesort_indicator($variables) {
$style = $variables['style'];
$theme_path = drupal_get_path('theme', 'seven');
if ($style == 'asc') {
- return theme('image', array('path' => $theme_path . '/images/arrow-asc.png', 'alt' => t('sort ascending'), 'width' => 13, 'height' => 13, 'title' => t('sort ascending')));
+ return theme('image', array('uri' => $theme_path . '/images/arrow-asc.png', 'alt' => t('sort ascending'), 'width' => 13, 'height' => 13, 'title' => t('sort ascending')));
}
else {
- return theme('image', array('path' => $theme_path . '/images/arrow-desc.png', 'alt' => t('sort descending'), 'width' => 13, 'height' => 13, 'title' => t('sort descending')));
+ return theme('image', array('uri' => $theme_path . '/images/arrow-desc.png', 'alt' => t('sort descending'), 'width' => 13, 'height' => 13, 'title' => t('sort descending')));
}
}