Skip to content

Commit 33c445e

Browse files
ahtesham-quraishAhtesham Quraish
andauthored
refactor: Replace of injectIntl with useIntl() part 6 - files-and-videos/generic/EditFileErrors (#2358)
Co-authored-by: Ahtesham Quraish <[email protected]>
1 parent 7825bcd commit 33c445e

File tree

1 file changed

+68
-69
lines changed

1 file changed

+68
-69
lines changed
Lines changed: 68 additions & 69 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import React from 'react';
22
import PropTypes from 'prop-types';
3-
import { injectIntl, intlShape } from '@edx/frontend-platform/i18n';
3+
import { useIntl } from '@edx/frontend-platform/i18n';
44
import { Alert } from '@openedx/paragon';
55
import ErrorAlert from '../../editors/sharedComponents/ErrorAlerts/ErrorAlert';
66
import { RequestStatus } from '../../data/constants';
@@ -13,71 +13,72 @@ const EditFileErrors = ({
1313
deleteFileStatus,
1414
updateFileStatus,
1515
loadingStatus,
16-
// injected
17-
intl,
18-
}) => (
19-
<>
20-
<ErrorAlert
21-
hideHeading={false}
22-
dismissError={() => resetErrors({ errorType: 'loading' })}
23-
isError={loadingStatus === RequestStatus.FAILED || loadingStatus === RequestStatus.PARTIAL_FAILURE}
24-
>
25-
{intl.formatMessage(messages.errorAlertMessage, { message: errorMessages.loading })}
26-
</ErrorAlert>
27-
<ErrorAlert
28-
hideHeading
29-
dismissError={() => resetErrors({ errorType: 'add' })}
30-
isError={addFileStatus === RequestStatus.FAILED}
31-
>
32-
<Alert.Heading>
33-
{intl.formatMessage(messages.uploadErrorAlertTitle)}
34-
</Alert.Heading>
35-
<ul className="p-0">
36-
{errorMessages.add.map(message => (
37-
<li key={`add-error-${message}`} style={{ listStyle: 'none' }}>
38-
{intl.formatMessage(messages.errorAlertMessage, { message })}
39-
</li>
40-
))}
41-
</ul>
42-
</ErrorAlert>
43-
<ErrorAlert
44-
hideHeading={false}
45-
dismissError={() => resetErrors({ errorType: 'delete' })}
46-
isError={deleteFileStatus === RequestStatus.FAILED}
47-
>
48-
<ul className="p-0">
49-
{errorMessages.delete.map(message => (
50-
<li key={`delete-error-${message}`} style={{ listStyle: 'none' }}>
51-
{intl.formatMessage(messages.errorAlertMessage, { message })}
52-
</li>
53-
))}
54-
</ul>
55-
</ErrorAlert>
56-
<ErrorAlert
57-
hideHeading={false}
58-
dismissError={() => resetErrors({ errorType: 'update' })}
59-
isError={updateFileStatus === RequestStatus.FAILED}
60-
>
61-
<ul className="p-0">
62-
{errorMessages.lock?.map(message => (
63-
<li key={`lock-error-${message}`} style={{ listStyle: 'none' }}>
64-
{intl.formatMessage(messages.errorAlertMessage, { message })}
65-
</li>
66-
))}
67-
{errorMessages.download.map(message => (
68-
<li key={`download-error-${message}`} style={{ listStyle: 'none' }}>
69-
{intl.formatMessage(messages.errorAlertMessage, { message })}
70-
</li>
71-
))}
72-
{errorMessages.thumbnail?.map(message => (
73-
<li key={`add-thumbnail-error-${message}`} style={{ listStyle: 'none' }}>
74-
{intl.formatMessage(messages.errorAlertMessage, { message })}
75-
</li>
76-
))}
77-
</ul>
78-
</ErrorAlert>
79-
</>
80-
);
16+
}) => {
17+
const intl = useIntl();
18+
return (
19+
<>
20+
<ErrorAlert
21+
hideHeading={false}
22+
dismissError={/* istanbul ignore next */ () => resetErrors({ errorType: 'loading' })}
23+
isError={loadingStatus === RequestStatus.FAILED || loadingStatus === RequestStatus.PARTIAL_FAILURE}
24+
>
25+
{intl.formatMessage(messages.errorAlertMessage, { message: errorMessages.loading })}
26+
</ErrorAlert>
27+
<ErrorAlert
28+
hideHeading
29+
dismissError={/* istanbul ignore next */ () => resetErrors({ errorType: 'add' })}
30+
isError={addFileStatus === RequestStatus.FAILED}
31+
>
32+
<Alert.Heading>
33+
{intl.formatMessage(messages.uploadErrorAlertTitle)}
34+
</Alert.Heading>
35+
<ul className="p-0">
36+
{errorMessages.add.map(message => (
37+
<li key={`add-error-${message}`} style={{ listStyle: 'none' }}>
38+
{intl.formatMessage(messages.errorAlertMessage, { message })}
39+
</li>
40+
))}
41+
</ul>
42+
</ErrorAlert>
43+
<ErrorAlert
44+
hideHeading={false}
45+
dismissError={/* istanbul ignore next */ () => resetErrors({ errorType: 'delete' })}
46+
isError={deleteFileStatus === RequestStatus.FAILED}
47+
>
48+
<ul className="p-0">
49+
{errorMessages.delete.map(message => (
50+
<li key={`delete-error-${message}`} style={{ listStyle: 'none' }}>
51+
{intl.formatMessage(messages.errorAlertMessage, { message })}
52+
</li>
53+
))}
54+
</ul>
55+
</ErrorAlert>
56+
<ErrorAlert
57+
hideHeading={false}
58+
dismissError={/* istanbul ignore next */ () => resetErrors({ errorType: 'update' })}
59+
isError={updateFileStatus === RequestStatus.FAILED}
60+
>
61+
<ul className="p-0">
62+
{errorMessages.lock?.map(message => (
63+
<li key={`lock-error-${message}`} style={{ listStyle: 'none' }}>
64+
{intl.formatMessage(messages.errorAlertMessage, { message })}
65+
</li>
66+
))}
67+
{errorMessages.download.map(message => (
68+
<li key={`download-error-${message}`} style={{ listStyle: 'none' }}>
69+
{intl.formatMessage(messages.errorAlertMessage, { message })}
70+
</li>
71+
))}
72+
{errorMessages.thumbnail?.map(message => (
73+
<li key={`add-thumbnail-error-${message}`} style={{ listStyle: 'none' }}>
74+
{intl.formatMessage(messages.errorAlertMessage, { message })}
75+
</li>
76+
))}
77+
</ul>
78+
</ErrorAlert>
79+
</>
80+
);
81+
};
8182

8283
EditFileErrors.propTypes = {
8384
resetErrors: PropTypes.func.isRequired,
@@ -93,8 +94,6 @@ EditFileErrors.propTypes = {
9394
deleteFileStatus: PropTypes.string.isRequired,
9495
updateFileStatus: PropTypes.string.isRequired,
9596
loadingStatus: PropTypes.string.isRequired,
96-
// injected
97-
intl: intlShape.isRequired,
9897
};
9998

100-
export default injectIntl(EditFileErrors);
99+
export default EditFileErrors;

0 commit comments

Comments
 (0)