JqueryEventsTrait::_change() - Code Metrics - Inspection of "JsUtils & Jquery refactoring with traits" - jcheron/phalcon-jquery - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 79b035...e37d9a )

by Jean-Christophe

created 2016-04-17 22:36 UTC

JqueryEventsTrait::_change() A

↳ Parent: JqueryEventsTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	3
rs	10
cc	1
eloc	2
nc	1
nop	2

<?php

namespace Ajax\common\traits;

trait JqueryEventsTrait {

	/**
	 * Blur
	 *
	 * Outputs a jQuery blur event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _blur($element='this', $js='') {
		return $this->_add_event($element, $js, 'blur');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Change
	 *
	 * Outputs a jQuery change event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _change($element='this', $js='') {
		return $this->_add_event($element, $js, 'change');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery click event
	 *
	 * @param string $element The element to attach the event to
	 * @param mixed $js The code to execute
	 * @param boolean $ret_false whether or not to return false
	 * @return string
	 */
	public function _click($element='this', $js=array(), $ret_false=TRUE) {
		if (!is_array($js)) {
			$js=array (
					$js
			);
		}

		if ($ret_false) {
			$js[]="return false;";
		}

		return $this->_add_event($element, $js, 'click');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	/**
	 * Outputs a jQuery contextmenu event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _contextmenu($element='this', $js='') {
		return $this->_add_event($element, $js, 'contextmenu');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery dblclick event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _dblclick($element='this', $js='') {
		return $this->_add_event($element, $js, 'dblclick');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery error event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _error($element='this', $js='') {
		return $this->_add_event($element, $js, 'error');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery focus event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _focus($element='this', $js='') {
		return $this->_add_event($element, $js, 'focus');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery hover event
	 *
	 * @param string - element
	 * @param string - Javascript code for mouse over
	 * @param string - Javascript code for mouse out
	 * @return string
	 */
	public function _hover($element='this', $over, $out) {
		$event="\n\t$(".$this->_prep_element($element).").hover(\n\t\tfunction()\n\t\t{\n\t\t\t{$over}\n\t\t}, \n\t\tfunction()\n\t\t{\n\t\t\t{$out}\n\t\t});\n";
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

		$this->jquery_code_for_compile[]=$event;
class MyClass { }

$x = new MyClass();
$x->foo = true;

		return $event;
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery keydown event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _keydown($element='this', $js='') {
		return $this->_add_event($element, $js, 'keydown');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery keypress event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _keypress($element='this', $js='') {
		return $this->_add_event($element, $js, 'keypress');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery keydown event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _keyup($element='this', $js='') {
		return $this->_add_event($element, $js, 'keyup');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery load event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _load($element='this', $js='') {
		return $this->_add_event($element, $js, 'load');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery mousedown event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _mousedown($element='this', $js='') {
		return $this->_add_event($element, $js, 'mousedown');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery mouseout event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _mouseout($element='this', $js='') {
		return $this->_add_event($element, $js, 'mouseout');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery mouseover event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _mouseover($element='this', $js='') {
		return $this->_add_event($element, $js, 'mouseover');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery mouseup event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _mouseup($element='this', $js='') {
		return $this->_add_event($element, $js, 'mouseup');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	/**
	 * Outputs a jQuery resize event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _resize($element='this', $js='') {
		return $this->_add_event($element, $js, 'resize');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery scroll event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _scroll($element='this', $js='') {
		return $this->_add_event($element, $js, 'scroll');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

	// --------------------------------------------------------------------

	/**
	 * Outputs a jQuery unload event
	 *
	 * @param string The element to attach the event to
	 * @param string The code to execute
	 * @return string
	 */
	public function _unload($element='this', $js='') {
		return $this->_add_event($element, $js, 'unload');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
	}

}

1			<?php
2
3			namespace Ajax\common\traits;
4
5			trait JqueryEventsTrait {
6
7			/**
8			* Blur
9			*
10			* Outputs a jQuery blur event
11			*
12			* @param string The element to attach the event to
13			* @param string The code to execute
14			* @return string
15			*/
16			public function _blur($element='this', $js='') {
17			return $this->_add_event($element, $js, 'blur');
			0 ignored issues – show Bug introduced 2016-04-17 22:39 UTC by Report Bug Copy Issue Report It seems like `_add_event()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
18			}
19
20			// --------------------------------------------------------------------
21
22			/**
23			* Change
24			*
25			* Outputs a jQuery change event
26			*
27			* @param string The element to attach the event to
28			* @param string The code to execute
29			* @return string
30			*/
31			public function _change($element='this', $js='') {
32			return $this->_add_event($element, $js, 'change');
			0 ignored issues – show Bug introduced 2016-04-17 22:39 UTC by Report Bug Copy Issue Report It seems like `_add_event()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
33			}
34
35			// --------------------------------------------------------------------
36
37			/**
38			* Outputs a jQuery click event

jcheron / phalcon-jquery

Push — master ( 79b035...e37d9a )

JqueryEventsTrait::_change() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like