Skip to content

Commit a49d3b2

Browse files
committed
remove use of tracked analytics object
1 parent bfbaf7c commit a49d3b2

File tree

3 files changed

+26
-22
lines changed

3 files changed

+26
-22
lines changed

lib-es5/utils/index.js

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -897,17 +897,19 @@ function url(public_id) {
897897
}
898898

899899
var urlAnalytics = ensureOption(options, 'urlAnalytics', false);
900-
var trackedAnalytics = ensureOption(options, 'trackedAnalytics', false);
901900

902-
if (urlAnalytics === true || trackedAnalytics) {
903-
var sdkVersions = getSDKVersions();
901+
if (urlAnalytics === true) {
902+
var { sdkCode, sdkSemver, techVersion } = getSDKVersions();
903+
var sdkVersions = {
904+
sdkCode: ensureOption(options, 'sdkCode', sdkCode),
905+
sdkSemver: ensureOption(options, 'sdkSemver', sdkSemver),
906+
techVersion: ensureOption(options, 'techVersion', techVersion)
907+
};
908+
909+
var analyticsOptions = getAnalyticsOptions(
910+
Object.assign({}, options, sdkVersions)
911+
);
904912

905-
if (trackedAnalytics) {
906-
sdkVersions = trackedAnalytics;
907-
sdkVersions.urlAnalytics = true;
908-
}
909-
910-
var analyticsOptions = getAnalyticsOptions(Object.assign({}, options, sdkVersions));
911913
var sdkAnalyticsSignature = getSDKAnalyticsSignature(analyticsOptions);
912914

913915
// url might already have a '?' query param

lib/utils/index.js

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -827,17 +827,19 @@ function url(public_id, options = {}) {
827827
}
828828

829829
let urlAnalytics = ensureOption(options, 'urlAnalytics', false);
830-
let trackedAnalytics = ensureOption(options, 'trackedAnalytics', false);
831830

832-
if (urlAnalytics === true || trackedAnalytics) {
833-
let sdkVersions = getSDKVersions();
831+
if (urlAnalytics === true) {
832+
let { sdkCode, sdkSemver, techVersion } = getSDKVersions();
833+
let sdkVersions = {
834+
sdkCode: ensureOption(options, 'sdkCode', sdkCode),
835+
sdkSemver: ensureOption(options, 'sdkSemver', sdkSemver),
836+
techVersion: ensureOption(options, 'techVersion', techVersion)
837+
};
834838

835-
if (trackedAnalytics) {
836-
sdkVersions = trackedAnalytics;
837-
sdkVersions.urlAnalytics = true;
838-
}
839+
let analyticsOptions = getAnalyticsOptions(
840+
Object.assign({}, options, sdkVersions)
841+
);
839842

840-
let analyticsOptions = getAnalyticsOptions(Object.assign({}, options, sdkVersions));
841843
let sdkAnalyticsSignature = getSDKAnalyticsSignature(analyticsOptions);
842844

843845
// url might already have a '?' query param

test/unit/sdkAnalytics/imageTagWithAnalytics.spec.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,11 @@ describe('Tests for sdk analytics through image tag', function () {
7373

7474
let imgStr = cloudinary.image("hello", {
7575
format: "png",
76-
trackedAnalytics: {
77-
sdkCode: "X",
78-
sdkSemver: "7.3.0",
79-
techVersion: "3.4.7"
80-
}
76+
urlAnalytics: true,
77+
sdkCode: "X",
78+
sdkSemver: "7.3.0",
79+
techVersion: "3.4.7"
80+
8181
});
8282

8383
expect(imgStr).to.contain(`src='http://res.cloudinary.com/${TEST_CLOUD_NAME}/image/upload/hello.png?_s=AXAEzGT0`);

0 commit comments

Comments
 (0)