From 6982b9565946e21c5ad29dd74916e5d207e1aea7 Mon Sep 17 00:00:00 2001
From: Pavel Lukandiy <bux.booga@gmail.com>
Date: Mon, 21 May 2018 21:09:56 +0300
Subject: [PATCH] Refactor and fix annotations

---
 .../PaginationDataLoading/PaginationWrapper.swift          | 7 +++----
 .../PaginationDataLoading/PaginationWrapperDelegate.swift  | 7 +++----
 .../PaginationWrapperUIDelegate.swift                      | 2 ++
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Sources/Classes/DataLoading/PaginationDataLoading/PaginationWrapper.swift b/Sources/Classes/DataLoading/PaginationDataLoading/PaginationWrapper.swift
index 7e9792d4..a804d3a3 100644
--- a/Sources/Classes/DataLoading/PaginationDataLoading/PaginationWrapper.swift
+++ b/Sources/Classes/DataLoading/PaginationDataLoading/PaginationWrapper.swift
@@ -177,15 +177,14 @@ final public class PaginationWrapper<Cursor: ResettableRxDataSourceCursor, Deleg
                 wrappedView.scrollView.support.refreshControl?.endRefreshing()
             }
 
-            let customErrorHandling = uiDelegate?.customInitialLoadingErrorHandling(for: error) ?? false
+            delegate?.clearData()
 
+            let customErrorHandling = uiDelegate?.customInitialLoadingErrorHandling(for: error) ?? false
             guard !customErrorHandling, let errorView = uiDelegate?.errorPlaceholder(for: error) else {
                 return
             }
 
             replacePlaceholderViewIfNeeded(with: errorView)
-
-            delegate?.clearView()
         } else if case .loadingMore = afterState {
             removeInfiniteScroll()
 
@@ -213,7 +212,7 @@ final public class PaginationWrapper<Cursor: ResettableRxDataSourceCursor, Deleg
             wrappedView.scrollView.support.refreshControl?.endRefreshing()
         }
 
-        delegate?.clearView()
+        delegate?.clearData()
 
         guard let emptyView = uiDelegate?.emptyPlaceholder() else {
             return
diff --git a/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperDelegate.swift b/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperDelegate.swift
index a372e74d..c33bde08 100644
--- a/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperDelegate.swift
+++ b/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperDelegate.swift
@@ -20,8 +20,7 @@
 //  THE SOFTWARE.
 //
 
-/// PaginationWrapper delegate used for pagination results handling and
-/// customization of bound states (loading, empty, error, etc.).
+/// PaginationWrapper delegate used for pagination results handling
 public protocol PaginationWrapperDelegate: class {
 
     associatedtype DataSourceType: DataSource
@@ -42,6 +41,6 @@ public protocol PaginationWrapperDelegate: class {
     func paginationWrapper(didReload allItems: DataSourceType.ResultType,
                            using dataSource: DataSourceType)
 
-    /// Clears view when placeholder is shown.
-    func clearView()
+    /// Handles empty data state.
+    func clearData()
 }
diff --git a/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperUIDelegate.swift b/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperUIDelegate.swift
index c3c32093..039eb2a4 100644
--- a/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperUIDelegate.swift
+++ b/Sources/Structures/DataLoading/PaginationDataLoading/PaginationWrapperUIDelegate.swift
@@ -20,6 +20,8 @@
 //  THE SOFTWARE.
 //
 
+/// PaginationWrapper UI delegate used for customization
+/// of bound states (loading, empty, error, etc.).
 public protocol PaginationWrapperUIDelegate: class {
 
     /// Returns placeholder view for empty state.