Change in ovirt-engine[master]: webadmin, userportal: Improve GIN bindings/injections
vszocs at redhat.com
vszocs at redhat.com
Mon Jan 13 16:00:40 UTC 2014
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: Improve GIN bindings/injections
......................................................................
webadmin,userportal: Improve GIN bindings/injections
1. added BaseSystemModule.bindTypeAndImplAsSingleton
method and used it for following bindings:
- place manager
- all resources (constants, messages, etc.)
i.e. injecting CommonApplicationConstants and
app-specific ApplicationConstants now yields the
same singleton (ApplicationConstants) instance
2. modified classes to inject ClientAgentType instead
of doing "new ClientAgentType" on their own
Change-Id: I09a0178171de3f4e698e616a78ca988f47436957
Signed-off-by: Vojtech Szocs <vszocs at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseSystemModule.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/utils/ConsoleUtilsImpl.java
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/uicommon/UserPortalConfigurator.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/uicommon/WebAdminConfigurator.java
4 files changed, 23 insertions(+), 14 deletions(-)
Approvals:
Vojtech Szocs: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/19797
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I09a0178171de3f4e698e616a78ca988f47436957
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list