Commit @c79fb9784f2e630817de2fc1ce59a3a66a10a0f0

Y
List All
Feedback

Yi EungJun 2015-02-10

Property: Make sure check() returns nonnull list

@c79fb9784f2e630817de2fc1ce59a3a66a10a0f0

c65b866

c79fb97

app/models/Property.java

--- app/models/Property.java

+++ app/models/Property.java


 import play.db.ebean.Model;
 import utils.Diagnostic;
 
+import javax.annotation.Nonnull;
 import javax.persistence.Entity;
 import javax.persistence.EnumType;
 import javax.persistence.Enumerated;

     public static void onStart() {
         Diagnostic.register(new Diagnostic() {
             @Override
-            public List<String> check() {
+            public @Nonnull List<String> check() {
                 List<String> errors = new ArrayList<>();
 
                 for (Property.Name name : Property.Name.values()) {

Add a comment

Open 0
Closed 0

List

...	...	@@ -24,6 +24,7 @@
24	24	import play.db.ebean.Model;
25	25	import utils.Diagnostic;
26	26
	27	+import javax.annotation.Nonnull;
27	28	import javax.persistence.Entity;
28	29	import javax.persistence.EnumType;
29	30	import javax.persistence.Enumerated;
...	...	@@ -91,7 +92,7 @@
91	92	public static void onStart() {
92	93	Diagnostic.register(new Diagnostic() {
93	94	@Override
94		- public List<String> check() {
	95	+ public @Nonnull List<String> check() {
95	96	List<String> errors = new ArrayList<>();
96	97
97	98	for (Property.Name name : Property.Name.values()) {

Delete comment