update access control

Author: zhongwuzw

.swiftlint.yml

@@ -11,6 +11,10 @@ custom_rules:
     regex: "override (open|public|private|internal|fileprivate)" # matching pattern
     message: "Use like open override or public override instead" # violation message. optional.
     severity: warning # violation severity. optional.
+opt_in_rules:
+  - explicit_acl
+  - explicit_top_level_acl
+explicit_acl: error
+explicit_top_level_acl: error
 included:
-  - Sources
-  - Tests
+  - Sources

Sources/Controllers/MessagesViewController+Keyboard.swift

@@ -72,7 +72,7 @@ extension MessagesViewController {
     }
 
     @objc
-    func adjustScrollViewInset() {
+    internal func adjustScrollViewInset() {
         if #available(iOS 11.0, *) {
             // No need to add to the top contentInset
         } else {

Sources/Extensions/Bundle+Extensions.swift

@@ -26,7 +26,7 @@ import Foundation
 
 extension Bundle {
 
-    public static func messageKitAssetBundle() -> Bundle {
+    internal static func messageKitAssetBundle() -> Bundle {
         let podBundle = Bundle(for: MessagesViewController.self)
 
         guard let resourceBundleUrl = podBundle.url(forResource: "MessageKitAssets", withExtension: "bundle") else {

Sources/Extensions/CGRect+Extensions.swift

@@ -26,7 +26,7 @@ import Foundation
 
 extension CGRect {
 
-    init(_ x: CGFloat, _ y: CGFloat, _ w: CGFloat, _ h: CGFloat) {
+    internal init(_ x: CGFloat, _ y: CGFloat, _ w: CGFloat, _ h: CGFloat) {
         self.init(x: x, y: y, width: w, height: h)
     }
 

Sources/Extensions/NSAttributedString+Extensions.swift

@@ -26,15 +26,15 @@ import Foundation
 
 extension NSAttributedString {
 
-    public func height(considering width: CGFloat) -> CGFloat {
+    internal func height(considering width: CGFloat) -> CGFloat {
 
         let constraintBox = CGSize(width: width, height: .greatestFiniteMagnitude)
         let rect = self.boundingRect(with: constraintBox, options: .usesLineFragmentOrigin, context: nil)
         return rect.height
 
     }
 
-    public func width(considering height: CGFloat) -> CGFloat {
+    internal func width(considering height: CGFloat) -> CGFloat {
 
         let constraintBox = CGSize(width: .greatestFiniteMagnitude, height: height)
         let rect = self.boundingRect(with: constraintBox, options: .usesLineFragmentOrigin, context: nil)

Sources/Extensions/UIColor+Extensions.swift

@@ -26,14 +26,14 @@ import Foundation
 
 extension UIColor {
 
-    public static let incomingGray = UIColor(red: 230/255, green: 230/255, blue: 235/255, alpha: 1.0)
+    internal static let incomingGray = UIColor(red: 230/255, green: 230/255, blue: 235/255, alpha: 1.0)
 
-    public static let outgoingGreen = UIColor(red: 69/255, green: 214/255, blue: 93/255, alpha: 1.0)
+    internal static let outgoingGreen = UIColor(red: 69/255, green: 214/255, blue: 93/255, alpha: 1.0)
 
-    public static let inputBarGray = UIColor(red: 247/255, green: 247/255, blue: 247/255, alpha: 1.0)
+    internal static let inputBarGray = UIColor(red: 247/255, green: 247/255, blue: 247/255, alpha: 1.0)
 
-    public static let playButtonLightGray = UIColor(red: 230/255, green: 230/255, blue: 230/255, alpha: 1.0)
+    internal static let playButtonLightGray = UIColor(red: 230/255, green: 230/255, blue: 230/255, alpha: 1.0)
 
-    public static let sendButtonBlue = UIColor(red: 15/255, green: 135/255, blue: 255/255, alpha: 1.0)
+    internal static let sendButtonBlue = UIColor(red: 15/255, green: 135/255, blue: 255/255, alpha: 1.0)
 
 }

Sources/Extensions/UIEdgeInsets+Extensions.swift

@@ -26,11 +26,11 @@ import Foundation
 
 extension UIEdgeInsets {
 
-    public var vertical: CGFloat {
+    internal var vertical: CGFloat {
         return top + bottom
     }
 
-    public var horizontal: CGFloat {
+    internal var horizontal: CGFloat {
         return left + right
     }
 

Sources/Extensions/UIView+Extensions.swift

@@ -26,7 +26,7 @@ import UIKit
 
 extension UIView {
 
-    public func fillSuperview() {
+    internal func fillSuperview() {
         guard let superview = self.superview else {
             return
         }
@@ -41,7 +41,7 @@ extension UIView {
 	    NSLayoutConstraint.activate(constraints)
     }
 
-    public func centerInSuperview() {
+    internal func centerInSuperview() {
         guard let superview = self.superview else {
             return
         }
@@ -53,7 +53,7 @@ extension UIView {
         NSLayoutConstraint.activate(constraints)
     }
 
-    public func constraint(equalTo size: CGSize) {
+    internal func constraint(equalTo size: CGSize) {
         guard superview != nil else { return }
         translatesAutoresizingMaskIntoConstraints = false
         let constraints: [NSLayoutConstraint] = [
@@ -65,7 +65,7 @@ extension UIView {
     }
 
     @discardableResult
-    public func addConstraints(_ top: NSLayoutYAxisAnchor? = nil, left: NSLayoutXAxisAnchor? = nil, bottom: NSLayoutYAxisAnchor? = nil, right: NSLayoutXAxisAnchor? = nil, topConstant: CGFloat = 0, leftConstant: CGFloat = 0, bottomConstant: CGFloat = 0, rightConstant: CGFloat = 0, widthConstant: CGFloat = 0, heightConstant: CGFloat = 0) -> [NSLayoutConstraint] {
+    internal func addConstraints(_ top: NSLayoutYAxisAnchor? = nil, left: NSLayoutXAxisAnchor? = nil, bottom: NSLayoutYAxisAnchor? = nil, right: NSLayoutXAxisAnchor? = nil, topConstant: CGFloat = 0, leftConstant: CGFloat = 0, bottomConstant: CGFloat = 0, rightConstant: CGFloat = 0, widthConstant: CGFloat = 0, heightConstant: CGFloat = 0) -> [NSLayoutConstraint] {
 
         if self.superview == nil {
             return []

Sources/Models/DetectorType.swift

@@ -38,7 +38,7 @@ public enum DetectorType {
     //case hashtag
     //case custom
 
-    public var textCheckingType: NSTextCheckingResult.CheckingType {
+    internal var textCheckingType: NSTextCheckingResult.CheckingType {
         switch self {
         case .address: return .address
         case .date: return .date

Sources/Models/MessageStyle.swift

@@ -35,7 +35,7 @@ public enum MessageStyle {
         case topRight
         case bottomRight
 
-        var imageOrientation: UIImageOrientation {
+        internal var imageOrientation: UIImageOrientation {
             switch self {
             case .bottomRight: return .up
             case .bottomLeft: return .upMirrored
@@ -52,7 +52,7 @@ public enum MessageStyle {
         case curved
         case pointedEdge
 
-        var imageNameSuffix: String {
+        internal var imageNameSuffix: String {
             switch self {
             case .curved:
                 return "_tail_v2"

Sources/Views/InputStackView.swift

@@ -46,7 +46,7 @@ open class InputStackView: UIStackView {
 
     // MARK: Initialization
 
-    convenience init(axis: UILayoutConstraintAxis, spacing: CGFloat) {
+    public convenience init(axis: UILayoutConstraintAxis, spacing: CGFloat) {
         self.init(frame: .zero)
         self.axis = axis
         self.spacing = spacing

Sources/Views/MessageLabel.swift

@@ -371,7 +371,7 @@ open class MessageLabel: UILabel {
 
     }
 
-  func handleGesture(_ touchLocation: CGPoint) -> Bool {
+  internal func handleGesture(_ touchLocation: CGPoint) -> Bool {
 
         guard let index = stringIndex(at: touchLocation) else { return false }