Skip to content

Commit 3834082

Browse files
committed
GPII-228 Minor refactoring of promise chains, documenting invokers
1 parent 5c985d0 commit 3834082

File tree

1 file changed

+18
-10
lines changed

1 file changed

+18
-10
lines changed

gpii/node_modules/flowManager/src/Capture.js

Lines changed: 18 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ fluid.defaults("gpii.flowManager.capture", {
4545
priority: "first"
4646
},
4747
"onCaptureSettingsForCurrentDevice.captureSystemSettings": {
48-
func: "{that}.captureSystemSettings",
49-
args: ["{arguments}.0", "{arguments}.1"],
48+
funcName: "gpii.flowManager.capture.captureSystemSettings",
49+
args: ["{lifecycleManager}.read", "{arguments}.0", "{arguments}.1"], // solutionsRegistryEntries, options
5050
priority: "after:getInstalledSolutionsForCurrentDevice"
5151
},
5252
"onCaptureSettingsForCurrentDevice.formatRawCapturedSettings": {
@@ -62,26 +62,34 @@ fluid.defaults("gpii.flowManager.capture", {
6262
},
6363
getSystemSettingsCapture: {
6464
funcName: "fluid.promise.fireTransformEvent",
65-
args: ["{that}.events.onCaptureSettingsForCurrentDevice", {}, "{arguments}.0"] // options
66-
},
67-
captureSystemSettings: {
68-
funcName: "gpii.flowManager.capture.captureSystemSettings",
69-
args: ["{lifecycleManager}.read", "{arguments}.0", "{arguments}.1"] // solutionsRegistryEntries, options
65+
args: ["{that}.events.onCaptureSettingsForCurrentDevice", null, "{arguments}.0"] // options
7066
}
7167
}
7268
});
7369

70+
/**
71+
* Invoker `{gpii.flowManager.capture}.getInstalledSolutionsForCurrentDevice`
72+
*
73+
* @method
74+
* @name {gpii.flowManager.capture}.getInstalledSolutionsForCurrentDevice
75+
*
76+
* This invoker method will return the solution registry entries, in their usual json format,
77+
* that are available on the current device.
78+
*
79+
* @return {Promise} A promise resolved with an object of solutions registry entries available on the
80+
* current device. As with the solutions registry itself, these are keyed by the solution ID.
81+
*/
82+
7483
/**
7584
* Invoker `{gpii.flowManager.capture}.getSystemSettingsCapture`
7685
*
77-
* @function
86+
* @method
7887
* @name {gpii.flowManager.capture}.getSystemSettingsCapture
7988
*
8089
* This main API entry point for capturing settings from a system or computer. This captures
8190
* the actual settings on the device, so it assumed to be running in a local untrusted flow
8291
* manager.
8392
*
84-
* @param {Event} onCaptureSettingsForCurrentDevice - The transforming promise chain
8593
* @param {Object} options - Options for this chain.
8694
* @param {Array} options.solutionsList - An array of solution IDs to filter by when
8795
* retreiving settings. If this option is not included, all available settings will be
@@ -183,7 +191,7 @@ gpii.flowManager.capture.formatRawCapturedSettings = function (data) {
183191
if (!togo[key]) {
184192
togo[key] = {};
185193
}
186-
fluid.each(fluid.get(item, "0.settings"), function (value, settingId) {
194+
fluid.each(fluid.get(item, [0, "settings"]), function (value, settingId) {
187195
togo[key][settingId] = value;
188196
});
189197
});

0 commit comments

Comments
 (0)