only warn if reserved key is directly used on this or self objects

.eslintignore

@@ -1,2 +1,3 @@
 node_modules/**
-build/**
+build/**
+coverage/**

lib/rules/reserved-keys.js

@@ -48,8 +48,10 @@ module.exports = function(context) {
 
         "MemberExpression": function(node){
             var reservedPropertyKey = reservedWords.indexOf(node.property.name) > -1;
+            var memberOfThisOrSelf = node.object.type === "ThisExpression" || node.object.name === "self";
+
+            if (reservedPropertyKey && memberOfThisOrSelf){
 
-            if (reservedPropertyKey){
                 failures.push(context.report.bind(null, node,
                     "Property name '{{name}}' is a reserved identifier name. It might be used by another framework in this context.",
                     { name: node.property.name }));

tests/lib/rules/reserved-keys.js

@@ -30,6 +30,12 @@ eslintTester.addRuleTest("lib/rules/reserved-keys", {
         {
             args: [1, {keywords: [":)"]}],
             code: "this.id = 1; expect();"
+        },
+        {
+            code: "this.selectedCard.id = 1; expect();"
+        },
+        {
+            code: "other.id = 1; expect();"
         }
     ],
 
@@ -40,6 +46,13 @@ eslintTester.addRuleTest("lib/rules/reserved-keys", {
                 message: "Property name 'id' is a reserved identifier name. It might be used by another framework in this context.",
                 type: "MemberExpression"
             }]
+        },
+        {
+            code: "self.id = 1; expect();",
+            errors: [{
+                message: "Property name 'id' is a reserved identifier name. It might be used by another framework in this context.",
+                type: "MemberExpression"
+            }]
         }
     ]
 });