From 1ff3b9458412ec14f7bda845809ce33abea677a5 Mon Sep 17 00:00:00 2001
From: Neil Blakey-Milner <nbm@knowledgetree.com>
Date: Wed, 30 Nov 2005 14:37:39 +0000
Subject: [PATCH] Be more defensive in terms of what is potentially getting passed to us.

---
 lib/documentmanagement/documentutil.inc.php | 6 ++++++
 1 file changed, 6 insertions(+), 0 deletions(-)

diff --git a/lib/documentmanagement/documentutil.inc.php b/lib/documentmanagement/documentutil.inc.php
index 4379ec9..3dcbbf4 100644
--- a/lib/documentmanagement/documentutil.inc.php
+++ b/lib/documentmanagement/documentutil.inc.php
@@ -214,6 +214,9 @@ class KTDocumentUtil {
         $aSimpleMetadata = array();
         foreach ($aMetadata as $aSingleMetadatum) {
             list($oField, $sValue) = $aSingleMetadatum;
+            if (is_null($oField)) {
+                continue;
+            }
             $aSimpleMetadata[$oField->getId()] = $sValue;
         }
         $aFailed = array();
@@ -262,6 +265,9 @@ class KTDocumentUtil {
         // XXX: Metadata refactor
         foreach ($aMetadata as $aInfo) {
             list($oMetadata, $sValue) = $aInfo;
+            if (is_null($oMetadata)) {
+                continue;
+            }
             $res = DBUtil::autoInsert($table, array(
                 "document_id" => $oDocument->getID(),
                 "document_field_id" => $oMetadata->getID(),
--
libgit2 0.21.4