test: add unformatted PHP to trigger transformer by sawez-deepsource · Pull Request #1 · sawez-deepsource/demo-php

sawez-deepsource · 2026-03-23T06:21:28Z

Summary

Adds an intentionally unformatted PHP file to trigger the transformer/formatter pipeline
Used for observing transformer logs in GCP

Test plan

Verify transformer runs on this PR
Check GCP logs for transformer output

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

deepsource-io · 2026-03-23T06:21:50Z

DeepSource Code Review

We reviewed changes in 21e5e1d...e75fda1 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade	Security Reliability Complexity Hygiene

Code Review Summary

Analyzer	Status	Updated (UTC)	Details
PHP		Mar 23, 2026 6:28a.m.	Review ↗

deepsource-development · 2026-03-23T06:24:51Z

DeepSource Code Review

We reviewed changes in 21e5e1d...e75fda1 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade	Security Reliability Complexity Hygiene

Code Review Summary

Analyzer	Status	Updated (UTC)	Details
PHP		Mar 23, 2026 6:28a.m.	Review ↗

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

This commit fixes the style issues introduced in a902146 according to the output from PHP CS Fixer. Details: #1

deepsource-io · 2026-03-23T06:25:38Z

src/Controller/ApiController.php

+
+private function  handleGet(  $path,  $params  )
+{
+$cacheKey='get_'.md5(  $path.serialize($params)  );


Use of insecure md5() function found

Using md5(), sha1() function is not recommended to generate secure passwords. Due to its fast nature to compute passwords too quickly, these functions can become really easy to crack a password using brute force attack.

It is recommended to use PHP's password hashing function password_hash() to create a secure password hash.

deepsource-io · 2026-03-23T06:25:38Z

src/Service/ValidationService.php

+return   empty(  $this->errors  );
+}
+
+private function   validateRequired(  $field,  $value,  $params  )


The private method validateRequired() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:38Z

src/Service/ValidationService.php

+}
+}
+
+private function validateMin(  $field,  $value,  $params  )


The private method validateMin() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:38Z

src/Service/ValidationService.php

+}
+}
+
+private function   validateMax(  $field,  $value,  $params  )


The private method validateMax() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:38Z

src/Service/ValidationService.php

+}
+}
+
+private function  validateEmail(  $field,  $value,  $params  )


The private method validateEmail() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:38Z

src/Service/ValidationService.php

+}
+}
+
+private function  validateNumeric(  $field,  $value,  $params  )


The private method validateNumeric() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:38Z

src/Service/ValidationService.php

+}
+}
+
+private function  validateIn($field,$value,$params)


The private method validateIn() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:43Z

src/Controller/ApiController.php

+
+    private function handleGet($path, $params)
+    {
+        $cacheKey = 'get_'.md5($path.serialize($params));


Use of insecure md5() function found

Using md5(), sha1() function is not recommended to generate secure passwords. Due to its fast nature to compute passwords too quickly, these functions can become really easy to crack a password using brute force attack.

It is recommended to use PHP's password hashing function password_hash() to create a secure password hash.

deepsource-io · 2026-03-23T06:25:43Z

src/Service/ValidationService.php

+        return   empty($this->errors);
+    }
+
+    private function validateRequired($field, $value, $params)


The private method validateRequired() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:43Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateMin($field, $value, $params)


The private method validateMin() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:43Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateMax($field, $value, $params)


The private method validateMax() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:43Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateEmail($field, $value, $params)


The private method validateEmail() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:43Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateNumeric($field, $value, $params)


The private method validateNumeric() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:25:43Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateIn($field, $value, $params)


The private method validateIn() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

This commit fixes the style issues introduced in d21b458 according to the output from PHP CS Fixer. Details: #1

deepsource-development · 2026-03-23T06:28:55Z

src/Controller/ApiController.php

+
+    private function handleGet($path, $params)
+    {
+        $cacheKey = 'get_'.md5($path.serialize($params));


Use of insecure md5() function found

Using md5(), sha1() function is not recommended to generate secure passwords. Due to its fast nature to compute passwords too quickly, these functions can become really easy to crack a password using brute force attack.

It is recommended to use PHP's password hashing function password_hash() to create a secure password hash.

deepsource-development · 2026-03-23T06:28:55Z

src/Service/ValidationService.php

+        return   empty($this->errors);
+    }
+
+    private function validateRequired($field, $value, $params)


The private method validateRequired() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-development · 2026-03-23T06:28:55Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateMin($field, $value, $params)


The private method validateMin() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-development · 2026-03-23T06:28:55Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateMax($field, $value, $params)


The private method validateMax() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-development · 2026-03-23T06:28:55Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateEmail($field, $value, $params)


The private method validateEmail() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-development · 2026-03-23T06:28:55Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateNumeric($field, $value, $params)


The private method validateNumeric() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-development · 2026-03-23T06:28:55Z

src/Service/ValidationService.php

+        }
+    }
+
+    private function validateIn($field, $value, $params)


The private method validateIn() is not used

The class contains unused private methods. These private class methods are dead code, and do nothing. It is recommended to remove these unused private methods.

It is also recommended that you review these unused private methods if they were added for later use in the code, but the author missed using them.

deepsource-io · 2026-03-23T06:29:28Z

src/Repository/EventRepository.php

+
+class EventRepository
+{
+    private $events = [];


The private property $events is not used

The class contains unused private property. These private class properties should be removed as they can sometimes lead to confusion while reading the code.

It is also recommended that you review these unused private properties, in case they were added for later use in the code, but the author missed using them.

deepsource-development · 2026-03-23T06:32:12Z

src/Repository/EventRepository.php

+
+class EventRepository
+{
+    private $events = [];


The private property $events is not used

The class contains unused private property. These private class properties should be removed as they can sometimes lead to confusion while reading the code.

It is also recommended that you review these unused private properties, in case they were added for later use in the code, but the author missed using them.

test: add unformatted PHP file to trigger transformer

85c0692

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

sawez-deepsource and others added 2 commits March 23, 2026 11:55

test: add unformatted PHP services and controller

a902146

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

style: format code with PHP CS Fixer

48c8568

This commit fixes the style issues introduced in a902146 according to the output from PHP CS Fixer. Details: #1

deepsource-io bot reviewed Mar 23, 2026

View reviewed changes

sawez-deepsource and others added 2 commits March 23, 2026 11:58

test: add more unformatted PHP files for transformer

d21b458

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

style: format code with PHP CS Fixer

e75fda1

This commit fixes the style issues introduced in d21b458 according to the output from PHP CS Fixer. Details: #1

deepsource-development bot reviewed Mar 23, 2026

View reviewed changes

deepsource-io bot reviewed Mar 23, 2026

View reviewed changes

deepsource-development bot reviewed Mar 23, 2026

View reviewed changes

Conversation

sawez-deepsource commented Mar 23, 2026

Summary

Test plan

Uh oh!

deepsource-io bot commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

DeepSource Code Review

PR Report Card

Code Review Summary

Uh oh!

deepsource-development bot commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

DeepSource Code Review

PR Report Card

Code Review Summary

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

Use of insecure md5() function found

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateRequired() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateMin() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateMax() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateEmail() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateNumeric() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateIn() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

Use of insecure md5() function found

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateRequired() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateMin() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateMax() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateEmail() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateNumeric() is not used

Uh oh!

deepsource-io bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateIn() is not used

Uh oh!

deepsource-development bot Mar 23, 2026

Choose a reason for hiding this comment

Use of insecure md5() function found

Uh oh!

deepsource-development bot Mar 23, 2026

Choose a reason for hiding this comment

The private method validateRequired() is not used

deepsource-io bot commented Mar 23, 2026 •

edited

Loading

deepsource-development bot commented Mar 23, 2026 •

edited

Loading