Merge pull request #39 from tractorcow/pulls/fix-status

BUG Fix dropdown not selecting correct status
symbiote · May 11, 2015 · d4d83e7 · d4d83e7
2 parents 00141d1 + a1e47b0
commit d4d83e7
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/code/dataobjects/QueuedJobDescriptor.php b/code/dataobjects/QueuedJobDescriptor.php
@@ -161,18 +161,19 @@ public function getCMSFields() {
 				QueuedJob::LARGE => 'Large'
 			))
 		);
+		$statuses = array(
+			QueuedJob::STATUS_NEW,
+			QueuedJob::STATUS_INIT,
+			QueuedJob::STATUS_RUN,
+			QueuedJob::STATUS_WAIT,
+			QueuedJob::STATUS_COMPLETE,
+			QueuedJob::STATUS_PAUSED,
+			QueuedJob::STATUS_CANCELLED,
+			QueuedJob::STATUS_BROKEN
+		);
 		$fields->replaceField(
 			'JobStatus',
-			new DropdownField('JobStatus', $this->fieldLabel('JobStatus'), array(
-				QueuedJob::STATUS_NEW,
-				QueuedJob::STATUS_INIT,
-				QueuedJob::STATUS_RUN,
-				QueuedJob::STATUS_WAIT,
-				QueuedJob::STATUS_COMPLETE,
-				QueuedJob::STATUS_PAUSED,
-				QueuedJob::STATUS_CANCELLED,
-				QueuedJob::STATUS_BROKEN
-			))
+			new DropdownField('JobStatus', $this->fieldLabel('JobStatus'), array_combine($statuses, $statuses))
 		);