[jira] [Resolved] (WICKET-6560) Improve serialization warnings in ChainingModel

["Sven Meier (JIRA)" <ji...@apache.org>]

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

Sven Meier resolved WICKET-6560.
--------------------------------
       Resolution: Fixed
         Assignee: Sven Meier
    Fix Version/s: 8.1.0

Thanks.

> Improve serialization warnings in ChainingModel
> -----------------------------------------------
>
>                 Key: WICKET-6560
>                 URL: https://issues.apache.org/jira/browse/WICKET-6560
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 8.0.0
>            Reporter: Thomas Heigl
>            Assignee: Sven Meier
>            Priority: Minor
>             Fix For: 8.1.0
>
>
> After porting our app to Wicket 8, I'm getting these warnings on my test environment:
> {code:java}
> WAR o.a.w.model.ChainingModel It is not a good idea to reference a non-serializable instance in models directly as it may lead to serialization problems{code}
> The warning is OK, but it gives me absolutely no hint where to look for the issue. Could you please add the (simple) class name of the model object to the warning message? Currently, the only way for me to find these issues is with a breakpoint in `ChainingModel` and clicking through my application until I hit the warning.
>   
>  Also the warning is logged for `null` objects, which probably shouldn't happen. 
> {code:java}
> public ChainingModel(final Object modelObject)
> {
>     if (modelObject instanceof Session)   
>     {    
>         LOG.warn("It is not a good idea to reference the .........");
>     }
>     else if (modelObject instanceof Serializable == false)
>     {         
>         LOG.warn("It is not a good idea to reference a non-serializable instance ....");     
>     }
>     target = modelObject;
> }{code}
>  
>  It should read something like:
> {code:java}
> if (modelObject != null && modelObject instanceof Serializable == false){code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)