search

wyona / yanel

http://www.yanel.org
Apache License 2.0
10 stars 5 forks source link

target-environment should also work for Realms properly #20

Closed michaelwechner closed 12 years ago

michaelwechner commented 12 years ago

It seems that something is still not 100% right about target-environment inside realms. In order to debug better one can use the modified class

src/core/java/org/wyona/yanel/core/map/RealmDefaultImpl.java

inside branch 'target-env-realm' which is dumping the filtered config file.

Whereas one will see that

src/realms/from-scratch-realm-template/realm.xml

and

src/realms/from-scratch-realm-template/realm.xml.filtered

contains the name element twice....

michaelwechner commented 12 years ago

Children of the root element "realm-config", such as for example "name" are still not removed properly.

Also please note that one can get confusing results in cases such as

src/realms/from-scratch-realm-template/realm.xml

  
    en
    en
    de

because the current algorithm will remove all but


en