Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA
    • Fix Version/s: 4.0
    • Component/s: Bridge, Framework
    • Labels:
      None
    • Environment:
      Portlets
    • Assignee Priority:
      P3
    • Affects:
      Compatibility/Configuration

      Description

      While this is mostly for portlets, there is a general move with the JSF implementation to ensure that all parameters are namespaced appropriately to the NamingContainer they are in. This is valid no matter what environment they are running in. See https://java.net/jira/browse/JAVASERVERFACES-3031.

      With the current version (6.2) of the LiferayFaces Bridge, it's necessary to set the following portlet configuration setting in order to get ICEpush working:

      <requires-namespaced-parameters>false</requires-namespaced-parameters>
      1. ICE-9953.patch
        19 kB
        Mircea Toma
      2. SamplesRegressions-ParameterNamespacing.pdf
        119 kB
        Ken Fyten
      1. after.PNG
        16 kB
      2. before.PNG
        11 kB
      3. multi.PNG
        14 kB

        Activity

        Hide
        Mircea Toma added a comment -

        Modified EnvUtils.getParameterNamespace method to return the container's ID only when the view root is a naming container. This behaviour follows what Mojarra is doing when it decides to run with namespaced parameters.

        Show
        Mircea Toma added a comment - Modified EnvUtils.getParameterNamespace method to return the container's ID only when the view root is a naming container. This behaviour follows what Mojarra is doing when it decides to run with namespaced parameters.
        Hide
        Ted Goddard added a comment - - edited

        Still tracking this bug down, but this typo "parametr" seems to be preventing Cloud Push from working on the dev.bridgeit.io deployment:

        core/src/main/javascript/application.js

        24302 mircea.tom onSend: function(callback) {
        41327 mircea.tom onSend(asyncConnection, function(query) {
        41327 mircea.tom //the callback parameters: function(addParameter)

        {...; addParameter('A', '123'); ...}

        24302 mircea.tom callback(function(name, value)

        { 41493 mircea.tom parametr(query, name, value); 24302 mircea.tom }

        );
        24302 mircea.tom });
        24302 mircea.tom },

        Show
        Ted Goddard added a comment - - edited Still tracking this bug down, but this typo "parametr" seems to be preventing Cloud Push from working on the dev.bridgeit.io deployment: core/src/main/javascript/application.js 24302 mircea.tom onSend: function(callback) { 41327 mircea.tom onSend(asyncConnection, function(query) { 41327 mircea.tom //the callback parameters: function(addParameter) {...; addParameter('A', '123'); ...} 24302 mircea.tom callback(function(name, value) { 41493 mircea.tom parametr(query, name, value); 24302 mircea.tom } ); 24302 mircea.tom }); 24302 mircea.tom },
        Hide
        Ted Goddard added a comment -

        Re-opening to address typo and verify.

        Show
        Ted Goddard added a comment - Re-opening to address typo and verify.
        Hide
        Mircea Toma added a comment -

        Fixed typo and verified successfully commit.

        Show
        Mircea Toma added a comment - Fixed typo and verified successfully commit.
        Hide
        Liana Munroe added a comment - - edited

        Verified Icefaces 4 trunk r42072 samples and dataTable test applications. All regression tests were run. No issues specific to Context-param: "com.sun.faces.namespaceParameters=true" were found.

        Show
        Liana Munroe added a comment - - edited Verified Icefaces 4 trunk r42072 samples and dataTable test applications. All regression tests were run. No issues specific to Context-param: "com.sun.faces.namespaceParameters=true" were found.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Deryk Sinotte
          • Votes:
            1 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: