From 977efe4e1d359757b14623fb5f64bdcd4d6206e6 Mon Sep 17 00:00:00 2001
From: mark_story <mark@mark-story.com>
Date: Sat, 28 Mar 2009 00:31:51 -0400
Subject: [PATCH] Fixing callback naming issues with mootools

---
 cake/libs/view/helpers/mootools_engine.php                  | 5 ++++-
 cake/tests/cases/libs/view/helpers/mootools_engine.test.php | 2 +-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/cake/libs/view/helpers/mootools_engine.php b/cake/libs/view/helpers/mootools_engine.php
index 67330d8c73d..3c8ed5629e2 100644
--- a/cake/libs/view/helpers/mootools_engine.php
+++ b/cake/libs/view/helpers/mootools_engine.php
@@ -44,6 +44,9 @@ class MootoolsEngineHelper extends JsBaseEngineHelper {
 		'sortable' => array(
 			'distance' => 'snap',
 			'containment' => 'constrain',
+			'sort' => 'onSort',
+			'complete' => 'onComplete',
+			'start' => 'onStart',
 		)
 	);
 /**
@@ -198,7 +201,7 @@ function request($url, $options = array()) {
  **/
 	function sortable($options = array()) {
 		$options = $this->_mapOptions('sortable', $options);
-		$callbacks = array('start', 'sort', 'complete');
+		$callbacks = array('onStart', 'onSort', 'onComplete');
 		$options = $this->_parseOptions($options, $callbacks);
 		return 'var mooSortable = new Sortables(' . $this->selection . ', {' . $options . '});';
 	}
diff --git a/cake/tests/cases/libs/view/helpers/mootools_engine.test.php b/cake/tests/cases/libs/view/helpers/mootools_engine.test.php
index 29e016cd14b..3f19e8a9484 100644
--- a/cake/tests/cases/libs/view/helpers/mootools_engine.test.php
+++ b/cake/tests/cases/libs/view/helpers/mootools_engine.test.php
@@ -184,7 +184,7 @@ function testSortable() {
 			'complete' => 'onStop',
 			'sort' => 'onSort',
 		));
-		$expected = 'var mooSortable = new Sortables($("myList"), {start:onStart, complete:onStop, sort:onSort, snap:5, constrain:"parent"});';
+		$expected = 'var mooSortable = new Sortables($("myList"), {constrain:"parent", onComplete:onStop, onSort:onSort, onStart:onStart, snap:5});';
 		$this->assertEqual($result, $expected);
 	}
 }