[PR #61] [MERGED] Implemented the wrapping JS array for .NET arrays/lists. #194

New issue

Closed

opened 2026-05-05 11:20:35 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 11:20:35 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/qmlnet/qmlnet/pull/61
Author: @pauldotknopf
Created: 8/19/2018
Status: ✅ Merged
Merged: 8/19/2018
Merged by: @pauldotknopf

Base: develop ← Head: jsarray

📝 Commits (10+)

663df41 Added a bool indicating if a type is an array.
4567f10 Added a method to wrap a .NET array to a JS array.
ab449ad Added a facade to help hide the complexity of invoking a .NET array.
7997adb Added length tests.
9def032 Added tests for the get indexer.
3e1c7de Added tests for the set indexer.
3715896 Verified we can call forEach.
bb656e0 Prevent array methods (currently push) that modify size to be called.
9ee1fa2 Made sure you can't pop a .NET array.
290ef5e Don't use the array prototype.

📊 Changes

25 files changed (+1326 additions, -60 deletions)

View changed files

➕ src/native/QmlNet/QmlNet/qml/JsNetArray.h (+45 -0)
➕ src/native/QmlNet/QmlNet/qml/JsNetArray510.cpp (+182 -0)
➕ src/native/QmlNet/QmlNet/qml/JsNetArray511.cpp (+203 -0)
📝 src/native/QmlNet/QmlNet/qml/JsNetObject.cpp (+49 -0)
📝 src/native/QmlNet/QmlNet/qml/JsNetObject.h (+2 -0)
📝 src/native/QmlNet/QmlNet/qml/NetValueMetaObject.cpp (+2 -2)
📝 src/native/QmlNet/QmlNet/qml/QQmlApplicationEngine.cpp (+24 -0)
📝 src/native/QmlNet/QmlNet/qml/qml.pri (+10 -1)
📝 src/native/QmlNet/QmlNet/types/Callbacks.cpp (+14 -34)
📝 src/native/QmlNet/QmlNet/types/Callbacks.h (+2 -2)
➕ src/native/QmlNet/QmlNet/types/NetTypeArrayFacade.cpp (+65 -0)
➕ src/native/QmlNet/QmlNet/types/NetTypeArrayFacade.h (+26 -0)
➕ src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeArray.cpp (+74 -0)
➕ src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeArray.h (+25 -0)
➕ src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeList.cpp (+94 -0)
➕ src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeList.h (+29 -0)
📝 src/native/QmlNet/QmlNet/types/NetTypeInfo.cpp (+55 -0)
📝 src/native/QmlNet/QmlNet/types/NetTypeInfo.h (+13 -0)
📝 src/native/QmlNet/QmlNet/types/types.pri (+8 -2)
➕ src/net/Qml.Net.Tests/Qml/ArrayTests.cs (+276 -0)

...and 5 more files

📄 Description

No description provided

_{🔄 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/61 **Author:** [@pauldotknopf](https://github.com/pauldotknopf) **Created:** 8/19/2018 **Status:** ✅ Merged **Merged:** 8/19/2018 **Merged by:** [@pauldotknopf](https://github.com/pauldotknopf) **Base:** `develop` ← **Head:** `jsarray` --- ### 📝 Commits (10+) - [`663df41`](https://github.com/qmlnet/qmlnet/commit/663df417c82bf32fe7f64350e3feac158fed6e70) Added a bool indicating if a type is an array. - [`4567f10`](https://github.com/qmlnet/qmlnet/commit/4567f10e471ae1c233db5cf0b9725a8a60d34ce6) Added a method to wrap a .NET array to a JS array. - [`ab449ad`](https://github.com/qmlnet/qmlnet/commit/ab449ad363df4cee68cfd93e73f9e3125aa3777e) Added a facade to help hide the complexity of invoking a .NET array. - [`7997adb`](https://github.com/qmlnet/qmlnet/commit/7997adbdc7fc25d27298bfbac546596c36b1f796) Added length tests. - [`9def032`](https://github.com/qmlnet/qmlnet/commit/9def032327ff68b7a5facc4dcec01e983a8c2565) Added tests for the get indexer. - [`3e1c7de`](https://github.com/qmlnet/qmlnet/commit/3e1c7de3782f5f86b90ffac312efba42cb14af9d) Added tests for the set indexer. - [`3715896`](https://github.com/qmlnet/qmlnet/commit/37158962e50bcab899c17427a79dbabc5a209187) Verified we can call forEach. - [`bb656e0`](https://github.com/qmlnet/qmlnet/commit/bb656e0780fff9bfd4c58ecc561a5ddb0fe7f84c) Prevent array methods (currently push) that modify size to be called. - [`9ee1fa2`](https://github.com/qmlnet/qmlnet/commit/9ee1fa26603ac0c3062d5c9ecb048cf18831a20c) Made sure you can't pop a .NET array. - [`290ef5e`](https://github.com/qmlnet/qmlnet/commit/290ef5e2f8fbb5db87d63bb0ac1292061b9138f4) Don't use the array prototype. ### 📊 Changes **25 files changed** (+1326 additions, -60 deletions) <details> <summary>View changed files</summary> ➕ `src/native/QmlNet/QmlNet/qml/JsNetArray.h` (+45 -0) ➕ `src/native/QmlNet/QmlNet/qml/JsNetArray510.cpp` (+182 -0) ➕ `src/native/QmlNet/QmlNet/qml/JsNetArray511.cpp` (+203 -0) 📝 `src/native/QmlNet/QmlNet/qml/JsNetObject.cpp` (+49 -0) 📝 `src/native/QmlNet/QmlNet/qml/JsNetObject.h` (+2 -0) 📝 `src/native/QmlNet/QmlNet/qml/NetValueMetaObject.cpp` (+2 -2) 📝 `src/native/QmlNet/QmlNet/qml/QQmlApplicationEngine.cpp` (+24 -0) 📝 `src/native/QmlNet/QmlNet/qml/qml.pri` (+10 -1) 📝 `src/native/QmlNet/QmlNet/types/Callbacks.cpp` (+14 -34) 📝 `src/native/QmlNet/QmlNet/types/Callbacks.h` (+2 -2) ➕ `src/native/QmlNet/QmlNet/types/NetTypeArrayFacade.cpp` (+65 -0) ➕ `src/native/QmlNet/QmlNet/types/NetTypeArrayFacade.h` (+26 -0) ➕ `src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeArray.cpp` (+74 -0) ➕ `src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeArray.h` (+25 -0) ➕ `src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeList.cpp` (+94 -0) ➕ `src/native/QmlNet/QmlNet/types/NetTypeArrayFacadeList.h` (+29 -0) 📝 `src/native/QmlNet/QmlNet/types/NetTypeInfo.cpp` (+55 -0) 📝 `src/native/QmlNet/QmlNet/types/NetTypeInfo.h` (+13 -0) 📝 `src/native/QmlNet/QmlNet/types/types.pri` (+8 -2) ➕ `src/net/Qml.Net.Tests/Qml/ArrayTests.cs` (+276 -0) _...and 5 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>