From 69247d7aad8052482d3feac7041ccca702226b03 Mon Sep 17 00:00:00 2001
From: Steven Wittens <steven@10.no-reply.drupal.org>
Date: Fri, 2 Sep 2005 02:11:41 +0000
Subject: [PATCH] - #30128: Call by reference left-overs

---
 modules/blogapi.module         | 6 +++---
 modules/blogapi/blogapi.module | 6 +++---
 modules/node.module            | 9 +++------
 modules/node/node.module       | 9 +++------
 modules/upload.module          | 2 +-
 modules/upload/upload.module   | 2 +-
 6 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/modules/blogapi.module b/modules/blogapi.module
index 3160539db86..81844de8710 100644
--- a/modules/blogapi.module
+++ b/modules/blogapi.module
@@ -218,10 +218,10 @@ function blogapi_blogger_new_post($appkey, $blogid, $username, $password, $conte
     return blogapi_error(t('You do not have permission to create the type of post you wanted to create.'));
   }
 
-  node_save(&$node);
+  node_save($node);
   if ($node->nid) {
     watchdog('content', t('%type: added %title using blog API.', array('%type' => '<em>'. t($node->type) .'</em>', '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), "node/$node->nid"));
-    return $nid;
+    return $node->nid;
   }
 
   return blogapi_error(t('Error storing post.'));
@@ -276,7 +276,7 @@ function blogapi_blogger_edit_post($appkey, $postid, $username, $password, $cont
   foreach ($terms as $term) {
     $node->taxonomy[] = $term->tid;
   }
-  node_save(&$node);
+  node_save($node);
   if ($node->nid) {
     watchdog('content', t('%type: updated %title using blog API.', array('%type' => '<em>'. t($node->type) .'</em>', '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), "node/$node->nid"));
     return true;
diff --git a/modules/blogapi/blogapi.module b/modules/blogapi/blogapi.module
index 3160539db86..81844de8710 100644
--- a/modules/blogapi/blogapi.module
+++ b/modules/blogapi/blogapi.module
@@ -218,10 +218,10 @@ function blogapi_blogger_new_post($appkey, $blogid, $username, $password, $conte
     return blogapi_error(t('You do not have permission to create the type of post you wanted to create.'));
   }
 
-  node_save(&$node);
+  node_save($node);
   if ($node->nid) {
     watchdog('content', t('%type: added %title using blog API.', array('%type' => '<em>'. t($node->type) .'</em>', '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), "node/$node->nid"));
-    return $nid;
+    return $node->nid;
   }
 
   return blogapi_error(t('Error storing post.'));
@@ -276,7 +276,7 @@ function blogapi_blogger_edit_post($appkey, $postid, $username, $password, $cont
   foreach ($terms as $term) {
     $node->taxonomy[] = $term->tid;
   }
-  node_save(&$node);
+  node_save($node);
   if ($node->nid) {
     watchdog('content', t('%type: updated %title using blog API.', array('%type' => '<em>'. t($node->type) .'</em>', '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), "node/$node->nid"));
     return true;
diff --git a/modules/node.module b/modules/node.module
index e35dc5d5695..331267551d7 100644
--- a/modules/node.module
+++ b/modules/node.module
@@ -400,7 +400,7 @@ function node_load($param = array(), $revision = NULL, $reset = NULL) {
 /**
  * Save a node object into the database.
  */
-function node_save($node) {
+function node_save(&$node) {
   global $user;
 
   $node->is_new = false;
@@ -501,9 +501,6 @@ function node_save($node) {
 
   // Clear the cache so an anonymous poster can see the node being added or updated.
   cache_clear_all();
-
-  // Return the node ID:
-  return $node;
 }
 
 /**
@@ -1573,7 +1570,7 @@ function node_submit(&$node) {
     // Check whether the current user has the proper access rights to
     // perform this operation:
     if (node_access('update', $node)) {
-      node_save(&$node);
+      node_save($node);
       watchdog('content', t('%type: updated %title.', array('%type' => theme('placeholder', t($node->type)), '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), 'node/'. $node->nid));
       $msg = t('The %post was updated.', array ('%post' => node_get_name($node)));
     }
@@ -1582,7 +1579,7 @@ function node_submit(&$node) {
     // Check whether the current user has the proper access rights to
     // perform this operation:
     if (node_access('create', $node)) {
-      node_save(&$node);
+      node_save($node);
       watchdog('content', t('%type: added %title.', array('%type' => theme('placeholder', t($node->type)), '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), "node/$node->nid"));
       $msg = t('Your %post was created.', array ('%post' => node_get_name($node)));
     }
diff --git a/modules/node/node.module b/modules/node/node.module
index e35dc5d5695..331267551d7 100644
--- a/modules/node/node.module
+++ b/modules/node/node.module
@@ -400,7 +400,7 @@ function node_load($param = array(), $revision = NULL, $reset = NULL) {
 /**
  * Save a node object into the database.
  */
-function node_save($node) {
+function node_save(&$node) {
   global $user;
 
   $node->is_new = false;
@@ -501,9 +501,6 @@ function node_save($node) {
 
   // Clear the cache so an anonymous poster can see the node being added or updated.
   cache_clear_all();
-
-  // Return the node ID:
-  return $node;
 }
 
 /**
@@ -1573,7 +1570,7 @@ function node_submit(&$node) {
     // Check whether the current user has the proper access rights to
     // perform this operation:
     if (node_access('update', $node)) {
-      node_save(&$node);
+      node_save($node);
       watchdog('content', t('%type: updated %title.', array('%type' => theme('placeholder', t($node->type)), '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), 'node/'. $node->nid));
       $msg = t('The %post was updated.', array ('%post' => node_get_name($node)));
     }
@@ -1582,7 +1579,7 @@ function node_submit(&$node) {
     // Check whether the current user has the proper access rights to
     // perform this operation:
     if (node_access('create', $node)) {
-      node_save(&$node);
+      node_save($node);
       watchdog('content', t('%type: added %title.', array('%type' => theme('placeholder', t($node->type)), '%title' => theme('placeholder', $node->title))), WATCHDOG_NOTICE, l(t('view'), "node/$node->nid"));
       $msg = t('Your %post was created.', array ('%post' => node_get_name($node)));
     }
diff --git a/modules/upload.module b/modules/upload.module
index 335fd8dfc00..d9f34d738c1 100644
--- a/modules/upload.module
+++ b/modules/upload.module
@@ -464,7 +464,7 @@ function _upload_image($file) {
 function upload_js() {
   // We only do the upload.module part of the node validation process.
   $node = array2object($_POST['edit']);
-  upload_nodeapi(&$node, 'validate', NULL);
+  upload_nodeapi($node, 'validate', NULL);
   $output = theme('status_messages') . _upload_form($node);
 
   // We send the updated file attachments form.
diff --git a/modules/upload/upload.module b/modules/upload/upload.module
index 335fd8dfc00..d9f34d738c1 100644
--- a/modules/upload/upload.module
+++ b/modules/upload/upload.module
@@ -464,7 +464,7 @@ function _upload_image($file) {
 function upload_js() {
   // We only do the upload.module part of the node validation process.
   $node = array2object($_POST['edit']);
-  upload_nodeapi(&$node, 'validate', NULL);
+  upload_nodeapi($node, 'validate', NULL);
   $output = theme('status_messages') . _upload_form($node);
 
   // We send the updated file attachments form.