You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by an...@apache.org on 2007/11/08 15:12:55 UTC
svn commit: r593160 - in /tapestry/tapestry4/trunk/tapestry-contrib/src:
java/org/apache/tapestry/contrib/table/model/simple/
test/org/apache/tapestry/contrib/table/
test/org/apache/tapestry/contrib/table/model/
test/org/apache/tapestry/contrib/table/m...
Author: andyhot
Date: Thu Nov 8 06:12:54 2007
New Revision: 593160
URL: http://svn.apache.org/viewvc?rev=593160&view=rev
Log:
TAPESTRY-1872: use java 1.4 method + add small test for the column and display name transformations
Added:
tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/
tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/
tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/simple/
tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/simple/TestSimpleTableColumn.java
Modified:
tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/table/model/simple/SimpleTableColumn.java
Modified: tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/table/model/simple/SimpleTableColumn.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/table/model/simple/SimpleTableColumn.java?rev=593160&r1=593159&r2=593160&view=diff
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/table/model/simple/SimpleTableColumn.java (original)
+++ tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/table/model/simple/SimpleTableColumn.java Thu Nov 8 06:12:54 2007
@@ -158,7 +158,7 @@
public void setDisplayName(String displayName)
{
if (displayName != null)
- displayName = displayName.replace("_", ".");
+ displayName = displayName.replace('_', '.');
m_strDisplayName = displayName;
}
Added: tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/simple/TestSimpleTableColumn.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/simple/TestSimpleTableColumn.java?rev=593160&view=auto
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/simple/TestSimpleTableColumn.java (added)
+++ tapestry/tapestry4/trunk/tapestry-contrib/src/test/org/apache/tapestry/contrib/table/model/simple/TestSimpleTableColumn.java Thu Nov 8 06:12:54 2007
@@ -0,0 +1,16 @@
+package org.apache.tapestry.contrib.table.model.simple;
+
+import org.apache.tapestry.TestBase;
+import org.testng.annotations.Test;
+
+@Test
+public class TestSimpleTableColumn extends TestBase
+{
+ public void test_display_column_names()
+ {
+ SimpleTableColumn column = new SimpleTableColumn("a.b_c");
+ assertEquals(column.getDisplayName(), "a.b.c");
+ assertEquals(column.getColumnName(), "a_b_c");
+
+ }
+}