[TAP5-1177] In a partial page update request, if the server returns an empty string for the content, the client-side is left as is, rather than cleared

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.2.0
• Fix Version/s: 5.2.1
• Component/s: tapestry-core
• Assignee: Howard Lewis Ship
• Reporter: Howard Lewis Ship
• Created: 04/Jun/10 17:53
• Updated: 20/Sep/10 22:44
• Resolved: 20/Sep/10 21:41

Description

Workaround: place an empty comment inside the content.

The issue is here:

processReply : function(reply) {
Tapestry.loadScriptsInReply(reply, function() {
/*

• In a multi-zone update, the reply.content may be blank or
• missing.
  */

reply.content && this.show(reply.content);

In this situation the reply looks like:

{
"content": ""
}

And the empty string is falsey; it short circuits the && operator.

A check for undefined is more appropriate (this may be the case in a multi-zone update).