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));