From d98e01596b1253f759a89311bb8142c0c21f6580 Mon Sep 17 00:00:00 2001
From: JJ Allaire <jj@rstudio.org>
Date: Mon, 26 Sep 2011 13:51:58 -0400
Subject: [PATCH] Revert "Revert "Revert "don't pass ServerError to
 RequestCallback.onFailure"""

This reverts commit 48ce0bf31149bc2932660fb3c79d33bac3b5ec72.
---
 .../rstudio/core/client/ResultCallback.java   | 10 ++++----
 .../client/workbench/views/source/Source.java | 24 +++++++++----------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/gwt/src/org/rstudio/core/client/ResultCallback.java b/src/gwt/src/org/rstudio/core/client/ResultCallback.java
index b818983fdb5..c2d76a7f99f 100644
--- a/src/gwt/src/org/rstudio/core/client/ResultCallback.java
+++ b/src/gwt/src/org/rstudio/core/client/ResultCallback.java
@@ -16,15 +16,15 @@
  * Provides a generic interface for handling success and/or failure of an
  * operation, especially an asynchronous one.
  */
-public abstract class ResultCallback<TSuccess>
+public abstract class ResultCallback<TSuccess, TFailure>
 {
    public void onSuccess(TSuccess result) {}
-   public void onFailure() {}
+   public void onFailure(TFailure info) {}
 
-   public static <TSuccess>
-   ResultCallback<TSuccess> create(final CommandWithArg<TSuccess> cmd)
+   public static <TSuccess, TFailure>
+   ResultCallback<TSuccess, TFailure> create(final CommandWithArg<TSuccess> cmd)
    {
-      return new ResultCallback<TSuccess>()
+      return new ResultCallback<TSuccess, TFailure>()
       {
          @Override
          public void onSuccess(TSuccess result)
diff --git a/src/gwt/src/org/rstudio/studio/client/workbench/views/source/Source.java b/src/gwt/src/org/rstudio/studio/client/workbench/views/source/Source.java
index 9044c93cdb8..47ef408724e 100644
--- a/src/gwt/src/org/rstudio/studio/client/workbench/views/source/Source.java
+++ b/src/gwt/src/org/rstudio/studio/client/workbench/views/source/Source.java
@@ -406,7 +406,7 @@ public void onNewSourceDoc()
    }
 
    private void newDoc(EditableFileType fileType,
-                       final ResultCallback<EditingTarget> resultCallback)
+                       final ResultCallback<EditingTarget, ServerError> resultCallback)
    {
       ensureVisible(true);
       server_.newDocument(
@@ -430,7 +430,7 @@ public void onError(ServerError error)
                   super.onError(error);
 
                   if (resultCallback != null)
-                     resultCallback.onFailure();
+                     resultCallback.onFailure(error);
                }
             });
    }
@@ -876,7 +876,7 @@ private void openFile(final FileSystemItem file,
                          final CommandWithArg<EditingTarget> executeOnSuccess)
    {
       openFile(file, fileType,
-               ResultCallback.<EditingTarget>create(executeOnSuccess));
+               ResultCallback.<EditingTarget, ServerError>create(executeOnSuccess));
    }
 
    // top-level wrapper for opening files. takes care of:
@@ -888,7 +888,7 @@ private void openFile(final FileSystemItem file,
    //    via the call to the lower level openFile method
    private void openFile(final FileSystemItem file,
                          final TextFileType fileType,
-                         final ResultCallback<EditingTarget> resultCallback)
+                         final ResultCallback<EditingTarget, ServerError> resultCallback)
    {
       ensureVisible(true);
 
@@ -918,7 +918,7 @@ private void openFile(final FileSystemItem file,
       if (file.getLength() > target.getFileSizeLimit())
       {
          if (resultCallback != null)
-            resultCallback.onFailure();
+            resultCallback.onFailure(null);
          showFileTooLargeWarning(file, target.getFileSizeLimit());
       }
       else if (file.getLength() > target.getLargeFileSize())
@@ -933,7 +933,7 @@ public void execute()
             {
                // user (wisely) cancelled
                if (resultCallback != null)
-                  resultCallback.onFailure();
+                  resultCallback.onFailure(null);
             }
          });
       }
@@ -946,7 +946,7 @@ public void execute()
    // variation of top-level file opening wrapper which automatically 
    // deduces the file type
    private void openFile(FileSystemItem file,
-                         ResultCallback<EditingTarget> callback)
+                         ResultCallback<EditingTarget, ServerError> callback)
    {
       TextFileType fileType = fileTypeRegistry_.getTextTypeForFile(file);
       openFile(file, fileType, callback);
@@ -986,7 +986,7 @@ private void confirmOpenLargeFile(FileSystemItem file,
    private void openFileFromServer(
          final FileSystemItem file,
          final TextFileType fileType,
-         final ResultCallback<EditingTarget> resultCallback)
+         final ResultCallback<EditingTarget, ServerError> resultCallback)
    {
       final Command dismissProgress = globalDisplay_.showProgress(
                                                          "Opening file...");
@@ -1003,7 +1003,7 @@ public void onError(ServerError error)
                   dismissProgress.execute();
                   Debug.logError(error);
                   if (resultCallback != null)
-                     resultCallback.onFailure();
+                     resultCallback.onFailure(error);
                   globalDisplay_.showMessage(GlobalDisplay.MSG_ERROR,
                                              "Error while opening file",
                                              error.getUserMessage());
@@ -1115,7 +1115,7 @@ public void onInsertSource(final InsertSourceEvent event)
       else
       {
          newDoc(FileTypeRegistry.R,
-                new ResultCallback<EditingTarget>()
+                new ResultCallback<EditingTarget, ServerError>()
          {
             public void onSuccess(EditingTarget arg)
             {
@@ -1358,7 +1358,7 @@ else if (navigation.getPath() != null)
       {
          suspendSourceNavigationAdding_ = true;
          openFile(FileSystemItem.createFile(navigation.getPath()),
-                  new ResultCallback<EditingTarget>() {
+                  new ResultCallback<EditingTarget, ServerError>() {
                      public void onSuccess(EditingTarget target)
                      {
                         try
@@ -1372,7 +1372,7 @@ public void onSuccess(EditingTarget target)
                      }
 
                      @Override
-                     public void onFailure()
+                     public void onFailure(ServerError info)
                      {
                         suspendSourceNavigationAdding_ = false;
                      }