Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jsf.ajax.response <update id='id'> clarification needed #646

Closed
eclipse-faces-bot opened this issue Oct 7, 2009 · 16 comments
Closed

jsf.ajax.response <update id='id'> clarification needed #646

eclipse-faces-bot opened this issue Oct 7, 2009 · 16 comments

Comments

@eclipse-faces-bot
Copy link

The JSDocs say:

  1. For any other element:

Find the DOM element with the identifier that matches the element
identifier, and replace its contents with the element's CDATA contents.

while true, this is misleading - the contents of the CDATA should only be a
single element. There's no way to know how to handle multiple elements in the
update id= case.

Environment

Operating System: All
Platform: All

Affected Versions

[2.0]

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Reported by driscoll

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Was assigned to rogerk

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Prepare to delete "spec" subcomponent.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
cat1

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
jsdocs

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
These are valid 2.0 Rev a issues

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
take ownership.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
rogerk said:
ownership

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
rogerk said:
starting

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
rogerk said:
Personally, I think the jsdocs are clear here, however, I've added the following
text to the update element processing section:

"The contents of the CDATA section is the data that will be used when updating
the contents of the DOM element as specified by the element identifier. "

There seemed to have been some confusion as to what the contents of the CDATA
section was used for, since some folks had specified multiple elements in the
CDATA section itself, expecting multiple elements to be updated.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
rogerk said:
checked in.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
rogerk said:
changelog

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Marked as fixed on Tuesday, June 22nd 2010, 12:02:35 am

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@manfredriem said:
Closing resolved issue out

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-646

@eclipse-faces-bot
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants