Revert "Don't throw error when using db_fetch_array on empty resultset"

dregad · dregad · commit 7b08792395b1 · 2012-06-04T13:13:24.000+02:00
As per grangeway's comment and confirmed by vincent_sels (original issue's reporter), this change is actually not necessary. This reverts commit 9b1a282. Affects #14336
diff --git a/core/database_api.php b/core/database_api.php
@@ -450,7 +450,7 @@ function db_affected_rows() {
 function db_fetch_array( &$p_result ) {
 	global $g_db, $g_db_type;
 
-	if( is_null( $p_result ) || $p_result->EOF ) {
+	if( $p_result->EOF ) {
 		return false;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ function db_affected_rows() {`
`450`	`450`	`function db_fetch_array( &$p_result ) {`
`451`	`451`	`global $g_db, $g_db_type;`
`452`	`452`
`453`		`- if( is_null( $p_result ) \|\| $p_result->EOF ) {`
	`453`	`+ if( $p_result->EOF ) {`
`454`	`454`	`return false;`
`455`	`455`	`}`
`456`	`456`