BMSAnalytics method initialize needs to be reworked

[cshunger]

The ONLY device event that is being picked up by the initialize method is LIFECYCLE.

There are 4 different types of events:
LIFECYLE
NETWORK
NONE
ALL

As of right now, the for loop does not account for the NONE and ALL events:

            for i in 0..<events.count {
                switch(i){
                case 2:
                    lifecycleFlag = true
                    deviceEvents.append(.lifecycle)
                    break
                case 3:
                    networkFlag = true
                    deviceEvents.append(.network)
                    break
                default:
                    lifecycleFlag = true
                    deviceEvents.append(.lifecycle)
                    break
                }

            }

The NONE and ALL will always go to the default in the switch statement.

The only way to collect on NETWORK is to send an array of 4 device events. For example:

var deviceEvents = [BMSAnalytics.NONE, BMSAnalytics.NONE, BMSAnalytics.NONE, BMSAnalytics.NONE];
BMSAnalytics.initialize(applicationName, clientApiKey, hasUserContext,  deviceEvents)

My device events list on the SDK will take on the array of:

• 0 : BMSAnalyticsAPI.DeviceEvent.lifecycle
  • 1 : BMSAnalyticsAPI.DeviceEvent.lifecycle
  • 2 : BMSAnalyticsAPI.DeviceEvent.lifecycle
  • 3 : BMSAnalyticsAPI.DeviceEvent.network

Logic needs to be reworked in the for loop of the initialize method.

[Aerex]

Please verify fix in BMS-Core 1.2.13