Users_ExportData_Action

Complexity

Total Complexity

15

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	0	10	1
C	setDataFromRequest()	0	29	12
A	checkPermission()	0	5	2

<?php
/**
 * Export data file.
 *
 * @package Action
 *
 * @copyright YetiForce S.A.
 * @license   YetiForce Public License 3.0 (licenses/LicenseEN.txt or yetiforce.com)
 * @author    Radosław Skrzypczak <r.skrzypczak@yetiforce.com>
 */
/**
 * Export data class.
 */
class Users_ExportData_Action extends Vtiger_ExportData_Action

The type Vtiger_ExportData_Action was not found. Maybe you did not declare it correctly or list all dependencies?

{
	/** {@inheritdoc} */
	public function checkPermission(App\Request $request)
	{
		$this->moduelName = $request->getModule();

The property moduelName does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		if (!\App\User::getCurrentUserModel()->isAdmin()) {
			throw new \App\Exceptions\NoPermitted('LBL_PERMISSION_DENIED', 406);
		}
	}

	/** {@inheritdoc} */
	public function process(App\Request $request)
	{
		$this->exportModel = \App\Export\ExportRecords::getInstance($this->moduelName, 'csv')

The property exportModel does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

			->setLimit(\App\Config::performance('MAX_NUMBER_EXPORT_RECORDS'))
			->setFormat(\App\Export\ExportRecords::EXPORT_FORMAT);
		$this->exportModel->fullData = true;

		$this->setDataFromRequest($request);
		$this->exportModel->sendHttpHeader();
		$this->exportModel->exportData();
	}

	/** {@inheritdoc} */
	public function setDataFromRequest(App\Request $request)
	{
		$searchParams = \App\Condition::validSearchParams($this->moduelName, $request->getArray('search_params'));
		$operator = $request->isEmpty('operator') ? '' : $request->getByType('operator');
		if ($operator && $searchValue = \App\Condition::validSearchValue($request->getByType('search_value', \App\Purifier::TEXT), $this->moduelName, $request->getByType('search_key', \App\Purifier::ALNUM), $operator)) {
			$searchKey = $request->getByType('search_key', \App\Purifier::ALNUM);
		}

		$queryGenerator = $this->exportModel->getQueryGenerator();
		$queryGenerator->addCondition('status', 'Active', 'e');

		switch ($request->getMode()) {
				case 'ExportAllData':
					break;
				case 'ExportSelectedRecords':
					$selectedIds = $request->getArray('selected_ids', \App\Purifier::ALNUM);
					if ($selectedIds && 'all' !== $selectedIds[0]) {

The expression $selectedIds of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

						$queryGenerator->addCondition('id', $selectedIds, 'e');
					}
					$queryGenerator->parseAdvFilter($queryGenerator->parseBaseSearchParamsToCondition($searchParams));
					if ($excludedIds = $request->getArray('excluded_ids', \App\Purifier::INTEGER)) {
						$queryGenerator->addCondition('id', $excludedIds, 'n');
					}
					if ($operator && $searchValue && $searchKey) {

The variable $searchValue does not seem to be defined for all execution paths leading up to this point.

The variable $searchKey does not seem to be defined for all execution paths leading up to this point.

						$queryGenerator->addCondition($searchKey, $searchValue, $operator);
					}
					break;
				default:
					throw new \App\Exceptions\IllegalValue('ERR_FIELD_NOT_FOUND||mode');
			}
	}
}