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

[Michael Wechner]

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?

Cheers

Michi

>Josias
>
>  
>
>>Cheers
>>
>>Michi
>>
>>    
>>
>
>
>_______________________________________________
>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