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

Michael MacFadden resolved WAVE-271.
------------------------------------

    Resolution: Fixed

This was fixed a while ago.
                
> Review / rename ConversationBlip.appendInlineReplyThread(int location)
> ----------------------------------------------------------------------
>
>                 Key: WAVE-271
>                 URL: https://issues.apache.org/jira/browse/WAVE-271
>             Project: Wave
>          Issue Type: Task
>            Reporter: Michael MacFadden
>            Assignee: Michael MacFadden
>            Priority: Minor
>
> The ConversationBlip defines these two methods: 
> ConversationThread appendReplyThread(); 
> ConversationThread appendInlineReplyThread(int location); 
> Since we have largely removed the distinction between inline threads and 
> non-inline threads.  These methods need to be reviewed.  At first I had 
> though of just renaming the second method to "ConversationThread 
> appendInlineReplyThread(int location)", thereby just overloading the method.  
> That still might be and option, but I thought we should actually review what 
> these two methods do to see if we need to do anything other than renaming 
> them.  At the very least some better JavaDoc is probably a good idea.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to