Method registerEventListeners
has 35 lines of code (exceeds 25 allowed). Consider refactoring. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed. Open
public function registerEventListeners(): void
- Read upRead up
- Exclude checks
Cognitive Complexity is a measure of how hard the control flow of a function is to understand. Functions with high Cognitive Complexity will be difficult to maintain.
See
Avoid using static access to class '\Illuminate\Support\Facades\Log' in method 'send'. Open
Log::error($exception, ['api_response' => (string) $exception->getResponse()->getBody()]);
- Read upRead up
- 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
Avoid using static access to class '\Illuminate\Support\Facades\Log' in method 'send'. Open
Log::error($t->getMessage());
- Read upRead up
- 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
The parameter $transaction_name is not named in camelCase. Open
protected function stopTransaction(string $transaction_name, int $result): void
{
// Stop the transaction and measure the time
$this->agent->stopTransaction($transaction_name, ['result' => $result]);
$this->agent->collectEvents($transaction_name);
- Read upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}
Source
The parameter $transaction_name is not named in camelCase. Open
protected function startTransaction(string $transaction_name): Transaction
{
return $this->agent->startTransaction(
$transaction_name,
[],
- Read upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}
Source
The parameter $transaction_name is not named in camelCase. Open
protected function setTransactionType(string $transaction_name): void
{
$this->agent->getTransaction($transaction_name)->setMeta([
'type' => 'job',
]);
- Read upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}
Source
The parameter $transaction_name is not named in camelCase. Open
protected function addMetadata(string $transaction_name, Job $job): void
{
$this->agent->getTransaction($transaction_name)->setCustomContext([
'job_id' => $job->getJobId(),
'max_tries' => $job->maxTries(),
- Read upRead up
- Exclude checks
CamelCaseParameterName
Since: 0.2
It is considered best practice to use the camelCase notation to name parameters.
Example
class ClassName {
public function doSomething($user_name) {
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function startTransaction(string $transaction_name): Transaction
{
return $this->agent->startTransaction(
$transaction_name,
[],
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function stopTransaction(string $transaction_name, int $result): void
{
// Stop the transaction and measure the time
$this->agent->stopTransaction($transaction_name, ['result' => $result]);
$this->agent->collectEvents($transaction_name);
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function getTransactionName($event): string
{
$transaction_name = $event->job->resolveName();
return $this->shouldIgnoreTransaction($transaction_name) ? '' : $transaction_name;
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function setTransactionType(string $transaction_name): void
{
$this->agent->getTransaction($transaction_name)->setMeta([
'type' => 'job',
]);
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function getTransactionName($event): string
{
$transaction_name = $event->job->resolveName();
return $this->shouldIgnoreTransaction($transaction_name) ? '' : $transaction_name;
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function stopTransaction(string $transaction_name, int $result): void
{
// Stop the transaction and measure the time
$this->agent->stopTransaction($transaction_name, ['result' => $result]);
$this->agent->collectEvents($transaction_name);
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function addMetadata(string $transaction_name, Job $job): void
{
$this->agent->getTransaction($transaction_name)->setCustomContext([
'job_id' => $job->getJobId(),
'max_tries' => $job->maxTries(),
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
public function registerEventListeners(): void
{
$this->app->events->listen(JobProcessing::class, function (JobProcessing $event) {
if ($this->app->runningInConsole()) {
// Since the application starts only once for async queues, make sure
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}
Source
The variable $transaction_name is not named in camelCase. Open
protected function getTransactionName($event): string
{
$transaction_name = $event->job->resolveName();
return $this->shouldIgnoreTransaction($transaction_name) ? '' : $transaction_name;
- Read upRead up
- Exclude checks
CamelCaseVariableName
Since: 0.2
It is considered best practice to use the camelCase notation to name variables.
Example
class ClassName {
public function doSomething() {
$data_module = new DataModule();
}
}