Merge pull request #116 from lencioni/hasownproperty

Use hasOwnProperty to check for object properties

Author: Robin Frischmann

modules/dynamic/createPrefixer.js

@@ -111,7 +111,7 @@ export default function createPrefixer(
           }
 
           // add prefixes to properties
-          if (this._requiresPrefix[property]) {
+          if (this._requiresPrefix.hasOwnProperty(property)) {
             style[this._browserInfo.jsPrefix + capitalizeString(property)] = value
             if (!this._keepUnprefixed) {
               delete style[property]

modules/dynamic/plugins/flexboxIE.js

@@ -36,7 +36,7 @@ export default function flexboxIE(
 PluginMetaData
 ): ?Array<any> | ?any {
   if (
-    (alternativeProps[property] ||
+    (alternativeProps.hasOwnProperty(property) ||
       property === 'display' && typeof value === 'string' && value.indexOf('flex') > -1) &&
       ((browserName === 'ie_mob' || browserName === 'ie') && browserVersion === 10)
   ) {
@@ -45,10 +45,10 @@ PluginMetaData
     if (!keepUnprefixed && !Array.isArray(style[property])) {
       delete style[property]
     }
-    if (property === 'display' && alternativeValues[value]) {
+    if (property === 'display' && alternativeValues.hasOwnProperty(value)) {
       return getPrefixedValue(cssPrefix + alternativeValues[value], value, keepUnprefixed)
     }
-    if (alternativeProps[property]) {
+    if (alternativeProps.hasOwnProperty(property)) {
       style[alternativeProps[property]] = alternativeValues[value] || value
     }
   }

modules/dynamic/plugins/flexboxOld.js

@@ -70,10 +70,10 @@ PluginMetaData
         style.WebkitBoxDirection = 'normal'
       }
     }
-    if (property === 'display' && alternativeValues[value]) {
+    if (property === 'display' && alternativeValues.hasOwnProperty(value)) {
       return getPrefixedValue(cssPrefix + alternativeValues[value], value, keepUnprefixed)
     }
-    if (alternativeProps[property]) {
+    if (alternativeProps.hasOwnProperty(property)) {
       style[alternativeProps[property]] = alternativeValues[value] || value
     }
   }

modules/dynamic/plugins/sizing.js

@@ -29,7 +29,7 @@ export default function sizing(
 ): ?Array<any> | ?any {
   // This might change in the future
   // Keep an eye on it
-  if (properties[property] && values[value]) {
+  if (properties.hasOwnProperty(property) && values.hasOwnProperty(value)) {
     return getPrefixedValue(cssPrefix + value, value, keepUnprefixed)
   }
 }

modules/dynamic/plugins/transition.js

@@ -19,7 +19,7 @@ export default function transition(
   style: Object,
   { cssPrefix, keepUnprefixed, requiresPrefix }: PluginMetaData
 ): ?Array<any> | ?any {
-  if (typeof value === 'string' && properties[property]) {
+  if (typeof value === 'string' && properties.hasOwnProperty(property)) {
     // memoize the prefix array for later use
     if (!requiresPrefixDashCased) {
       requiresPrefixDashCased = Object.keys(requiresPrefix).map(prop => hyphenateProperty(prop))

modules/generator/generateDynamicPrefixMap.js

@@ -37,7 +37,7 @@ export default function generateDynamicPrefixMap(browserList: Object): Object {
 
   for (let i = 0, len = browsers.length; i < len; ++i) {
     const browser = browsers[i]
-    if (!prefixMap[browser]) {
+    if (!prefixMap.hasOwnProperty(browser)) {
       prefixMap[browser] = {}
     }
 

modules/generator/generatePluginList.js

@@ -8,7 +8,7 @@ export default function getRecommendedPlugins(browserList: Object): Array<string
     const browserSupportByPlugin = pluginMap[plugin]
 
     for (const browser in browserSupportByPlugin) {
-      if (browserList[browser]) {
+      if (browserList.hasOwnProperty(browser)) {
         const browserVersion = browserSupportByPlugin[browser]
 
         if (browserList[browser] < browserVersion) {

modules/static/plugins/cursor.js

@@ -9,7 +9,7 @@ const values = {
 }
 
 export default function cursor(property: string, value: any): ?Array<string> {
-  if (property === 'cursor' && values[value]) {
+  if (property === 'cursor' && values.hasOwnProperty(value)) {
     return prefixes.map(prefix => prefix + value)
   }
 }

modules/static/plugins/flex.js

@@ -5,7 +5,7 @@ const values = {
 }
 
 export default function flex(property: string, value: any): ?Array<string> {
-  if (property === 'display' && values[value]) {
+  if (property === 'display' && values.hasOwnProperty(value)) {
     return ['-webkit-box', '-moz-box', `-ms-${value}box`, `-webkit-${value}`, value]
   }
 }

modules/static/plugins/flexboxIE.js

@@ -17,7 +17,7 @@ const alternativeProps = {
 }
 
 export default function flexboxIE(property: string, value: any, style: Object): void {
-  if (alternativeProps[property]) {
+  if (alternativeProps.hasOwnProperty(property)) {
     style[alternativeProps[property]] = alternativeValues[value] || value
   }
 }