]> source.dussan.org Git - vaadin-framework.git/commitdiff
Change getColumnProperty to getPropertyId in Grid Column (#15512)
authorTeemu Suo-Anttila <teemusa@vaadin.com>
Wed, 7 Jan 2015 13:58:40 +0000 (15:58 +0200)
committerVaadin Code Review <review@vaadin.com>
Wed, 7 Jan 2015 16:36:48 +0000 (16:36 +0000)
Change-Id: I89a8ccbb918301abed66adcb0d6246c8db0585a5

server/src/com/vaadin/data/RpcDataProviderExtension.java
server/src/com/vaadin/ui/Grid.java
uitest/src/com/vaadin/tests/components/grid/GridColumnExpand.java

index 2b3e00e6f062a2e6722d63ac11c20ea2e78a6846..c968cb1888e9b878a6cce65b12ca7effc1fc202e 100644 (file)
@@ -762,7 +762,7 @@ public class RpcDataProviderExtension extends AbstractExtension {
         Grid grid = getGrid();
 
         for (Column column : columns) {
-            Object propertyId = column.getColumnProperty();
+            Object propertyId = column.getPropertyId();
 
             Object propertyValue = item.getItemProperty(propertyId).getValue();
             JsonValue encodedValue = encodeValue(propertyValue,
@@ -794,7 +794,7 @@ public class RpcDataProviderExtension extends AbstractExtension {
             JsonObject rowObject, Collection<Column> columns) {
         JsonObject cellStyles = null;
         for (Column column : columns) {
-            Object propertyId = column.getColumnProperty();
+            Object propertyId = column.getPropertyId();
             cellReference.set(propertyId);
             String style = generator.getStyle(cellReference);
             if (style != null) {
index d61458297a912ee01c362947a297deb574ec14c7..5b29d1f10b8557a842c25134c0bcb38150ff38f3 100644 (file)
@@ -1781,7 +1781,7 @@ public class Grid extends AbstractComponent implements SelectionNotifier,
         /**
          * Backing property for column
          */
-        private final Object columnProperty;
+        private final Object propertyId;
 
         private Converter<?, Object> converter;
 
@@ -1799,13 +1799,13 @@ public class Grid extends AbstractComponent implements SelectionNotifier,
          *            The grid this column belongs to. Should not be null.
          * @param state
          *            the shared state of this column
-         * @param columnProperty
+         * @param propertyId
          *            the backing property id for this column
          */
-        Column(Grid grid, GridColumnState state, Object columnProperty) {
+        Column(Grid grid, GridColumnState state, Object propertyId) {
             this.grid = grid;
             this.state = state;
-            this.columnProperty = columnProperty;
+            this.propertyId = propertyId;
             internalSetRenderer(new TextRenderer());
         }
 
@@ -1824,8 +1824,8 @@ public class Grid extends AbstractComponent implements SelectionNotifier,
          * 
          * @return property id
          */
-        public Object getColumnProperty() {
-            return columnProperty;
+        public Object getPropertyId() {
+            return propertyId;
         }
 
         /**
@@ -4387,7 +4387,7 @@ public class Grid extends AbstractComponent implements SelectionNotifier,
         editedItemId = itemId;
 
         for (Column column : getColumns()) {
-            Object propertyId = column.getColumnProperty();
+            Object propertyId = column.getPropertyId();
 
             Field<?> editor = getEditorField(propertyId);
 
index eb0c14ae416d0bb694f173f13301cabced6aa467..f8338f991a2b7e8d8f656cecaf3a0e82a433cc64 100644 (file)
@@ -151,7 +151,7 @@ public class GridColumnExpand extends AbstractTestUI {
         double minimumWidth = Math.round(column.getMinimumWidth() * 100) / 100;
         double maximumWidth = Math.round(column.getMaximumWidth() * 100) / 100;
         double width = Math.round(column.getWidth() * 100) / 100;
-        Object propertyId = column.getColumnProperty();
+        Object propertyId = column.getPropertyId();
         label.setValue(String.format(
                 "[%s] Expand ratio: %s - min: %s - max: %s - width: %s",
                 propertyId, expandRatio, minimumWidth, maximumWidth, width));