a3398843c2
Frequently used; top in perf profile Most of the time, you already have the claim object in the current context. `selectClaimIsMineForUri` will retrieve the claim again, which is wasteful, even if it is memoized (looking up the cache still takes time). Break apart the logic and added the alternative `selectClaimIsMine` for faster lookup. Co-authored-by: infinite-persistence <inf.persistence@gmail.com>
28 lines
1,017 B
JavaScript
28 lines
1,017 B
JavaScript
import { connect } from 'react-redux';
|
|
import { doDeleteFileAndMaybeGoBack } from 'redux/actions/file';
|
|
import {
|
|
makeSelectTitleForUri,
|
|
selectClaimForUri,
|
|
makeSelectIsAbandoningClaimForUri,
|
|
makeSelectClaimIsMine,
|
|
} from 'redux/selectors/claims';
|
|
import { doResolveUri } from 'redux/actions/claims';
|
|
import { doHideModal } from 'redux/actions/app';
|
|
import ModalRemoveFile from './view';
|
|
|
|
const select = (state, props) => ({
|
|
claimIsMine: makeSelectClaimIsMine(props.uri)(state),
|
|
title: makeSelectTitleForUri(props.uri)(state),
|
|
claim: selectClaimForUri(state, props.uri),
|
|
isAbandoning: makeSelectIsAbandoningClaimForUri(props.uri)(state),
|
|
});
|
|
|
|
const perform = (dispatch) => ({
|
|
closeModal: () => dispatch(doHideModal()),
|
|
doResolveUri: (uri) => dispatch(doResolveUri(uri)),
|
|
deleteFile: (uri, deleteFromComputer, abandonClaim, doGoBack) => {
|
|
dispatch(doDeleteFileAndMaybeGoBack(uri, deleteFromComputer, abandonClaim, doGoBack));
|
|
},
|
|
});
|
|
|
|
export default connect(select, perform)(ModalRemoveFile);
|