click-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "WarnerJan Veldhuis (JIRA)" <>
Subject [jira] Created: (CLK-561) Page#setForward(Class) silently fails for a non-existing .htm
Date Fri, 26 Jun 2009 09:47:07 GMT
Page#setForward(Class) silently fails for a non-existing .htm

                 Key: CLK-561
             Project: Click
          Issue Type: Bug
          Components: core
    Affects Versions: 2.1.0
            Reporter: WarnerJan Veldhuis

When setting a forward in a page, and the corresponding .htm is missing, Page#setForward(Class)
will silently fail, and Page#getForward() will return null. Eventually, you will remain on
the current page, since ClickServlet  wasn't told otherwise.

public boolean onLogin() {
   if ( isLoggedIn ) {
        return false;

If home.htm is missing, setForward will set the forward field to null, while I would expect
it to barf with an IllegalArgumentException. The javadoc of ConfigService#getPagePath(Class)
also says: "@throws IllegalArgumentException if the Page Class is not configured"

Proposed solution:
Instead of returning null in XmlConfigService#getPagePage(Class), throw an IAE.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message