Harmonize convertToBean and getRow close #758.

Author: gabordemooij

RedBeanPHP/Facade.php

@@ -1443,10 +1443,11 @@ public static function convertToBeans( $type, $rows, $metamask = NULL )
 	 * @param array  $row       one row from the database
 	 * @param string $metamask  metamask (see convertToBeans)
 	 *
-	 * @return OODBBean
+	 * @return OODBBean|NULL
 	 */
 	public static function convertToBean( $type, $row, $metamask = NULL )
 	{
+		if ( !count( $row ) ) return NULL;
 		$beans = self::$redbean->convertToBeans( $type, array( $row ), $metamask );
 		$bean  = reset( $beans );
 		return $bean;

testing/RedUNIT/Base/Database.php

@@ -40,6 +40,24 @@ public function getTargetDrivers()
 		return array( 'mysql', 'pgsql', 'sqlite', 'CUBRID' );
 	}
 
+	/**
+	 * Make ConvertToBean work together with getRow #759.
+	 * When no results are found for getRow it returns []
+	 * Then when you give that to convertToBean it wraps your
+	 * single row into an array of multiple rows, so you get [[]].
+	 * Then this loop has something to
+	 * iterate on foreach ( $rows as $row ) { ...
+	 * And then it crashes on: $id = $row['id'];
+	 */
+	public function testHarmonizeConvertToBeanAndGetRow()
+	{
+		R::nuke();
+		$book = R::convertToBean( 'book', R::getRow( 'SELECT * FROM book' ) );
+		asrt( is_null( $book ), TRUE );
+		$book = R::convertToBean( 'book', array() );
+		asrt( is_null( $book ), TRUE );
+	}
+
 	/**
 	 * Test for bugfix:
 	 * adhere to return type specification for R::getRow #728