Replace instances of NSLog with ABPLogger.debugLog()

[dzhang]

Background

There's a couple things motivating this issue.

1. NSLog is bad because it can expose unintended information and is unprofessional to leave in release builds.
2. Having an 'ABPLogger' will reduce the need to refactor every single file that uses logging when it is needed to refactor logging.

Transitioning to os_log may be logical as it is the next step in logging by Apple. If we use something like Crashlytics, that has its own logging calls.

Making our own logging API will insulate the code from these changing needs. For now, having

class ABPLogger
{
    func debugLog(message: String) 
    {
        ...
    }
}

will be a good replacement.

This is one of the same reasons for using RxSwift as it helps keep the code stable across changing iOS SDK changes.

What to change

Replace instances of NSLog with ABPLogger.debugLog() or equivalent function.

[ashephard]

Interesting topic. How do you envision the os_log system working? Would we have a class for logging (i.e. LoggingManager.swift) that would define the subsystem/category, or would each class have their own instance of the log?

[dzhang]

Replying to ashephard:

Interesting topic. How do you envision the os_log system working? Would we have a class for logging (i.e. LoggingManager.swift) that would define the subsystem/category, or would each class have their own instance of the log?

I've updated the issue to better explain the background behind the change. A single class should be enough.

[ashephard]

Replying to dzhang:

Replying to ashephard:

Interesting topic. How do you envision the os_log system working? Would we have a class for logging (i.e. LoggingManager.swift) that would define the subsystem/category, or would each class have their own instance of the log?

I've updated the issue to better explain the background behind the change. A single class should be enough.

You could also add types of log message so that if we needed to, we could limit logs to Errors only, Warnings only, Verbose etc.
I believe you could do that by extending os_log with your ABPLogger class. Also, with os_log you could also specify the category. For a personal project, I split networking, internal functions, UI etc.