| <html> |
| <head> |
| <META http-equiv="Content-Type" content="text/html; charset=UTF-8"> |
| <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> |
| <link rel="stylesheet" href="../../../default_style.css" type="text/css"> |
| <link rel="stylesheet" href="../../../webtools/wtp.css" type="text/css"> |
| <title>the javaserver faces tools subproject</title> |
| </head> |
| <body> |
| <table width="100%" cellspacing="5" cellpadding="2" border="0"> |
| <tbody> |
| <tr> |
| <td width="60%" align="left"><font class="indextop">jsf</font> |
| <br> |
| <font class="indexsub">the javaserver faces tools subproject</font></td><td width="40%"><img width="207" hspace="50" height="129" align="middle" src="../../../webtools/images/wtplogosmall.jpg"></td> |
| </tr> |
| </tbody> |
| </table> |
| <table width="100%" cellspacing="5" cellpadding="2" border="0"> |
| <col width="16"> |
| <col width="*"> |
| <tbody> |
| <tr> |
| <td valign="top" bgcolor="#0080c0" align="left" colspan="2"><b><font face="Arial,Helvetica" color="#ffffff">Design discussion meeting notes: 2006-03-03.</font></b></td> |
| </tr> |
| <tr> |
| <td valign="top" align="right"> </td><td valign="top"> |
| <h4>Attendees</h4> |
| </td> |
| </tr> |
| <tr> |
| <td valign="top" align="right"> </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 valign="top" bgcolor="#0080c0" align="left" colspan="2"><b><font face="Arial,Helvetica" color="#ffffff">Talking points for the discussion</font></b></td> |
| </tr> |
| <tr> |
| <td valign="top" align="right"> </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 valign="top" align="right"> </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 valign="top" align="right"> </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 valign="top" align="right"> </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 valign="top" align="right"> </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> |