Merge pull request #18533 from wordpress-mobile/feature/18523-list_fetch_prompts

Blogging Prompts List: show real prompts

Author: ScoutHarris

WordPress/Classes/ViewRelated/Blog/Blogging Prompts/BloggingPromptTableViewCell.swift

@@ -27,12 +27,12 @@ class BloggingPromptTableViewCell: UITableViewCell, NibReusable {
         configureView()
     }
 
-    // TODO: remove answered param. Add BloggingPrompt, configure with its properties.
-    func configure(answered: Bool) {
-        titleLabel.text = "Cast the movie of your life."
-        dateLabel.text = dateFormatter.string(from: Date())
+    func configure(_ prompt: BloggingPrompt) {
+        self.prompt = prompt
+        titleLabel.text = prompt.text.stringByDecodingXMLCharacters().trim()
+        dateLabel.text = dateFormatter.string(from: prompt.date)
         answerCountLabel.text = answerInfoText
-        answeredStateView.isHidden = !answered
+        answeredStateView.isHidden = !prompt.answered
     }
 }
 
@@ -52,11 +52,7 @@ private extension BloggingPromptTableViewCell {
     }
 
     var answerInfoText: String {
-        // TODO: remove when we have a prompt.
-        guard let answerCount = prompt?.answerCount else {
-            return "99 answers"
-        }
-
+        let answerCount = prompt?.answerCount ?? 0
         let stringFormat = (answerCount == 1 ? Strings.answerInfoSingularFormat : Strings.answerInfoPluralFormat)
         return String(format: stringFormat, answerCount)
     }

WordPress/Classes/ViewRelated/Blog/Blogging Prompts/BloggingPromptsViewController.swift

@@ -9,11 +9,24 @@ class BloggingPromptsViewController: UIViewController, NoResultsViewHost {
     @IBOutlet private weak var filterTabBar: FilterTabBar!
 
     private var blog: Blog?
+    private var prompts: [BloggingPrompt] = [] {
+        didSet {
+            tableView.reloadData()
+            showNoResultsViewIfNeeded()
+        }
+    }
 
-    // TODO: remove when prompts are fetched, use fetched prompts count.
-    private var promptCount: Int = 10
-    // TODO: set when prompt fetching is added.
-    private var isLoading: Bool = false
+    private lazy var bloggingPromptsService: BloggingPromptsService? = {
+        return BloggingPromptsService(blog: blog)
+    }()
+
+    private var isLoading: Bool = false {
+        didSet {
+            if isLoading != oldValue {
+                showNoResultsViewIfNeeded()
+            }
+        }
+    }
 
     // MARK: - Init
 
@@ -35,9 +48,12 @@ class BloggingPromptsViewController: UIViewController, NoResultsViewHost {
         title = Strings.viewTitle
         configureFilterTabBar()
         configureTableView()
-        showNoResultsViewIfNeeded()
     }
 
+    override func viewWillAppear(_ animated: Bool) {
+        super.viewWillAppear(animated)
+        fetchPrompts()
+    }
 }
 
 // MARK: - Private Methods
@@ -54,28 +70,28 @@ private extension BloggingPromptsViewController {
     }
 
     func showNoResultsViewIfNeeded() {
-        hideNoResults()
-
         guard !isLoading else {
             showLoadingView()
             return
         }
 
-        // TODO: use fetched prompts count.
-        guard promptCount == 0 else {
+        guard prompts.isEmpty else {
+            hideNoResults()
             return
         }
 
         showNoResultsView()
     }
 
     func showNoResultsView() {
+        hideNoResults()
         configureAndDisplayNoResults(on: tableView,
                                      title: NoResults.emptyTitle,
                                      image: NoResults.imageName)
     }
 
     func showLoadingView() {
+        hideNoResults()
         configureAndDisplayNoResults(on: tableView,
                                      title: NoResults.loadingTitle,
                                      accessoryView: NoResultsViewController.loadingAccessoryView())
@@ -89,6 +105,27 @@ private extension BloggingPromptsViewController {
                                      image: NoResults.imageName)
     }
 
+    func fetchPrompts() {
+        // TODO: show cached prompts first.
+
+        guard let bloggingPromptsService = bloggingPromptsService else {
+            DDLogError("Failed creating BloggingPromptsService instance.")
+            showErrorView()
+            return
+        }
+
+        isLoading = true
+
+        bloggingPromptsService.fetchPrompts(success: { [weak self] (prompts) in
+            self?.isLoading = false
+            self?.prompts = prompts.sorted(by: { $0.date.compare($1.date) == .orderedDescending })
+        }, failure: { [weak self] (error) in
+            DDLogError("Failed fetching blogging prompts: \(String(describing: error))")
+            self?.isLoading = false
+            self?.showErrorView()
+        })
+    }
+
     enum Strings {
         static let viewTitle = NSLocalizedString("Prompts", comment: "View title for Blogging Prompts list.")
     }
@@ -108,17 +145,16 @@ private extension BloggingPromptsViewController {
 extension BloggingPromptsViewController: UITableViewDataSource, UITableViewDelegate {
 
     func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
-        // TODO: use fetched prompts count.
-        return promptCount
+        return prompts.count
     }
 
     func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
-        guard let cell = tableView.dequeueReusableCell(withIdentifier: BloggingPromptTableViewCell.defaultReuseID) as? BloggingPromptTableViewCell else {
+        guard let cell = tableView.dequeueReusableCell(withIdentifier: BloggingPromptTableViewCell.defaultReuseID) as? BloggingPromptTableViewCell,
+              let prompt = prompts[safe: indexPath.row] else {
             return UITableViewCell()
         }
 
-        // TODO: replace answered with BloggingPrompt.
-        cell.configure(answered: indexPath.row > 4)
+        cell.configure(prompt)
         return cell
     }
 
@@ -152,7 +188,6 @@ private extension BloggingPromptsViewController {
         // TODO:
         // - track selected filter changed
         // - refresh view for selected filter
-        showNoResultsViewIfNeeded()
     }
 
 }