Allon Mureinik has submitted this change and it was merged. Change subject: webadmin: isOptionEnabledOnVolume filters ...................................................................... webadmin: isOptionEnabledOnVolume filters isOptionEnabledOnVolume receives a list of filters, but is always called with a singlton list containing a single element. Taking into consideration that this code was not touched for over a year, the method can be simplified to receive a single filter, in order to make it easier to read. In the future, if additional filters are required, this patch could always be reverted. Change-Id: Ide95e55e72f082e3969857b8e3ba13d668316130 Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java 1 file changed, 5 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/49617 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide95e55e72f082e3969857b8e3ba13d668316130 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>