Change in ovirt-engine[master]: userportal, webadmin: fix LocaleFilter

vszocs at redhat.com vszocs at redhat.com
Fri Jun 21 14:20:29 UTC 2013


Vojtech Szocs has submitted this change and it was merged.

Change subject: userportal,webadmin: fix LocaleFilter
......................................................................


userportal,webadmin: fix LocaleFilter

- Fixed a potential resource leak in LocaleFilter. The inputstream was
opened but never closed.
- Conformed the location of languages.properties to the same layout
as other properties files.

Change-Id: I57d895d44c463c835d396bf2c83e2a580d079688
Signed-off-by: Alexander Wels <awels at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/LocaleFilter.java
R backend/manager/modules/utils/src/main/resources/languages.properties
2 files changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Vojtech Szocs: Verified; Looks good to me, approved


-- 
To view, visit http://gerrit.ovirt.org/15703
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I57d895d44c463c835d396bf2c83e2a580d079688
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels 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