Jetspeed 2
  1. Jetspeed 2
  2. JS2-316

JSF Component javascript not being rendered

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.0-M3
    • Fix Version/s: 2.1
    • Component/s: Demo Portlets
    • Labels:
      None
    • Environment:
      Windows XP, Jakarta Tomcat 5.5.9, Jetspeed 2.0 M3, JDK 1.5.0_04, MyFaces 1.09

      Description

      The x:swapImage component is not generating the link to it's javascript file. I had a look at the myfaces render class for it (HtmlSwapImageRenderer), and it includes the following line:

      Line 57: AddResource.addJavaScriptToHeader(HtmlSwapImage.class, "swapimage.js", facesContext);

      I assume that the JSF framework will then add a link to this javascript file in the header. But it is not being included in the portal page.

        Activity

        Hide
        Shinsuke Sugaya added a comment -

        J2 has HeaderResource component. Using it, if you extends MyFacesGenericPortlet, it will work. Anyway, I'll add demo later.

        Show
        Shinsuke Sugaya added a comment - J2 has HeaderResource component. Using it, if you extends MyFacesGenericPortlet, it will work. Anyway, I'll add demo later.
        Hide
        Ronald Holshausen added a comment -

        I'm using the org.apache.portals.bridges.jsf.FacesPortlet, but I'll switch to MyFacesGenericPortlet.

        Should this be reassigned to the JSF Bridge component?

        Show
        Ronald Holshausen added a comment - I'm using the org.apache.portals.bridges.jsf.FacesPortlet, but I'll switch to MyFacesGenericPortlet. Should this be reassigned to the JSF Bridge component?
        Hide
        Shinsuke Sugaya added a comment -

        I'll add extended MyFacesGenericPortlet to handle HeaderResource component. Since this is J2 specific portlet, it should not be added to portals-bridge. So, moved to Demo Portlets.

        Show
        Shinsuke Sugaya added a comment - I'll add extended MyFacesGenericPortlet to handle HeaderResource component. Since this is J2 specific portlet, it should not be added to portals-bridge. So, moved to Demo Portlets.
        Hide
        Shinsuke Sugaya added a comment -

        Filed MYFACES-434. If it's fixed, the componet will work on J2.

        Show
        Shinsuke Sugaya added a comment - Filed MYFACES-434 . If it's fixed, the componet will work on J2.
        Hide
        Shinsuke Sugaya added a comment -

        I'm also working on MYFACES-434. I'll add some classes into J2 after fixed MYFACES-434.

        Show
        Shinsuke Sugaya added a comment - I'm also working on MYFACES-434 . I'll add some classes into J2 after fixed MYFACES-434 .
        Hide
        Michael Lipp added a comment -

        Shinsuke wrote: "This issue depends on other issues(portlet filter, common-fileupload
        1.1, myfaces release..). So, I do not think it may be fixed by 2.0 final
        even if marked 2.0-FINAL."

        Show
        Michael Lipp added a comment - Shinsuke wrote: "This issue depends on other issues(portlet filter, common-fileupload 1.1, myfaces release..). So, I do not think it may be fixed by 2.0 final even if marked 2.0-FINAL."
        Hide
        Shinsuke Sugaya added a comment -

        I put my comment to MYFACES-434.
        If you use MyFaces Bridge 0.8 I provided, you can use Tomahawk's portlet on J2. So, J2 does not need to take care of this issue. If you have any problems and comments, let me know.

        Show
        Shinsuke Sugaya added a comment - I put my comment to MYFACES-434 . If you use MyFaces Bridge 0.8 I provided, you can use Tomahawk's portlet on J2. So, J2 does not need to take care of this issue. If you have any problems and comments, let me know.

          People

          • Assignee:
            Shinsuke Sugaya
            Reporter:
            Ronald Holshausen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development