[PR #154] [MERGED] Fix parameter bug in NetQObject::invokeMethod #227

Closed
opened 2026-05-05 11:21:11 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/qmlnet/qmlnet/pull/154
Author: @andy-fillebrown
Created: 7/20/2019
Status: Merged
Merged: 7/21/2019
Merged by: @pauldotknopf

Base: developHead: develop


📝 Commits (1)

  • 0874f7f Fix parameter bug in NetQObject::invokeMethod

📊 Changes

1 file changed (+9 additions, -9 deletions)

View changed files

📝 src/native/QmlNet/QmlNet/qml/NetQObject.cpp (+9 -9)

📄 Description

When setting up the QObject meta method arguments in
NetQObject::invokeMethod the 1st parameter is getting overwritten by
parameters 2 to 10 and parameters 2 to 10 are getting ignored. This
change fixes the issue and gets calling NetQObject::invokeMethod with
multiple parameters working as expected.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/qmlnet/qmlnet/pull/154 **Author:** [@andy-fillebrown](https://github.com/andy-fillebrown) **Created:** 7/20/2019 **Status:** ✅ Merged **Merged:** 7/21/2019 **Merged by:** [@pauldotknopf](https://github.com/pauldotknopf) **Base:** `develop` ← **Head:** `develop` --- ### 📝 Commits (1) - [`0874f7f`](https://github.com/qmlnet/qmlnet/commit/0874f7f69eb87e61c3bcb7f76e72127a39eb6e16) Fix parameter bug in NetQObject::invokeMethod ### 📊 Changes **1 file changed** (+9 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `src/native/QmlNet/QmlNet/qml/NetQObject.cpp` (+9 -9) </details> ### 📄 Description When setting up the QObject meta method arguments in NetQObject::invokeMethod the 1st parameter is getting overwritten by parameters 2 to 10 and parameters 2 to 10 are getting ignored. This change fixes the issue and gets calling NetQObject::invokeMethod with multiple parameters working as expected. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 11:21:11 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/qmlnet#227
No description provided.