fix: removes callbacks and fixes segment and sonarqube issues (#9)

* fix: removes callbacks from the ios sdk and other bug fixes

* fix: removes recaptcha type as its not longer needed

* fix: sonar issues

* fix: sonar issues

Author: pandeymangg

FormbricksSDK.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name             = "FormbricksSDK"
-  s.version          = "1.0.1"                              
+  s.version          = "1.1.0"                              
   s.summary          = "iOS SDK for Formbricks" 
   s.homepage         = "https://github.com/formbricks/ios"   
   s.license          = { :type => "MIT", :file => "LICENSE" }

Sources/FormbricksSDK/Extension/Error+Message.swift

@@ -1,6 +1,6 @@
 import Foundation
 
-extension Error {
+public extension Error {
     var message: String {
         if let error = self as? RuntimeError {
             return error.message

Sources/FormbricksSDK/Formbricks.swift

@@ -41,12 +41,15 @@ import Network
      */
     @objc public static func setup(with config: FormbricksConfig, force: Bool = false) {
         logger = Logger()
+        apiQueue = OperationQueue()
 
-        if (force == true) {
+        if force {
             isInitialized = false
         }
+        
         guard !isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsAlreadyInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsAlreadyInitialized)
+            Formbricks.logger?.error(error.message)
             return
         }
 
@@ -87,7 +90,8 @@ import Network
      */
     @objc public static func setUserId(_ userId: String) {
         guard Formbricks.isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsNotInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsNotInitialized)
+                        Formbricks.logger?.error(error.message)
             return
         }
 
@@ -110,7 +114,8 @@ import Network
      */
     @objc public static func setAttribute(_ attribute: String, forKey key: String) {
         guard Formbricks.isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsNotInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsNotInitialized)
+            Formbricks.logger?.error(error.message)
             return
         }
 
@@ -128,7 +133,8 @@ import Network
      */
     @objc public static func setAttributes(_ attributes: [String : String]) {
         guard Formbricks.isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsNotInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsNotInitialized)
+            Formbricks.logger?.error(error.message)
             return
         }
 
@@ -146,7 +152,8 @@ import Network
      */
     @objc public static func setLanguage(_ language: String) {
         guard Formbricks.isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsNotInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsNotInitialized)
+            Formbricks.logger?.error(error.message)
             return
         }
 
@@ -169,7 +176,8 @@ import Network
      */
     @objc public static func track(_ action: String) {
         guard Formbricks.isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsNotInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsNotInitialized)
+            Formbricks.logger?.error(error.message)
             return
         }
 
@@ -194,7 +202,8 @@ import Network
      */
     @objc public static func logout() {
         guard Formbricks.isInitialized else {
-            Formbricks.logger?.error(FormbricksSDKError(type: .sdkIsNotInitialized).message)
+            let error = FormbricksSDKError(type: .sdkIsNotInitialized)
+            Formbricks.logger?.error(error.message)
             return
         }
 
@@ -235,6 +244,7 @@ import Network
     }
 
     private static func performCleanup() {
+        userManager?.logout()
         userManager?.cleanupUpdateQueue()
         presentSurveyManager?.dismissView()
         presentSurveyManager = nil

Sources/FormbricksSDK/Helpers/AnyCodable/AnyCodable.swift

@@ -144,4 +144,4 @@ extension AnyCodable: Hashable {
             break
         }
     }
-}
+}

Sources/FormbricksSDK/Helpers/AnyCodable/AnyDecodable.swift

@@ -186,4 +186,4 @@ extension AnyDecodable: Hashable {
             break
         }
     }
-}
+}

Sources/FormbricksSDK/Helpers/AnyCodable/AnyEncodable.swift

@@ -289,4 +289,4 @@ extension AnyEncodable: Hashable {
             break
         }
     }
-}
+}

Sources/FormbricksSDK/Helpers/FormbricksEnvironment.swift

@@ -1,10 +1,32 @@
 import Foundation
 
-class FormbricksEnvironment {
-    public static let baseApiUrl: String = Formbricks.appUrl ?? "http://localhost:3000"
-    public static let surveyScriptUrl: String = "\(baseApiUrl)/js/surveys.umd.cjs"
-    /// Endpoint for getting environment data. Replace {environmentId} with the actual environment ID.
-    public static let getEnvironmentRequestEndpoint: String = "/api/v2/client/{environmentId}/environment"
-    /// Endpoint for posting user data. Replace {environmentId} with the actual environment ID.
-    public static let postUserRequestEndpoint: String = "/api/v2/client/{environmentId}/user"
+internal enum FormbricksEnvironment {
+
+  /// Only `appUrl` is user-supplied. Crash early if it’s missing.
+  fileprivate static var baseApiUrl: String {
+    guard let url = Formbricks.appUrl else {
+      fatalError("Formbricks.setup must be called before using the SDK.")
+    }
+    return url
+  }
+
+  /// Returns the full survey‐script URL as a String
+  static var surveyScriptUrlString: String {
+    let path = "/" + ["js", "surveys.umd.cjs"].joined(separator: "/") // NOSONAR we can hard-code "/" here
+    return baseApiUrl + path
+  }
+
+  /// Returns the full environment‐fetch URL as a String for the given ID
+    static var getEnvironmentRequestEndpoint: String {
+      let path = "/" + ["api", "v2", "client", "{environmentId}", "environment"] // NOSONAR we can hard-code "/" here
+      .joined(separator: "/")
+    return path
+  }
+
+  /// Returns the full post-user URL as a String for the given ID
+    static var postUserRequestEndpoint: String {
+    let path = "/" + ["api", "v2", "client", "{environmentId}", "user"] // NOSONAR we can hard-code "/" here
+      .joined(separator: "/")
+    return path
+  }
 }

Sources/FormbricksSDK/Manager/SurveyManager.swift

@@ -62,7 +62,19 @@ final class SurveyManager {
 
         // Display percentage
         let shouldDisplay = shouldDisplayBasedOnPercentage(firstSurveyWithActionClass?.displayPercentage)
-                
+        let isMultiLangSurvey = firstSurveyWithActionClass?.languages?.count ?? 0 > 1
+
+        if isMultiLangSurvey {
+            guard let survey = firstSurveyWithActionClass else {return}
+            let currentLanguage = Formbricks.language
+            guard let languageCode = getLanguageCode(survey: survey, language: currentLanguage) else {
+                Formbricks.logger?.error("Survey \(survey.name) is not available in language “\(currentLanguage)”. Skipping.")
+                return
+            }
+            
+            Formbricks.language = languageCode
+        }
+
         // Display and delay it if needed
         if let surveyId = firstSurveyWithActionClass?.id, shouldDisplay {
             isShowingSurvey = true
@@ -93,7 +105,8 @@ extension SurveyManager {
                 self?.filterSurveys()
             case .failure:
                 self?.hasApiError = true
-                Formbricks.logger?.error(FormbricksSDKError(type: .unableToRefreshEnvironment).message)
+                let error = FormbricksSDKError(type: .unableToRefreshEnvironment)
+                Formbricks.logger?.error(error.message)
                 self?.startErrorTimer()
             }
         }
@@ -171,7 +184,8 @@ extension SurveyManager {
                 if let data = UserDefaults.standard.data(forKey: SurveyManager.environmentResponseObjectKey) {
                     return try? JSONDecoder().decode(EnvironmentResponse.self, from: data)
                 } else {
-                    Formbricks.logger?.error(FormbricksSDKError(type: .unableToRetrieveEnvironment).message)
+                    let error = FormbricksSDKError(type: .unableToRetrieveEnvironment)
+                    Formbricks.logger?.error(error.message)
                     return nil
                 }
             }
@@ -180,7 +194,8 @@ extension SurveyManager {
                 UserDefaults.standard.set(data, forKey: SurveyManager.environmentResponseObjectKey)
                 backingEnvironmentResponse = newValue
             } else {
-                Formbricks.logger?.error(FormbricksSDKError(type: .unableToPersistEnvironment).message)
+                let error = FormbricksSDKError(type: .unableToPersistEnvironment)
+                Formbricks.logger?.error(error.message)
             }
         }
     }
@@ -212,7 +227,8 @@ private extension SurveyManager {
                 }
 
             default:
-                Formbricks.logger?.error(FormbricksSDKError(type: .invalidDisplayOption).message)
+                let error = FormbricksSDKError(type: .invalidDisplayOption)
+                Formbricks.logger?.error(error.message)
                 return false
             }
 
@@ -234,6 +250,47 @@ private extension SurveyManager {
         }
     }
 
+    func getLanguageCode(
+        survey: Survey,
+        language: String?
+    ) -> String? {
+        // 1) Collect all codes
+        let availableLanguageCodes = survey.languages?
+            .map { $0.language.code }
+        
+        // 2) If no language was passed or it's the explicit "default" token → default
+        guard let raw = language?.lowercased(), !raw.isEmpty else {
+            return "default"
+        }
+        
+        if raw == "default" {
+            return "default"
+        }
+    
+        // 3) Find matching entry by code or alias
+        let selected = survey.languages?.first { entry in
+            entry.language.code.lowercased() == raw ||
+            entry.language.alias?.lowercased() == raw
+        }
+        
+        // 4) If that entry is marked default → default
+        if selected?.isDefault == true {
+            return "default"
+        }
+        
+        // 5) If no entry, or not enabled, or code not in the available list → nil
+        guard
+            let entry = selected,
+            entry.enabled,
+            availableLanguageCodes?.contains(entry.language.code) == true
+        else {
+            return nil
+        }
+        
+        // 6) Otherwise return its code
+        return entry.language.code
+    }
+    
     /// Filters the surveys based on the user's segments.
     func filterSurveysBasedOnSegments(_ surveys: [Survey], segments: [String]) -> [Survey] {
         return surveys.filter { survey in

Sources/FormbricksSDK/Manager/UserManager.swift

@@ -132,7 +132,10 @@ final class UserManager: UserManagerSyncable {
         backingLastDisplayedAt = nil
         backingExpiresAt = nil
         Formbricks.language = "default"
-        updateQueue?.reset()
+        
+        syncTimer?.invalidate()
+        syncTimer = nil
+        updateQueue?.cleanup()
 
         if isUserIdDefined {
             Formbricks.logger?.debug("Successfully logged out user and reset the user state.")

Sources/FormbricksSDK/Model/Environment/EnvironmentData.swift

@@ -2,4 +2,5 @@ struct EnvironmentData: Codable {
     let surveys: [Survey]?
     let actionClasses: [ActionClass]?
     let project: Project
+    let recaptchaSiteKey: String?
 }