Fixes #3914. Pulled avatar code out into its own library.

2 files changed, 50 insertions, 37 deletions

diff --git a/js/lib/avatar_cropper.js b/js/lib/avatar_cropper.js
new file mode 100644
index 000000000..bfd76225b
--- /dev/null
+++ b/js/lib/avatar_cropper.js
@@ -0,0 +1,50 @@
+/**
+ * Avatar cropping
+ */
+
+elgg.provide('elgg.avatarCropper');
+
+/**
+ * Register the avatar cropper.
+ */
+elgg.avatarCropper.init = function() {
+	$('#user-avatar-cropper').imgAreaSelect({
+		selectionOpacity: 0,
+		aspectRatio: '1:1',
+		onSelectEnd: elgg.avatarCropper.selectChange,
+		onSelectChange: elgg.avatarCropper.preview
+	});
+}
+
+/**
+ * Handler for changing select area.
+ */
+elgg.avatarCropper.preview = function(img, selection) {
+	// catch for the first click on the image
+	if (selection.width == 0 || selection.height == 0) {
+		return;
+	}
+
+	var origWidth = $("#user-avatar-cropper").width();
+	var origHeight = $("#user-avatar-cropper").height();
+	var scaleX = 100 / selection.width;
+	var scaleY = 100 / selection.height;
+	$('#user-avatar-preview > img').css({
+		width: Math.round(scaleX * origWidth) + 'px',
+		height: Math.round(scaleY * origHeight) + 'px',
+		marginLeft: '-' + Math.round(scaleX * selection.x1) + 'px',
+		marginTop: '-' + Math.round(scaleY * selection.y1) + 'px'
+	});
+}
+
+/**
+ * Handler for updating the form inputs after select ends
+ */
+elgg.avatarCropper.selectChange = function(img, selection) {
+	$('input[name=x1]').val(selection.x1);
+	$('input[name=x2]').val(selection.x2);
+	$('input[name=y1]').val(selection.y1);
+	$('input[name=y2]').val(selection.y2);
+}
+
+elgg.register_hook_handler('init', 'system', elgg.avatarCropper.init);
\ No newline at end of file
diff --git a/js/lib/ui.js b/js/lib/ui.js
index 8cee500d0..166ca16bc 100644
--- a/js/lib/ui.js
+++ b/js/lib/ui.js
@@ -25,14 +25,6 @@ elgg.ui.init = function () {
 	if ($('.elgg-input-date').length) {
 		elgg.ui.initDatePicker();
 	}
-
-	// avatar cropper
-	$('#user-avatar-cropper').imgAreaSelect({
-		selectionOpacity: 0,
-		aspectRatio: '1:1',
-		onSelectEnd: elgg.ui.avatarSelectChange,
-		onSelectChange: elgg.ui.avatarPreview
-	});
 }
 
 /**
@@ -286,34 +278,5 @@ elgg.ui.initDatePicker = function() {
 	});
 }
 
-// Avatar cropping
-
-// display a preview of the users cropped section
-elgg.ui.avatarPreview = function(img, selection) {
-	// catch for the first click on the image
-	if (selection.width == 0 || selection.height == 0) {
-		return;
-	}
-
-	var origWidth = $("#user-avatar-cropper").width();
-	var origHeight = $("#user-avatar-cropper").height();
-	var scaleX = 100 / selection.width;
-	var scaleY = 100 / selection.height;
-	$('#user-avatar-preview > img').css({
-		width: Math.round(scaleX * origWidth) + 'px',
-		height: Math.round(scaleY * origHeight) + 'px',
-		marginLeft: '-' + Math.round(scaleX * selection.x1) + 'px',
-		marginTop: '-' + Math.round(scaleY * selection.y1) + 'px'
-	});
-}
-
-// populate the form with the correct coordinates once a user has cropped their image
-elgg.ui.avatarSelectChange = function(img, selection) {
-	$('input[name=x1]').val(selection.x1);
-	$('input[name=x2]').val(selection.x2);
-	$('input[name=y1]').val(selection.y1);
-	$('input[name=y2]').val(selection.y2);
-}
-
 elgg.register_hook_handler('init', 'system', elgg.ui.init);
 elgg.register_hook_handler('getOptions', 'ui.popup', elgg.ui.LoginHandler);
\ No newline at end of file