Fix url selectors
No memo required.
This commit is contained in:
parent
6382238834
commit
75bde149cf
4 changed files with 18 additions and 12 deletions
|
@ -1,10 +1,10 @@
|
||||||
import { connect } from 'react-redux';
|
import { connect } from 'react-redux';
|
||||||
import { makeSelectCanonicalUrlForUri } from 'redux/selectors/claims';
|
import { selectCanonicalUrlForUri } from 'redux/selectors/claims';
|
||||||
import { doToast } from 'redux/actions/notifications';
|
import { doToast } from 'redux/actions/notifications';
|
||||||
import ClaimUri from './view';
|
import ClaimUri from './view';
|
||||||
|
|
||||||
const select = (state, props) => ({
|
const select = (state, props) => ({
|
||||||
shortUrl: makeSelectCanonicalUrlForUri(props.uri)(state),
|
shortUrl: selectCanonicalUrlForUri(state, props.uri),
|
||||||
});
|
});
|
||||||
|
|
||||||
export default connect(select, {
|
export default connect(select, {
|
||||||
|
|
|
@ -5,7 +5,7 @@ import {
|
||||||
selectFirstRunCompleted,
|
selectFirstRunCompleted,
|
||||||
makeSelectNotificationsDisabled,
|
makeSelectNotificationsDisabled,
|
||||||
} from 'redux/selectors/subscriptions';
|
} from 'redux/selectors/subscriptions';
|
||||||
import { makeSelectPermanentUrlForUri } from 'redux/selectors/claims';
|
import { selectPermanentUrlForUri } from 'redux/selectors/claims';
|
||||||
import { selectUser } from 'redux/selectors/user';
|
import { selectUser } from 'redux/selectors/user';
|
||||||
import { doToast } from 'redux/actions/notifications';
|
import { doToast } from 'redux/actions/notifications';
|
||||||
import SubscribeButton from './view';
|
import SubscribeButton from './view';
|
||||||
|
@ -13,7 +13,7 @@ import SubscribeButton from './view';
|
||||||
const select = (state, props) => ({
|
const select = (state, props) => ({
|
||||||
isSubscribed: selectIsSubscribedForUri(state, props.uri),
|
isSubscribed: selectIsSubscribedForUri(state, props.uri),
|
||||||
firstRunCompleted: selectFirstRunCompleted(state),
|
firstRunCompleted: selectFirstRunCompleted(state),
|
||||||
permanentUrl: makeSelectPermanentUrlForUri(props.uri)(state),
|
permanentUrl: selectPermanentUrlForUri(state, props.uri),
|
||||||
notificationsDisabled: makeSelectNotificationsDisabled(props.uri)(state),
|
notificationsDisabled: makeSelectNotificationsDisabled(props.uri)(state),
|
||||||
user: selectUser(state),
|
user: selectUser(state),
|
||||||
});
|
});
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
import { connect } from 'react-redux';
|
import { connect } from 'react-redux';
|
||||||
import InvitedPage from './view';
|
import InvitedPage from './view';
|
||||||
import { makeSelectPermanentUrlForUri } from 'redux/selectors/claims';
|
import { selectPermanentUrlForUri } from 'redux/selectors/claims';
|
||||||
import { withRouter } from 'react-router';
|
import { withRouter } from 'react-router';
|
||||||
|
|
||||||
const select = (state, props) => {
|
const select = (state, props) => {
|
||||||
|
@ -10,7 +10,7 @@ const select = (state, props) => {
|
||||||
const sanitizedReferrer = referrer ? referrer.replace(':', '#') : '';
|
const sanitizedReferrer = referrer ? referrer.replace(':', '#') : '';
|
||||||
const uri = `lbry://${sanitizedReferrer}`;
|
const uri = `lbry://${sanitizedReferrer}`;
|
||||||
return {
|
return {
|
||||||
fullUri: makeSelectPermanentUrlForUri(uri)(state),
|
fullUri: selectPermanentUrlForUri(state, uri),
|
||||||
referrer: referrer,
|
referrer: referrer,
|
||||||
};
|
};
|
||||||
};
|
};
|
||||||
|
|
|
@ -647,14 +647,20 @@ export const selectClaimSearchByQueryLastPageReached = createSelector(
|
||||||
(state) => state.claimSearchByQueryLastPageReached || {}
|
(state) => state.claimSearchByQueryLastPageReached || {}
|
||||||
);
|
);
|
||||||
|
|
||||||
export const makeSelectShortUrlForUri = (uri: string) =>
|
export const selectShortUrlForUri = (state: State, uri: string) => {
|
||||||
createSelector(makeSelectClaimForUri(uri), (claim) => claim && claim.short_url);
|
const claim = selectClaimForUri(state, uri);
|
||||||
|
return claim && claim.short_url;
|
||||||
|
};
|
||||||
|
|
||||||
export const makeSelectCanonicalUrlForUri = (uri: string) =>
|
export const selectCanonicalUrlForUri = (state: State, uri: string) => {
|
||||||
createSelector(makeSelectClaimForUri(uri), (claim) => claim && claim.canonical_url);
|
const claim = selectClaimForUri(state, uri);
|
||||||
|
return claim && claim.canonical_url;
|
||||||
|
};
|
||||||
|
|
||||||
export const makeSelectPermanentUrlForUri = (uri: string) =>
|
export const selectPermanentUrlForUri = (state: State, uri: string) => {
|
||||||
createSelector(makeSelectClaimForUri(uri), (claim) => claim && claim.permanent_url);
|
const claim = selectClaimForUri(state, uri);
|
||||||
|
return claim && claim.permanent_url;
|
||||||
|
};
|
||||||
|
|
||||||
export const makeSelectSupportsForUri = (uri: string) =>
|
export const makeSelectSupportsForUri = (uri: string) =>
|
||||||
createSelector(selectSupportsByOutpoint, makeSelectClaimForUri(uri), (byOutpoint, claim: ?StreamClaim) => {
|
createSelector(selectSupportsByOutpoint, makeSelectClaimForUri(uri), (byOutpoint, claim: ?StreamClaim) => {
|
||||||
|
|
Loading…
Reference in a new issue