Eliraz Levi has posted comments on this change.

Change subject: backend: adding @Mask annotation
......................................................................


Patch Set 21:

(2 comments)

https://gerrit.ovirt.org/#/c/35155/21/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/ValidatorConstraint.java
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/ValidatorConstraint.java:

Line 36:         final private boolean validValue;
Line 37:         final private String badFormatErrorMessage;
Line 38:         final private String badValueErrorMessage;
Line 39: 
Line 40:         public ValidatorConstraintArgsFormatValue(boolean 
isValidFormat,
> s/isValidFormat/validFormat
Done
Line 41:                 boolean isValidValue,
Line 42:                 String badFormatErrorMessage,
Line 43:                 String badValueErrorMessage) {
Line 44:             this.validFormat = isValidFormat;


Line 37:         final private String badFormatErrorMessage;
Line 38:         final private String badValueErrorMessage;
Line 39: 
Line 40:         public ValidatorConstraintArgsFormatValue(boolean 
isValidFormat,
Line 41:                 boolean isValidValue,
> s/isValidValue/validValue
Done
Line 42:                 String badFormatErrorMessage,
Line 43:                 String badValueErrorMessage) {
Line 44:             this.validFormat = isValidFormat;
Line 45:             this.validValue = isValidValue;


-- 
To view, visit https://gerrit.ovirt.org/35155
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If92f2ec51f9cb647d6008b2fd2b1ab4c3db1d3f1
Gerrit-PatchSet: 21
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eliraz Levi <el...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Eliraz Levi <el...@redhat.com>
Gerrit-Reviewer: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzasp...@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

Reply via email to