myfaces-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ir. ing. Jan Dockx" <>
Subject Re: [jira] Commented: (MYFACES-447) tree2 TreeNode interface has too many methods
Date Thu, 25 Aug 2005 23:25:51 GMT
Now we are on the list, and we can discuss this ;-).

The first suggested patch doesn't influence anybody, actually, since  
the methods that are removed from the interface are not used by any  
tree2 code. And TreeNodeBase keeps behaving as it did before (actually,  
for it should better be changed too, but it should stay in now at least  
for backward compatibility reasons).

The second suggested patch takes cleaning up the interface further. But  
since the differences between isLeaf(), getChildren().isEmpty() and  
getChildCount() == 0 on the one hand, and getChildren().size() and  
getChildCount() on the other hand, are rather obscure (and, as far as I  
can see, not applied consistently in the existing tree2 code either), I  
bet that there are not that many users out there that depend on the  
differences. If so, please let me know here: it might increase my  
understanding of the differences.
And if your current implementation of a TreeNode is consistent  
(getChildCount() == getChildren().size() and isLeaf() <==>  
getChildren().isEmpty() <==> getChildCount() == 0), this change will  
not affect your code either, since that is how the code is changed in  
tree2 implementation.

On 26 Aug 2005, at 1:10, sean schofield (JIRA) wrote:

>     [  
> page=comments#action_12320067 ]
> sean schofield commented on MYFACES-447:
> ----------------------------------------
> No you did not miss a discussion.  Don't worry this patch will not be  
> applied in the short term for a few reasons.  The first reason is that  
> it is too close to release time to change an interface.  The second  
> reason is that we will need to do extensive review and consultation  
> with users before a major change such as this.
> I'm definitely interested in improving tree2 so I will take a close  
> look at this patch when I get a chance.  We can discuss the changes  
> now even if we don't make them for a while.
> @Jan
> JIRA is a good place to discuss matters like this (especially when  
> patches are provided to help illustrate the requestor's point.)  In  
> the future can you also email the user list if you have a *major*  
> change of this nature?  Let them know the JIRA link.  That way nobody  
> gets left out of a crucial discussion.  Bug fixes or minor feature  
> additions (that's don't impact existing features, interfaces) can go  
> directly to JIRA though.  This is kind of a special case b/c its a  
> major change.
> @Dennis
> You may want to add yourself as a watcher to issues when you do spot  
> them on the dev list.  Now that you know you're interested, why take  
> the chance on missing out on an important discussion?
>> tree2 TreeNode interface has too many methods
>> ---------------------------------------------
>>          Key: MYFACES-447
>>          URL:
>>      Project: MyFaces
>>         Type: Improvement
>>   Components: Tomahawk
>>     Versions: Nightly Build
>>     Reporter: Jan Dockx
>>  Attachments: tree2_TreeNode_extensive_patch.txt,  
>> tree2_TreeNode_extensive_patch_examples.txt, tree2_TreeNode_patch.txt
>> The TreeNode interface is the hook for the backing bean developer,  
>> the user, into the tree2 framework. As such, the interface should  
>> pose minimal requirements for the user. It should only require from  
>> the user to implement those methods that are strictly needed for the  
>> framework to work, and leave as much freedom for the user as  
>> possible.
>> The patch included removes unnecessary methods. Apart from a minor  
>> change in TreeNodeBase, the change doesn't affect the existing tree2  
>> code, nor the example, at all.
>> The patch also contains some documentation for the interface, and  
>> some suggestions for further changes.
> -- 
> This message is automatically generated by JIRA.
> -
> If you think it was sent incorrectly contact one of the administrators:
> -
> For more information on JIRA, see:
Met vriendelijke groeten,

Jan Dockx

PeopleWare NV - Head Office
Cdt.Weynsstraat 85
B-2660 Hoboken
Tel: +32 3 448.33.38
Fax: +32 3 448.32.66

PeopleWare NV - Branch Office Geel
Kleinhoefstraat 5
B-2440 Geel
Tel: +32 14 57.00.90
Fax: +32 14 58.13.25

View raw message