Fix notice/warning for file-less items. Fix #470.

omeka · Jan 24, 2013 · f5e5fbe · f5e5fbe
1 parent 5d52dab
commit f5e5fbe
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/application/models/Item.php b/application/models/Item.php
@@ -178,17 +178,18 @@ protected function afterSave($args)
             $post = $args['post'];
 
             // Update file order for this item.
-            foreach ($post['order'] as $fileId => $fileOrder) {
-
-                // File order must be an integer or NULL.
-                $fileOrder = (int) $fileOrder;
-                if (!$fileOrder) {
-                    $fileOrder = null;
+            if (isset($post['order'])) {
+                foreach ($post['order'] as $fileId => $fileOrder) {
+                    // File order must be an integer or NULL.
+                    $fileOrder = (int) $fileOrder;
+                    if (!$fileOrder) {
+                        $fileOrder = null;
+                    }
+
+                    $file = $this->getTable('File')->find($fileId);
+                    $file->order = $fileOrder;
+                    $file->save();
                 }
-
-                $file = $this->getTable('File')->find($fileId);
-                $file->order = $fileOrder;
-                $file->save();
             }
 
             // Delete files that have been designated by passing an array of IDs