Wicket
  1. Wicket
  2. WICKET-4317

Logic or Exception Message is incorrect in ModalWindow.setCookieName

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.4.18
    • Fix Version/s: 1.5.4, 6.0.0-beta1
    • Component/s: wicket
    • Labels:
    • Environment:
      software

      Description

      The logic of the compound conditional statement is incorrect or the message passed to the Exception is incorrect.
      /**

      • Sets the name of the cookie that is used to remember window position (and size if the window
      • is resizable).
      • @param cookieName
      • Name of the cookie
      • @return this
        */
        public StatelessModalWindow setCookieName(String cookieName)
        Unknown macro: { if (cookieName != null && cookieName.indexOf(",") != -1 && cookieName.indexOf("|") != -1) { throw new IllegalArgumentException("Cookie name may not contain ',' or '|' characters."); } this.cookieName = cookieName; return this; }

      The logic reads:
      If the String is not null and it contains a comma and it contains a pipe, then throw the exception.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        9d 10h 55m 1 Martin Grigorov 07/Jan/12 09:44
        Martin Grigorov made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Martin Grigorov [ mgrigorov ]
        Fix Version/s 1.5.4 [ 12319051 ]
        Fix Version/s 6.0.0 [ 12315431 ]
        Resolution Fixed [ 1 ]
        R. Brian Amesbury created issue -

          People

          • Assignee:
            Martin Grigorov
            Reporter:
            R. Brian Amesbury
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 3h
              3h
              Remaining:
              Remaining Estimate - 3h
              3h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development