Просмотр исходного кода

ref(ui): Remove pure render mixin (#6950)

Lyn Nagara 7 лет назад
Родитель
Сommit
058b04a3cf

+ 0 - 1
package.json

@@ -62,7 +62,6 @@
     "raven-js": "3.19.1",
     "react": "15.6.2",
     "react-addons-css-transition-group": "15.6.2",
-    "react-addons-pure-render-mixin": "15.6.2",
     "react-addons-test-utils": "15.6.2",
     "react-bootstrap": "^0.29.5",
     "react-document-title": "1.0.4",

+ 1 - 2
src/sentry/static/sentry/app/components/activity/noteInput.jsx

@@ -3,7 +3,6 @@ import React from 'react';
 import createReactClass from 'create-react-class';
 import marked from 'marked';
 import {MentionsInput, Mention} from 'react-mentions';
-import PureRenderMixin from 'react-addons-pure-render-mixin';
 
 import ApiMixin from '../../mixins/apiMixin';
 import GroupStore from '../../stores/groupStore';
@@ -30,7 +29,7 @@ const NoteInput = createReactClass({
     sessionUser: PropTypes.object.isRequired,
   },
 
-  mixins: [PureRenderMixin, ApiMixin],
+  mixins: [ApiMixin],
 
   getInitialState() {
     let {item, group} = this.props;

+ 1 - 2
src/sentry/static/sentry/app/components/alerts.jsx

@@ -1,14 +1,13 @@
 import React from 'react';
 import createReactClass from 'create-react-class';
 import Reflux from 'reflux';
-import PureRenderMixin from 'react-addons-pure-render-mixin';
 
 import AlertStore from '../stores/alertStore';
 import AlertMessage from './alertMessage';
 
 const Alerts = createReactClass({
   displayName: 'Alerts',
-  mixins: [PureRenderMixin, Reflux.connect(AlertStore, 'alerts')],
+  mixins: [Reflux.connect(AlertStore, 'alerts')],
 
   getInitialState() {
     return {

+ 1 - 2
src/sentry/static/sentry/app/components/shortId.jsx

@@ -1,7 +1,6 @@
 import PropTypes from 'prop-types';
 import React from 'react';
 import createReactClass from 'create-react-class';
-import PureRenderMixin from 'react-addons-pure-render-mixin';
 import ProjectState from '../mixins/projectState';
 
 import AutoSelectText from './autoSelectText';
@@ -14,7 +13,7 @@ const ShortId = createReactClass({
     project: PropTypes.object,
   },
 
-  mixins: [PureRenderMixin, ProjectState],
+  mixins: [ProjectState],
 
   preventPropagation(e) {
     // this is a hack for the stream so the click handler doesn't

+ 5 - 11
src/sentry/static/sentry/app/views/organizationStats/projectTable.jsx

@@ -1,8 +1,6 @@
 import PropTypes from 'prop-types';
 import React from 'react';
-import createReactClass from 'create-react-class';
 import {Link} from 'react-router';
-import PureRenderMixin from 'react-addons-pure-render-mixin';
 
 import Count from '../../components/count';
 import {t} from '../../locale';
@@ -17,17 +15,13 @@ let getPercent = (item, total) => {
   return parseInt(item / total * 100, 10) + '%';
 };
 
-const ProjectTable = createReactClass({
-  displayName: 'ProjectTable',
-
-  propTypes: {
+class ProjectTable extends React.PureComponent {
+  static propTypes = {
     projectMap: PropTypes.object.isRequired,
     projectTotals: PropTypes.array.isRequired,
     orgTotal: PropTypes.object.isRequired,
     organization: PropTypes.object.isRequired,
-  },
-
-  mixins: [PureRenderMixin],
+  };
 
   render() {
     let projectMap = this.props.projectMap;
@@ -104,7 +98,7 @@ const ProjectTable = createReactClass({
         </tbody>
       </table>
     );
-  },
-});
+  }
+}
 
 export default ProjectTable;

+ 0 - 2
src/sentry/static/sentry/app/views/stream/actions.jsx

@@ -1,7 +1,6 @@
 import PropTypes from 'prop-types';
 import React from 'react';
 import createReactClass from 'create-react-class';
-import PureRenderMixin from 'react-addons-pure-render-mixin';
 import Reflux from 'reflux';
 import {capitalize} from 'lodash';
 
@@ -129,7 +128,6 @@ const StreamActions = createReactClass({
       ],
     }),
     Reflux.listenTo(SelectedGroupStore, 'onSelectedGroupChange'),
-    PureRenderMixin,
   ],
 
   getDefaultProps() {

+ 0 - 2
src/sentry/static/sentry/app/views/stream/searchBar.jsx

@@ -2,7 +2,6 @@ import PropTypes from 'prop-types';
 import React from 'react';
 import createReactClass from 'create-react-class';
 import ReactDOM from 'react-dom';
-import PureRenderMixin from 'react-addons-pure-render-mixin';
 import Reflux from 'reflux';
 import _ from 'lodash';
 import classNames from 'classnames';
@@ -34,7 +33,6 @@ const SearchBar = createReactClass({
 
   mixins: [
     ApiMixin,
-    PureRenderMixin,
     Reflux.listenTo(MemberListStore, 'onMemberListStoreChange'),
   ],
 

+ 0 - 7
yarn.lock

@@ -7228,13 +7228,6 @@ react-addons-css-transition-group@15.6.2:
   dependencies:
     react-transition-group "^1.2.0"
 
-react-addons-pure-render-mixin@15.6.2:
-  version "15.6.2"
-  resolved "https://registry.yarnpkg.com/react-addons-pure-render-mixin/-/react-addons-pure-render-mixin-15.6.2.tgz#6b83f40b6b36ee40735cbd6125eb3f13ce1cddc9"
-  dependencies:
-    fbjs "^0.8.4"
-    object-assign "^4.1.0"
-
 react-addons-shallow-compare@^15.0.2:
   version "15.6.2"
   resolved "https://registry.yarnpkg.com/react-addons-shallow-compare/-/react-addons-shallow-compare-15.6.2.tgz#198a00b91fc37623db64a28fd17b596ba362702f"