diff --git a/app/javascript/mastodon/actions/boosts.js b/app/javascript/mastodon/actions/boosts.js index c0f0f3acc5..6d3eed1dc5 100644 --- a/app/javascript/mastodon/actions/boosts.js +++ b/app/javascript/mastodon/actions/boosts.js @@ -7,7 +7,7 @@ export function initBoostModal(props) { return (dispatch, getState) => { const default_privacy = getState().getIn(['compose', 'default_privacy']); - const privacy = props.status.get('visibility') === 'private' ? 'private' : default_privacy; + const privacy = props.status.get('visibility_ex') === 'private' ? 'private' : default_privacy; dispatch({ type: BOOSTS_INIT_MODAL, diff --git a/app/javascript/mastodon/actions/compose.js b/app/javascript/mastodon/actions/compose.js index 331c634897..6000970512 100644 --- a/app/javascript/mastodon/actions/compose.js +++ b/app/javascript/mastodon/actions/compose.js @@ -222,11 +222,11 @@ export function submitCompose(routerHistory) { dispatch(importFetchedStatus({ ...response.data })); } - if (statusId === null && response.data.visibility !== 'direct') { + if (statusId === null && response.data.visibility_ex !== 'direct') { insertIfOnline('home'); } - if (statusId === null && response.data.in_reply_to_id === null && response.data.visibility === 'public') { + if (statusId === null && response.data.in_reply_to_id === null && response.data.visibility_ex === 'public') { insertIfOnline('community'); insertIfOnline('public'); insertIfOnline(`account:${response.data.account.id}`); diff --git a/app/javascript/mastodon/components/status.jsx b/app/javascript/mastodon/components/status.jsx index 78cbd22eae..7f387d29dc 100644 --- a/app/javascript/mastodon/components/status.jsx +++ b/app/javascript/mastodon/components/status.jsx @@ -512,7 +512,7 @@ class Status extends ImmutablePureComponent { 'direct': { icon: 'at', text: intl.formatMessage(messages.direct_short) }, }; - const visibilityIcon = visibilityIconInfo[status.get('visibility')]; + const visibilityIcon = visibilityIconInfo[status.get('visibility_ex')]; let emojiReactionsBar = null; if (!this.props.withoutEmojiReactions && status.get('emoji_reactions')) { @@ -524,10 +524,10 @@ class Status extends ImmutablePureComponent { return ( -
+
{prepend} -
+
diff --git a/app/javascript/mastodon/components/status_action_bar.jsx b/app/javascript/mastodon/components/status_action_bar.jsx index 1a6be65eef..b9509910b8 100644 --- a/app/javascript/mastodon/components/status_action_bar.jsx +++ b/app/javascript/mastodon/components/status_action_bar.jsx @@ -251,8 +251,8 @@ class StatusActionBar extends ImmutablePureComponent { const { signedIn, permissions } = this.context.identity; const anonymousAccess = !signedIn; - const publicStatus = ['public', 'unlisted', 'public_unlisted'].includes(status.get('visibility')); - const pinnableStatus = ['public', 'unlisted', 'public_unlisted', 'private'].includes(status.get('visibility')); + const publicStatus = ['public', 'unlisted', 'public_unlisted'].includes(status.get('visibility_ex')); + const pinnableStatus = ['public', 'unlisted', 'public_unlisted', 'private'].includes(status.get('visibility_ex')); const mutingConversation = status.get('muted'); const account = status.get('account'); const writtenByMe = status.getIn(['account', 'id']) === me; @@ -351,7 +351,7 @@ class StatusActionBar extends ImmutablePureComponent { replyTitle = intl.formatMessage(messages.replyAll); } - const reblogPrivate = status.getIn(['account', 'id']) === me && status.get('visibility') === 'private'; + const reblogPrivate = status.getIn(['account', 'id']) === me && status.get('visibility_ex') === 'private'; let reblogTitle = ''; if (status.get('reblogged')) { diff --git a/app/javascript/mastodon/features/picture_in_picture/components/footer.jsx b/app/javascript/mastodon/features/picture_in_picture/components/footer.jsx index 10e18e504f..3ccd9eb77e 100644 --- a/app/javascript/mastodon/features/picture_in_picture/components/footer.jsx +++ b/app/javascript/mastodon/features/picture_in_picture/components/footer.jsx @@ -154,8 +154,8 @@ class Footer extends ImmutablePureComponent { render () { const { status, intl, withOpenButton } = this.props; - const publicStatus = ['public', 'unlisted', 'public_unlisted'].includes(status.get('visibility')); - const reblogPrivate = status.getIn(['account', 'id']) === me && status.get('visibility') === 'private'; + const publicStatus = ['public', 'unlisted', 'public_unlisted'].includes(status.get('visibility_ex')); + const reblogPrivate = status.getIn(['account', 'id']) === me && status.get('visibility_ex') === 'private'; let replyIcon, replyTitle; diff --git a/app/javascript/mastodon/features/report/components/status_check_box.jsx b/app/javascript/mastodon/features/report/components/status_check_box.jsx index ba44d4a0ed..1ddeaad450 100644 --- a/app/javascript/mastodon/features/report/components/status_check_box.jsx +++ b/app/javascript/mastodon/features/report/components/status_check_box.jsx @@ -49,7 +49,7 @@ class StatusCheckBox extends React.PureComponent { 'direct': { icon: 'at', text: intl.formatMessage(messages.direct_short) }, }; - const visibilityIcon = visibilityIconInfo[status.get('visibility')]; + const visibilityIcon = visibilityIconInfo[status.get('visibility_ex')]; const labelComponent = (
diff --git a/app/javascript/mastodon/features/status/components/action_bar.jsx b/app/javascript/mastodon/features/status/components/action_bar.jsx index 7ae8dc97a3..5c0f36ea10 100644 --- a/app/javascript/mastodon/features/status/components/action_bar.jsx +++ b/app/javascript/mastodon/features/status/components/action_bar.jsx @@ -192,8 +192,8 @@ class ActionBar extends React.PureComponent { const { status, relationship, intl } = this.props; const { signedIn, permissions } = this.context.identity; - const publicStatus = ['public', 'unlisted', 'public_unlisted'].includes(status.get('visibility')); - const pinnableStatus = ['public', 'unlisted', 'public_unlisted', 'private'].includes(status.get('visibility')); + const publicStatus = ['public', 'unlisted', 'public_unlisted'].includes(status.get('visibility_ex')); + const pinnableStatus = ['public', 'unlisted', 'public_unlisted', 'private'].includes(status.get('visibility_ex')); const mutingConversation = status.get('muted'); const account = status.get('account'); const writtenByMe = status.getIn(['account', 'id']) === me; @@ -280,7 +280,7 @@ class ActionBar extends React.PureComponent { replyIcon = 'reply-all'; } - const reblogPrivate = status.getIn(['account', 'id']) === me && status.get('visibility') === 'private'; + const reblogPrivate = status.getIn(['account', 'id']) === me && status.get('visibility_ex') === 'private'; let reblogTitle; if (status.get('reblogged')) { diff --git a/app/javascript/mastodon/features/status/components/detailed_status.jsx b/app/javascript/mastodon/features/status/components/detailed_status.jsx index 1a46c19f95..4a863035ae 100644 --- a/app/javascript/mastodon/features/status/components/detailed_status.jsx +++ b/app/javascript/mastodon/features/status/components/detailed_status.jsx @@ -219,7 +219,7 @@ class DetailedStatus extends ImmutablePureComponent { 'direct': { icon: 'at', text: intl.formatMessage(messages.direct_short) }, }; - const visibilityIcon = visibilityIconInfo[status.get('visibility')]; + const visibilityIcon = visibilityIconInfo[status.get('visibility_ex')]; const visibilityLink = · ; const searchabilityIconInfo = { @@ -232,7 +232,7 @@ class DetailedStatus extends ImmutablePureComponent { const searchabilityIcon = searchabilityIconInfo[status.get('searchability')]; const searchabilityLink = · ; - if (['private', 'direct'].includes(status.get('visibility'))) { + if (['private', 'direct'].includes(status.get('visibility_ex'))) { reblogLink = ''; } else if (this.context.router) { reblogLink = ( @@ -307,7 +307,7 @@ class DetailedStatus extends ImmutablePureComponent { return (
-
+
diff --git a/app/javascript/mastodon/features/ui/components/boost_modal.jsx b/app/javascript/mastodon/features/ui/components/boost_modal.jsx index f9b36907c6..0a6347db23 100644 --- a/app/javascript/mastodon/features/ui/components/boost_modal.jsx +++ b/app/javascript/mastodon/features/ui/components/boost_modal.jsx @@ -93,12 +93,12 @@ class BoostModal extends ImmutablePureComponent { 'direct': { icon: 'at', text: intl.formatMessage(messages.direct_short) }, }; - const visibilityIcon = visibilityIconInfo[status.get('visibility')]; + const visibilityIcon = visibilityIconInfo[status.get('visibility_ex')]; return (
-
+
@@ -127,7 +127,7 @@ class BoostModal extends ImmutablePureComponent {
Shift + }} />
- {status.get('visibility') !== 'private' && !status.get('reblogged') && ( + {status.get('visibility_ex') !== 'private' && !status.get('reblogged') && ( { map.set('text', action.raw_text || unescapeHTML(expandMentions(action.status))); map.set('in_reply_to', action.status.get('in_reply_to_id')); - map.set('privacy', action.status.get('visibility')); + map.set('privacy', action.status.get('visibility_ex')); map.set('media_attachments', action.status.get('media_attachments').map((media) => media.set('unattached', true))); map.set('focusDate', new Date()); map.set('caretPosition', null); @@ -511,7 +511,7 @@ export default function compose(state = initialState, action) { map.set('id', action.status.get('id')); map.set('text', action.text); map.set('in_reply_to', action.status.get('in_reply_to_id')); - map.set('privacy', action.status.get('visibility')); + map.set('privacy', action.status.get('visibility_ex')); map.set('media_attachments', action.status.get('media_attachments')); map.set('focusDate', new Date()); map.set('caretPosition', null); diff --git a/app/serializers/rest/status_serializer.rb b/app/serializers/rest/status_serializer.rb index d4677a2dbe..e86e07387b 100644 --- a/app/serializers/rest/status_serializer.rb +++ b/app/serializers/rest/status_serializer.rb @@ -4,7 +4,7 @@ class REST::StatusSerializer < ActiveModel::Serializer include FormattingHelper attributes :id, :created_at, :in_reply_to_id, :in_reply_to_account_id, - :sensitive, :spoiler_text, :visibility, :language, :translatable, + :sensitive, :spoiler_text, :visibility, :visibility_ex, :language, :translatable, :uri, :url, :replies_count, :reblogs_count, :searchability, :favourites_count, :emoji_reactions, :edited_at @@ -60,11 +60,17 @@ class REST::StatusSerializer < ActiveModel::Serializer # UX differences if object.limited_visibility? 'private' + elsif object.public_unlisted_visibility? + 'public' else object.visibility end end + def visibility_ex + object.visibility + end + def searchability object.compute_searchability end