Skip to content

Commit

Permalink
Making $data parameter required in Set::matches.
Browse files Browse the repository at this point in the history
Refs #993.

This has been a leftover of a previous refactor (9084d69).
This patch changes signature as per documentation.

It can be expected that as the second parameter must be given, the first
must also always be given, so this doesn't break BC.
  • Loading branch information
mariuswilms committed Jan 7, 2014
1 parent 0338402 commit 7b1f156
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion util/Set.php
Original file line number Diff line number Diff line change
Expand Up @@ -584,7 +584,7 @@ public static function isNumeric($array = null) {
* @param integer $length
* @return boolean
*/
public static function matches($data = array(), $conditions, $i = null, $length = null) {
public static function matches($data, $conditions, $i = null, $length = null) {
if (!$conditions) {
return true;
}
Expand Down

0 comments on commit 7b1f156

Please sign in to comment.