fix(ios): make removeAllListeners return a promise (#5526)

jcesarmobile · web-flow · commit 815f71b6b62f · 2022-03-29T17:01:25.000+02:00
diff --git a/ios/Capacitor/Capacitor/CAPPlugin.m b/ios/Capacitor/Capacitor/CAPPlugin.m
@@ -109,6 +109,7 @@ - (void)removeListener:(CAPPluginCall *)call {
 
 - (void)removeAllListeners:(CAPPluginCall *)call {
   [self.eventListeners removeAllObjects];
+  [call resolve];
 }
 
 - (NSArray<CAPPluginCall *>*)getListeners:(NSString *)eventName {
diff --git a/ios/Capacitor/Capacitor/JSExport.swift b/ios/Capacitor/Capacitor/JSExport.swift
@@ -97,7 +97,7 @@ internal class JSExport {
             let methods = [
                 PluginHeaderMethod(name: "addListener", rtype: nil),
                 PluginHeaderMethod(name: "removeListener", rtype: nil),
-                PluginHeaderMethod(name: "removeAllListeners", rtype: nil),
+                PluginHeaderMethod(name: "removeAllListeners", rtype: "promise"),
                 PluginHeaderMethod(name: "checkPermissions", rtype: "promise"),
                 PluginHeaderMethod(name: "requestPermissions", rtype: "promise")
             ]

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,7 @@ - (void)removeListener:(CAPPluginCall *)call {`
`109`	`109`
`110`	`110`	`- (void)removeAllListeners:(CAPPluginCall *)call {`
`111`	`111`	`[self.eventListeners removeAllObjects];`
	`112`	`+ [call resolve];`
`112`	`113`	`}`
`113`	`114`
`114`	`115`	`- (NSArray<CAPPluginCall >)getListeners:(NSString *)eventName {`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ internal class JSExport {`
`97`	`97`	`let methods = [`
`98`	`98`	`PluginHeaderMethod(name: "addListener", rtype: nil),`
`99`	`99`	`PluginHeaderMethod(name: "removeListener", rtype: nil),`
`100`		`- PluginHeaderMethod(name: "removeAllListeners", rtype: nil),`
	`100`	`+ PluginHeaderMethod(name: "removeAllListeners", rtype: "promise"),`
`101`	`101`	`PluginHeaderMethod(name: "checkPermissions", rtype: "promise"),`
`102`	`102`	`PluginHeaderMethod(name: "requestPermissions", rtype: "promise")`
`103`	`103`	`]`