GroupBundle/Controller/Api/GroupController.php
Avoid using static access to class '\OpenOrchestra\Pagination\Configuration\PaginateFinderConfiguration' in method 'listAction'. Open
Open
$configuration = PaginateFinderConfiguration::generateFromRequest($request, $mapping);
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
StaticAccess
Since: 1.4.0
Static access causes unexchangeable dependencies to other classes and leads to hard to test code. Avoid using static access at all costs and instead inject dependencies through the constructor. The only case when static access is acceptable is when used for factory methods.
Example
class Foo
{
public function bar()
{
Bar::baz();
}
}
Source https://phpmd.org/rules/cleancode.html#staticaccess
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$facade = $this->get('jms_serializer')->deserialize(
- Create a ticketCreate a ticket
- Exclude checks
Parameter $request
has undeclared type \Symfony\Component\HttpFoundation\Request
Open
Open
public function listAction(Request $request, $withCount)
- Create a ticketCreate a ticket
- Exclude checks
Return type of duplicateAction()
is undeclared type \Symfony\Component\HttpFoundation\Response
Open
Open
public function duplicateAction(Request $request)
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$currentSite = $this->get('open_orchestra_model.repository.site')->findOneBySiteId($currentSiteId);
- Create a ticketCreate a ticket
- Exclude checks
Return type of listAction()
is undeclared type \OpenOrchestra\BaseApi\Facade\FacadeInterface
Open
Open
public function listAction(Request $request, $withCount)
- Create a ticketCreate a ticket
- Exclude checks
Call to method __construct
from undeclared class \OpenOrchestra\UserBundle\Event\GroupEvent
Open
Open
$this->dispatchEvent(GroupEvents::GROUP_DELETE, new GroupEvent($group));
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant GROUP_DELETE
from undeclared class \OpenOrchestra\UserBundle\GroupEvents
Open
Open
$this->dispatchEvent(GroupEvents::GROUP_DELETE, new GroupEvent($group));
- Create a ticketCreate a ticket
- Exclude checks
Return type of deleteGroupsAction()
is undeclared type \Symfony\Component\HttpFoundation\Response
Open
Open
public function deleteGroupsAction(Request $request)
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$groupRepository = $this->get('open_orchestra_user.repository.group');
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$repository = $this->get('open_orchestra_user.repository.group');
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$facade = $this->get('open_orchestra_api.transformer_manager')->transform('group_collection', $collection, array('nbrGroupsUsers' => $nbrGroupsUsers));
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$this->get('open_orchestra_backoffice.business_rules_manager')->isGranted(BusinessActionInterface::DELETE, $group, $nbrGroupsUsers)
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::denyAccessUnlessGranted
Open
Open
$this->denyAccessUnlessGranted(ContributionActionInterface::DELETE, $group);
- Create a ticketCreate a ticket
- Exclude checks
Class extends undeclared class \OpenOrchestra\BaseApiBundle\Controller\BaseController
Open
Open
class GroupController extends BaseController
- Create a ticketCreate a ticket
- Exclude checks
Call to method getContent
from undeclared class \Symfony\Component\HttpFoundation\Request
Open
Open
$request->getContent(),
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$nbrGroupsUsers = $this->get('open_orchestra_user.repository.user')->getCountsUsersByGroups($filter);
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::dispatchEvent
Open
Open
$this->dispatchEvent(GroupEvents::GROUP_DELETE, new GroupEvent($group));
- Create a ticketCreate a ticket
- Exclude checks
Call to method __construct
from undeclared class \OpenOrchestra\UserBundle\Event\GroupEvent
Open
Open
$this->dispatchEvent(GroupEvents::GROUP_DELETE, new GroupEvent($group));
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::getParameter
Open
Open
$this->getParameter('open_orchestra_group.facade.group_collection.class'),
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$objectManager = $this->get('object_manager');
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::isGranted
Open
Open
if ($this->isGranted(ContributionActionInterface::DELETE, $group) &&
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$group = $this->get('open_orchestra_user.repository.group')->find($groupId);
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::denyAccessUnlessGranted
Open
Open
$this->denyAccessUnlessGranted(ContributionActionInterface::CREATE, GroupInterface::ENTITY_TYPE);
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$group = $this->get('open_orchestra_api.transformer_manager')->reverseTransform('group', $facade);
- Create a ticketCreate a ticket
- Exclude checks
Returning type array{}
but duplicateAction()
is declared to return \Symfony\Component\HttpFoundation\Response
Open
Open
return array();
- Create a ticketCreate a ticket
- Exclude checks
Call to method generateFromRequest
from undeclared class \OpenOrchestra\Pagination\Configuration\PaginateFinderConfiguration
Open
Open
$configuration = PaginateFinderConfiguration::generateFromRequest($request, $mapping);
- Create a ticketCreate a ticket
- Exclude checks
Reference to constant GROUP_DELETE
from undeclared class \OpenOrchestra\UserBundle\GroupEvents
Open
Open
$this->dispatchEvent(GroupEvents::GROUP_DELETE, new GroupEvent($group));
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$facade = $this->get('jms_serializer')->deserialize(
- Create a ticketCreate a ticket
- Exclude checks
Parameter $request
has undeclared type \Symfony\Component\HttpFoundation\Request
Open
Open
public function deleteGroupsAction(Request $request)
- Create a ticketCreate a ticket
- Exclude checks
Call to method get
from undeclared class \Symfony\Component\HttpFoundation\Request
Open
Open
$format = $request->get('_format', 'json');
- Create a ticketCreate a ticket
- Exclude checks
Return type of deleteAction()
is undeclared type \Symfony\Component\HttpFoundation\Response
Open
Open
public function deleteAction($groupId)
- Create a ticketCreate a ticket
- Exclude checks
Returning type array{}
but deleteAction()
is declared to return \Symfony\Component\HttpFoundation\Response
Open
Open
return array();
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::denyAccessUnlessGranted
Open
Open
$this->denyAccessUnlessGranted(ContributionActionInterface::READ, GroupInterface::ENTITY_TYPE);
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$nbrGroupsUsers = $this->get('open_orchestra_user.repository.user')->getCountsUsersByGroups($filter);
- Create a ticketCreate a ticket
- Exclude checks
Call to method get
from undeclared class \Symfony\Component\HttpFoundation\Request
Open
Open
$format = $request->get('_format', 'json');
- Create a ticketCreate a ticket
- Exclude checks
Call to method getContent
from undeclared class \Symfony\Component\HttpFoundation\Request
Open
Open
$request->getContent(),
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$currentSiteId = $this->get('open_orchestra_backoffice.context_backoffice_manager')->getSiteId();
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$objectManager = $this->get('object_manager');
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$groups = $this->get('open_orchestra_api.transformer_manager')->reverseTransform('group_collection', $facade);
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::getParameter
Open
Open
$this->getParameter('open_orchestra_group.facade.group.class'),
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
if ($group instanceof GroupInterface && $this->get('open_orchestra_backoffice.business_rules_manager')->isGranted(BusinessActionInterface::DELETE, $group)) {
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::dispatchEvent
Open
Open
$this->dispatchEvent(GroupEvents::GROUP_DELETE, new GroupEvent($group));
- Create a ticketCreate a ticket
- Exclude checks
Possibly zero references to use statement for classlike/namespace Api
(\OpenOrchestra\BaseApiBundle\Controller\Annotation)
Open
Open
use OpenOrchestra\BaseApiBundle\Controller\Annotation as Api;
- Create a ticketCreate a ticket
- Exclude checks
Parameter $request
has undeclared type \Symfony\Component\HttpFoundation\Request
Open
Open
public function duplicateAction(Request $request)
- Create a ticketCreate a ticket
- Exclude checks
Call to undeclared method \OpenOrchestra\GroupBundle\Controller\Api\GroupController::get
Open
Open
$availableSites = $this->get('open_orchestra_backoffice.context_backoffice_manager')->getAvailableSites();
- Create a ticketCreate a ticket
- Exclude checks
Possibly zero references to use statement for classlike/namespace Config
(\Sensio\Bundle\FrameworkExtraBundle\Configuration)
Open
Open
use Sensio\Bundle\FrameworkExtraBundle\Configuration as Config;
- Create a ticketCreate a ticket
- Exclude checks
Returning type array{}
but deleteGroupsAction()
is declared to return \Symfony\Component\HttpFoundation\Response
Open
Open
return array();
- Create a ticketCreate a ticket
- Exclude checks
Each PHP statement must be on a line by itself Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Expected 1 space after FUNCTION keyword; 0 found Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Expected 1 space after FUNCTION keyword; 0 found Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Each PHP statement must be on a line by itself Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Line exceeds 120 characters; contains 165 characters Open
Open
if ($group instanceof GroupInterface && $this->get('open_orchestra_backoffice.business_rules_manager')->isGranted(BusinessActionInterface::DELETE, $group)) {
- Create a ticketCreate a ticket
- Exclude checks
Line exceeds 120 characters; contains 147 characters Open
Open
$this->get('open_orchestra_backoffice.business_rules_manager')->isGranted(BusinessActionInterface::DELETE, $group, $nbrGroupsUsers)
- Create a ticketCreate a ticket
- Exclude checks
Line exceeds 120 characters; contains 159 characters Open
Open
$facade = $this->get('open_orchestra_api.transformer_manager')->transform('group_collection', $collection, array('nbrGroupsUsers' => $nbrGroupsUsers));
- Create a ticketCreate a ticket
- Exclude checks
Closing brace must be on a line by itself Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Opening brace must be the last content on the line Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Closing brace must be on a line by itself Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks
Opening brace must be the last content on the line Open
Open
array_walk($filter, function(&$item) {$item = $item->getId();});
- Create a ticketCreate a ticket
- Exclude checks