diff options
| author | Neil Drumm <drumm@3064.no-reply.drupal.org> | 2006-06-11 23:58:55 +0000 |
|---|---|---|
| committer | Neil Drumm <drumm@3064.no-reply.drupal.org> | 2006-06-11 23:58:55 +0000 |
| commit | 7c05eaa9ac3a42f3ea7075e20ba07543abc16a90 (patch) | |
| tree | 033bdb7ddb2dd2a0214a4a317aea9d290654a5ef | |
| parent | 24d04502c9571e553bdafd9c1c25366e5f68ef4d (diff) | |
| download | brdo-7c05eaa9ac3a42f3ea7075e20ba07543abc16a90.tar.gz brdo-7c05eaa9ac3a42f3ea7075e20ba07543abc16a90.tar.bz2 | |
#66662 by stefan nagtegaal, Remove PHP notices
| -rw-r--r-- | modules/upload.module | 8 | ||||
| -rw-r--r-- | modules/upload/upload.module | 8 |
2 files changed, 8 insertions, 8 deletions
diff --git a/modules/upload.module b/modules/upload.module index c003d098c..c2e31c482 100644 --- a/modules/upload.module +++ b/modules/upload.module @@ -45,7 +45,7 @@ function upload_link($type, $node = 0, $main = 0) { $links = array(); // Display a link with the number of attachments - if ($main && $type == 'node' && is_array($node->files) && user_access('view uploaded files')) { + if ($main && $type == 'node' && isset($node->files) && user_access('view uploaded files')) { $num_files = 0; foreach ($node->files as $file) { if ($file->list) { @@ -81,7 +81,7 @@ function upload_menu($may_cache) { } else { // Add handlers for previewing new uploads. - if ($_SESSION['file_previews']) { + if (isset($_SESSION['file_previews'])) { foreach ($_SESSION['file_previews'] as $fid => $file) { $filename = file_create_filename($file->filename, file_create_path()); if (variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC) == FILE_DOWNLOADS_PRIVATE) { @@ -405,8 +405,8 @@ function upload_nodeapi(&$node, $op, $teaser) { case 'load': if (variable_get("upload_$node->type", 1) == 1) { $output['files'] = upload_load($node); + return $output; } - return $output; break; case 'prepare': @@ -418,7 +418,7 @@ function upload_nodeapi(&$node, $op, $teaser) { break; case 'view': - if (is_array($node->files) && user_access('view uploaded files')) { + if (isset($node->files) && user_access('view uploaded files')) { // Manipulate so that inline references work in preview if (!variable_get('clean_url', 0)) { $previews = array(); diff --git a/modules/upload/upload.module b/modules/upload/upload.module index c003d098c..c2e31c482 100644 --- a/modules/upload/upload.module +++ b/modules/upload/upload.module @@ -45,7 +45,7 @@ function upload_link($type, $node = 0, $main = 0) { $links = array(); // Display a link with the number of attachments - if ($main && $type == 'node' && is_array($node->files) && user_access('view uploaded files')) { + if ($main && $type == 'node' && isset($node->files) && user_access('view uploaded files')) { $num_files = 0; foreach ($node->files as $file) { if ($file->list) { @@ -81,7 +81,7 @@ function upload_menu($may_cache) { } else { // Add handlers for previewing new uploads. - if ($_SESSION['file_previews']) { + if (isset($_SESSION['file_previews'])) { foreach ($_SESSION['file_previews'] as $fid => $file) { $filename = file_create_filename($file->filename, file_create_path()); if (variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC) == FILE_DOWNLOADS_PRIVATE) { @@ -405,8 +405,8 @@ function upload_nodeapi(&$node, $op, $teaser) { case 'load': if (variable_get("upload_$node->type", 1) == 1) { $output['files'] = upload_load($node); + return $output; } - return $output; break; case 'prepare': @@ -418,7 +418,7 @@ function upload_nodeapi(&$node, $op, $teaser) { break; case 'view': - if (is_array($node->files) && user_access('view uploaded files')) { + if (isset($node->files) && user_access('view uploaded files')) { // Manipulate so that inline references work in preview if (!variable_get('clean_url', 0)) { $previews = array(); |