BUG: refs #0349. uploadPerform with revision=head now only creates 1 new rev.

Michael Grauer · Michael Grauer · commit 0940fc610a3e · 2011-11-17T18:23:21.000-05:00
If uploadPerform was called with revision=head on an item without any revs,
two new revs would be created and the upload would happen on the 2nd rev.
diff --git a/modules/api/controllers/components/ApiComponent.php b/modules/api/controllers/components/ApiComponent.php
@@ -329,7 +329,7 @@ function uploadPerform($args)
           $revision->setUser_id($userDao->getKey());
           $revision->setDate(date('c'));
           $revision->setLicense(null);
-          $revision = $itemModel->addRevision($item, $revision);
+          $itemModel->addRevision($item, $revision);
           }
         }
       else

Original file line number	Diff line number	Diff line change
`@@ -329,7 +329,7 @@ function uploadPerform($args)`
`329`	`329`	`$revision->setUser_id($userDao->getKey());`
`330`	`330`	`$revision->setDate(date('c'));`
`331`	`331`	`$revision->setLicense(null);`
`332`		`- $revision = $itemModel->addRevision($item, $revision);`
	`332`	`+ $itemModel->addRevision($item, $revision);`
`333`	`333`	`}`
`334`	`334`	`}`
`335`	`335`	`else`