[WICKET-4317] Logic or Exception Message is incorrect in ModalWindow.setCookieName

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.4.18
• Fix Version/s: 1.5.4, 6.0.0-beta1
• Component/s: wicket
• Labels: patch
• Environment: software
• Assignee: Martin Tzvetanov Grigorov
• Reporter: R. Brian Amesbury
• Created: 28/Dec/11 22:49
• Updated: 07/Jan/12 09:44
• Resolved: 07/Jan/12 09:44

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.