refactor: unblocked against laravel#57467

Author: NickSdot

src/Illuminate/View/ComponentAttributeBag.php

@@ -491,25 +491,14 @@ public function __toString()
                 continue;
             }
 
-            // Empty "alt" explicitly marks an image as decorative; empty "value" and "data-*"
-            // are valid. For everything else we don't want to render the attribute at all.
-            if (!in_array($key, ['alt', 'value']) && !str_starts_with($key, 'data') && (is_string($value) && trim($value) === '')) {
-                continue;
-            }
-
             if ($value === true) {
                 $value = $key === 'x-data' || str_starts_with($key, 'wire:') ? '' : $key;
             }
 
-            // Same as above. We want everything but data and value to be trimmed
-            $value = !str_starts_with($key, 'data') && $key !== 'value'
-                ? trim($value)
-                : $value;
-
             if($key === $value) {
                 $string .= ' '.$key;
             } else {
-                $string .= ' '.$key.'="'.str_replace('"', '\\"', $value).'"';
+                $string .= ' ' . $key . '="' . str_replace('"', '\\"', trim($value)) . '"';
             }
         }
 

tests/View/ViewComponentAttributeBagTest.php

@@ -51,15 +51,10 @@ public function testAttributeRetrieval()
             'test-0' => 0,
             'test-0-string' => '0',
             'test-empty-string' => '',
-            'test-whitespace-string' => ' ',
-            'data-test-empty-string' => '',
-            'data-test-whitespace-string' => ' ',
-            'value' => ' ',
-            'alt' => '',
         ]);
 
-        $this->assertSame('test-string="ok" test-true test-0="0" test-0-string="0" data-test-empty-string="" data-test-whitespace-string=" " value=" " alt=""', (string) $bag);
-        $this->assertSame('test-string="ok" test-true test-0="0" test-0-string="0" data-test-empty-string="" data-test-whitespace-string=" " value=" " alt=""', (string) $bag->merge());
+        $this->assertSame('test-string="ok" test-true test-0="0" test-0-string="0" test-empty-string=""', (string) $bag);
+        $this->assertSame('test-string="ok" test-true test-0="0" test-0-string="0" test-empty-string=""', (string) $bag->merge());
 
         $bag = (new ComponentAttributeBag)
             ->merge([
@@ -77,14 +72,10 @@ public function testAttributeRetrieval()
                 'test-0' => 0,
                 'test-0-string' => '0',
                 'test-empty-string' => '',
-                'test-whitespace-string' => ' ',
-                'data-test-empty-string' => '',
-                'data-test-whitespace-string' => ' ',
-                'value' => ' ',
-                'alt' => '',
             ]);
 
-        $this->assertSame('test-string="ok" test-true test-0="0" test-0-string="0" data-test-empty-string="" data-test-whitespace-string=" " value=" " alt=""', (string) $bag);
+        $this->assertSame('test-string="ok" test-true test-0="0" test-0-string="0" test-empty-string=""', (string) $bag);
+
         $bag = (new ComponentAttributeBag)
             ->merge([
                 'test-extract-1' => 'extracted-1',