From d82d95ef78e0cc31f08e833ef6f1cee776177d26 Mon Sep 17 00:00:00 2001
From: Lukas Eder <lukas.eder@gmail.com>
Date: Fri, 28 Sep 2012 17:41:09 +0200
Subject: [PATCH] [#1848] Add Record.changed() to indicate whether a Record
 contains "dirty" values

---
 .../org/jooq/test/_/testcases/RecordTests.java    | 15 +++++++++++++++
 jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java |  5 +++++
 jOOQ/src/main/java/org/jooq/Record.java           | 11 +++++++++++
 .../main/java/org/jooq/impl/AbstractRecord.java   | 11 +++++++++++
 4 files changed, 42 insertions(+)

diff --git a/jOOQ-test/src/org/jooq/test/_/testcases/RecordTests.java b/jOOQ-test/src/org/jooq/test/_/testcases/RecordTests.java
index 2944fe9175..634d4ae8e3 100644
--- a/jOOQ-test/src/org/jooq/test/_/testcases/RecordTests.java
+++ b/jOOQ-test/src/org/jooq/test/_/testcases/RecordTests.java
@@ -36,6 +36,8 @@
 package org.jooq.test._.testcases;
 
 import static junit.framework.Assert.assertEquals;
+import static junit.framework.Assert.assertFalse;
+import static junit.framework.Assert.assertTrue;
 
 import org.jooq.TableRecord;
 import org.jooq.UpdatableRecord;
@@ -80,13 +82,26 @@ extends BaseTest<A, AP, B, S, B2S, BS, L, X, DATE, BOOL, D, T, U, I, IPK, T658,
         assertEquals(book.getValue(TBook_TITLE()), orig.getValue(TBook_TITLE()));
 
         book.setValue(TBook_TITLE(), "abc");
+        assertFalse(book.equals(orig));
+        assertFalse(book.equals(book.original()));
         assertEquals("abc", book.getValue(TBook_TITLE()));
         assertEquals(BOOK_TITLES.get(0), orig.getValue(TBook_TITLE()));
 
         book = orig;
         orig = orig.original();
         book.setValue(TBook_TITLE(), "abc");
+        assertFalse(book.equals(orig));
+        assertFalse(book.equals(book.original()));
         assertEquals("abc", book.getValue(TBook_TITLE()));
         assertEquals(BOOK_TITLES.get(0), orig.getValue(TBook_TITLE()));
     }
+
+    @Test
+    public void testRecordChanged() throws Exception {
+        B book = create().selectFrom(TBook()).where(TBook_ID().eq(1)).fetchOne();
+
+        assertFalse(book.changed());
+        book.setValue(TBook_TITLE(), "abc");
+        assertTrue(book.changed());
+    }
 }
diff --git a/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java b/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java
index cb7c39433b..3756800c0a 100644
--- a/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java
+++ b/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java
@@ -1162,6 +1162,11 @@ public abstract class jOOQAbstractTest<
         new RecordTests(this).testRecordOriginals();
     }
 
+    @Test
+    public void testRecordChanged() throws Exception {
+        new RecordTests(this).testRecordChanged();
+    }
+
     @Test
     public void testConcurrentExecution() throws Exception {
         new ThreadSafetyTests(this).testConcurrentExecution();
diff --git a/jOOQ/src/main/java/org/jooq/Record.java b/jOOQ/src/main/java/org/jooq/Record.java
index 654a29c092..55d79270f6 100644
--- a/jOOQ/src/main/java/org/jooq/Record.java
+++ b/jOOQ/src/main/java/org/jooq/Record.java
@@ -1118,6 +1118,17 @@ public interface Record extends FieldProvider, Store<Object> {
      */
     Record original();
 
+    /**
+     * Check if this record has been changed from its original as fetched from
+     * the database.
+     * <p>
+     * If this returns <code>false</code>, then it can be said that
+     * <code>record.equals(record.original())</code> is true.
+     *
+     * @see #original()
+     */
+    boolean changed();
+
     /**
      * Convert this record into an array.
      * <p>
diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractRecord.java b/jOOQ/src/main/java/org/jooq/impl/AbstractRecord.java
index 8202044dd8..077c0b5a9d 100644
--- a/jOOQ/src/main/java/org/jooq/impl/AbstractRecord.java
+++ b/jOOQ/src/main/java/org/jooq/impl/AbstractRecord.java
@@ -609,6 +609,17 @@ abstract class AbstractRecord extends AbstractStore<Object> implements Record {
         return result;
     }
 
+    @Override
+    public final boolean changed() {
+        for (Value<?> value : getValues()) {
+            if (value.isChanged()) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     @Override
     public final Object[] intoArray() {
         return into(Object[].class);