Updated UnfollowModal

• Updated:
- UnfollowModal to fix the issue with people with it enabled being unable to unfollow others
- All instances of instantiating the UnfollowModal to pass in account instead of accountId

app/javascript/gabsocial/components/account.js

@@ -31,7 +31,7 @@ const mapDispatchToProps = (dispatch) => ({
     if (account.getIn(['relationship', 'following']) || account.getIn(['relationship', 'requested'])) {
       if (unfollowModal) {
         dispatch(openModal('UNFOLLOW', {
-          accountId: account.get('id'),
+          account,
         }))
       } else {
         dispatch(unfollowAccount(account.get('id')))

app/javascript/gabsocial/components/account_action_button.js

@@ -30,7 +30,7 @@ const mapDispatchToProps = (dispatch) => ({
     if (account.getIn(['relationship', 'following']) || account.getIn(['relationship', 'requested'])) {
       if (unfollowModal) {
         dispatch(openModal('UNFOLLOW', {
-          accountId: account.get('id'),
+          account,
         }));
       } else {
         dispatch(unfollowAccount(account.get('id')));

app/javascript/gabsocial/components/modal/unfollow_modal.js

@@ -1,43 +1,29 @@
-import { injectIntl, defineMessages } from 'react-intl'
-import { muteAccount } from '../../actions/accounts'
-
-const messages = defineMessages({
-  muteMessage: { id: 'confirmations.mute.message', defaultMessage: 'Are you sure you want to mute {name}?' },
-  cancel: { id: 'confirmation_modal.cancel', defaultMessage: 'Cancel' },
-  confirm: { id: 'confirmations.mute.confirm', defaultMessage: 'Mute' },
-})
-
-const mapStateToProps = (state) => ({
-  isSubmitting: state.getIn(['reports', 'new', 'isSubmitting']),
-  account: state.getIn(['mutes', 'new', 'account']),
-})
+import { FormattedMessage } from 'react-intl'
+import ImmutablePureComponent from 'react-immutable-pure-component'
+import ImmutablePropTypes from 'react-immutable-proptypes'
+import { unfollowAccount } from '../../actions/accounts'
+import ConfirmationModal from './confirmation_modal'
 
 const mapDispatchToProps = (dispatch) => ({
-  onConfirm(account, notifications) {
-    dispatch(muteAccount(account.get('id'), notifications))
+  onConfirm(accountId) {
+    dispatch(unfollowAccount(accountId))
   },
 })
 
 export default
-@connect(mapStateToProps, mapDispatchToProps)
-@injectIntl
-class UnfollowModal extends PureComponent {
+@connect(null, mapDispatchToProps)
+class UnfollowModal extends ImmutablePureComponent {
 
   static propTypes = {
     isSubmitting: PropTypes.bool.isRequired,
-    account: PropTypes.object.isRequired,
+    account: ImmutablePropTypes.map.isRequired,
     onConfirm: PropTypes.func.isRequired,
     onClose: PropTypes.func.isRequired,
-    intl: PropTypes.object.isRequired,
-  }
-
-  componentDidMount() {
-    this.button.focus()
   }
 
   handleClick = () => {
     this.props.onClose()
-    this.props.onConfirm(this.props.account, this.props.notifications)
+    this.props.onConfirm(this.props.account.get('id'))
   }
 
   handleCancel = () => {
@@ -45,25 +31,15 @@ class UnfollowModal extends PureComponent {
   }
 
   render() {
-    const { account, intl } = this.props
-
-    // : TODO :
-    // , {
-    //   message: <FormattedMessage id='confirmations.unfollow.message' defaultMessage='Are you sure you want to unfollow {name}?' values={{ name: <strong>@{account.get('acct')}</strong> }} />,
-    //   confirm: intl.formatMessage(messages.unfollowConfirm),
-    //   onConfirm: () => dispatch(unfollowAccount(account.get('id'))),
-    // }));
+    const { account } = this.props
 
     return (
       <ConfirmationModal
-        title={`Mute @${account.get('acct')}`}
-        message={<FormattedMessage id='confirmations.mute.message' defaultMessage='Are you sure you want to mute @{name}?' values={{ name: account.get('acct') }} />}
-        confirm={<FormattedMessage id='mute' defaultMessage='Mute' />}
-        onClose={onClose}
-        onConfirm={() => {
-          // dispatch(blockDomain(domain))
-          // dispatch(blockDomain(domain))
-        }}
+        title={`Unfollow @${account.get('acct')}`}
+        message={<FormattedMessage id='confirmations.unfollow.message' defaultMessage='Are you sure you want to unfollow {name}?' values={{ name: <strong>@{account.get('acct')}</strong> }} />}
+        confirm={<FormattedMessage id='confirmations.unfollow.confirm' defaultMessage='Unfollow' />}
+        onClose={this.handleCancel}
+        onConfirm={this.handleClick}
       />
     )
   }

app/javascript/gabsocial/components/popover/profile_options_popover.js

@@ -22,7 +22,6 @@ import Text from '../text'
 import List from '../list'
 
 const messages = defineMessages({
-  unfollowConfirm: { id: 'confirmations.unfollow.confirm', defaultMessage: 'Unfollow' },
   blockAndReport: { id: 'confirmations.block.block_and_report', defaultMessage: 'Block & Report' },
   unfollow: { id: 'account.unfollow', defaultMessage: 'Unfollow' },
   follow: { id: 'account.follow', defaultMessage: 'Follow' },
@@ -66,7 +65,7 @@ const mapDispatchToProps = (dispatch, { intl }) => ({
     if (account.getIn(['relationship', 'following']) || account.getIn(['relationship', 'requested'])) {
       if (unfollowModal) {
         dispatch(openModal('UNFOLLOW', {
-          accountId: account.get('id'),
+          account,
         }))
       } else {
         dispatch(unfollowAccount(account.get('id')))