[Engine-devel] AppErrors properties

Einav Cohen ecohen at redhat.com
Thu Jan 30 14:09:51 UTC 2014


> ----- Original Message -----
> From: "Michal Skrivanek" <michal.skrivanek at redhat.com>
> Sent: Thursday, January 30, 2014 8:56:07 AM
> 
> 
> On Jan 30, 2014, at 14:52 , Einav Cohen <ecohen at redhat.com> wrote:
> 
> > - we used to have a lot of differences between the admin messages
> > and the user portal messages; unfortunately we lost them somehow,
> > I can't remember how or when, it was quite a long time ago. so
> > the fact that the admin and user messages are identical is actually
> > a bug/regression.
> 
> yeah, it would make sense if e.g. UP messages were simpler/reduced…but it
> doesn't look like anymore. E.g. there are some gluster-related mesages
> present in UP but missing from WA which makes no sense at all.
> 
> > 
> > - once the GUI will move to REST-API, we won't need any of the
> > frontend files since we will get the translated message directly
> > from the backend. no point in doing any changes before that - it
> > is too much effort for too little gain.
> 
> right, so doesn't it make sense to keep just the backend ones?
> (and remove defaults which are useless and just requires you to update 2
> places instead of one)

you would still need to update the "defaults" file with the key (just not 
with the value) - it is needed from GWT perspective. 

> and only if REST GUI is still far away by then….copying the same file to
> UP/WA is a matter of a single makefile line…
> 
> Thanks,
> michal
> 
> > 
> > Einav
> > 
> > ----- Original Message -----
> >> From: "Michal Skrivanek" <michal.skrivanek at redhat.com>
> >> To: engine-devel at ovirt.org
> >> Sent: Thursday, January 30, 2014 8:35:22 AM
> >> Subject: [Engine-devel] AppErrors properties
> >> 
> >> Hi,
> >> while trying to add a new message I couldn't help notice some ugliness in
> >> the
> >> code regarding our error messages in UI
> >> 
> >> - why do we have @DefaultStringValue in
> >> frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
> >> ? All strings are defined in individual properties anyway.
> >> 
> >> - so we have individual properties files….3 of them
> >> REST -
> >> backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
> >> UP -
> >> frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
> >> WA -
> >> frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
> >> 
> >> funnily enough it's not that there are more messages in WA than UP, no,
> >> it's
> >> just full of random omissions, typos and discrepancies which do not make
> >> any
> >> logical sense
> >> 
> >> Can't we get rid of the defaults in the interface (useless) and have just
> >> one
> >> property containing all the messages and use the sam file in all 3 places.
> >> And only if there is intentionally a different message in e.g. UP than in
> >> WA
> >> we can have an override/additional property file there (just by briefly
> >> going through most of the differences there's not a single case)
> >> 
> >> Thanks,
> >> michal
> >> 
> >> _______________________________________________
> >> Engine-devel mailing list
> >> Engine-devel at ovirt.org
> >> http://lists.ovirt.org/mailman/listinfo/engine-devel
> >> 
> >> 
> >> 
> 
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 
> 
> 



More information about the Devel mailing list