refactor: up to date

Author: mydearxym

containers/AccountEditor/index.js

@@ -78,7 +78,7 @@ const FormItem = ({ label, textarea, value, onChange }) => (
 )
 
 class AccountEditorContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { accountEditor } = this.props
     logic.init(accountEditor)
   }

containers/AccountViewer/index.js

@@ -39,16 +39,13 @@ const ThemeSection = ({ themeKeys, curTheme }) => {
 }
 
 class AccountViewerContainer extends React.Component {
-  componentWillMount() {
-    const { accountViewer } = this.props
-    logic.init(accountViewer)
-    logic.loadAccount()
-  }
-
   componentDidMount() {
     /* force rebuild the tooltip, otherwise it won't work in some async cases */
     /* if you want to custom see: */
     /* https://github.com/wwayne/react-tooltip/blob/2364dc61332aa947b106dd4bbdd1f2b0e4b1e51d/src/index.scss */
+    const { accountViewer, user } = this.props
+    logic.init(accountViewer, user)
+
     setTimeout(() => {
       ReactTooltip.rebuild()
     }, 2000)

containers/AccountViewer/logic.js

@@ -11,6 +11,7 @@ import {
 } from '../../utils'
 import SR71 from '../../utils/network/sr71'
 import S from './schema'
+
 /* eslint-disable no-unused-vars */
 const debug = makeDebugger('L:AccountViewer')
 /* eslint-enable no-unused-vars */
@@ -20,6 +21,7 @@ const sr71$ = new SR71({
 })
 
 let store = null
+let sub$ = null
 
 export function loadUser() {}
 
@@ -80,7 +82,23 @@ const ErrSolver = [
   },
 ]
 
-export function init(selectedStore) {
-  store = selectedStore
-  sr71$.data().subscribe($solver(DataSolver, ErrSolver))
+export const loadUserInfo = user => {
+  if (user) return loadUser(user)
+  loadAccount()
+}
+
+export function init(_store, user) {
+  store = _store
+
+  if (sub$) return loadUserInfo(user)
+  sub$ = sr71$.data().subscribe($solver(DataSolver, ErrSolver))
+
+  return loadUserInfo(user)
+}
+
+export function uninit() {
+  if (!sub$) return false
+  debug('===== do uninit')
+  sub$.unsubscribe()
+  sub$ = null
 }

containers/ApiLayout/index.js

@@ -19,7 +19,7 @@ const debug = makeDebugger('C:ApiLayout')
 /* eslint-enable no-unused-vars */
 
 class ApiLayoutContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { apiLayout } = this.props
     logic.init(apiLayout)
   }

containers/ArticleViwer/index.js

@@ -40,7 +40,7 @@ const Viwer = ({ type, data, loading, onReaction }) => {
 }
 
 class ArticleViwerContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { articleViwer } = this.props
     logic.init(articleViwer)
   }

containers/Banner/index.js

@@ -39,7 +39,7 @@ const DomainBanner = ({ curRoute }) => {
    for some unkown reasion, pages/index will always be the entry in dev mode
  */
 class BannerContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { banner } = this.props
     logic.init(banner)
   }

containers/BodyLayout/index.js

@@ -17,7 +17,7 @@ import Body from './styles'
 import * as logic from './logic'
 
 class BodyLayoutContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { bodylayout } = this.props
     logic.init(bodylayout)
   }

containers/CategoryEditor/index.js

@@ -28,7 +28,7 @@ const debug = makeDebugger('C:CategoryEditor')
 /* eslint-enable no-unused-vars */
 
 class CategoryEditorContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { categoryEditor, editData } = this.props
 
     logic.init(categoryEditor, editData)

containers/CategorySetter/index.js

@@ -32,7 +32,7 @@ const CategoriesList = ({ communityId, categories, selectedids }) => (
 )
 
 class CategorySetterContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { categorySetter } = this.props
     logic.init(categorySetter)
   }

containers/CheatSheetContent/index.js

@@ -81,7 +81,7 @@ const Langs = ({ base }) => {
 }
 
 class CheatSheetContentContainer extends React.Component {
-  componentWillMount() {
+  componentDidMount() {
     const { cheatSheetContent } = this.props
     logic.init(cheatSheetContent)
   }