ICEfaces
  1. ICEfaces
  2. ICE-9160

JavascriptRunner alert with \n

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.2.0.GA
          • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
          • Component/s: Bridge
          • Labels:
            None
          • Environment:
            JBoss EAP6
          • Assignee Priority:
            P1

            Description

            I am sending warnings to the the user via an alert sent with JavaScriptRunner:

            JavaScriptRunner.runScript(FacesContext.getCurrentInstance(), "alert('" +
                               warningString.toString() + "');");

            This works, except for when I add a line break '\n' to the alert text. When I do that I don't see the alert and get a js error in the browser (this is the Chrome output):

            [window] Error [status: malformedXML code: 200]: Unexpected token ILLEGAL bridge.uncompressed.js.jsf:1310
            ConsoleLogHandler.errorPrimitive

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              JavascriptRunner escapes now Javascript special characters before sending them to the bridge

              Show
              Mircea Toma added a comment - - edited JavascriptRunner escapes now Javascript special characters before sending them to the bridge
              Hide
              Permalink
              Ken Fyten added a comment -

              Re-opened due to general runtime failures.

              Reverted commit changes for JavaScriptRunner.java on trunk for now to restore general functioning.

              [13:31:28] - svn commit -m "ICE-9160 - Revert commit for 'Escape Javascript special characters before sending them to the bridge' due to new regressions." 1 packet
              [13:31:28] - Sending /svn/icefaces3-trunk/icefaces/core/src/main/java/org/icefaces/util/JavaScriptRunner.java
              [13:31:29] - Committed revision 34472.

              Show
              Ken Fyten added a comment - Re-opened due to general runtime failures. Reverted commit changes for JavaScriptRunner.java on trunk for now to restore general functioning. [13:31:28] - svn commit -m " ICE-9160 - Revert commit for 'Escape Javascript special characters before sending them to the bridge' due to new regressions." 1 packet [13:31:28] - Sending /svn/icefaces3-trunk/icefaces/core/src/main/java/org/icefaces/util/JavaScriptRunner.java [13:31:29] - Committed revision 34472.
              Hide
              Permalink
              Mircea Toma added a comment -

              Escaping now only text formatting characters: \n, \t , \r, \b, \f.

              Show
              Mircea Toma added a comment - Escaping now only text formatting characters: \n, \t , \r, \b, \f.

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Brad Kroeger
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: