From 2f57697c326a1bb33d42ac5d8210194937d88728 Mon Sep 17 00:00:00 2001 From: Michael Bayne Date: Mon, 15 Sep 2008 18:22:18 +0000 Subject: [PATCH] We're going to allow FieldOverride on @Computed records, just not on @Computed fields. Maybe Zell will remind me why this was disallowed, but it seems that to disallow this is to make FieldOverride useless. --- src/java/com/samskivert/jdbc/depot/BuildVisitor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/java/com/samskivert/jdbc/depot/BuildVisitor.java b/src/java/com/samskivert/jdbc/depot/BuildVisitor.java index b3fc74c..6303488 100644 --- a/src/java/com/samskivert/jdbc/depot/BuildVisitor.java +++ b/src/java/com/samskivert/jdbc/depot/BuildVisitor.java @@ -509,7 +509,7 @@ public abstract class BuildVisitor implements ExpressionVisitor if (override != null) { boolean useOverride; if (override instanceof FieldOverride) { - if (fm.getComputed() != null || dm.getComputed() != null) { + if (fm.getComputed() != null) { throw new IllegalArgumentException( "FieldOverride cannot be used on @Computed field: " + field); }