Use Array.includes in place of Array.indexOf

js/models/conversations.js

@@ -2728,7 +2728,7 @@
         const knownMembers = this.get('members');
 
         if (knownMembers) {
-          const fromMember = knownMembers.indexOf(sender) !== -1;
+          const fromMember = knownMembers.includes(sender);
 
           if (!fromMember) {
             window.log.warn(

js/models/messages.js

@@ -1744,7 +1744,7 @@
       const knownMembers = conversation.get('members');
 
       if (!newGroup && knownMembers) {
-        const fromMember = knownMembers.indexOf(source) !== -1;
+        const fromMember = knownMembers.includes(source);
 
         if (!fromMember) {
           window.log.warn(`Ignoring group message from non-member: ${source}`);
@@ -1766,8 +1766,7 @@
           );
         }
 
-        const fromAdmin =
+        const fromAdmin = conversation.get('groupAdmins').includes(source);
-          conversation.get('groupAdmins').indexOf(source) !== -1;
 
         if (!fromAdmin) {
           // Make sure the message is not removing members / renaming the group
@@ -1832,12 +1831,11 @@
 
       if (message.isFriendRequest() && backgroundFrReq) {
         // Check if the contact is a member in one of our private groups:
-        const groupMember =
+        const groupMember = window
-          window
+          .getConversations()
-            .getConversations()
+          .models.filter(c => c.get('members'))
-            .models.filter(c => c.get('members'))
+          .reduce((acc, x) => window.Lodash.concat(acc, x.get('members')), [])
-            .reduce((acc, x) => window.Lodash.concat(acc, x.get('members')), [])
+          .includes(source);
-            .indexOf(source) !== -1;
 
         if (groupMember) {
           window.log.info(
@@ -1914,9 +1912,7 @@
 
               if (removedMembers.length > 0) {
                 if (
-                  removedMembers.indexOf(
+                  removedMembers.includes(textsecure.storage.user.getNumber())
-                    textsecure.storage.user.getNumber()
-                  ) !== -1
                 ) {
                   groupUpdate.kicked = 'You';
                   attributes.isKickedFromGroup = true;

js/modules/data.js

@@ -3,7 +3,7 @@
 const electron = require('electron');
 
 // TODO: this results in poor readability, would be
-// much better to implicitly call with `_`.
+// much better to explicitly call with `_`.
 const {
   cloneDeep,
   forEach,

js/views/create_group_dialog_view.js

@@ -102,7 +102,7 @@
 
       const ourPK = textsecure.storage.user.getNumber();
 
-      this.isAdmin = groupConvo.get('groupAdmins').indexOf(ourPK) !== -1;
+      this.isAdmin = groupConvo.get('groupAdmins').includes(ourPK);
 
       const convos = window.getConversations().models;
 

ts/components/conversation/UpdateGroupDialog.tsx

@@ -45,7 +45,7 @@ export class UpdateGroupDialog extends React.Component<Props, State> {
       const lokiProfile = d.getLokiProfile();
       const name = lokiProfile ? lokiProfile.displayName : 'Anonymous';
 
-      const existingMember = this.props.existingMembers.indexOf(d.id) !== -1;
+      const existingMember = this.props.existingMembers.includes(d.id);
 
       return {
         id: d.id,