1 /*
  2  * code review by Pavel Yaschenko
  3  * 
  4  * 1. No need to save DOM element (this.indicator). We can use id to get dom element. It helps to avoid memory leaks :)
  5  * 
  6  * 2. Name refactoring: change names acceptClass, rejectClass, draggingClass 
  7  * 						to more readable names: getAcceptClass, getRejectClass, getDragClass
  8  * 
  9  */
 10 
 11 (function ($, rf) {
 12 
 13     rf.ui = rf.ui || {};
 14 
 15     rf.ui.DragIndicator = function(id, options) {
 16         $super.constructor.call(this, id);
 17         this.attachToDom(id);
 18 
 19         this.indicator = $(document.getElementById(id));
 20         this.options = options;
 21     };
 22 
 23     var defaultOptions = {
 24     };
 25 
 26     rf.BaseComponent.extend(rf.ui.DragIndicator);
 27     var $super = rf.ui.DragIndicator.$super;
 28 
 29     $.extend(rf.ui.DragIndicator.prototype, ( function () {
 30         return {
 31             show : function() {
 32                 this.indicator.show();
 33             },
 34 
 35             hide: function() {
 36                 this.indicator.hide();
 37             },
 38 
 39             getAcceptClass: function() {
 40                 return this.options.acceptClass;
 41             },
 42 
 43             getRejectClass: function() {
 44                 return this.options.rejectClass;
 45             },
 46 
 47             getDraggingClass: function() {
 48                 return this.options.draggingClass;
 49             },
 50 
 51             getElement: function() {
 52                 return this.indicator;
 53             }
 54         }
 55     })());
 56 
 57 })(RichFaces.jQuery, window.RichFaces);
 58 
 59