From 2fe8f98e4e4303fec7fc39a849aecd9c1c1f09db Mon Sep 17 00:00:00 2001
From: David Huynh <dfhuynh@gmail.com>
Date: Thu, 4 Mar 2010 00:00:46 +0000
Subject: [PATCH] Added repeat and repeatCount options for text transform
 operation. This lets us fix those &amp;amp;amp; repeated encoding problems
 easily.

git-svn-id: http://google-refine.googlecode.com/svn/trunk@179 7d457c2a-affb-35e4-300a-418c747d4874
---
 .../commands/edit/DoTextTransformCommand.java | 13 +++-
 .../operations/TextTransformOperation.java    | 36 ++++++++++-
 .../dialogs/expression-preview-dialog.js      |  8 +--
 .../views/data-table-column-header-ui.js      | 62 ++++++++++++-------
 src/main/webapp/styles/common.css             |  6 +-
 5 files changed, 95 insertions(+), 30 deletions(-)

diff --git a/src/main/java/com/metaweb/gridworks/commands/edit/DoTextTransformCommand.java b/src/main/java/com/metaweb/gridworks/commands/edit/DoTextTransformCommand.java
index a35da29e7..79a194b50 100644
--- a/src/main/java/com/metaweb/gridworks/commands/edit/DoTextTransformCommand.java
+++ b/src/main/java/com/metaweb/gridworks/commands/edit/DoTextTransformCommand.java
@@ -17,11 +17,22 @@ public class DoTextTransformCommand extends EngineDependentCommand {
         String columnName = request.getParameter("columnName");
         String expression = request.getParameter("expression");
         String onError = request.getParameter("onError");
+        boolean repeat = "true".equals(request.getParameter("repeat"));
+        
+        int repeatCount = 10;
+        String repeatCountString = request.getParameter("repeatCount");
+        try {
+            repeatCount = Math.max(Math.min(Integer.parseInt(repeatCountString), 10), 0);
+        } catch (Exception e) {
+        }
         
         return new TextTransformOperation(
         	engineConfig, 
         	columnName, 
         	expression, 
-        	TextTransformOperation.stringToOnError(onError));
+        	TextTransformOperation.stringToOnError(onError),
+        	repeat,
+        	repeatCount
+        );
     }
 }
diff --git a/src/main/java/com/metaweb/gridworks/operations/TextTransformOperation.java b/src/main/java/com/metaweb/gridworks/operations/TextTransformOperation.java
index fd44aca9b..28befc626 100644
--- a/src/main/java/com/metaweb/gridworks/operations/TextTransformOperation.java
+++ b/src/main/java/com/metaweb/gridworks/operations/TextTransformOperation.java
@@ -27,8 +27,10 @@ public class TextTransformOperation extends EngineDependentMassCellOperation {
     	StoreError
     }
 
-    final protected String _expression;
+    final protected String  _expression;
     final protected OnError _onError;
+    final protected boolean _repeat;
+    final protected int     _repeatCount;
     
     static public AbstractOperation reconstruct(Project project, JSONObject obj) throws Exception {
         JSONObject engineConfig = obj.getJSONObject("engineConfig");
@@ -37,7 +39,9 @@ public class TextTransformOperation extends EngineDependentMassCellOperation {
             engineConfig,
             obj.getString("columnName"),
             obj.getString("expression"),
-            stringToOnError(obj.getString("onError"))
+            stringToOnError(obj.getString("onError")),
+            obj.getBoolean("repeat"),
+            obj.getInt("repeatCount")
         );
     }
     
@@ -60,10 +64,19 @@ public class TextTransformOperation extends EngineDependentMassCellOperation {
     	}
     }
     
-    public TextTransformOperation(JSONObject engineConfig, String columnName, String expression, OnError onError) {
+    public TextTransformOperation(
+            JSONObject engineConfig, 
+            String columnName, 
+            String expression, 
+            OnError onError,
+            boolean repeat,
+            int repeatCount
+        ) {
         super(engineConfig, columnName, true);
         _expression = expression;
         _onError = onError;
+        _repeat = repeat;
+        _repeatCount = repeatCount;
     }
 
     public void write(JSONWriter writer, Properties options)
@@ -76,6 +89,8 @@ public class TextTransformOperation extends EngineDependentMassCellOperation {
         writer.key("columnName"); writer.value(_columnName);
         writer.key("expression"); writer.value(_expression);
         writer.key("onError"); writer.value(onErrorToString(_onError));
+        writer.key("repeat"); writer.value(_repeat);
+        writer.key("repeatCount"); writer.value(_repeatCount);
         writer.endObject();
     }
 
@@ -127,6 +142,21 @@ public class TextTransformOperation extends EngineDependentMassCellOperation {
                 
                 if (!ExpressionUtils.sameValue(oldValue, newValue)) {
                     Cell newCell = new Cell(newValue, (cell != null) ? cell.recon : null);
+                    
+                    if (_repeat) {
+                        for (int i = 0; i < _repeatCount; i++) {
+                            ExpressionUtils.bind(bindings, row, rowIndex, newCell);
+                            
+                            newValue = eval.evaluate(bindings);
+                            if (ExpressionUtils.isError(newValue)) {
+                                break;
+                            } else if (ExpressionUtils.sameValue(newCell.value, newValue)) {
+                                break;
+                            }
+                            
+                            newCell = new Cell(newValue, newCell.recon);
+                        }
+                    }
                 
                     CellChange cellChange = new CellChange(rowIndex, cellIndex, cell, newCell);
                     cellChanges.add(cellChange);
diff --git a/src/main/webapp/scripts/dialogs/expression-preview-dialog.js b/src/main/webapp/scripts/dialogs/expression-preview-dialog.js
index 296da327c..1e38c223b 100644
--- a/src/main/webapp/scripts/dialogs/expression-preview-dialog.js
+++ b/src/main/webapp/scripts/dialogs/expression-preview-dialog.js
@@ -32,7 +32,7 @@ function ExpressionPreviewDialog(title, cellIndex, rowIndices, values, expressio
 };
 
 ExpressionPreviewDialog.generateWidgetHtml = function() {
-    return '<table class="expression-preview-layout">' +
+    return '<table class="expression-preview-layout" rows="4" cols="2">' +
             '<tr>' +
                 '<td>Expression</td>' +
                 '<td>Language</td>' +
@@ -48,7 +48,7 @@ ExpressionPreviewDialog.generateWidgetHtml = function() {
                 '</td>' +
             '</tr>' +
             '<tr>' +
-                '<td class="expression-preview-error-container" bind="expressionPreviewErrorContainer" width="150"></td>' +
+                '<td class="expression-preview-error-container" bind="expressionPreviewErrorContainer" width="150" style="vertical-align: top;"></td>' +
             '</tr>' +
             '<tr>' +
                 '<td colspan="2">' +
@@ -61,7 +61,7 @@ ExpressionPreviewDialog.generateWidgetHtml = function() {
                         '<div id="expression-preview-tabs-preview">' +
                             '<div class="expression-preview-container" bind="expressionPreviewPreviewContainer"></div>' +
                         '</div>' +
-                        '<div id="expression-preview-tabs-history">' +
+                        '<div id="expression-preview-tabs-history" style="display: none;">' +
                             '<div class="expression-preview-container" bind="expressionPreviewHistoryContainer"></div>' +
                         '</div>' +
                         '<div id="expression-preview-tabs-help" style="display: none;">' +
@@ -90,7 +90,7 @@ ExpressionPreviewDialog.Widget = function(
     this._timerID = null;
     
     $("#expression-preview-tabs").tabs();
-    $("#expression-preview-tabs-preview").css("display", "");
+    $("#expression-preview-tabs-history").css("display", "");
     $("#expression-preview-tabs-help").css("display", "");
         
     var self = this;
diff --git a/src/main/webapp/scripts/views/data-table-column-header-ui.js b/src/main/webapp/scripts/views/data-table-column-header-ui.js
index ebceb4200..fdfc4e542 100644
--- a/src/main/webapp/scripts/views/data-table-column-header-ui.js
+++ b/src/main/webapp/scripts/views/data-table-column-header-ui.js
@@ -59,15 +59,15 @@ DataTableColumnHeaderUI.prototype._createMenuForColumnHeader = function(elmt) {
             submenu: [
                 {
                     label: "To Titlecase",
-                    click: function() { self._doTextTransform("toTitlecase(value)", "store-blank"); }
+                    click: function() { self._doTextTransform("toTitlecase(value)", "store-blank", false, ""); }
                 },
                 {
                     label: "To Uppercase",
-                    click: function() { self._doTextTransform("toUppercase(value)", "store-blank"); }
+                    click: function() { self._doTextTransform("toUppercase(value)", "store-blank", false, ""); }
                 },
                 {
                     label: "To Lowercase",
-                    click: function() { self._doTextTransform("toLowercase(value)", "store-blank"); }
+                    click: function() { self._doTextTransform("toLowercase(value)", "store-blank", false, ""); }
                 },
                 {
                     label: "Custom Transform ...",
@@ -399,10 +399,16 @@ DataTableColumnHeaderUI.prototype._doFilterByExpressionPrompt = function(express
     );
 };
 
-DataTableColumnHeaderUI.prototype._doTextTransform = function(expression, onError) {
+DataTableColumnHeaderUI.prototype._doTextTransform = function(expression, onError, repeat, repeatCount) {
     Gridworks.postProcess(
         "do-text-transform",
-        { columnName: this._column.headerLabel, expression: expression, onError: onError },
+        {
+            columnName: this._column.headerLabel, 
+            expression: expression, 
+            onError: onError,
+            repeat: repeat,
+            repeatCount: repeatCount
+        },
         null,
         { cellsChanged: true }
     );
@@ -417,24 +423,36 @@ DataTableColumnHeaderUI.prototype._doTextTransformPrompt = function() {
     var body = $('<div></div>').addClass("dialog-body").appendTo(frame);
     var footer = $('<div></div>').addClass("dialog-footer").appendTo(frame);
     
-    body.html(ExpressionPreviewDialog.generateWidgetHtml());
+    body.html(
+        '<table class="expression-preview-layout">' +
+            '<tr>' +
+                '<td colspan="4">' + ExpressionPreviewDialog.generateWidgetHtml() + '</td>' +
+            '</tr>' +
+            '<tr style="white-space: pre;">' +
+                '<td width="1%">' +
+                    'On error' +
+                '</td>' +
+                '<td>' +
+                    '<input type="radio" name="text-transform-dialog-onerror-choice" value="set-to-blank" checked /> set to blank<br/>' +
+                    '<input type="radio" name="text-transform-dialog-onerror-choice" value="store-error" /> store error<br/>' +
+                    '<input type="radio" name="text-transform-dialog-onerror-choice" value="keep-original" /> keep original' +
+                '</td>' +
+                '<td width="1%">' +
+                    '<input type="checkbox" bind="repeatCheckbox" />' +
+                '</td>' +
+                '<td>' +
+                    'Re-transform until no change<br/>' +
+                    'up to <input bind="repeatCountInput" value="10" size="3" /> times' +
+                '</td>' +
+            '</tr>' +
+        '</table>'
+    );
     var bodyElmts = DOM.bind(body);
     
     footer.html(
-        '<table class="expression-preview-layout">' +
-            '<tr>' +
-                '<td align="left">' +
-                    'On error ' +
-                    '<input type="radio" name="text-transform-dialog-onerror-choice" value="set-to-blank" checked /> set to blank ' +
-                    '<input type="radio" name="text-transform-dialog-onerror-choice" value="store-error" /> store error ' +
-                    '<input type="radio" name="text-transform-dialog-onerror-choice" value="keep-original" /> keep original' +
-                '</td>' +
-                '<td align="right">' +
-                    '<button bind="okButton">&nbsp;&nbsp;OK&nbsp;&nbsp;</button>' +
-                    '<button bind="cancelButton">Cancel</button>' +
-                '</td>' +
-            '</tr>' +
-        '</table>');
+        '<button bind="okButton">&nbsp;&nbsp;OK&nbsp;&nbsp;</button>' +
+        '<button bind="cancelButton">Cancel</button>'
+    );
     var footerElmts = DOM.bind(footer);
         
     var level = DialogSystem.showDialog(frame);
@@ -445,7 +463,9 @@ DataTableColumnHeaderUI.prototype._doTextTransformPrompt = function() {
     footerElmts.okButton.click(function() {
         self._doTextTransform(
             previewWidget.getExpression(true),
-            $('input[name="text-transform-dialog-onerror-choice"]:checked')[0].value
+            $('input[name="text-transform-dialog-onerror-choice"]:checked')[0].value,
+            bodyElmts.repeatCheckbox[0].checked,
+            bodyElmts.repeatCountInput[0].value
         );
         dismiss();
     })
diff --git a/src/main/webapp/styles/common.css b/src/main/webapp/styles/common.css
index 09c935cf6..a41f44da8 100644
--- a/src/main/webapp/styles/common.css
+++ b/src/main/webapp/styles/common.css
@@ -14,7 +14,11 @@ table {
 }
 
 tr, td {
-    vertical-align: top;
+    vertical-align: baseline;
+}
+
+input {
+    vertical-align: middle;
 }
 
 #header {