Skip to content

Commit 20b2b1b

Browse files
committed
Follow JS styleguide and use camelCase functions
1 parent dc57ced commit 20b2b1b

File tree

1 file changed

+14
-14
lines changed

1 file changed

+14
-14
lines changed
Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,21 @@
11
(function($) {
22

3-
var create_new_id = function() {
3+
var createNewResourceID = function() {
44
return new Date().getTime();
55
}
66

77
$(document).on('click', '.add_fields', function(e) {
88
e.preventDefault();
9-
9+
1010
var $link = $(this);
1111
var assoc = $link.data('association');
1212
var content = $link.data('association-insertion-template');
1313
var insertionMethod = $link.data('association-insertion-method') || $link.data('association-insertion-position') || 'before';
1414
var insertionNode = $link.data('association-insertion-node');
1515
var insertionTraversal = $link.data('association-insertion-traversal');
16-
var new_id = create_new_id();
16+
var newId = createNewResourceID();
1717
var regex = new RegExp("new_" + assoc, "g");
18-
var new_content = content.replace(regex, new_id);
18+
var newContent = content.replace(regex, newId);
1919

2020
if (insertionNode){
2121
if (insertionTraversal){
@@ -27,7 +27,7 @@
2727
insertionNode = $link.parent();
2828
}
2929

30-
var contentNode = $(new_content);
30+
var contentNode = $(newContent);
3131
insertionNode.trigger('before-insert', [contentNode]);
3232

3333
var addedContent = insertionNode[insertionMethod](contentNode);
@@ -39,23 +39,23 @@
3939
e.preventDefault();
4040

4141
var $link = $(this);
42-
var wrapper_class = $link.data('wrapper-class') || 'nested-fields';
43-
var node_to_delete = $link.closest('.' + wrapper_class);
44-
var trigger_node = node_to_delete.parent();
42+
var wrapperClass = $link.data('wrapper-class') || 'nested-fields';
43+
var nodeToDelete = $link.closest('.' + wrapperClass);
44+
var triggerNode = nodeToDelete.parent();
4545

46-
trigger_node.trigger('before-remove', [node_to_delete]);
46+
triggerNode.trigger('before-remove', [nodeToDelete]);
4747

48-
var timeout = trigger_node.data('remove-timeout') || 0;
48+
var timeout = triggerNode.data('remove-timeout') || 0;
4949

5050
setTimeout(function() {
5151
if ($link.hasClass('dynamic')) {
52-
node_to_delete.remove();
52+
nodeToDelete.remove();
5353
} else {
5454
$link.prev("input[type=hidden]").val("1");
55-
node_to_delete.hide();
55+
nodeToDelete.hide();
5656
}
57-
trigger_node.trigger('after-remove', [node_to_delete]);
57+
triggerNode.trigger('after-remove', [nodeToDelete]);
5858
}, timeout);
5959
});
6060

61-
})(jQuery);
61+
})(jQuery);

0 commit comments

Comments
 (0)