<div dir="ltr">response inline<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 5:29 AM, Martin Sivak <span dir="ltr"><<a href="mailto:msivak@redhat.com" target="_blank">msivak@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
We still have three almost identical files. Can we somehow keep just<br>
one and generate the other two? I was actually playing a bit with a<br>
change in the opposite direction - keeping just the EngineMessages<br>
enum with added default english translations and generating all other<br>
files from it.<br></blockquote><div><br></div><div>Are you talking about some files in specific, and if so which ones? <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Do you think it would make sense? It would not require a test then as<br>
the consistency would be checked during compilation phase directly.<br></blockquote><div><br></div><div>Sure, generating some code from a key/message file could be useful. The difficulty comes in when we have Messages or Constants interface that inherit from a common ancestor. That construct is used a few times to share messages between both the user portal and the admin portal.<br><br></div><div>The primary reasons this change was done was to simplify translation and to better manage workflow between language files in gerrit and documents in Zanata. With this change, there is now a 1 to 1 mapping of properties files to zanata documents. Well, for the front end i18n code at least. The current translations for 4.0 can be seen here: <br><br> <a href="https://translate.zanata.org/iteration/view/ovirt/master/documents">https://translate.zanata.org/iteration/view/ovirt/master/documents</a> <br></div><div><br></div><div>We will be making additional i18n changes for 4.1, so any input is very welcome. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Martin<br>
<div><div class="h5"><br>
On Mon, May 23, 2016 at 7:38 PM, Scott Dickerson <<a href="mailto:sdickers@redhat.com">sdickers@redhat.com</a>> wrote:<br>
> Hi,<br>
><br>
> In order to resolve bug [1] and prep [2], the default English text for I18N<br>
> Constants and Messages have been moved to their corresponding properties<br>
> files. Going forward, if a new constant or message needs to be added,<br>
> please add the key method to the proper interface and then add the key plus<br>
> English text to the interface's corresponding properties file. Checks will<br>
> be added, in the next few days, to fail the build process if the GWT<br>
> annotations (@DefaultStringValue, @DefaultMessage) are used.<br>
><br>
> This change has been made to simplify the translation process [1] and to<br>
> prepare for the replacement of the GWT i18n engine in 4.1 [2].<br>
><br>
> For example, in the webadmin project, the ApplicationConstants [3] interface<br>
> previously looked like:<br>
> package org.ovirt.engine.ui.webadmin;<br>
><br>
> import org.ovirt.engine.ui.common.CommonApplicationConstants;<br>
><br>
> public interface ApplicationConstants extends CommonApplicationConstants<br>
> {<br>
><br>
> @DefaultStringValue("oVirt Engine Web Administration")<br>
> String applicationTitle();<br>
><br>
> @DefaultStringValue("About")<br>
> String aboutPopupCaption();<br>
><br>
> @DefaultStringValue("oVirt Engine Version:")<br>
> String ovirtVersionAbout();<br>
><br>
> Now, the interface looks like this [4]:<br>
> package org.ovirt.engine.ui.webadmin;<br>
><br>
> import org.ovirt.engine.ui.common.CommonApplicationConstants;<br>
><br>
> public interface ApplicationConstants extends CommonApplicationConstants<br>
> {<br>
> String applicationTitle();<br>
><br>
> String aboutPopupCaption();<br>
><br>
> String ovirtVersionAbout();<br>
><br>
> With a new properties file created, ApplicationContants.properties [5], with<br>
> this kind of content :<br>
> #<br>
> #<br>
> # Bundle name: org.ovirt.engine.ui.webadmin.ApplicationConstants<br>
> #<br>
> #<br>
> #Thu May 19 01:24:21 EDT 2016<br>
> aboutPopupCaption=About<br>
> applicationTitle=oVirt Engine Web Administration<br>
> ovirtVersionAbout=oVirt Engine Version\:<br>
><br>
> Let me know if there are any questions.<br>
><br>
> Regards,<br>
> Scott<br>
><br>
> --<br>
> Scott Dickerson<br>
> Senior Software Engineer<br>
> RHEV-M Engineering - UX Team<br>
> Red Hat, Inc<br>
><br>
><br>
> [1] - <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1224423" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1224423</a><br>
> [2] - <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1287408" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1287408</a><br>
> [3] -<br>
> <a href="https://github.com/oVirt/ovirt-engine/blob/569695916d2325d0c1e1fdabb6f85cd9e97d7232/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java" rel="noreferrer" target="_blank">https://github.com/oVirt/ovirt-engine/blob/569695916d2325d0c1e1fdabb6f85cd9e97d7232/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java</a><br>
> [4] -<br>
> <a href="https://github.com/oVirt/ovirt-engine/blob/master/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java" rel="noreferrer" target="_blank">https://github.com/oVirt/ovirt-engine/blob/master/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java</a><br>
> [5] -<br>
> <a href="https://github.com/oVirt/ovirt-engine/blob/master/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants.properties" rel="noreferrer" target="_blank">https://github.com/oVirt/ovirt-engine/blob/master/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants.properties</a><br>
><br>
</div></div>> _______________________________________________<br>
> Devel mailing list<br>
> <a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
> <a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/devel</a><br>
</blockquote></div><br></div></div></div>