| <html><head><META http-equiv="Content-Type" content="text/html; charset=UTF-8"><link type="text/css" href="../../../default_style.css" rel="stylesheet"><link type="text/css" href="../../../webtools/wtp.css" rel="stylesheet"><title>the javaserver faces tools subproject</title></head><body><table border="0" cellpadding="2" cellspacing="5" width="100%"><tbody><tr><td align="left" width="60%"><font class="indextop">jsf</font><br><font class="indexsub">the javaserver faces tools subproject</font></td><td width="40%"><img src="../../../webtools/images/wtplogosmall.jpg" align="middle" height="129" hspace="50" width="207" alt="WTP Logo" usemap="logomap"><map id="logomap" name="logomap"><area coords="0,0,207,129" href="/webtools/" alt="WTP Home"></map></td></tr></tbody></table><table border="0" cellpadding="2" cellspacing="5" width="100%"><col width="16"><col width="*"><tbody><tr><td colspan="2" align="left" bgcolor="#0080c0" valign="top"><b><font color="#ffffff" face="Arial,Helvetica">Design discussion meeting notes: 2006-03-03.</font></b></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><h4>Attendees</h4></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><ul> | |
| <li> | |
| From Oracle: Raghu Srinivasan, Gerry Kessler, Ian | |
| Trimble, Justin Chen, Cameron Bateman | |
| </li> | |
| <li> | |
| From IBM: David Williams, Amy Wu, Phillip Avery, Nitin | |
| Dahyabhai | |
| </li> | |
| <li>From Sybase: Jean Choi</li> | |
| </ul></td></tr><tr><td colspan="2" align="left" bgcolor="#0080c0" valign="top"><b><font color="#ffffff" face="Arial,Helvetica">Talking points for the discussion</font></b></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><p> | |
| Purpose: To allow JSF-specific content assist to be hosted | |
| in relevant attribute value within the JSP editor. | |
| <br> | |
| The bug <a class="external" href="https://bugs.eclipse.org/bugs/show_bug.cgi?id=130041"> 130041</a> | |
| logs the proposed change request to support content | |
| assist for JSF EL in the JSP editor including a patch for | |
| the existing configuration. Proposal: | |
| </p></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><p> | |
| Summary: <br> | |
| 1) For expression language (EL) attribute values (those of | |
| the form "#{...}"), a patch would be applied to | |
| StructuredTextViewerConfigurationJSP.getContentAssistProcessors | |
| that handles the currently unhandled case for the partition | |
| IJSPPartitions.JSP_DEFAULT_EL2. This new logic would call | |
| into a new class called ContentAssistantProcessorLocator. | |
| This new class would use a new extension point (or | |
| alternatively a new provisional element of the | |
| editorConfiguration extension) to find all | |
| ContentAssistProcessors that wish to contribute to | |
| content-assist for late-bound (JSF) EL attribute values. JSF | |
| content assist would use this extension point to contribute | |
| it's logic. The impact to existing logic is almost nil. All | |
| new logic will be provided by JSF team as part of the patch. | |
| </p></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><p> | |
| NOTE: this scheme could extended in the future to support | |
| content assist contributors for other partition type as | |
| well. Most of the delta would be localized to the same | |
| configuration method. | |
| </p></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><p> | |
| 2) For non-EL attributes the existing Model Query extension | |
| point would be used. While this is not a public interface, | |
| it was the only way found that content assist could provided | |
| unobtrusively for partitions not already handled by the JSP | |
| editor configuration. | |
| </p></td></tr><tr><td align="right" valign="top"> | |
| | |
| </td><td valign="top"><p> | |
| An alternative would be to add similar extension point | |
| support as discussed in 1), but add it for partition types | |
| already handled. Again, JSF team would provide all new logic | |
| required in this area. | |
| </p></td></tr></tbody></table></body></html> |