Bump Style Rules and Linting

app/components/Comments/Comment.js

@@ -74,7 +74,7 @@ export default class Comment extends Component<Props, State> {
                 <Button flat onClick={this.toggleReply}>
                   {this.state.replyOpen ? 'Lukk svar' : 'Svar'}
                 </Button>
-                {user && author.id == user.id && (
+                {user && author.id === user.id && (
                   <Button
                     flat
                     className={styles.delete}

app/components/Comments/__tests__/fixtures/comments.js

@@ -1,4 +1,4 @@
-export default [
+const comments = [
   {
     id: 2,
     text: 'blaargarhgh',
@@ -40,3 +40,5 @@ export default [
     },
   },
 ];
+
+export default comments;

app/components/Form/legoForm.js

@@ -56,7 +56,7 @@ const legoForm = ({
       try {
         // We should instead check if error is SubmissionError. Does not work now
         let [firstErrorField] = Object.keys(errors);
-        if (firstErrorField == '_error') {
+        if (firstErrorField === '_error') {
           // This is because of the stange usage of SubmissionError in
           // app/routes/surveys/components/SubmissionEditor/SubmissionEditor.js
           // That code should instead use redux-form FieldArray :smile:

app/components/Header/index.js

@@ -85,7 +85,13 @@ function AccountDropdownItems({
       </Dropdown.ListItem>
       <Dropdown.Divider />
       <Dropdown.ListItem>
-        <Button flat onClick={() => (logout(), onClose())}>
+        <Button
+          flat
+          onClick={() => {
+            logout();
+            onClose();
+          }}
+        >
           Logg ut
           <Icon name="log-out" size={24} />
         </Button>
@@ -121,7 +127,7 @@ class Header extends Component<Props, State> {
     const isLogin = this.state.mode === 'login';
     let title, form;
 
-    if (loggedIn && currentUser && currentUser.selectedTheme != getTheme()) {
+    if (loggedIn && currentUser && currentUser.selectedTheme !== getTheme()) {
       applySelectedTheme(currentUser.selectedTheme || 'light');
     }
 
@@ -138,6 +144,8 @@ class Header extends Component<Props, State> {
         title = 'Glemt passord';
         form = <ForgotPasswordForm />;
         break;
+      default:
+        break;
     }
 
     const MeetingButton = withRouter(({ history }) => (

app/components/LegoReactions/index.js

@@ -36,7 +36,8 @@ const LegoReactions = (props: Props) => {
       const foundReaction =
         parentEntity.reactionsGrouped &&
         parentEntity.reactionsGrouped.find(
-          (reaction) => emoji.shortCode == reaction.emoji && reaction.hasReacted
+          (reaction) =>
+            emoji.shortCode === reaction.emoji && reaction.hasReacted
         );
       return {
         ...emoji,

app/components/LoginForm/LoginPage.js

@@ -52,6 +52,8 @@ class LoginPage extends Component<Props, State> {
         title = 'Glemt passord';
         form = <ForgotPasswordForm />;
         break;
+      default:
+        break;
     }
     return (
       <Content>

app/components/Poll/index.js

@@ -19,16 +19,16 @@ type Props = {
   details?: boolean,
 };
 
+type OptionEntityRatio = OptionEntity & {
+  ratio: number,
+};
+
 type State = {
   truncateOptions: boolean,
   shuffledOptions: Array<OptionEntityRatio>,
   expanded: boolean,
 };
 
-type OptionEntityRatio = OptionEntity & {
-  ratio: number,
-};
-
 class Poll extends React.Component<Props, State> {
   constructor(props: Props) {
     super(props);

app/components/Reactions/ReactionPicker.js

@@ -39,9 +39,7 @@ const searchEmojis = (emojis, searchString) => {
       matchingEmojis.push(emoji);
       return false;
     }
-    {
-      return true;
-    }
+    return true;
   });
 
   /*
@@ -105,7 +103,7 @@ const searchEmojis = (emojis, searchString) => {
   currentEmojis = currentEmojis.filter((emoji) => {
     if (
       emoji.keywords.find(
-        (keyword) => keyword.toLowerCase() == searchString
+        (keyword) => keyword.toLowerCase() === searchString
       ) !== undefined
     ) {
       matchingEmojis.push(emoji);
@@ -178,9 +176,10 @@ const ReactionPicker = ({
   const onCategoryClick = useCallback((category) => {
     setActiveCategory(category);
     setSearchString(null);
-  });
-  const onSearch = useCallback((searchString) =>
-    setSearchString(searchString.trim().toLowerCase())
+  }, []);
+  const onSearch = useCallback(
+    (searchString) => setSearchString(searchString.trim().toLowerCase()),
+    []
   );
 
   return (

app/components/Reactions/ReactionPickerHeader.js

@@ -20,7 +20,7 @@ const ReactionPickerHeader = ({
       <ReactionPickerCategory
         key={category}
         name={category}
-        isActive={category == activeCategory}
+        isActive={category === activeCategory}
         onCategoryClick={onCategoryClick}
       />
     ))}

app/components/Search/SearchPage.js

@@ -64,6 +64,9 @@ class SearchPage extends Component<Props, State> {
         e.preventDefault();
         this.handleSelect(this.props.results[this.state.selectedIndex]);
         break;
+
+      default:
+        break;
     }
   };
 

app/components/Search/utils.js

@@ -146,7 +146,7 @@ function retrieveAllowed(links: Array<Link>, { allowed, loggedIn }: Options) {
   return links
     .filter(({ key, requireLogin }) => {
       // If we have a mapping for this from the server, check that:
-      if (allowed.hasOwnProperty(key)) {
+      if (Object.prototype.hasOwnProperty.call(allowed, key)) {
         return allowed[key];
       }
 

app/components/Search/withAutocomplete.js

@@ -55,7 +55,7 @@ function withAutocomplete<Props>({
           // Set the result to the response result
           let finalResult = result;
           // Retain a query with no match
-          if (retainFailedQuery && result.length == 0) {
+          if (retainFailedQuery && result.length === 0) {
             finalResult = [{ title: query, label: query }];
           }
 

app/components/Table/index.js

@@ -273,7 +273,7 @@ export default class Table extends Component<Props, State> {
 
     const match = Object.keys(this.state.filters).filter((key) => {
       const { inlineFiltering = true, filterMapping = (val) => val } =
-        this.props.columns.find((col) => col.dataIndex == key) || {};
+        this.props.columns.find((col) => col.dataIndex === key) || {};
 
       if (!inlineFiltering) return true;
 

app/components/Tooltip/index.js

@@ -89,6 +89,8 @@ export default class Tooltip extends Component<Props, State> {
         case 'right':
           renderDirectionClass = styles.renderDirectionRight;
           break;
+        default:
+          break;
       }
       switch (pointerPosition) {
         case 'left':
@@ -97,6 +99,8 @@ export default class Tooltip extends Component<Props, State> {
         case 'right':
           startPointChildren = 10 / 9;
           break;
+        default:
+          break;
       }
     }
     const tooltipClass = this.state.hovered

app/components/Upload/ImageUpload.js

@@ -164,7 +164,7 @@ export default class ImageUpload extends Component<Props, State> {
   };
 
   componentDidUpdate = (props: Object) => {
-    if (props.img != this.props.img) {
+    if (props.img !== this.props.img) {
       this.setState({ img: this.props.img });
     }
   };