[Yanel-dev] RealmConfiguration is kind of misleading as a name

Michael Wechner michael.wechner at wyona.com
Tue May 15 23:35:08 CEST 2007


Josias Thöny wrote:

>On Mon, 2007-05-14 at 09:50 +0200, Michael Wechner wrote:
>  
>
>>Josias Thöny wrote:
>>
>>    
>>
>>>On Sat, 2007-05-12 at 13:35 +0200, Michael Wechner wrote:
>>> 
>>>
>>>      
>>>
>>>>Hi
>>>>
>>>>I think the class name RealmConfiguration is a bit misleading, because 
>>>>one want think that it is re the configuration of one realm and not of 
>>>>all the realms being registered within Yanel. Hence I think it would 
>>>>make sense to rename it.
>>>>
>>>>WDYT?
>>>>   
>>>>
>>>>        
>>>>
>>>+1 for renaming.
>>>This class is not only about configuration, it's also provides methods
>>>to e.g. add a new realm, so maybe we could rename it to "RealmManager".
>>>Any better ideas?
>>> 
>>>
>>>      
>>>
>>RealmManager sounds good to me
>>
>>    
>>
>>>And maybe we could also move this class and Realm.java from
>>>org.wyona.yanel.core.map to org.wyona.yanel.core,
>>>
>>>      
>>>
>>makes sense, because I would say it's on the same "purpose level" as a 
>>resource
>>
>>    
>>
>>>or is there a reason
>>>why these two classes belong to the map package?
>>> 
>>>
>>>      
>>>
>>I guess because a realm is part of the map layout, but do not really 
>>remember ;-)
>>
>>The question is if we move this two classes, will it then break the 
>>implemented resources
>>I guess it should't because resource are accessing the realm through 
>>getRealm(), right?
>>    
>>
>
>Hm, some resources are importing org.wyona.yanel.core.map.Realm, so they
>would break.
>The other resources which just use getRealm() without using the package
>name might still need to be recompiled.
>Should we keep the current package structure to avoid problems?
>  
>

yes, whereas we could set it to deprecated and copy it to the core dir
and refactor the resources which are being part of Yanel distribution.

WDYT?

Cheers

Michi

>Josias
>
>  
>
>>Cheers
>>
>>Michi
>>
>>    
>>
>>>Josias
>>>
>>> 
>>>
>>>      
>>>
>>>>Cheers
>>>>
>>>>Michi
>>>>
>>>>   
>>>>
>>>>        
>>>>
>>>_______________________________________________
>>>Yanel-development mailing list
>>>Yanel-development at wyona.com
>>>http://wyona.com/cgi-bin/mailman/listinfo/yanel-development
>>>
>>> 
>>>
>>>      
>>>
>>    
>>
>
>
>_______________________________________________
>Yanel-development mailing list
>Yanel-development at wyona.com
>http://wyona.com/cgi-bin/mailman/listinfo/yanel-development
>
>  
>


-- 
Michael Wechner
Wyona      -   Open Source Content Management   -    Apache Lenya
http://www.wyona.com                      http://lenya.apache.org
michael.wechner at wyona.com                        michi at apache.org
+41 44 272 91 61




More information about the Yanel-development mailing list