lint

Author: Saket Hatwar

modules/conflict_resolver/jsx/CandidateConflictsWidget.js

@@ -2,11 +2,8 @@ import '../../../node_modules/c3/c3.css';
 import c3 from 'c3';
 import React, {useEffect} from 'react';
 import PropTypes from 'prop-types';
-import i18n from 'I18nSetup';
 import {withTranslation} from 'react-i18next';
 
-import hiStrings from '../locale/hi/LC_MESSAGES/conflict_resolver.json';
-
 /**
  * Renders a representation of the candidate conflicts as a React
  * component
@@ -15,7 +12,7 @@ import hiStrings from '../locale/hi/LC_MESSAGES/conflict_resolver.json';
  * @return {object}
  */
 function CandidateConflictsWidget(props) {
-  const { t } = props;
+  const {t} = props;
   const visits = getVisits(props.Conflicts);
   const instruments = getInstruments(props.Conflicts);
 
@@ -69,10 +66,14 @@ function CandidateConflictsWidget(props) {
     <div id='conflictschart' />
     <ul>
       <li>
-        {t('Click on instrument in legend to visit conflict resolver for that instrument across all visits.', {ns: 'conflict_resolver'})}
+        {t('Click on instrument in legend to visit conflict resolver' +
+        ' for that instrument across all visits.',
+        {ns: 'conflict_resolver'})}
       </li>
       <li>
-        {t('Click on bar in graph to visit conflict resolver for that visit and instrument combination.', {ns: 'conflict_resolver'})}
+        {t('Click on bar in graph to visit conflict resolver' +
+        ' for that visit and instrument combination.',
+        {ns: 'conflict_resolver'})}
       </li>
     </ul>
   </div>;
@@ -154,4 +155,5 @@ function getDataBreakdown(visits, instruments, conflicts) {
   return data;
 }
 
-export default withTranslation(['conflict_resolver', 'loris'])(CandidateConflictsWidget);
+export default withTranslation(
+  ['conflict_resolver', 'loris'])(CandidateConflictsWidget);

modules/conflict_resolver/jsx/conflict_resolver.js

@@ -3,6 +3,7 @@ import React, {Component} from 'react';
 import {Tabs, TabPane} from 'Tabs';
 import i18n from 'I18nSetup';
 import {withTranslation} from 'react-i18next';
+import PropTypes from 'prop-types';
 
 import UnresolvedFilterableDataTable from './unresolved_filterabledatatable';
 import ResolvedFilterableDataTable from './resolved_filterabledatatable';
@@ -42,7 +43,7 @@ class ConflictResolver extends Component {
    * @return {JSX}
    */
   render() {
-    const { t } = this.props;
+    const {t} = this.props;
     const tabs = [
       {id: 'unresolved', label: t('Unresolved', {ns: 'conflict_resolver'})},
       {id: 'resolved', label: t('Resolved', {ns: 'conflict_resolver'})},
@@ -87,5 +88,10 @@ window.addEventListener('load', () => {
   ).render(<Index />);
 });
 
-export default withTranslation(['conflict_resolver', 'loris'])(ConflictResolver);
+ConflictResolver.propTypes = {
+  t: PropTypes.func,
+};
+
+export default withTranslation(
+  ['conflict_resolver', 'loris'])(ConflictResolver);
 

modules/conflict_resolver/jsx/fix_conflict_form.js

@@ -5,9 +5,7 @@
 import {Component} from 'react';
 import PropTypes from 'prop-types';
 import {SelectElement} from 'jsx/Form';
-import i18n from 'I18nSetup';
 import {withTranslation} from 'react-i18next';
-import hiStrings from '../locale/hi/LC_MESSAGES/conflict_resolver.json';
 
 /**
  * The fix FixConflictForm renders a <form> within a <td>. The form as a select
@@ -56,7 +54,7 @@ class FixConflictForm extends Component {
    * @param {string} value
    */
   resolveConflict(name, value) {
-    const { t } = this.props;
+    const {t} = this.props;
     fetch(loris.BaseURL.concat('/conflict_resolver/unresolved'), {
       method: 'POST',
       credentials: 'same-origin',

modules/conflict_resolver/jsx/resolved_filterabledatatable.js

@@ -1,10 +1,8 @@
 import React, {Component} from 'react';
 import {withTranslation} from 'react-i18next';
 import Loader from 'Loader';
-import i18n from 'I18nSetup';
 import FilterableDataTable from 'FilterableDataTable';
-
-import hiStrings from '../locale/hi/LC_MESSAGES/conflict_resolver.json';
+import PropTypes from 'prop-types';
 
 /**
  * Filterable Datatable for resolved conflicts.
@@ -112,12 +110,13 @@ class ResolvedFilterableDataTable extends Component {
    * @return {JSX}
    */
   render() {
-    const { t } = this.props;
+    const {t} = this.props;
     // If error occurs, return a message.
     if (this.state.error) {
       return (
         <div className="alert alert-danger" role="alert">
-          <h4>{t('An error occured while loading the page.', {ns: 'conflict_resolver'})}</h4>
+          <h4>{t('An error occured while loading the page.',
+            {ns: 'conflict_resolver'})}</h4>
           {this.state.error.message}
         </div>
       );
@@ -131,13 +130,15 @@ class ResolvedFilterableDataTable extends Component {
     const options = this.state.data.fieldOptions;
 
     const fields = [
-      {label: t('Resolved ID', {ns: 'conflict_resolver'}), show: false},
+      {label: t('Resolved ID',
+        {ns: 'conflict_resolver'}), show: false},
       {label: t('Project', {ns: 'loris'}), show: true, filter: {
         name: 'Project',
         type: 'select',
         options: options.project,
       }},
-      {label: t('Cohort', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Cohort',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'Cohort',
         type: 'select',
         options: options.cohort,
@@ -166,35 +167,43 @@ class ResolvedFilterableDataTable extends Component {
         type: 'select',
         options: options.instrument,
       }},
-      {label: t('Question', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Question',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'Question',
         type: 'text',
       }},
-      {label: t('Description', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Description',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'Description',
         type: 'text',
       }},
-      {label: t('Incorrect Answer', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Incorrect Answer',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'OldValue',
         type: 'text',
       }},
-      {label: t('Correct Answer', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Correct Answer',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'CorrectAnswer',
         type: 'text',
       }},
-      {label: t('User 1', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('User 1',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'User1',
         type: 'text',
       }},
-      {label: t('User 2', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('User 2',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'User2',
         type: 'text',
       }},
-      {label: t('Resolver', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Resolver',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'Resolver',
         type: 'text',
       }},
-      {label: t('Resolution Timestamp', {ns: 'conflict_resolver'}), show: true, filter: {
+      {label: t('Resolution Timestamp',
+        {ns: 'conflict_resolver'}), show: true, filter: {
         name: 'ResolutionTimestamp',
         type: 'text',
       }},
@@ -211,6 +220,10 @@ class ResolvedFilterableDataTable extends Component {
   }
 }
 
+ResolvedFilterableDataTable.propTypes = {
+  t: PropTypes.func.isRequired,
+};
+
 export default withTranslation(
   ['conflict_resolver', 'loris']
 )(ResolvedFilterableDataTable);

modules/conflict_resolver/jsx/unresolved_filterabledatatable.js

@@ -1,11 +1,9 @@
 import React, {Component} from 'react';
 import Loader from 'Loader';
-import i18n from 'I18nSetup';
 import {withTranslation} from 'react-i18next';
 import FilterableDataTable from 'FilterableDataTable';
 import FixConflictForm from './fix_conflict_form';
-
-import hiStrings from '../locale/hi/LC_MESSAGES/conflict_resolver.json';
+import PropTypes from 'prop-types';
 
 /**
  * Filterable database for unresolved conflicts.
@@ -125,11 +123,12 @@ class UnresolvedFilterableDataTable extends Component {
    */
   render() {
     // If error occurs, return a message.
-    const { t } = this.props;
+    const {t} = this.props;
     if (this.state.error) {
       return (
         <div className="alert alert-danger" role="alert">
-          <h4>{t('An error occured while loading the page.', {ns: 'conflict_resolver'})}</h4>
+          <h4>{t('An error occured while loading the page.',
+            {ns: 'conflict_resolver'})}</h4>
           {this.state.error.toString()}
         </div>
       );
@@ -202,6 +201,10 @@ class UnresolvedFilterableDataTable extends Component {
   }
 }
 
+UnresolvedFilterableDataTable.propTypes = {
+  t: PropTypes.func.isRequired,
+};
+
 export default withTranslation(
   ['conflict_resolver', 'loris']
 )(UnresolvedFilterableDataTable);