[ 
https://issues.apache.org/jira/browse/ISIS-333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13578239#comment-13578239
 ] 

Dan Haywood commented on ISIS-333:
----------------------------------

Kevin,
I'm not convinced this is the right fix.  Having the HTML viewer (or any 
viewer) persist objects just because it can't handle the rendering of them as 
transient objects seems incorrect.  

Instead - if we are agreed that the contract for choices methods is that the 
objects should be persistent or should be values - then I think that the viewer 
should throw an exception if it finds itself being asked to render transient 
objects.  This tells the programmer that they've made a mistake in their code.

I'm re-opening the issue for further discussion.
                
> HTML Viewer is unable to save an object if one of the properties still 
> contains a transient
> -------------------------------------------------------------------------------------------
>
>                 Key: ISIS-333
>                 URL: https://issues.apache.org/jira/browse/ISIS-333
>             Project: Isis
>          Issue Type: Bug
>          Components: Viewer: HTML
>    Affects Versions: viewer-html-1.0.0
>            Reporter: Kevin Meyer
>            Assignee: Dan Haywood
>            Priority: Minor
>             Fix For: viewer-html-1.0.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> As discussed in the mail thread [1], there is an issue with the HTML viewer 
> that when you trying to edit an existing, persisted, object and one of the 
> properties is unpersisted (transient), the EditTask dies with an exception.
> [1] http://isis.markmail.org/message/k4szznni5ycn4h4v

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to