Skip to content

Commit 4fa1432

Browse files
changes to pass cocoapods validation
1 parent 00a0ac6 commit 4fa1432

File tree

9 files changed

+112
-79
lines changed

9 files changed

+112
-79
lines changed

Diff for: DockerSwift.podspec

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ Pod::Spec.new do |s|
1313
s.ios.deployment_target = '9.0'
1414
s.swift_version = '4.2'
1515
s.social_media_url = 'https://www.facebook.com/sysdata.it/'
16-
s.summary = 'DockerSwift handles in some easy steps all connections with your remote servers. Offers you some classes to call Web Services defining http method, request, response, .... and some classes to handle resources download.'
16+
s.summary = 'DockerSwift handles in some easy steps all connections with your remote servers.'
1717
s.description = <<-DESC
1818
DockerSwift handles in some easy steps all connections with your remote servers. Offers you some classes to call Web Services defining http method, request, response, .... and some classes to handle resources download.
1919
DESC

Diff for: DockerSwift/Classes/Service/Docker+Blabber.swift

+17
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,23 @@ public func SDLogModuleVerbose(_ message: @autoclosure () -> String, module: Str
3535
SDLogger.shared().log(with: .verbose, module: module, file: String(describing: file), function: String(describing: function), line: line, message: message())
3636
}
3737
#else
38+
public func SDLogModuleError(_ message: @autoclosure () -> String, module: String, file: StaticString = #file , function: StaticString = #function, line: UInt = #line)
39+
{
40+
print(message())
41+
}
42+
public func SDLogModuleInfo(_ message: @autoclosure () -> String, module: String,file: StaticString = #file , function: StaticString = #function, line: UInt = #line)
43+
{
44+
print(message())
45+
}
46+
public func SDLogModuleWarning(_ message: @autoclosure () -> String, module: String,file: StaticString = #file , function: StaticString = #function, line: UInt = #line)
47+
{
48+
print(message())
49+
}
50+
public func SDLogModuleVerbose(_ message: @autoclosure () -> String, module: String,file: StaticString = #file , function: StaticString = #function, line: UInt = #line)
51+
{
52+
print(message())
53+
}
54+
3855
public func SDLogError(_ message: @autoclosure () -> String, file: StaticString = #file , function: StaticString = #function, line: UInt = #line)
3956
{
4057
print(message())

Diff for: DockerSwift/Classes/Service/Request.swift

+3-3
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ extension Request {
9191

9292
// search for "/:" to find the start of a path parameter
9393
while let paramRange = findNextPathParamPlaceholderRange(in: composedUrl) {
94-
let paramName = composedUrl.substring(with: composedUrl.index(after: paramRange.lowerBound)..<paramRange.upperBound)
94+
let paramName = String(composedUrl[composedUrl.index(after: paramRange.lowerBound)..<paramRange.upperBound])
9595
let param = try findPathParam(with: paramName, in: pathParameters)
9696
composedUrl.replaceSubrange(paramRange, with: param)
9797
}
@@ -158,7 +158,7 @@ open class RequestPList: Request {
158158
extension Request {
159159

160160
open override var description: String {
161-
var string = "REQUEST URL: \(service.baseUrl)\(service.path)\nMETHOD:\(method.rawValue)\nHEADERS:\(headers ?? [:]))"
161+
var string = "REQUEST URL: \(service.baseUrl)\(service.path)\nMETHOD:\(method.rawValue)\nHEADERS:\(headers))"
162162
if !urlParameters.isEmpty {
163163
string.append("\nPARAMETERS:\(urlParameters)")
164164
}
@@ -172,7 +172,7 @@ extension Request {
172172
}
173173

174174
open var shortDescription: String {
175-
var string = "REQUEST \(method.rawValue) at \(urlStringDescription)"
175+
let string = "REQUEST \(method.rawValue) at \(urlStringDescription)"
176176
return string
177177
}
178178

Diff for: DockerSwift/Classes/Service/Response.swift

+2-3
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ open class Response<Val, ErrVal>: Responsable {
7272
//MARK: CustomStringConvertible
7373
extension Response {
7474
public var description: String {
75-
var received = response != nil || request.sentInDemoMode
75+
let received = response != nil || request.sentInDemoMode
7676
var d = ""
7777
if received {
7878
d.append("RESPONSE RECEIVED - URL= \(request.urlStringDescription)")
@@ -83,7 +83,6 @@ extension Response {
8383
d.append("\nBODY=\n\(body)")
8484
}
8585
} else {
86-
var urlString: String = ""
8786
d.append("RESPONSE NOT RECEIVED - URL= \(request.urlStringDescription)")
8887
}
8988
return d
@@ -150,7 +149,7 @@ open class DownloadResponse: Response<Any, Any> {
150149
if let localURL = localURL {
151150
let data = try Data(contentsOf: localURL)
152151
let value = UIImage(data: data)
153-
result = .success(value)
152+
result = .success(value as Any)
154153
} else {
155154
SDLogModuleWarning("🌍⚠️ localURL not defined", module: DockerServiceLogModuleName)
156155
}

Diff for: DockerSwift/Classes/Service/ServiceManager.swift

+5-5
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ open class ServiceManager { // : Singleton, Initializable
7171
private func upload<Resp: Responsable>(serviceCall: ServiceCall<Resp>, fileURL: URL) throws {
7272
let urlRequest = try serviceCall.request.buildUrlRequest()
7373
SDLogModuleInfo("🌍▶️ Service Manager: start upload \(serviceCall.request.shortDescription)", module: DockerServiceLogModuleName)
74-
var request = serviceCall.service.sessionManager.upload(fileURL, with: urlRequest as URLRequestConvertible).validate()
74+
let request = serviceCall.service.sessionManager.upload(fileURL, with: urlRequest as URLRequestConvertible).validate()
7575
serviceCall.request.internalRequest = request
7676
sendRequest(request: request, serviceCall: serviceCall)
7777
}
@@ -121,7 +121,7 @@ open class ServiceManager { // : Singleton, Initializable
121121
private func download<Resp: Responsable>(serviceCall: ServiceCall<Resp>, to destination: @escaping DownloadRequest.DownloadFileDestination) throws {
122122
let urlRequest = try serviceCall.request.buildUrlRequest()
123123
SDLogModuleInfo("🌍▶️ Service Manager: start download \(serviceCall.request.shortDescription)", module: DockerServiceLogModuleName)
124-
var request = serviceCall.service.sessionManager.download(urlRequest as URLRequestConvertible, to: destination).validate()
124+
let request = serviceCall.service.sessionManager.download(urlRequest as URLRequestConvertible, to: destination).validate()
125125
serviceCall.request.internalRequest = request
126126
sendRequest(request: request, serviceCall: serviceCall)
127127
}
@@ -166,7 +166,7 @@ open class ServiceManager { // : Singleton, Initializable
166166
default:
167167
response = responseClass.init(statusCode: 0, data: data ?? Data(), request: serviceCall.request, response: urlResponse)
168168
var httpErrorCode = NSURLErrorUnknown
169-
if let err = error as? NSError {
169+
if let err = error as NSError? {
170170
httpErrorCode = err.code
171171
}
172172
responseError = DockerError.missingResponse(error ?? NSError(domain: NSURLErrorDomain, code: NSURLErrorUnknown, userInfo: nil), httpErrorCode)
@@ -181,7 +181,7 @@ open class ServiceManager { // : Singleton, Initializable
181181

182182
open func completeServiceCall<Resp: Responsable>(_ serviceCall: ServiceCall<Resp>, with response: Resp, error: DockerError?) {
183183
if error != nil || (serviceCall.request.useDifferentResponseForErrors && serviceCall.request.httpErrorStatusCodeRange.contains(response.httpStatusCode)) {
184-
SDLogModuleError("🌍‼️ Service completed service with error \(error)", module: DockerServiceLogModuleName)
184+
SDLogModuleError("🌍‼️ Service completed service with error \(String(describing: error))", module: DockerServiceLogModuleName)
185185

186186
// errori da mappare eventualmente
187187
SDLogModuleVerbose("🌍‼️ Trying to map error response", module: DockerServiceLogModuleName)
@@ -271,7 +271,7 @@ public protocol ServiceCallable {
271271
var completion: ServiceCompletion<Resp> { get }
272272
var progressBlock: ProgressHandler? { get }
273273

274-
weak var delegate: ServiceManager? { get }
274+
var delegate: ServiceManager? { get }
275275

276276
var isProcessing: Bool { get }
277277

Diff for: Example/DockerSwift/AppDelegate.swift

+4
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@
88

99
import UIKit
1010
import DockerSwift
11+
#if BLABBER
1112
import Blabber
13+
#endif
1214

1315
@UIApplicationMain
1416
class AppDelegate: UIResponder, UIApplicationDelegate {
@@ -18,7 +20,9 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
1820

1921
func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
2022
// Override point for customization after application launch.
23+
#if BLABBER
2124
SDLogger.shared().setup()
25+
#endif
2226
return true
2327
}
2428

0 commit comments

Comments
 (0)