Fix incorrect annotations in Livewire components using "@return void" (#1467)

coreymcmahon · driesvints · web-flow · commit e9ca098e74b9 · 2024-04-03T09:41:53.000-05:00
* Fix type annotations in Livewire components using "@return void" when a return value is possible * Update src/Http/Livewire/UpdateProfileInformationForm.php * Update src/Http/Livewire/TeamMemberManager.php * Update src/Http/Livewire/UpdateProfileInformationForm.php --------- Co-authored-by: Dries Vints <dries@vints.be>
diff --git a/src/Http/Livewire/CreateTeamForm.php b/src/Http/Livewire/CreateTeamForm.php
@@ -22,7 +22,7 @@ class CreateTeamForm extends Component
      * Create a new team.
      *
      * @param  \Laravel\Jetstream\Contracts\CreatesTeams  $creator
-     * @return void
+     * @return mixed
      */
     public function createTeam(CreatesTeams $creator)
     {
diff --git a/src/Http/Livewire/TeamMemberManager.php b/src/Http/Livewire/TeamMemberManager.php
@@ -183,7 +183,7 @@ public function stopManagingRole()
      * Remove the currently authenticated user from the team.
      *
      * @param  \Laravel\Jetstream\Contracts\RemovesTeamMembers  $remover
-     * @return void
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function leaveTeam(RemovesTeamMembers $remover)
     {
diff --git a/src/Http/Livewire/UpdateProfileInformationForm.php b/src/Http/Livewire/UpdateProfileInformationForm.php
@@ -50,7 +50,7 @@ public function mount()
      * Update the user's profile information.
      *
      * @param  \Laravel\Fortify\Contracts\UpdatesUserProfileInformation  $updater
-     * @return void
+     * @return \Illuminate\Http\RedirectResponse|null
      */
     public function updateProfileInformation(UpdatesUserProfileInformation $updater)
     {

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ class CreateTeamForm extends Component`
`22`	`22`	`* Create a new team.`
`23`	`23`	`*`
`24`	`24`	`* @param \Laravel\Jetstream\Contracts\CreatesTeams $creator`
`25`		`- * @return void`
	`25`	`+ * @return mixed`
`26`	`26`	`*/`
`27`	`27`	`public function createTeam(CreatesTeams $creator)`
`28`	`28`	`{`
Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,7 @@ public function stopManagingRole()`
`183`	`183`	`* Remove the currently authenticated user from the team.`
`184`	`184`	`*`
`185`	`185`	`* @param \Laravel\Jetstream\Contracts\RemovesTeamMembers $remover`
`186`		`- * @return void`
	`186`	`+ * @return \Illuminate\Http\RedirectResponse`
`187`	`187`	`*/`
`188`	`188`	`public function leaveTeam(RemovesTeamMembers $remover)`
`189`	`189`	`{`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function mount()`
`50`	`50`	`* Update the user's profile information.`
`51`	`51`	`*`
`52`	`52`	`* @param \Laravel\Fortify\Contracts\UpdatesUserProfileInformation $updater`
`53`		`- * @return void`
	`53`	`+ * @return \Illuminate\Http\RedirectResponse\|null`
`54`	`54`	`*/`
`55`	`55`	`public function updateProfileInformation(UpdatesUserProfileInformation $updater)`
`56`	`56`	`{`