Closed Bug 1734731 Opened 4 years ago Closed 4 years ago

Glean.shutdown in qt errors.

Categories

(Data Platform and Tools Graveyard :: Glean.js, defect, P1)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sarah, Assigned: brizental)

Details

Attachments

(1 file)

Warning: qrc:/glean/org/mozilla/Glean/glean.js:102: TypeError: Property 'shutdown' of object [object Object] is not a function (glean.js:102)

The function in glean.js is

/**
 * Finishes executing any ongoing tasks and shuts down Glean.
 *
 * This will attempt to send pending pings before resolving.
 *
 * @returns A promise which resolves once shutdown is complete.
 */
function shutdown() {
  return Glean.shutdown();
}

I have a suspicion it should be Glean.Glean.default.shutdown() in the way that Glean.initialize looks like this

/**
 * Initialize Glean. This method should only be called once, subsequent calls will be no-op.
 *
 * @param applicationId The application ID (will be sanitized during initialization).
 * @param uploadEnabled Determines whether telemetry is enabled.
 *        If disabled, all persisted metrics, events and queued pings (except
 *        first_run_date) are cleared.
 * @param config Glean configuration options.
 */
function initialize(applicationId, uploadEnabled, config) {
  Glean.Glean.default.initialize(applicationId, uploadEnabled, config);
}

I have a suspicion it should be Glean.Glean.default.shutdown() in the way that Glean.initialize looks like this

Yes you are 100% right here. Woof that was an ooversight, I will have a fix later today.

Assignee: nobody → brizental
Priority: P4 → P1
Whiteboard: [telemetry:glean-js:m?]
Attached file GitHub Pull Request

Work on Bug 1708194 uncovered this issue as well so I applied the fixes for it there.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Product: Data Platform and Tools → Data Platform and Tools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: