Tal Nisan has posted comments on this change. Change subject: webadmin: Date formatting when deleting a snapshot ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/34837/1/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java File frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java: Line 52: @SuppressWarnings("unused") Line 53: public class VmSnapshotListModel extends SearchableListModel Line 54: { Line 55: // This constant is intendend to be exported to a generic UTILS class later on Line 56: private final String DATE_FORMAT = "yyyy-MM-dd, HH:mm"; //$NON-NLS-1$ > should be static. Yes, findbugs also says so, you can see it in the failed job and click on the new warning: http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/19530/findbugsResult/new/ Line 57: Line 58: private UICommand privateNewCommand; Line 59: Line 60: public UICommand getNewCommand() { -- To view, visit http://gerrit.ovirt.org/34837 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d6a3f43557696fd35256b4a3fc55a5fbf59fdba Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aavi...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Amit Aviram <aavi...@redhat.com> Gerrit-Reviewer: Daniel Erez <de...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
