Skip to content

Commit 77124af

Browse files
committed
minor #10717 unified return null usages (fabpot)
This PR was merged into the 2.3 branch. Discussion ---------- unified return null usages | Q | A | ------------- | --- | License | MIT This PR unifies the way we return `null` from a function or method: * always use `return;` instead of `return null;` (the current code base uses both); * never use `return;` at the end of a function/method. Commits ------- d1d569b unified return null usages
2 parents d9f9e48 + 8a9d682 commit 77124af

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

Firewall/ContextListener.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ private function refreshUser(TokenInterface $token)
171171
$this->logger->warning(sprintf('Username "%s" could not be found.', $notFound->getUsername()));
172172
}
173173

174-
return null;
174+
return;
175175
}
176176
}
177177

RememberMe/AbstractRememberMeServices.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,6 @@ final public function autoLogin(Request $request)
142142
}
143143

144144
$this->cancelCookie($request);
145-
146-
return null;
147145
}
148146

149147
/**

0 commit comments

Comments
 (0)