for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
namespace OCA\Calendar\Service;
use Exception;
use OCP\AppFramework\Http;
use OCP\AppFramework\Db\DoesNotExistException;
use OCP\AppFramework\Db\MultipleObjectsReturnedException;
use OCA\Calendar\Db\OtoConfirmation;
use OCA\Calendar\Db\OtoConfirmationMapper;
class OtoConfirmationService {
private $mapper;
/**
* @NoAdminRequired
* @param string $appName
$appName
This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.
Consider the following example. The parameter $italy is not defined by the method finale(...).
$italy
finale(...)
/** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; }
The most likely cause is that the parameter was removed, but the annotation was not.
* @param IRequest $request an instance of the request
$request
* @param OtoConfirmationMapper $mapper the db mapper
*/
public function __construct(OtoConfirmationMapper $mapper){
$this->mapper = $mapper;
}
*
* @NoCSRFRequired
* @throws \OCP\AppFramework\Db\DoesNotExistException if not found
* @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException if more than one result
* @param integer $otoLayerId
$otoLayerId
* @param string $password
$password
public function getConfirmationsByUser($userId){
$foo = $this->mapper->getConfirmationsForUser($userId);
return($foo);
public function deleteConfirmationsByOtoLayer($otoLayerId){
$this->mapper->deleteConfirmationsByOtoLayer($otoLayerId);
public function deleteBySourceId($sourceId,$userId){
$this->mapper->deleteBySourceId($sourceId,$userId);
This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.
Consider the following example. The parameter
$italyis not defined by the methodfinale(...).The most likely cause is that the parameter was removed, but the annotation was not.