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

[Josias Thöny]

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?

Josias

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