You are reviewing a merge request, and see the following code in the OtterService class:
protected $otterSession;
public function __construct(SessionManagerInterface $otterSession) {
$this-＞otterSession = $otterSession;
}
public function clearOtterSession() {
$this-＞otterSession-＞unsetData('otter_data');
}
❓ What (if any) changes do you request?
ℹ️ Looking for ways to improve your, or someone else's code?
Make sure you are utilizing the resources and features provided in the platform/language you are working with. PHP 8's promoted properties are a great example!
---
Connect with Joseph:
LinkedIn: / maxwelljoseph
Twitter: / swiftotter_joe
swiftotter.com
/ swift_otter
/ swiftotterinc
Have you joined the free SwiftOtter Slack community? It's exploding and we don't want you to miss out. Go to swiftotter.com/slack to join for free and get plugged into what might be the best group of collaborating developers around!