Fixed issue with user delete ownership not migrating

Caused by input not being part of the submitted form.
Updated test to ensure the input is within a form.
For #4124

diff --git a/resources/views/users/delete.blade.php b/resources/views/users/delete.blade.php
index b18c182ebde971b6016d47836d333ad0a2b8fc6f..b2f08b6412a64f7a623280fa1c967c670d4c50dc 100644 (file)
--- a/resources/views/users/delete.blade.php
+++ b/resources/views/users/delete.blade.php
@@ -5,40 +5,40 @@
 
         @include('settings.parts.navbar', ['selected' => 'users'])
 
-        <div class="card content-wrap auto-height">
-            <h1 class="list-heading">{{ trans('settings.users_delete') }}</h1>
+        <form action="{{ url("/settings/users/{$user->id}") }}" method="POST">
+            {!! csrf_field() !!}
 
-            <p>{{ trans('settings.users_delete_warning', ['userName' => $user->name]) }}</p>
+            <div class="card content-wrap auto-height">
+                <h1 class="list-heading">{{ trans('settings.users_delete') }}</h1>
 
-            @if(userCan('users-manage'))
-                <hr class="my-l">
+                <p>{{ trans('settings.users_delete_warning', ['userName' => $user->name]) }}</p>
 
-                <div class="grid half gap-xl v-center">
-                    <div>
-                        <label class="setting-list-label">{{ trans('settings.users_migrate_ownership') }}</label>
-                        <p class="small">{{ trans('settings.users_migrate_ownership_desc') }}</p>
-                    </div>
-                    <div>
-                        @include('form.user-select', ['name' => 'new_owner_id', 'user' => null])
-                    </div>
-                </div>
-            @endif
+                @if(userCan('users-manage'))
+                    <hr class="my-l">
 
-            <hr class="my-l">
+                    <div class="grid half gap-xl v-center">
+                        <div>
+                            <label class="setting-list-label">{{ trans('settings.users_migrate_ownership') }}</label>
+                            <p class="small">{{ trans('settings.users_migrate_ownership_desc') }}</p>
+                        </div>
+                        <div>
+                            @include('form.user-select', ['name' => 'new_owner_id', 'user' => null])
+                        </div>
+                    </div>
+                @endif
 
-            <div class="grid half">
-                <p class="text-neg"><strong>{{ trans('settings.users_delete_confirm') }}</strong></p>
-                <div>
-                    <form action="{{ url("/settings/users/{$user->id}") }}" method="POST" class="text-right">
-                        {!! csrf_field() !!}
+                <hr class="my-l">
 
+                <div class="grid half">
+                    <p class="text-neg"><strong>{{ trans('settings.users_delete_confirm') }}</strong></p>
+                    <div class="text-right">
                         <input type="hidden" name="_method" value="DELETE">
                         <a href="{{ url("/settings/users/{$user->id}") }}" class="button outline">{{ trans('common.cancel') }}</a>
                         <button type="submit" class="button">{{ trans('common.confirm') }}</button>
-                    </form>
+                    </div>
                 </div>
-            </div>
 
-        </div>
+            </div>
+        </form>
     </div>
 @stop

diff --git a/tests/User/UserManagementTest.php b/tests/User/UserManagementTest.php
index af17db52b525086d9be73ee577651f09712f13e7..d5cd7ac7963d671636a6b4ea12f9ca656fcfb6b7 100644 (file)
--- a/tests/User/UserManagementTest.php
+++ b/tests/User/UserManagementTest.php
@@ -145,6 +145,7 @@ class UserManagementTest extends TestCase
 
         $resp = $this->asEditor()->get("settings/users/{$editor->id}/delete");
         $resp->assertSee('Migrate Ownership');
+        $this->withHtml($resp)->assertElementExists('form input[name="new_owner_id"]');
         $resp->assertSee('new_owner_id');
     }