diff options
| author | Dries Buytaert <dries@buytaert.net> | 2008-06-06 01:50:20 +0000 |
|---|---|---|
| committer | Dries Buytaert <dries@buytaert.net> | 2008-06-06 01:50:20 +0000 |
| commit | 4f1af19d101c1940235850a7ea67a0ee7af85880 (patch) | |
| tree | 2c969805cabfbdf49d5b6b1fda1dc89477ffb092 | |
| parent | b87b7c2d01f1200d65311fce4b249cbd27f0bab9 (diff) | |
| download | brdo-4f1af19d101c1940235850a7ea67a0ee7af85880.tar.gz brdo-4f1af19d101c1940235850a7ea67a0ee7af85880.tar.bz2 | |
- Patch #267203 by Rob Loach: attributes of theme_item_list can be deceiving. If you have a look at theme_item_list you see that the $attributes variable takes a NULL value. This makes you believe that it wants a string, rather then a full array of attributes. If you make the default parameter "array()" it will make the programmer expect to pass an array of attributes rather then a string of attributes.
| -rw-r--r-- | includes/theme.inc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/includes/theme.inc b/includes/theme.inc index 18e7e160f..142226b85 100644 --- a/includes/theme.inc +++ b/includes/theme.inc @@ -1480,7 +1480,7 @@ function theme_mark($type = MARK_NEW) { * @return * A string containing the list output. */ -function theme_item_list($items = array(), $title = NULL, $type = 'ul', $attributes = NULL) { +function theme_item_list($items = array(), $title = NULL, $type = 'ul', $attributes = array()) { $output = '<div class="item-list">'; if (isset($title)) { $output .= '<h3>' . $title . '</h3>'; |