Simplify the batch api by having the core event handlers for

start_batch and end_batch add and remove the batch id from the
session.  Modules wishing to do batch processing, just need to fire
the start_batch and end_batch events.  Other modules that need to be
aware of batches (i.e. notifications) just check the session for "batch_id".

1 files changed, 3 insertions, 10 deletions

diff --git a/core/controllers/simple_uploader.php b/core/controllers/simple_uploader.php
index 005071b0..5ae4bb2a 100644
--- a/core/controllers/simple_uploader.php
+++ b/core/controllers/simple_uploader.php
@@ -48,12 +48,7 @@ class Simple_Uploader_Controller extends Controller {
     $file_validation = new Validation($_FILES);
     $file_validation->add_rules("file", "upload::valid", "upload::type[gif,jpg,png,flv,mp4]");
     if ($file_validation->validate()) {
-      $batch_id = Session::instance()->get("batch_id");
-      if (empty($batch_id)) {
-        $batch_id = mt_rand();
-        module::event("start_add_batch", $batch_id);
-        Session::instance()->set("batch_id", $batch_id);
-      }
+      module::event("start_batch");
       $temp_filename = upload::save("file");
       $title = substr(basename($temp_filename), 10);  // Skip unique identifier Kohana adds
       $path_info = pathinfo($temp_filename);
@@ -70,10 +65,8 @@ class Simple_Uploader_Controller extends Controller {
   }
 
   public function finish() {
-    $batch_id = Session::instance()->get_once("batch_id");
-    if (!empty($batch_id)) {
-      module::event("end_add_batch", $batch_id);
-    }
+    module::event("end_batch");
+
     print json_encode(array("result" => "success"));
   }
 }