Github user sagar15795 commented on a diff in the pull request:

    
https://github.com/apache/incubator-taverna-mobile/pull/56#discussion_r169214143
  
    --- Diff: 
app/src/main/java/org/apache/taverna/mobile/ui/workflowdetail/WorkflowDetailPresenter.java
 ---
    @@ -155,28 +155,21 @@ public void onError(Throwable e) {
                         }
     
                         @Override
    -                    public void onNext(Boolean b) {
    -                        if (b) {
    -                            getMvpView().setFavouriteIcon();
    -                        } else {
    -                            getMvpView().showErrorSnackBar("Something went 
wrong please try after" +
    -                                    "sometime");
    -                        }
    +                    public void onComplete() {
     
                         }
                     }));
         }
     
         public void getFavourite(String id) {
    -
    -
    -        mCompositeSubscription.add(mDataManager.getFavoriteWorkflow(id)
    +        checkViewAttached();
    +        compositeDisposable.add(mDataManager.getFavoriteWorkflow(id)
                     .observeOn(AndroidSchedulers.mainThread())
                     .subscribeOn(Schedulers.io())
    -                .subscribe(new Observer<Boolean>() {
    +                .subscribeWith(new DisposableObserver<Boolean>() {
                         @Override
    -                    public void onCompleted() {
    -
    +                    public void onNext(Boolean b) {
    --- End diff --
    
    This Boolean `b` name should be `favoriteStatus`


---

Reply via email to