ts3RenderChannels()

Complexity

Conditions	6
Paths	5

Size

Total Lines	26
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function ts3RenderChannels(array $channels, array $clients, int $cid = null, int $depth = null):string{
	$str = '';

	foreach($channels as $channel){

		if($channel->pid === null){
			$depth = 0;
		}

		if($channel->pid === $cid){
			$str .= str_repeat('  ', $depth).$channel->channel_name."\n";

It seems like $depth can also be of type null; however, parameter $times of str_repeat() does only seem to accept integer, maybe add an additional type check?

			foreach($clients as $client){
				if($client->cid === $channel->cid){
					$str .= str_repeat('  ', $depth).'  - '.$client->client_nickname."\n";
				}
			}

			// parse nested channels recursively
			$str .= ts3RenderChannels($channels, $clients, $channel->cid, $depth);
		}

		$depth++;
	}

	return $str;
}