]> gitweb.fluxo.info Git - lorea/elgg.git/commitdiff
added semi-colons after function expressions
authorcash <cash.costello@gmail.com>
Tue, 1 Nov 2011 23:36:27 +0000 (19:36 -0400)
committercash <cash.costello@gmail.com>
Tue, 1 Nov 2011 23:36:27 +0000 (19:36 -0400)
16 files changed:
documentation/coding_standards/javascript_coding_standards.txt
js/lib/avatar_cropper.js
js/lib/configuration.js
js/lib/elgglib.js
js/lib/hooks.js
js/lib/ui.js
js/lib/ui.widgets.js
js/lib/userpicker.js
mod/blog/views/default/js/blog/save_draft.php
mod/bookmarks/views/default/bookmarks/js.php
mod/developers/views/default/js/developers/developers.php
mod/embed/views/default/js/embed/embed.php
mod/messageboard/views/default/messageboard/js.php
mod/thewire/views/default/js/thewire.php
mod/tinymce/views/default/js/tinymce.php
mod/uservalidationbyemail/views/default/uservalidationbyemail/js.php

index 7d3b842ec93bb58670bee1258666e69c8b144c09..9939e80ab3a5260a8f16ed890894b7278ed09858 100644 (file)
@@ -1,2 +1,13 @@
 *** JAVASCRIPT CODING STANDARDS ***
 
+*      Same formatting standards as PHP.
+
+*      All functions should be in the elgg namespace.
+
+*      Function expressions should end with a semi-colon:
+
+       elgg.ui.toggles = function(event) {
+               event.preventDefault();
+               $(target).slideToggle('medium');
+       };
+
index df6ba78660584534e3161f48f93915c5da491700..fc32a08329605e12a3369a42429693fe3fd18634 100644 (file)
@@ -32,7 +32,7 @@ elgg.avatarCropper.init = function() {
                var selection = ias.getSelection();
                elgg.avatarCropper.preview($('#user-avatar-cropper'), selection);
        }
-}
+};
 
 /**
  * Handler for changing select area.
@@ -57,7 +57,7 @@ elgg.avatarCropper.preview = function(img, selection) {
                marginLeft: '-' + Math.round(scaleX * selection.x1) + 'px',
                marginTop: '-' + Math.round(scaleY * selection.y1) + 'px'
        });
-}
+};
 
 /**
  * Handler for updating the form inputs after select ends
@@ -71,6 +71,6 @@ elgg.avatarCropper.selectChange = function(img, selection) {
        $('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
index f724a2f01c328edf74fa682cc9a1f19a918a2902..6e221c957041d7a13f54db9bc11c091626430882 100644 (file)
@@ -7,4 +7,4 @@ elgg.provide('elgg.config');
  */
 elgg.get_site_url = function() {
        return elgg.config.wwwroot;
-}
\ No newline at end of file
+};
\ No newline at end of file
index 628adccfcc2cffa3986cbb0b9fba43b09de2bbe2..d963a62be9a947ccbb56deba40a8893d277b4a8e 100644 (file)
@@ -467,7 +467,7 @@ elgg.parse_url = function(url, component, expand) {
                }
        }
        return results;
-}
+};
 
 /**
  * Returns an object with key/values of the parsed query string.
@@ -540,7 +540,7 @@ elgg.push_to_object_array = function(object, parent, value) {
        }
 
        return false;
-}
+};
 
 /**
  * Tests if object[parent] contains child
@@ -554,7 +554,7 @@ elgg.is_in_object_array = function(object, parent, value) {
        elgg.assertTypeOf('string', parent);
 
        return typeof(object[parent]) != 'undefined' && object[parent].indexOf(value) >= 0;
-}
+};
 
 /**
  * Triggers the init hook when the library is ready
@@ -569,4 +569,4 @@ elgg.initWhenReady = function() {
                elgg.trigger_hook('init', 'system');
                elgg.trigger_hook('ready', 'system');
        }
-}
\ No newline at end of file
+};
\ No newline at end of file
index edfd28f2468f066964a21cd7991047c430d4f60c..7bac471f6cfe53efd431e1fc6f3f502cc3f01fbd 100644 (file)
@@ -136,7 +136,7 @@ elgg.register_instant_hook = function(name, type) {
        elgg.assertTypeOf('string', type);
 
        return elgg.push_to_object_array(elgg.config.instant_hooks, name, type);
-}
+};
 
 /**
  * Is this hook registered as an instant hook?
@@ -146,7 +146,7 @@ elgg.register_instant_hook = function(name, type) {
  */
 elgg.is_instant_hook = function(name, type) {
        return elgg.is_in_object_array(elgg.config.instant_hooks, name, type);
-}
+};
 
 /**
  * Records that a hook has been triggered.
@@ -156,7 +156,7 @@ elgg.is_instant_hook = function(name, type) {
  */
 elgg.set_triggered_hook = function(name, type) {
        return elgg.push_to_object_array(elgg.config.triggered_hooks, name, type);
-}
+};
 
 /**
  * Has this hook been triggered yet?
@@ -166,7 +166,7 @@ elgg.set_triggered_hook = function(name, type) {
  */
 elgg.is_triggered_hook = function(name, type) {
        return elgg.is_in_object_array(elgg.config.triggered_hooks, name, type);
-}
+};
 
 elgg.register_instant_hook('init', 'system');
 elgg.register_instant_hook('ready', 'system');
index 6cc1bc78ac329886e51652eabcce7ea431635ce0..16ef2dc9630f5fd5a63c8b698b77a41f94f65c3e 100644 (file)
@@ -25,7 +25,7 @@ elgg.ui.init = function () {
        if ($('.elgg-input-date').length) {
                elgg.ui.initDatePicker();
        }
-}
+};
 
 /**
  * Toggles an element based on clicking a separate element
@@ -43,7 +43,7 @@ elgg.ui.toggles = function(event) {
        var target = $(this).toggleClass('elgg-state-active').attr('href');
 
        $(target).slideToggle('medium');
-}
+};
 
 /**
  * Pops up an element based on clicking a separate element
@@ -105,7 +105,7 @@ elgg.ui.popsUp = function(event) {
        $('body')
                .die('click', elgg.ui.popupClose)
                .live('click', elgg.ui.popupClose);
-}
+};
 
 /**
  * Catches clicks that aren't in a popup and closes all popups.
@@ -143,7 +143,7 @@ elgg.ui.popupClose = function(event) {
 
                $('body').die('click', elgg.ui.popClose);
        }
-}
+};
 
 /**
  * Toggles a child menu when the parent is clicked
@@ -155,7 +155,7 @@ elgg.ui.toggleMenu = function(event) {
        $(this).siblings().slideToggle('medium');
        $(this).toggleClass('elgg-menu-closed elgg-menu-opened');
        event.preventDefault();
-}
+};
 
 /**
  * Initialize the hover menu
@@ -215,7 +215,7 @@ elgg.ui.initHoverMenu = function(parent) {
                        $(".elgg-menu-hover").fadeOut();
                }
        });
-}
+};
 
 /**
  * Calls a confirm() and prevents default if denied.
@@ -276,7 +276,7 @@ elgg.ui.initDatePicker = function() {
                        }
                }
        });
-}
+};
 
 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
index fb256672a41eb5de82b4edb62aae0dd13bd90150..6435d2147b73bc50edef68dcfdfd943ceb14c8f5 100644 (file)
@@ -65,7 +65,7 @@ elgg.ui.widgets.add = function(event) {
                }
        });
        event.preventDefault();
-}
+};
 
 /**
  * Persist the widget's new position
@@ -96,7 +96,7 @@ elgg.ui.widgets.move = function(event, ui) {
        // @hack fixes jquery-ui/opera bug where draggable elements jump
        ui.item.css('top', 0);
        ui.item.css('left', 0);
-}
+};
 
 /**
  * Removes a widget from the layout
@@ -134,7 +134,7 @@ elgg.ui.widgets.remove = function(event) {
                }
        });
        event.preventDefault();
-}
+};
 
 /**
  * Toggle the collapse state of the widget
@@ -146,7 +146,7 @@ elgg.ui.widgets.collapseToggle = function(event) {
        $(this).toggleClass('elgg-widget-collapsed');
        $(this).parent().parent().find('.elgg-body').slideToggle('medium');
        event.preventDefault();
-}
+};
 
 /**
  * Save a widget's settings
@@ -178,7 +178,7 @@ elgg.ui.widgets.saveSettings = function(event) {
                }
        });
        event.preventDefault();
-}
+};
 
 /**
  * Make all elements have the same min-height
@@ -197,6 +197,6 @@ elgg.ui.widgets.equalHeight = function(selector) {
                }
        })
        $(selector).css('min-height', maxHeight);
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.ui.widgets.init);
index ae2add53f8ab002697f971a713d7ae96269077e4..8287ba91cb4a478c401e22e7401b278329b6a06f 100644 (file)
@@ -34,7 +34,7 @@ elgg.userpicker.init = function() {
        });
 
        $('.elgg-userpicker-remove').live('click', elgg.userpicker.removeUser);
-}
+};
 
 /**
  * Adds a user to the select user list
@@ -59,7 +59,7 @@ elgg.userpicker.addUser = function(event, ui) {
 
        $(this).val('');
        event.preventDefault();
-}
+};
 
 /**
  * Remove a user from the selected user list
@@ -75,7 +75,7 @@ elgg.userpicker.removeUser = function(event) {
 
        item.remove();
        event.preventDefault();
-}
+};
 
 /**
  * Render the list item for insertion into the selected user list
@@ -96,7 +96,7 @@ elgg.userpicker.viewUser = function(info) {
        html += "</div";
        
        return html;
-}
+};
 
 /**
  * Get the parameters to use for autocomplete
@@ -112,6 +112,6 @@ elgg.userpicker.getSearchParams = function(obj) {
        } else {
                return {'match_on[]': 'users', 'term' : obj.term};
        }
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.userpicker.init);
\ No newline at end of file
index fd76c870c598866bc37ac9f4eaf16111ed7a3f2b..8a994ffb085a8767e88155b993d85c54b9ea81c3 100644 (file)
@@ -28,7 +28,7 @@ elgg.blog.saveDraftCallback = function(data, textStatus, XHR) {
        } else {
                $(".blog-save-status-time").html(elgg.echo('error'));
        }
-}
+};
 
 elgg.blog.saveDraft = function() {
        if (typeof(tinyMCE) != 'undefined') {
@@ -55,7 +55,7 @@ elgg.blog.saveDraft = function() {
        });
 
        $.post(draftURL, postData, elgg.blog.saveDraftCallback, 'json');
-}
+};
 
 elgg.blog.init = function() {
        // get a copy of the body to compare for auto save
index 0e55c510a157eafcf0ca041803600815d5cffe38..c36823c0934e796f3f3f8dad732e160eefb5bf0f 100644 (file)
@@ -7,6 +7,6 @@ elgg.bookmarks.init = function() {
        var e = $('a.elgg-bookmark-page');
        var link = e.attr('href') + '&title=' + encodeURIComponent(title);
        e.attr('href', link);
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.bookmarks.init);
index 09e6ddd12fce0512a57cb93d7d4568103df00045..6e82295aad6a16b185dc31c7551864b549580786 100644 (file)
@@ -8,7 +8,7 @@ elgg.provide('elgg.dev');
 
 elgg.dev.init = function() {
        $('.developers-form-inspect').live('submit', elgg.dev.inspectSubmit);
-}
+};
 
 /**
  * Submit the inspect form through Ajax
@@ -39,6 +39,6 @@ elgg.dev.inspectSubmit = function(event) {
        });
 
        event.preventDefault();
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.dev.init);
\ No newline at end of file
index 3126e12f786f7d2aa5e54a2dab58e78e04f69ee0..8e543ac37af6dc0b4df4137fe5a3081de63d6dd7 100644 (file)
@@ -19,7 +19,7 @@ elgg.embed.init = function() {
        $('.embed-section').live('click', elgg.embed.forward);
 
        $('.elgg-form-embed').live('submit', elgg.embed.submit);
-}
+};
 
 /**
  * Inserts data attached to an embed list item in textarea
@@ -48,7 +48,7 @@ elgg.embed.insert = function(event) {
        $.fancybox.close();
 
        event.preventDefault();
-}
+};
 
 /**
  * Submit an upload form through Ajax
@@ -85,7 +85,7 @@ elgg.embed.submit = function(event) {
        // this was bubbling up the DOM causing a submission
        event.preventDefault();
        event.stopPropagation();
-}
+};
 
 /**
  * Loads content within the lightbox
@@ -96,6 +96,6 @@ elgg.embed.submit = function(event) {
 elgg.embed.forward = function(event) {
        $('.embed-wrapper').parent().load($(this).attr('href'));
        event.preventDefault();
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.embed.init);
index e5495409a2f6d4c0b7aab345d61a6e3fae2977be..3295c68de3069072ae69a28a2df6dc8cb1cfff82 100644 (file)
@@ -13,7 +13,7 @@ elgg.messageboard.init = function() {
                // double whammy for in case the load order changes.
                .unbind('click', elgg.ui.requiresConfirmation)
                .removeClass('elgg-requires-confirmation');
-}
+};
 
 elgg.messageboard.submit = function(e) {
        var form = $(this).parents('form');
@@ -35,7 +35,7 @@ elgg.messageboard.submit = function(e) {
        });
 
        e.preventDefault();
-}
+};
 
 elgg.messageboard.deletePost = function(e) {
        var link = $(this);
@@ -50,6 +50,6 @@ elgg.messageboard.deletePost = function(e) {
        }
 
        e.preventDefault();
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.messageboard.init);
index d9bf6d10e09c76781e51809ba54c2b3496ce6249..0a6eba134a4eb90e5b2aeb376da5c51ce49f7f08 100644 (file)
@@ -18,7 +18,7 @@ elgg.thewire.init = function() {
        });
 
        $(".thewire-previous").live('click', elgg.thewire.viewPrevious);
-}
+};
 
 /**
  * Update the number of characters left with every keystroke
@@ -42,7 +42,7 @@ elgg.thewire.textCounter = function(textarea, status, limit) {
                $("#thewire-submit-button").removeAttr('disabled', 'disabled');
                $("#thewire-submit-button").removeClass('elgg-state-disabled');
        }
-}
+};
 
 /**
  * Display the previous wire post
@@ -81,6 +81,6 @@ elgg.thewire.viewPrevious = function(event) {
        }
 
        event.preventDefault();
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.thewire.init);
index 55e0c7ae137b7a1bf33c47280201e09bb1401c3c..c6973d87837d70a827205f7d5f3e2613e3139811 100644 (file)
@@ -18,7 +18,7 @@ elgg.tinymce.toggleEditor = function(event) {
                tinyMCE.execCommand('mceRemoveControl', false, id);
                $(this).html(elgg.echo('tinymce:add'));
        }
-}
+};
 
 /**
  * TinyMCE initialization script
@@ -69,6 +69,6 @@ elgg.tinymce.init = function() {
                content_css: elgg.config.wwwroot + 'mod/tinymce/css/elgg_tinymce.css'
        });
 
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.tinymce.init);
\ No newline at end of file
index abd29c0e113f4ce16bb2a2b863536a14c08a7e01..50c98406e653d02122fdf00ccf9eef66a360b62a 100644 (file)
@@ -23,6 +23,6 @@ elgg.uservalidationbyemail.init = function() {
 
                $form.attr('action', $(this).attr('href')).submit();
        });
-}
+};
 
 elgg.register_hook_handler('init', 'system', elgg.uservalidationbyemail.init);