Fix comments

Author: SBiOSoftWhare

MatrixSDK/Aggregations/LocationSharing/MXBeaconAggregations.swift

@@ -164,6 +164,7 @@ public class MXBeaconAggregations: NSObject {
         }
     }
 
+    /// Get MXBeaconInfoSummary class instead of MXBeaconInfoSummaryProtocol to have access to internal methods
     private func getBeaconInfoSummary(withIdentifier identifier: String, inRoomWithId roomId: String) -> MXBeaconInfoSummary? {
         return self.beaconInfoSummaryStore.getBeaconInfoSummary(withIdentifier: identifier, inRoomWithId: roomId)
     }

MatrixSDK/Aggregations/LocationSharing/MXBeaconInfoSummaryStore.swift

@@ -34,7 +34,7 @@ public class MXBeaconInfoSummaryMemoryStore: NSObject, MXBeaconInfoSummaryStoreP
         }
 
         if let existingIndex = existingIndex {
-            beaconInfoSummaries.insert(beaconInfoSummary, at: existingIndex)
+            beaconInfoSummaries[existingIndex] = beaconInfoSummary
         } else {
             beaconInfoSummaries.append(beaconInfoSummary)
         }

MatrixSDK/JSONModels/Location/MXBeacon.m

@@ -66,14 +66,7 @@ - (nullable instancetype)initWithMXEvent:(MXEvent*)event
         return nil;
     }
 
-    MXBeacon *beacon = [MXBeacon modelFromJSON:event.content];
-    
-    if (!beacon)
-    {
-        return nil;
-    }
-
-    return [self initWithLocation:beacon.location timestamp:beacon.timestamp beaconInfoEventId:beacon.beaconInfoEventId];
+    return [MXBeacon modelFromJSON:event.content];
 }
 
 #pragma mark - Overrides
@@ -101,8 +94,6 @@ + (instancetype)modelFromJSON:(NSDictionary *)JSONDictionary
 
     MXJSONModelSetNumber(timestampNumber, JSONDictionary[kMXMessageContentKeyExtensibleTimestampMSC3488])
 
-    MXJSONModelSetMXJSONModel(location, MXLocation, locationDictionary);
-    
     MXJSONModelSetMXJSONModel(relatesTo, MXEventContentRelatesTo, JSONDictionary[kMXEventRelationRelatesToKey]);
 
     if (relatesTo && [relatesTo.relationType isEqualToString:MXEventRelationTypeReference])

MatrixSDK/LocationSharing/MXBeaconInfoSummary.swift

@@ -85,6 +85,8 @@ public class MXBeaconInfoSummary: NSObject, MXBeaconInfoSummaryProtocol {
         return true
     }
 
+    /// Only set the device id after the beacon start if needed for current user only.
+    /// There is no reason to change it twice.
     @discardableResult
     func updateWithDeviceId(_ deviceId: String) -> Bool {
         guard self.deviceId != nil else {