fix(#19010): [autofocus] update autofocus behavior for undefined values

Author: tHe-AK

packages/primeng/src/autofocus/autofocus.spec.ts

@@ -33,6 +33,15 @@ class TestAutofocusDynamicComponent {
     autofocusEnabled = false;
 }
 
+@Component({
+    standalone: false,
+    selector: 'test-autofocus-undefined',
+    template: `<input type="text" [pAutoFocus]="autofocus" />`
+})
+class TestAutofocusUndefinedComponent {
+    autofocus: any = undefined;
+}
+
 @Component({
     standalone: false,
     selector: 'test-autofocus-button',
@@ -198,6 +207,7 @@ describe('AutoFocus', () => {
                 TestAutofocusDisabledComponent,
                 TestAutofocusEnabledComponent,
                 TestAutofocusDynamicComponent,
+                TestAutofocusUndefinedComponent,
                 TestAutofocusButtonComponent,
                 TestAutofocusDivComponent,
                 TestAutofocusMultipleElementsComponent,
@@ -300,6 +310,38 @@ describe('AutoFocus', () => {
         });
     });
 
+    describe('Undefined Binding', () => {
+        it('should not set autofocus attribute when bound value is undefined', async () => {
+            const fixture = TestBed.createComponent(TestAutofocusUndefinedComponent);
+            const component = fixture.componentInstance;
+            await fixture.whenStable();
+
+            const element = fixture.debugElement.query(By.css('input')).nativeElement;
+
+            spyOn(element, 'focus');
+
+            expect(element.hasAttribute('autofocus')).toBe(false);
+
+            await new Promise((resolve) => setTimeout(resolve, 10));
+            expect(element.focus).not.toHaveBeenCalled();
+        });
+
+        it('should set autofocus attribute and focus when property changes to true', async () => {
+            const fixture = TestBed.createComponent(TestAutofocusUndefinedComponent);
+            const component = fixture.componentInstance;
+            await fixture.whenStable();
+
+            const element = fixture.debugElement.query(By.css('input')).nativeElement;
+
+            component.autofocus = true;
+            fixture.changeDetectorRef.markForCheck();
+            await fixture.whenStable();
+
+            expect(element.hasAttribute('autofocus')).toBe(true);
+            expect(element.getAttribute('autofocus')).toBe('true');
+        });
+    });
+
     describe('Focus Behavior - Browser Platform', () => {
         it('should focus input element when autofocus is enabled', async () => {
             const fixture = TestBed.createComponent(TestAutofocusEnabledComponent);

packages/primeng/src/autofocus/autofocus.ts

@@ -28,7 +28,7 @@ export class AutoFocus extends BaseComponent {
 
     onAfterContentChecked() {
         // This sets the `attr.autofocus` which is different than the Input `autofocus` attribute.
-        if (this.autofocus === false) {
+        if (this.autofocus) {
             this.host.nativeElement.removeAttribute('autofocus');
         } else {
             this.host.nativeElement.setAttribute('autofocus', true);