From 6f5b765c97c8616e4a79719bfe835cb03dc0a236 Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@status.net>
Date: Wed, 6 Jan 2010 13:08:56 -0800
Subject: [PATCH 1/2] suppress notice for undefined prompt variable when
 console.php is used from non-interactive terminal

---
 scripts/console.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/scripts/console.php b/scripts/console.php
index 329caf4724..8b62a3a967 100755
--- a/scripts/console.php
+++ b/scripts/console.php
@@ -128,6 +128,8 @@ function console_help()
 if (CONSOLE_INTERACTIVE) {
     print "StatusNet interactive PHP console... type ctrl+D or enter 'exit' to exit.\n";
     $prompt = common_config('site', 'name') . '> ';
+} else {
+    $prompt = '';
 }
 while (!feof(STDIN)) {
     $line = read_input_line($prompt);

From 85554d0840642f4c1b47b50202dd648db565781c Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@status.net>
Date: Wed, 6 Jan 2010 13:23:39 -0800
Subject: [PATCH 2/2] Rearrange Memcached_DataObject::staticGet() to avoid
 "only variables can be passed by reference" warnings when DB lookup fails and
 we return false. (We need to keep it returning a reference because the extlib
 parent class is stuck in PHP 4-land and uses references everywhere, including
 this function's return value. Yuck!)

Also changed pkeyGet to drop the reference, since it doesn't have an upstream equivalent.
---
 classes/Memcached_DataObject.php | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/classes/Memcached_DataObject.php b/classes/Memcached_DataObject.php
index d11bd63682..04f75b775c 100644
--- a/classes/Memcached_DataObject.php
+++ b/classes/Memcached_DataObject.php
@@ -90,17 +90,16 @@ class Memcached_DataObject extends DB_DataObject
             unset($i);
         }
         $i = Memcached_DataObject::getcached($cls, $k, $v);
-        if ($i !== false) { // false == cache miss
-            return $i;
-        } else {
+        if ($i === false) { // false == cache miss
             $i = DB_DataObject::factory($cls);
             if (empty($i)) {
-                return false;
+                $i = false;
+                return $i;
             }
             $result = $i->get($k, $v);
             if ($result) {
+                // Hit!
                 $i->encache();
-                return $i;
             } else {
                 // save the fact that no such row exists
                 $c = self::memcache();
@@ -108,12 +107,16 @@ class Memcached_DataObject extends DB_DataObject
                     $ck = self::cachekey($cls, $k, $v);
                     $c->set($ck, null);
                 }
-                return false;
+                $i = false;
             }
         }
+        return $i;
     }
 
-    function &pkeyGet($cls, $kv)
+    /**
+     * @fixme Should this return false on lookup fail to match staticGet?
+     */
+    function pkeyGet($cls, $kv)
     {
         $i = Memcached_DataObject::multicache($cls, $kv);
         if ($i !== false) { // false == cache miss