OCA\Chat\OCH\Data\GetUsers

Complexity

Total Complexity

4

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setRequestData()	0	3	1
A	__construct()	0	7	1
A	execute()	0	14	2

<?php
/**
 * Copyright (c) 2014, Tobia De Koninck hey--at--ledfan.be
 * This file is licensed under the AGPL version 3 or later.
 * See the COPYING file.
 */

namespace OCA\Chat\OCH\Data;

use \OCA\Chat\OCH\ChatAPI;
use \OCA\Chat\App\Chat;
use \OCA\Chat\OCH\Db\UserMapper;


class GetUsers extends ChatAPI {

	/**
	 * @var $userMapper \OCA\Chat\OCH\Db\UserMapper
	 */
	private $userMapper;

	public function __construct(
		Chat $app,
		UserMapper $userMapper
	){
		$this->app = $app;

The property app does not exist. Did you maybe forget to declare it?

		$this->userMapper = $userMapper;
	}

	/*
	 * @param $requestData['user'] String user id of the client
	 * @param $requestData['convid'] String session_id of the client
	 */
	public function setRequestData(array $requestData){
		$this->requestData = $requestData;
	}

	public function execute(){
		$contacts = $this->app->getContacts();
		$contacts = $contacts['contactsObj'];
		
		$users = $this->userMapper->findUsersInConv($this->requestData['conv_id']);
		
		$return = array();
		foreach($users as $user){
			$return[] = $contacts[$user];
		}

		// Note: users are full contacts
		return array("users" => $return);
	}
}