This is an automated email from the ASF dual-hosted git repository.

thiagohp pushed a commit to branch feature/coffeescript-to-typescript
in repository https://gitbox.apache.org/repos/asf/tapestry-5.git


The following commit(s) were added to 
refs/heads/feature/coffeescript-to-typescript by this push:
     new 03230ed0c TAP5-2804: fixed ElementWrapper.attr() in jQuery.
03230ed0c is described below

commit 03230ed0ce8131fdbaef28130b6c6f21bfdc3ee0
Author: Thiago H. de Paula Figueiredo <thi...@arsmachina.com.br>
AuthorDate: Tue Jun 3 10:33:23 2025 -0300

    TAP5-2804: fixed ElementWrapper.attr() in jQuery.
---
 tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts 
b/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
index 073a170fa..379a79809 100644
--- a/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
+++ b/tapestry-core/src/main/typescript/src/t5/core/t5-core-dom-jquery.ts
@@ -133,7 +133,7 @@ class JQueryElementWrapper implements ElementWrapper {
     }
     current = this.$.attr(name);
     if (arguments.length > 1) {
-      if (value === null) {
+      if (value === null || value === undefined) {
         this.$.removeAttr(name);
       } else {
         this.$.attr(name, _.isString(value) ? value : String(value));

Reply via email to