Merge branch 'master' of https://github.com/Lickability/PinpointKit

Author: Twigz

Example/PinpointKitExample.xcodeproj/project.pbxproj

@@ -357,7 +357,7 @@
 				ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
 				INFOPLIST_FILE = PinpointKitExample/Info.plist;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
-				MARKETING_VERSION = 1.4.0;
+				MARKETING_VERSION = 1.5.0;
 				PRODUCT_BUNDLE_IDENTIFIER = net.Lickability.PinpointKitExample;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 			};
@@ -370,7 +370,7 @@
 				ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
 				INFOPLIST_FILE = PinpointKitExample/Info.plist;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
-				MARKETING_VERSION = 1.4.0;
+				MARKETING_VERSION = 1.5.0;
 				PRODUCT_BUNDLE_IDENTIFIER = net.Lickability.PinpointKitExample;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SWIFT_OPTIMIZATION_LEVEL = "-Owholemodule";

Example/PinpointKitExample/Info.plist

@@ -36,6 +36,8 @@
 		<string>UIInterfaceOrientationLandscapeLeft</string>
 		<string>UIInterfaceOrientationLandscapeRight</string>
 	</array>
+	<key>NSPhotoLibraryUsageDescription</key>
+	<string>The example application accesses your photo library.</string>
 	<key>UISupportedInterfaceOrientations~ipad</key>
 	<array>
 		<string>UIInterfaceOrientationPortrait</string>

Example/Podfile.lock

@@ -1,7 +1,7 @@
 PODS:
-  - PinpointKit (1.4.0):
-    - PinpointKit/Core (= 1.4.0)
-  - PinpointKit/Core (1.4.0)
+  - PinpointKit (1.5.0):
+    - PinpointKit/Core (= 1.5.0)
+  - PinpointKit/Core (1.5.0)
   - SwiftLint (0.25.1)
 
 DEPENDENCIES:
@@ -17,7 +17,7 @@ EXTERNAL SOURCES:
     :path: "../"
 
 SPEC CHECKSUMS:
-  PinpointKit: a33c929c2e9c3b7d9a1a343fd309161b71c02d06
+  PinpointKit: a513c4623502581067f3f8b60b9cffef6c8e1025
   SwiftLint: ce933681be10c3266e82576dad676fa815a602e9
 
 PODFILE CHECKSUM: 24af8a2ca63352a72ca1cd13fc8977477eae3a12

Example/Pods/Local Podspecs/PinpointKit.podspec.json

@@ -1,10 +1,10 @@
 {
   "name": "PinpointKit",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "homepage": "https://github.com/Lickability/PinpointKit",
   "source": {
       "git": "https://github.com/Lickability/PinpointKit.git",
-      "tag": "1.4.0"
+      "tag": "1.5.0"
   },
   "summary": "A library that makes bug reporting simple for your users by allowing them to send feedback with annotated screenshots and logs.",
   "authors" : {

Example/Pods/Manifest.lock

@@ -70,6 +70,7 @@
 		DA0DA60D1C53049B0012ADBE /* PinpointKit.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = DA0DA6021C53049B0012ADBE /* PinpointKit.framework */; };
 		F24381121D54CBAB004CC87F /* SystemLogCollectorTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = F24381111D54CBAB004CC87F /* SystemLogCollectorTests.swift */; };
 		F24381151D54ECD2004CC87F /* XCTestCaseExpectationExtensions.swift in Sources */ = {isa = PBXBuildFile; fileRef = F24381141D54ECD2004CC87F /* XCTestCaseExpectationExtensions.swift */; };
+		F2619BA32512C6100083286C /* RequestScreenshotCell.swift in Sources */ = {isa = PBXBuildFile; fileRef = F2619BA22512C6100083286C /* RequestScreenshotCell.swift */; };
 /* End PBXBuildFile section */
 
 /* Begin PBXContainerItemProxy section */
@@ -147,6 +148,7 @@
 		DA0DA6131C53049B0012ADBE /* Info.plist */ = {isa = PBXFileReference; lastKnownFileType = text.plist.xml; path = Info.plist; sourceTree = "<group>"; };
 		F24381111D54CBAB004CC87F /* SystemLogCollectorTests.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = SystemLogCollectorTests.swift; sourceTree = "<group>"; };
 		F24381141D54ECD2004CC87F /* XCTestCaseExpectationExtensions.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = XCTestCaseExpectationExtensions.swift; sourceTree = "<group>"; };
+		F2619BA22512C6100083286C /* RequestScreenshotCell.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; name = RequestScreenshotCell.swift; path = Core/RequestScreenshotCell.swift; sourceTree = "<group>"; };
 /* End PBXFileReference section */
 
 /* Begin PBXFrameworksBuildPhase section */
@@ -206,6 +208,7 @@
 				4C4037F11D9EB01800305A6E /* FeedbackViewController.swift */,
 				4C4037EF1D9EB01800305A6E /* FeedbackNavigationController.swift */,
 				4C4037FC1D9EB04400305A6E /* Screenshotter.swift */,
+				F2619BA22512C6100083286C /* RequestScreenshotCell.swift */,
 				4C4037F91D9EB02300305A6E /* ScreenshotCell.swift */,
 				4C4037EE1D9EB01800305A6E /* FeedbackConfiguration.swift */,
 				4C4037F01D9EB01800305A6E /* FeedbackTableViewDataSource.swift */,
@@ -490,6 +493,7 @@
 				4C4038091D9EB0CB00305A6E /* UIView+PinpointKit.swift in Sources */,
 				4CFB58801E29464B00B64D0D /* EditImageViewControllerBarButtonItemProviding.swift in Sources */,
 				4C40382C1D9EB7A800305A6E /* ScreenshotDetector.swift in Sources */,
+				F2619BA32512C6100083286C /* RequestScreenshotCell.swift in Sources */,
 				4C4038011D9EB07000305A6E /* LogViewer.swift in Sources */,
 				4C4037FF1D9EB06900305A6E /* BasicLogViewController.swift in Sources */,
 				4C4037D51D9EAF0D00305A6E /* Editor.swift in Sources */,
@@ -686,7 +690,7 @@
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
 				IPHONEOS_DEPLOYMENT_TARGET = 9.0;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @loader_path/Frameworks";
-				MARKETING_VERSION = 1.4.0;
+				MARKETING_VERSION = 1.5.0;
 				PRODUCT_BUNDLE_IDENTIFIER = net.Lickability.PinpointKit;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SKIP_INSTALL = YES;
@@ -709,7 +713,7 @@
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
 				IPHONEOS_DEPLOYMENT_TARGET = 9.0;
 				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks @loader_path/Frameworks";
-				MARKETING_VERSION = 1.4.0;
+				MARKETING_VERSION = 1.5.0;
 				PRODUCT_BUNDLE_IDENTIFIER = net.Lickability.PinpointKit;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				SKIP_INSTALL = YES;

Example/Pods/Pods.xcodeproj/project.pbxproj

@@ -24,10 +24,10 @@ public protocol FeedbackCollector: class, LogSupporting, InterfaceCustomizable {
     /**
      Begins feedback collection about a screenshot from a view controller.
 
-     - parameter screenshot:     The screenshot the user will be providing feedback on.
+     - parameter screenshot:     The screenshot the user will be providing feedback on. If the screenshot is nil, the user will be presented with a button to select a screenshot from their photo library.
      - parameter viewController: The view controller from which to present.
      */
-    func collectFeedback(with screenshot: UIImage, from viewController: UIViewController)
+    func collectFeedback(with screenshot: UIImage?, from viewController: UIViewController)
 }
 
 extension FeedbackCollector where Self: UIViewController {

Example/Pods/Target Support Files/PinpointKit/Info.plist

@@ -114,9 +114,9 @@ public final class FeedbackNavigationController: UINavigationController, Feedbac
 
     // MARK: - FeedbackCollector
 
-    public func collectFeedback(with screenshot: UIImage, from viewController: UIViewController) {
+    public func collectFeedback(with screenshot: UIImage?, from viewController: UIViewController) {
         guard presentingViewController == nil else {
-            NSLog("Unable to present FeedbackNavigationController because it is already being presetned")
+            NSLog("Unable to present FeedbackNavigationController because it is already being presented")
             return
         }
 

PinpointKit.podspec.json

@@ -18,12 +18,12 @@ final class FeedbackTableViewDataSource: NSObject, UITableViewDataSource {
      Initializes the data source with a configuration and a boolean value indicating whether the user has enabled log collection.
 
      - parameter interfaceCustomization:   The interface customization used to set up the data source.
-     - parameter screenshot:               The screenshot to display for annotating.
+     - parameter screenshot:               The screenshot to display for annotating. If nil, a button will appear allowing the user to select a screenshot.
      - parameter logSupporting:            The object the controls the support of logging.
      - parameter userEnabledLogCollection: A boolean value indicating whether the user has enabled log collection.
      - parameter delegate:                 The object informed when a screenshot is tapped.
      */
-    init(interfaceCustomization: InterfaceCustomization, screenshot: UIImage, logSupporting: LogSupporting, userEnabledLogCollection: Bool, delegate: FeedbackTableViewDataSourceDelegate? = nil) {
+    init(interfaceCustomization: InterfaceCustomization, screenshot: UIImage?, logSupporting: LogSupporting, userEnabledLogCollection: Bool, delegate: FeedbackTableViewDataSourceDelegate? = nil) {
         sections = type(of: self).sectionsFromConfiguration(interfaceCustomization, screenshot: screenshot, logSupporting: logSupporting, userEnabledLogCollection: userEnabledLogCollection)
         self.delegate = delegate
     }
@@ -40,19 +40,28 @@ final class FeedbackTableViewDataSource: NSObject, UITableViewDataSource {
     }
 
     private enum Row {
+        case selectScreenshot(text: String, font: UIFont)
         case screenshot(screensot: UIImage, hintText: String?, hintFont: UIFont)
         case collectLogs(enabled: Bool, title: String, font: UIFont, canView: Bool)
     }
 
     // MARK: - FeedbackTableViewDataSource
 
-    private static func sectionsFromConfiguration(_ interfaceCustomization: InterfaceCustomization, screenshot: UIImage, logSupporting: LogSupporting, userEnabledLogCollection: Bool) -> [Section] {
+    private static func sectionsFromConfiguration(_ interfaceCustomization: InterfaceCustomization, screenshot: UIImage?, logSupporting: LogSupporting, userEnabledLogCollection: Bool) -> [Section] {
         var sections: [Section] = []
 
-        let screenshotRow = Row.screenshot(screensot: screenshot, hintText: interfaceCustomization.interfaceText.feedbackEditHint, hintFont: interfaceCustomization.appearance.feedbackEditHintFont)
-        let screenshotSection = Section.feedback(rows: [screenshotRow])
-        
-        sections.append(screenshotSection)
+        if let screenshot = screenshot {
+            let screenshotRow = Row.screenshot(screensot: screenshot, hintText: interfaceCustomization.interfaceText.feedbackEditHint, hintFont: interfaceCustomization.appearance.feedbackEditHintFont)
+            let screenshotSection = Section.feedback(rows: [screenshotRow])
+            
+            sections.append(screenshotSection)
+        } else {
+            let requestScreenshotRow = Row.selectScreenshot(text: interfaceCustomization.interfaceText.selectScreenshotButtonTitle, font: interfaceCustomization.appearance.selectScreenshotButtonFont)
+            
+            let screenshotSection = Section.feedback(rows: [requestScreenshotRow])
+            
+            sections.append(screenshotSection)
+        }
 
         if logSupporting.logCollector != nil {
             let collectLogsRow = Row.collectLogs(enabled: userEnabledLogCollection, title: interfaceCustomization.interfaceText.logCollectionPermissionTitle, font: interfaceCustomization.appearance.logCollectionPermissionFont, canView: logSupporting.logViewer != nil)
@@ -98,6 +107,26 @@ final class FeedbackTableViewDataSource: NSObject, UITableViewDataSource {
         return cell
     }
 
+    private func requestScreenshotCell(for row: Row) -> UITableViewCell {
+        let cell = RequestScreenshotCell()
+        
+        guard case let .selectScreenshot(text, font) = row else {
+            assertionFailure("Found unexpected row type when creating screenshot cell.")
+            return cell
+        }
+                
+        cell.viewModel = RequestScreenshotCell.ViewModel(buttonText: text, buttonFont: font)
+        cell.screenshotButtonTapHandler = { [weak self] _ in
+            guard let self = self else { return }
+            
+            if #available(iOS 14, *) {
+                self.delegate?.feedbackTableViewDataSourceDidRequestScreenshot(feedbackTableViewDataSource: self)
+            }
+        }
+        
+        return cell
+    }
+    
     // MARK: - UITableViewDataSource
 
     func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
@@ -115,6 +144,8 @@ final class FeedbackTableViewDataSource: NSObject, UITableViewDataSource {
         case let .feedback(rows):
             let row = rows[indexPath.row]
             switch row {
+            case .selectScreenshot:
+                return requestScreenshotCell(for: row)
             case .screenshot:
                 return screenshotCell(for: row)
             case .collectLogs:
@@ -134,4 +165,12 @@ protocol FeedbackTableViewDataSourceDelegate: class {
      - parameter screenshot:                  The screenshot that was tapped.
      */
     func feedbackTableViewDataSource(feedbackTableViewDataSource: FeedbackTableViewDataSource, didTapScreenshot screenshot: UIImage)
+    
+    /**
+     Notifies the delegate that a screenshot is being requested.
+     
+     - parameter feedbackTableViewDataSource: The feedback table view data source that sent the message.
+     */
+    @available(iOS 14, *)
+    func feedbackTableViewDataSourceDidRequestScreenshot(feedbackTableViewDataSource: FeedbackTableViewDataSource)
 }

PinpointKit/PinpointKit.xcodeproj/project.pbxproj

@@ -7,6 +7,7 @@
 //
 
 import UIKit
+import PhotosUI
 
 /// A `UITableViewController` that conforms to `FeedbackCollector` in order to display an interface that allows the user to see, change, and send feedback.
 public final class FeedbackViewController: UITableViewController {
@@ -108,7 +109,6 @@ public final class FeedbackViewController: UITableViewController {
 
     private func updateDataSource() {
         guard let interfaceCustomization = interfaceCustomization else { assertionFailure(); return }
-        guard let screenshot = screenshot else { assertionFailure(); return }
         let screenshotToDisplay = annotatedScreenshot ?? screenshot
 
         dataSource = FeedbackTableViewDataSource(interfaceCustomization: interfaceCustomization, screenshot: screenshotToDisplay, logSupporting: self, userEnabledLogCollection: userEnabledLogCollection, delegate: self)
@@ -190,7 +190,7 @@ public final class FeedbackViewController: UITableViewController {
 // MARK: - FeedbackCollector
 
 extension FeedbackViewController: FeedbackCollector {
-    public func collectFeedback(with screenshot: UIImage, from viewController: UIViewController) {
+    public func collectFeedback(with screenshot: UIImage?, from viewController: UIViewController) {
         self.screenshot = screenshot
         annotatedScreenshot = nil
         viewController.showDetailViewController(self, sender: viewController)
@@ -249,4 +249,38 @@ extension FeedbackViewController: FeedbackTableViewDataSourceDelegate {
         editImageViewController.modalPresentationStyle = feedbackConfiguration?.presentationStyle ?? .fullScreen
         present(editImageViewController, animated: true, completion: nil)
     }
+    
+    @available(iOS 14, *)
+    func feedbackTableViewDataSourceDidRequestScreenshot(feedbackTableViewDataSource: FeedbackTableViewDataSource) {
+        var configuration = PHPickerConfiguration(photoLibrary: .shared())
+        configuration.filter = .images
+        
+        let pickerController = PHPickerViewController(configuration: configuration)
+        pickerController.delegate = self
+        viewController.present(pickerController, animated: true, completion: nil)
+    }
+}
+
+@available(iOS 14, *)
+extension FeedbackViewController: PHPickerViewControllerDelegate {
+    
+    public func picker(_ picker: PHPickerViewController, didFinishPicking results: [PHPickerResult]) {
+        guard let result = results.first else {
+            picker.presentingViewController?.dismiss(animated: true)
+            return
+        }
+        
+        result.itemProvider.loadObject(ofClass: UIImage.self, completionHandler: { image, _ in
+            OperationQueue.main.addOperation {
+                defer {
+                    picker.presentingViewController?.dismiss(animated: true)
+                }
+                
+                guard let image = image as? UIImage else { return }
+                self.screenshot = image
+                
+                self.tableView.reloadData()
+            }
+        })
+    }
 }