Skip to content

fix: Can't register multiple plugin listeners for an event#13360

Merged
lucasfernog merged 2 commits intotauri-apps:devfrom
velocitysystems:fix/ios_plugin_listener
May 2, 2025
Merged

fix: Can't register multiple plugin listeners for an event#13360
lucasfernog merged 2 commits intotauri-apps:devfrom
velocitysystems:fix/ios_plugin_listener

Conversation

@velocitysystems
Copy link
Contributor

Reference: #13359

Arrays and Dictionaries are value types in Swift, not reference types. So when if var eventListeners = listeners[args.event] executes, then eventListeners is a copy and therefore executing .append will only affect the copy, not the original array value in the listeners dictionary.

@velocitysystems velocitysystems requested a review from a team as a code owner May 1, 2025 14:16
@github-project-automation github-project-automation bot moved this to 📬Proposal in Roadmap May 1, 2025
@FabianLars FabianLars requested a review from lucasfernog May 1, 2025 14:19
@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

Package Changes Through b3bb271

There are 5 changes which include tauri with minor, @tauri-apps/api with minor, tauri-bundler with patch, tauri-cli with patch, @tauri-apps/cli with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package current next
@tauri-apps/api 2.5.0 2.6.0
tauri-bundler 2.4.0 2.4.1
tauri 2.5.1 2.6.0
@tauri-apps/cli 2.5.0 2.5.1
tauri-cli 2.5.0 2.5.1

Add another change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

@lucasfernog lucasfernog merged commit dfacb65 into tauri-apps:dev May 2, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from 📬Proposal to 🔎 In audit in Roadmap May 2, 2025
@velocitysystems velocitysystems deleted the fix/ios_plugin_listener branch May 24, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🔎 In audit

Development

Successfully merging this pull request may close these issues.

2 participants