Fixes a couple of build breaks in the samples/expenses app.
Review at http://gwt-code-reviews.appspot.com/872801
Review by: rjrjr@google.com
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@8774 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseDetails.java b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseDetails.java
index 7a114d9..f62ce7a 100644
--- a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseDetails.java
+++ b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseDetails.java
@@ -867,14 +867,7 @@
*/
private String getErrorMessageFromSync(Set<SyncResult> response) {
String errorMessage = "";
- for (SyncResult result : response) {
- if (result.hasViolations()) {
- Map<String, String> violations = result.getViolations();
- for (String message : violations.values()) {
- errorMessage += message + " ";
- }
- }
- }
+ // TODO: Remove this method..
return errorMessage;
}
diff --git a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java
index efce7b6..ae89ac0 100644
--- a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java
+++ b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/ExpenseTree.java
@@ -111,8 +111,7 @@
/**
* The {@link ListDataProvider} used for Employee lists.
*/
- private class EmployeeListDataProvider extends AsyncDataProvider<
- EmployeeProxy> extends Receiver<List<EmployeeProxy>> {
+ private class EmployeeListDataProvider extends AsyncDataProvider<EmployeeProxy> {
private final String department;
@@ -133,17 +132,17 @@
});
}
- public void onSuccess(
- List<EmployeeProxy> response, Set<SyncResult> syncResults) {
- updateRowData(0, response);
- }
-
@Override
protected void onRangeChanged(HasData<EmployeeProxy> view) {
Range range = view.getVisibleRange();
requestFactory.employeeRequest().findEmployeeEntriesByDepartment(
department, range.getStart(), range.getLength()).with(
- getEmployeeMenuProperties()).fire(this);
+ getEmployeeMenuProperties()).fire(new Receiver<List<EmployeeProxy>>(){
+ @Override
+ public void onSuccess(List<EmployeeProxy> response,
+ Set<SyncResult> syncResults) {
+ updateRowData(0, response);
+ }});
}
}
diff --git a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseEntry.java b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseEntry.java
index 9b69f49..4b426c1 100644
--- a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseEntry.java
+++ b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileExpenseEntry.java
@@ -16,6 +16,7 @@
package com.google.gwt.sample.expenses.client;
import com.google.gwt.core.client.GWT;
+
import com.google.gwt.dom.client.Element;
import com.google.gwt.requestfactory.shared.Receiver;
import com.google.gwt.requestfactory.shared.RequestObject;
@@ -31,7 +32,6 @@
import com.google.gwt.user.client.ui.Widget;
import java.util.Date;
-import java.util.Map;
import java.util.Set;
/**
@@ -124,14 +124,7 @@
public void onSuccess(Void ignore, Set<SyncResult> response) {
// Check for commit errors.
String errorMessage = "";
- for (SyncResult result : response) {
- if (result.hasViolations()) {
- Map<String, String> violations = result.getViolations();
- for (String message : violations.values()) {
- errorMessage += message + " ";
- }
- }
- }
+ // FIXME: Error handling disabled...
if (errorMessage.length() > 0) {
errorText.setInnerText(errorMessage);
} else {
diff --git a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileReportEntry.java b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileReportEntry.java
index 34d8a1d..9d7ad6f 100644
--- a/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileReportEntry.java
+++ b/samples/expenses/src/main/java/com/google/gwt/sample/expenses/client/MobileReportEntry.java
@@ -16,6 +16,7 @@
package com.google.gwt.sample.expenses.client;
import com.google.gwt.core.client.GWT;
+
import com.google.gwt.dom.client.Element;
import com.google.gwt.requestfactory.shared.Receiver;
import com.google.gwt.requestfactory.shared.RequestObject;
@@ -31,7 +32,6 @@
import com.google.gwt.user.client.ui.Widget;
import java.util.Date;
-import java.util.Map;
import java.util.Set;
/**
@@ -126,14 +126,7 @@
public void onSuccess(Void ignore, Set<SyncResult> response) {
// Check for commit errors.
String errorMessage = "";
- for (SyncResult result : response) {
- if (result.hasViolations()) {
- Map<String, String> violations = result.getViolations();
- for (String message : violations.values()) {
- errorMessage += message + " ";
- }
- }
- }
+ // FIXME: Error handling disabled
if (errorMessage.length() > 0) {
errorText.setInnerText(errorMessage);
} else {
