@applicaster/zapp-react-native-utils 14.0.39 → 14.0.40-alpha.2931525716
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/analyticsUtils/PlayerAnalyticsManager.ts +1 -1
- package/appUtils/playerManager/__tests__/playerFactory.test.ts +1 -1
- package/appUtils/playerManager/{conts.ts → const.ts} +14 -0
- package/appUtils/playerManager/index.ts +1 -1
- package/appUtils/playerManager/player.ts +1 -1
- package/appUtils/playerManager/playerFactory.ts +1 -1
- package/appUtils/playerManager/playerNative.ts +1 -1
- package/appUtils/playerManager/usePlayerControllerSetup.tsx +1 -1
- package/appUtils/playerManager/userLanguagePreference.ts +1 -1
- package/bottomSheet/__tests__/scrollContext.test.tsx +35 -0
- package/bottomSheet/index.ts +17 -0
- package/package.json +2 -2
- package/reactHooks/feed/__tests__/useInflatedUrl.test.tsx +25 -0
- package/stringUtils/__tests__/stringUtils.test.js +42 -0
- package/stringUtils/index.ts +2 -2
|
@@ -8,7 +8,7 @@ import {
|
|
|
8
8
|
} from "../appUtils/playerManager";
|
|
9
9
|
import { Player } from "../appUtils/playerManager/player";
|
|
10
10
|
import { AnalyticPlayerListener } from "./AnalyticPlayerListener";
|
|
11
|
-
import { PlayerRole } from "../appUtils/playerManager/
|
|
11
|
+
import { PlayerRole } from "../appUtils/playerManager/const";
|
|
12
12
|
|
|
13
13
|
enum PlayerAnalyticsTrackerType {
|
|
14
14
|
PlayerAnalyticsTracker = "PlayerAnalyticsTracker",
|
|
@@ -10,7 +10,7 @@ jest.mock("@applicaster/zapp-react-native-utils/pluginUtils", () => ({
|
|
|
10
10
|
mockFindPluginByIdentifier(id, plugins),
|
|
11
11
|
}));
|
|
12
12
|
|
|
13
|
-
import { PlayerRole } from "../
|
|
13
|
+
import { PlayerRole } from "../const";
|
|
14
14
|
import { playerFactory } from "../playerFactory";
|
|
15
15
|
|
|
16
16
|
const fakeComponent = () => null;
|
|
@@ -67,3 +67,17 @@ export enum PlayerRole {
|
|
|
67
67
|
Chromecast = "Chromecast",
|
|
68
68
|
Unspecified = "Unspecified",
|
|
69
69
|
}
|
|
70
|
+
|
|
71
|
+
/**
|
|
72
|
+
* The playback engine backing a web player. Published on
|
|
73
|
+
* `window.applicaster.playerType` when a player is created, so that plugins
|
|
74
|
+
* observing the player can tell which engine they are looking at instead of
|
|
75
|
+
* inferring it from the API the instance happens to expose.
|
|
76
|
+
*
|
|
77
|
+
* Keep in sync with `ApplicasterGlobals["playerType"]` in applicaster-types.
|
|
78
|
+
*/
|
|
79
|
+
export enum WebPlayerEngine {
|
|
80
|
+
Videojs = "videojs",
|
|
81
|
+
Hlsjs = "hlsjs",
|
|
82
|
+
Dashjs = "dashjs",
|
|
83
|
+
}
|
|
@@ -9,7 +9,7 @@ import {
|
|
|
9
9
|
unsubscribeToDeviceNetworkChanges,
|
|
10
10
|
} from "@applicaster/quick-brick-core/App/NetworkStatusProvider/utils";
|
|
11
11
|
import { setUserLanguagePreference } from "./userLanguagePreference";
|
|
12
|
-
import { PlayerRole } from "./
|
|
12
|
+
import { PlayerRole } from "./const";
|
|
13
13
|
import { RNPlayerManager } from "./nativePlayerManager";
|
|
14
14
|
|
|
15
15
|
const { log_error, log_debug, log_warning } = createLogger({
|
|
@@ -3,7 +3,7 @@ import { BehaviorSubject } from "rxjs";
|
|
|
3
3
|
import { isEmptyOrNil } from "@applicaster/zapp-react-native-utils/cellUtils";
|
|
4
4
|
import { createLogger, utilsLogger } from "../../logger";
|
|
5
5
|
import { getBoolFromConfigValue } from "../../configurationUtils";
|
|
6
|
-
import { PlayerRole, SharedPlayerCallBacksKeys } from "./
|
|
6
|
+
import { PlayerRole, SharedPlayerCallBacksKeys } from "./const";
|
|
7
7
|
import { OverlaysObserver } from "./OverlayObserver/OverlaysObserver";
|
|
8
8
|
import { TVSeekController } from "../../reactHooks/player/TVSeekControlller/TVSeekController";
|
|
9
9
|
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
import { appStore } from "@applicaster/zapp-react-native-redux/AppStore";
|
|
2
2
|
import { findPluginByIdentifier } from "@applicaster/zapp-react-native-utils/pluginUtils";
|
|
3
3
|
import { Player } from "./player";
|
|
4
|
-
import { PlayerRole } from "./
|
|
4
|
+
import { PlayerRole } from "./const";
|
|
5
5
|
|
|
6
6
|
type PlayerFactoryItem = {
|
|
7
7
|
Component: any;
|
|
@@ -3,7 +3,7 @@ import { isEmptyOrNil } from "@applicaster/zapp-react-native-utils/cellUtils";
|
|
|
3
3
|
import { createLogger, utilsLogger } from "../../logger";
|
|
4
4
|
import { Player } from "./player";
|
|
5
5
|
import { findNodeHandle } from "react-native";
|
|
6
|
-
import { PlayerRole } from "./
|
|
6
|
+
import { PlayerRole } from "./const";
|
|
7
7
|
import { toBooleanWithDefaultFalse } from "../../booleanUtils";
|
|
8
8
|
import { RNPlayerManager } from "./nativePlayerManager";
|
|
9
9
|
|
|
@@ -3,7 +3,7 @@ import { useConfiguration } from "../../reactHooks/useConfiguration";
|
|
|
3
3
|
import { useEffect } from "react";
|
|
4
4
|
import { playerManager } from "./index";
|
|
5
5
|
import { useValidatePlayerConfig } from "../../playerUtils/useValidatePlayerConfig";
|
|
6
|
-
import { PlayerRole } from "./
|
|
6
|
+
import { PlayerRole } from "./const";
|
|
7
7
|
import { isAppleTV } from "@applicaster/zapp-react-native-ui-components/Helpers/Platform";
|
|
8
8
|
import { TVSeekController } from "../../reactHooks/player/TVSeekControlller/TVSeekController";
|
|
9
9
|
import { KeyInputHandler } from "../keyInputHandler/KeyInputHandler";
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
import { localStorage } from "@applicaster/zapp-react-native-bridge/ZappStorage/LocalStorage";
|
|
2
2
|
import { sessionStorage } from "@applicaster/zapp-react-native-bridge/ZappStorage/SessionStorage";
|
|
3
3
|
import { createLogger, utilsLogger } from "../../logger";
|
|
4
|
-
import { userPreferencesNamespace } from "./
|
|
4
|
+
import { userPreferencesNamespace } from "./const";
|
|
5
5
|
|
|
6
6
|
export const preferredAudioLanguageStorageKey = "preferred_audio_language";
|
|
7
7
|
|
|
@@ -0,0 +1,35 @@
|
|
|
1
|
+
import React from "react";
|
|
2
|
+
import { Text } from "react-native";
|
|
3
|
+
import { render, screen } from "@testing-library/react-native";
|
|
4
|
+
import { BottomSheetContext, useBottomSheet } from "../index";
|
|
5
|
+
|
|
6
|
+
const Probe = () => {
|
|
7
|
+
const ctx = useBottomSheet();
|
|
8
|
+
|
|
9
|
+
return <Text testID="probe">{ctx ? "has-context" : "no-context"}</Text>;
|
|
10
|
+
};
|
|
11
|
+
|
|
12
|
+
describe("useBottomSheet", () => {
|
|
13
|
+
it("returns null when no provider is present", () => {
|
|
14
|
+
render(<Probe />);
|
|
15
|
+
expect(screen.getByTestId("probe")).toHaveTextContent("no-context");
|
|
16
|
+
});
|
|
17
|
+
|
|
18
|
+
it("returns the provided value inside a provider", () => {
|
|
19
|
+
const value = {
|
|
20
|
+
scrollRef: React.createRef<any>(),
|
|
21
|
+
masterDrawerRef: React.createRef<any>(),
|
|
22
|
+
setBodyDragEnabled: jest.fn(),
|
|
23
|
+
onDragGestureEvent: jest.fn(),
|
|
24
|
+
onDragStateChange: jest.fn(),
|
|
25
|
+
};
|
|
26
|
+
|
|
27
|
+
render(
|
|
28
|
+
<BottomSheetContext.Provider value={value}>
|
|
29
|
+
<Probe />
|
|
30
|
+
</BottomSheetContext.Provider>
|
|
31
|
+
);
|
|
32
|
+
|
|
33
|
+
expect(screen.getByTestId("probe")).toHaveTextContent("has-context");
|
|
34
|
+
});
|
|
35
|
+
});
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
import { createContext, useContext, RefObject } from "react";
|
|
2
|
+
|
|
3
|
+
export type BottomSheetContextValue = {
|
|
4
|
+
/** The sheet's master TapGestureHandler ref — the scrollable waits for it. */
|
|
5
|
+
masterDrawerRef: RefObject<any>;
|
|
6
|
+
/** Header drag-zone wiring: the sheet's pan gesture event handler. */
|
|
7
|
+
onDragGestureEvent: (...args: any[]) => void;
|
|
8
|
+
/** Header drag-zone wiring: the sheet's pan state-change handler. */
|
|
9
|
+
onDragStateChange: (event: { nativeEvent: any }) => void;
|
|
10
|
+
};
|
|
11
|
+
|
|
12
|
+
export const BottomSheetContext = createContext<BottomSheetContextValue | null>(
|
|
13
|
+
null
|
|
14
|
+
);
|
|
15
|
+
|
|
16
|
+
export const useBottomSheet = (): BottomSheetContextValue | null =>
|
|
17
|
+
useContext(BottomSheetContext);
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@applicaster/zapp-react-native-utils",
|
|
3
|
-
"version": "14.0.
|
|
3
|
+
"version": "14.0.40-alpha.2931525716",
|
|
4
4
|
"description": "Applicaster Zapp React Native utilities package",
|
|
5
5
|
"main": "index.js",
|
|
6
6
|
"types": "index.d.ts",
|
|
@@ -27,7 +27,7 @@
|
|
|
27
27
|
},
|
|
28
28
|
"homepage": "https://github.com/applicaster/quickbrick#readme",
|
|
29
29
|
"dependencies": {
|
|
30
|
-
"@applicaster/applicaster-types": "14.0.
|
|
30
|
+
"@applicaster/applicaster-types": "14.0.40-alpha.2931525716",
|
|
31
31
|
"buffer": "^5.2.1",
|
|
32
32
|
"camelize": "^1.0.0",
|
|
33
33
|
"dayjs": "^1.11.10",
|
|
@@ -242,4 +242,29 @@ describe("useGetUrlInflater", () => {
|
|
|
242
242
|
const url = result.current(source, mapping);
|
|
243
243
|
expect(url).toBe("https://foo.com/shows/A123?q=user%20query");
|
|
244
244
|
});
|
|
245
|
+
|
|
246
|
+
it("handles search queries with apostrophes without HTML entity escaping or URL truncation", () => {
|
|
247
|
+
const source =
|
|
248
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/mobile/search?q={{q}}";
|
|
249
|
+
|
|
250
|
+
const mapping = {
|
|
251
|
+
q: { source: "search", property: "q" },
|
|
252
|
+
};
|
|
253
|
+
|
|
254
|
+
const searchContext = getSearchContext("L'aventure", mapping);
|
|
255
|
+
|
|
256
|
+
const result = getInflatedDataSourceUrl({
|
|
257
|
+
source,
|
|
258
|
+
contexts: {
|
|
259
|
+
search: searchContext,
|
|
260
|
+
},
|
|
261
|
+
mapping,
|
|
262
|
+
});
|
|
263
|
+
|
|
264
|
+
expect(result).toBe(
|
|
265
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/mobile/search?q=L'aventure"
|
|
266
|
+
);
|
|
267
|
+
|
|
268
|
+
expect(result).not.toContain("'");
|
|
269
|
+
});
|
|
245
270
|
});
|
|
@@ -122,6 +122,48 @@ describe("inflate string", () => {
|
|
|
122
122
|
"this is a string with custom separators"
|
|
123
123
|
);
|
|
124
124
|
});
|
|
125
|
+
|
|
126
|
+
it("does not HTML-escape search variables containing single quotes or apostrophes", () => {
|
|
127
|
+
const string = "https://api.example.com/search?q={{search.query}}";
|
|
128
|
+
const data = { search: { query: "L'aventure" } };
|
|
129
|
+
|
|
130
|
+
expect(inflateString({ string, data })).toBe(
|
|
131
|
+
"https://api.example.com/search?q=L'aventure"
|
|
132
|
+
);
|
|
133
|
+
});
|
|
134
|
+
|
|
135
|
+
it("search context with straight apostrophe", () => {
|
|
136
|
+
const string =
|
|
137
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/tv/search?q={{search.q}}";
|
|
138
|
+
|
|
139
|
+
const data = { search: { q: "l'a" } };
|
|
140
|
+
|
|
141
|
+
expect(inflateString({ string, data })).toBe(
|
|
142
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/tv/search?q=l'a"
|
|
143
|
+
);
|
|
144
|
+
});
|
|
145
|
+
|
|
146
|
+
it("search context with backtick apostrophe", () => {
|
|
147
|
+
const string =
|
|
148
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/tv/search?q={{search.q}}";
|
|
149
|
+
|
|
150
|
+
const data = { search: { q: "l`a" } };
|
|
151
|
+
|
|
152
|
+
expect(inflateString({ string, data })).toBe(
|
|
153
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/tv/search?q=l`a"
|
|
154
|
+
);
|
|
155
|
+
});
|
|
156
|
+
|
|
157
|
+
it("search context with right single quotation mark", () => {
|
|
158
|
+
const string =
|
|
159
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/tv/search?q={{search.q}}";
|
|
160
|
+
|
|
161
|
+
const data = { search: { q: "l’a" } };
|
|
162
|
+
|
|
163
|
+
expect(inflateString({ string, data })).toBe(
|
|
164
|
+
"https://api.pc-cms.tele.quebec/api/applicaster/v1/tv/search?q=l’a"
|
|
165
|
+
);
|
|
166
|
+
});
|
|
125
167
|
});
|
|
126
168
|
|
|
127
169
|
describe("objectToReadableString", () => {
|
package/stringUtils/index.ts
CHANGED
|
@@ -72,8 +72,8 @@ const valueIsNull = (value) => () => value === null;
|
|
|
72
72
|
const encodeHelper = "encodeURL";
|
|
73
73
|
|
|
74
74
|
const encodeHandlebarsExpressions = R.compose(
|
|
75
|
-
R.replace(/{{([^}]+)}}/g, (
|
|
76
|
-
p1.includes("search") ?
|
|
75
|
+
R.replace(/{{([^}]+)}}/g, (_, p1) =>
|
|
76
|
+
p1.includes("search") ? `{{{${p1}}}}` : `{{{${encodeHelper} ${p1}}}}`
|
|
77
77
|
)
|
|
78
78
|
);
|
|
79
79
|
|