Code fix for pull request 165
This commit is contained in:
parent
646c2e263a
commit
48352fce63
@ -36,14 +36,14 @@ class Topic implements \IteratorAggregate, \Countable {
|
|||||||
* @param array $eligible
|
* @param array $eligible
|
||||||
* @return Topic
|
* @return Topic
|
||||||
*/
|
*/
|
||||||
public function broadcast($msg, array $exclude, array $eligible) {
|
public function broadcast($msg, array $exclude = array(), array $eligible = array()) {
|
||||||
$useEligible = count($eligible);
|
$useEligible = count($eligible);
|
||||||
foreach ($this->subscribers as $client) {
|
foreach ($this->subscribers as $client) {
|
||||||
if(in_array($client->getSessionId(), $exclude)) {
|
if(in_array($client->WAMP->sessionId, $exclude)) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
if($useEligible && !in_array($client->getSessionId(), $eligible)) {
|
if($useEligible && !in_array($client->WAMP->sessionId, $eligible)) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -100,10 +100,4 @@ class WampConnection extends AbstractConnectionDecorator {
|
|||||||
$this->getConnection()->close($opt);
|
$this->getConnection()->close($opt);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Get session ID
|
|
||||||
*/
|
|
||||||
public function getSessionId() {
|
|
||||||
return $this->WAMP->sessionId;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
@ -57,7 +57,7 @@ class TopicTest extends \PHPUnit_Framework_TestCase {
|
|||||||
$topic->add($first);
|
$topic->add($first);
|
||||||
$topic->add($second);
|
$topic->add($second);
|
||||||
|
|
||||||
$topic->broadcast($msg, array(), array());
|
$topic->broadcast($msg);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testIterator() {
|
public function testIterator() {
|
||||||
|
Loading…
Reference in New Issue
Block a user