[ 
https://issues.apache.org/jira/browse/TAPESTRY-352?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jesse Kuhnert updated TAPESTRY-352:
-----------------------------------

    Fix Version/s:     (was: 4.1.3)
                   4.1.4

> Catch common invalid abstract setter
> ------------------------------------
>
>                 Key: TAPESTRY-352
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-352
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: Framework
>    Affects Versions: 4.0
>            Reporter: Howard M. Lewis Ship
>            Priority: Minor
>             Fix For: 4.1.4
>
>
> I constantly attempt to define properties as follows:
> public abstract String setMessage(String message);
> Which is not valid; it should be a void method.
> However, I always spend a good amount of time parsing the following to 
> determine that:
> Error at context:/WEB-INF/Home.page, line 6, column 21: Method 'public 
> abstract java.lang.String com.vaisala.ww.ui.pages.Home.setMessage()' 
> (declared in class com.vaisala.ww.ui.pages.Home) has no implementation in 
> class com.vaisala.ww.ui.pages.Home (or enhanced subclass $Home_0).
> location:     context:/WEB-INF/Home.page, line 6, column 21
> 1     <?xml version="1.0"?>
> 2     <!DOCTYPE page-specification PUBLIC
> 3     "-//Apache Software Foundation//Tapestry Specification 4.0//EN"
> 4     "http://jakarta.apache.org/tapestry/dtd/Tapestry_4_0.dtd";>
> 5     
> 6     <page-specification>
> 7     
> 8     <component id="register" type="DirectLink">
> 9     <binding name="listener" value="doRegister"/>
> 10    </component>
> 11    </page-specification>
> Stack Trace:
>     * 
> org.apache.hivemind.impl.StrictErrorHandler.error(StrictErrorHandler.java:39)
>     * org.apache.hivemind.impl.ErrorLogImpl.error(ErrorLogImpl.java:40)
>     * 
> org.apache.tapestry.enhance.EnhancedClassValidatorImpl.validate(EnhancedClassValidatorImpl.java:60)
>     * 
> $EnhancedClassValidator_1049ff23ffb.validate($EnhancedClassValidator_1049ff23ffb.java)
>     * 
> $EnhancedClassValidator_1049ff23ffa.validate($EnhancedClassValidator_1049ff23ffa.java)
>     * 
> org.apache.tapestry.services.impl.ComponentConstructorFactoryImpl.getComponentConstructor(ComponentConstructorFactoryImpl.java:85)
>     * 
> $ComponentConstructorFactory_1049ff23fe6.getComponentConstructor($ComponentConstructorFactory_1049ff23fe6.java)
>     * 
> $ComponentConstructorFactory_1049ff23fe5.getComponentConstructor($ComponentConstructorFactory_1049ff23fe5.java)
>     * 
> org.apache.tapestry.pageload.PageLoader.instantiatePage(PageLoader.java:611)
>     * org.apache.tapestry.pageload.PageLoader.loadPage(PageLoader.java:639)
>     * $IPageLoader_1049ff23fe1.loadPage($IPageLoader_1049ff23fe1.java)
>     * $IPageLoader_1049ff23fe2.loadPage($IPageLoader_1049ff23fe2.java)
>     * org.apache.tapestry.pageload.PageSource.getPage(PageSource.java:118)
>     * $IPageSource_1049ff23f4f.getPage($IPageSource_1049ff23f4f.java)
>     * $IPageSource_1049ff23f4e.getPage($IPageSource_1049ff23f4e.java)
>     * org.apache.tapestry.engine.RequestCycle.loadPage(RequestCycle.java:252)
> This is a very common mistake (I think); Tapestry should have a special 
> message for this case, such as "Setter methods must return void.".

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to