Page MenuHomePhabricator
Files F3148649

D7554.id25559.diff
No OneTemporary
Actions

D7554.id25559.diff
View Options

diff --git a/native/avatars/avatar-hooks.js b/native/avatars/avatar-hooks.js
--- a/native/avatars/avatar-hooks.js
+++ b/native/avatars/avatar-hooks.js
@@ -26,6 +26,7 @@
ENSAvatarDBContent,
UpdateUserAvatarRemoveRequest,
} from 'lib/types/avatar-types.js';
+import type { SetState } from 'lib/types/hook-types.js';
import type { LoadingStatus } from 'lib/types/loading-types.js';
import type {
NativeMediaSelection,
@@ -134,6 +135,67 @@
return selectFromGallery;
}
+function useUploadSelectedMedia(
+ setProcessingOrUploadInProgress: SetState<boolean>,
+): (selection: ?NativeMediaSelection) => Promise<?string> {
+ const processSelectedMedia = useProcessSelectedMedia();
+ const uploadProcessedMedia = useUploadProcessedMedia();
+
+ return React.useCallback(
+ async (selection: ?NativeMediaSelection) => {
+ if (!selection) {
+ Alert.alert(
+ 'Media selection failed',
+ 'Unable to select media from Media Library.',
+ );
+ return;
+ }
+
+ setProcessingOrUploadInProgress(true);
+
+ let processedMedia;
+ try {
+ processedMedia = await processSelectedMedia(selection);
+ } catch (e) {
+ Alert.alert(
+ 'Media processing failed',
+ 'Unable to process selected media.',
+ );
+ setProcessingOrUploadInProgress(false);
+ return;
+ }
+
+ if (!processedMedia.success) {
+ Alert.alert(
+ 'Media processing failed',
+ 'Unable to process selected media.',
+ );
+ setProcessingOrUploadInProgress(false);
+ return;
+ }
+
+ let uploadedMedia: UploadMultimediaResult;
+ try {
+ uploadedMedia = await uploadProcessedMedia(processedMedia);
+ } catch {
+ Alert.alert(
+ 'Media upload failed',
+ 'Unable to upload selected media. Please try again.',
+ );
+ setProcessingOrUploadInProgress(false);
+ return;
+ }
+
+ return uploadedMedia.id;
+ },
+ [
+ processSelectedMedia,
+ setProcessingOrUploadInProgress,
+ uploadProcessedMedia,
+ ],
+ );
+}
+
const updateUserAvatarLoadingStatusSelector = createLoadingStatusSelector(
updateUserAvatarActionTypes,
);
@@ -145,8 +207,6 @@
const updateUserAvatarCall = useServerCall(updateUserAvatar);
const selectFromGallery = useSelectFromGallery();
- const processSelectedMedia = useProcessSelectedMedia();
- const uploadProcessedMedia = useUploadProcessedMedia();
const [processingOrUploadInProgress, setProcessingOrUploadInProgress] =
React.useState(false);
@@ -162,62 +222,22 @@
[processingOrUploadInProgress, updateUserAvatarLoadingStatus],
);
+ const uploadSelectedMedia = useUploadSelectedMedia(
+ setProcessingOrUploadInProgress,
+ );
+
const selectFromGalleryAndUpdateUserAvatar = React.useCallback(async () => {
const selection: ?MediaLibrarySelection = await selectFromGallery();
- if (!selection) {
- Alert.alert(
- 'Media selection failed',
- 'Unable to select media from Media Library.',
- );
- return;
- }
- setProcessingOrUploadInProgress(true);
- let processedMedia;
- try {
- processedMedia = await processSelectedMedia(selection);
- } catch (e) {
- Alert.alert(
- 'Media processing failed',
- 'Unable to process selected media.',
- );
- setProcessingOrUploadInProgress(false);
- return;
- }
-
- if (!processedMedia || !processedMedia.success) {
- Alert.alert(
- 'Media processing failed',
- 'Unable to process selected media.',
- );
- setProcessingOrUploadInProgress(false);
- return;
- }
-
- let uploadedMedia: ?UploadMultimediaResult;
- try {
- uploadedMedia = await uploadProcessedMedia(processedMedia);
- } catch {
- Alert.alert(
- 'Media upload failed',
- 'Unable to upload selected media. Please try again.',
- );
- setProcessingOrUploadInProgress(false);
- return;
- }
+ const uploadedMediaID = await uploadSelectedMedia(selection);
- if (!uploadedMedia) {
- Alert.alert(
- 'Media upload failed',
- 'Unable to upload selected media. Please try again.',
- );
- setProcessingOrUploadInProgress(false);
+ if (!uploadedMediaID) {
return;
}
const imageAvatarUpdateRequest: ImageAvatarDBContent = {
type: 'image',
- uploadID: uploadedMedia.id,
+ uploadID: uploadedMediaID,
};
dispatchActionPromise(
@@ -233,10 +253,9 @@
);
}, [
dispatchActionPromise,
- processSelectedMedia,
selectFromGallery,
updateUserAvatarCall,
- uploadProcessedMedia,
+ uploadSelectedMedia,
]);
return React.useMemo(
@@ -256,8 +275,6 @@
const changeThreadSettingsCall = useServerCall(changeThreadSettings);
const selectFromGallery = useSelectFromGallery();
- const processSelectedMedia = useProcessSelectedMedia();
- const uploadProcessedMedia = useUploadProcessedMedia();
const [processingOrUploadInProgress, setProcessingOrUploadInProgress] =
React.useState(false);
@@ -273,62 +290,22 @@
[processingOrUploadInProgress, updateThreadAvatarLoadingStatus],
);
+ const uploadSelectedMedia = useUploadSelectedMedia(
+ setProcessingOrUploadInProgress,
+ );
+
const selectFromGalleryAndUpdateThreadAvatar = React.useCallback(async () => {
const selection: ?MediaLibrarySelection = await selectFromGallery();
- if (!selection) {
- Alert.alert(
- 'Media selection failed',
- 'Unable to select media from Media Library.',
- );
- return;
- }
- setProcessingOrUploadInProgress(true);
- let processedMedia;
- try {
- processedMedia = await processSelectedMedia(selection);
- } catch (e) {
- Alert.alert(
- 'Media processing failed',
- 'Unable to process selected media.',
- );
- setProcessingOrUploadInProgress(false);
- return;
- }
+ const uploadedMediaID = await uploadSelectedMedia(selection);
- if (!processedMedia || !processedMedia.success) {
- Alert.alert(
- 'Media processing failed',
- 'Unable to process selected media.',
- );
- setProcessingOrUploadInProgress(false);
- return;
- }
-
- let uploadedMedia: ?UploadMultimediaResult;
- try {
- uploadedMedia = await uploadProcessedMedia(processedMedia);
- } catch {
- Alert.alert(
- 'Media upload failed',
- 'Unable to upload selected media. Please try again.',
- );
- setProcessingOrUploadInProgress(false);
- return;
- }
-
- if (!uploadedMedia) {
- Alert.alert(
- 'Media upload failed',
- 'Unable to upload selected media. Please try again.',
- );
- setProcessingOrUploadInProgress(false);
+ if (!uploadedMediaID) {
return;
}
const imageAvatarUpdateRequest: ImageAvatarDBContent = {
type: 'image',
- uploadID: uploadedMedia.id,
+ uploadID: uploadedMediaID,
};
const updateThreadRequest: UpdateThreadRequest = {
@@ -353,10 +330,9 @@
}, [
changeThreadSettingsCall,
dispatchActionPromise,
- processSelectedMedia,
selectFromGallery,
threadID,
- uploadProcessedMedia,
+ uploadSelectedMedia,
]);
return React.useMemo(

File Metadata

Mime Type
text/plain
Expires
Tue, Nov 5, 11:15 AM (5 h, 30 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
2425505
Default Alt Text
D7554.id25559.diff (7 KB)

Event Timeline