Revision: 10510
Author: [email protected]
Date: Mon Aug 8 13:19:39 2011
Log: Change AbstractHasData.getValueKey() to be visible to CellTable
API users, since this is needed in creating custom CellTableBuilder.
Review at http://gwt-code-reviews.appspot.com/1509807
Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=10510
Modified:
/trunk/user/src/com/google/gwt/user/cellview/client/AbstractHasData.java
=======================================
---
/trunk/user/src/com/google/gwt/user/cellview/client/AbstractHasData.java
Mon Aug 8 08:03:35 2011
+++
/trunk/user/src/com/google/gwt/user/cellview/client/AbstractHasData.java
Mon Aug 8 13:19:39 2011
@@ -642,6 +642,19 @@
return tabIndex;
}
+ /**
+ * Get the key for the specified value. If a keyProvider is not
specified or the value is null,
+ * the value is returned. If the key provider is specified, it is used
to get the key from
+ * the value.
+ *
+ * @param value the value
+ * @return the key
+ */
+ public Object getValueKey(T value) {
+ ProvidesKey<T> keyProvider = getKeyProvider();
+ return (keyProvider == null || value == null) ? value :
keyProvider.getKey(value);
+ }
+
@Override
public T getVisibleItem(int indexOnPage) {
checkRowBounds(indexOnPage);
@@ -935,7 +948,7 @@
this.tabIndex = index;
setKeyboardSelected(getKeyboardSelectedRow(), true, false);
}
-
+
@Override
public final void setVisibleRange(int start, int length) {
setVisibleRange(new Range(start, length));
@@ -1018,17 +1031,6 @@
* @return the keyboard selected element
*/
protected abstract Element getKeyboardSelectedElement();
-
- /**
- * Get the key for the specified value.
- *
- * @param value the value
- * @return the key
- */
- protected Object getValueKey(T value) {
- ProvidesKey<T> keyProvider = getKeyProvider();
- return (keyProvider == null || value == null) ? value :
keyProvider.getKey(value);
- }
/**
* Check if keyboard navigation is being suppressed, such as when the
user is
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
