[jsfm] add type check for register modules and components

runtime/api/component.js

@@ -17,6 +17,7 @@
  * under the License.
  */
 
+import { isPlainObject } from '../shared/utils'
 import { registerElement } from '../vdom/WeexElement'
 
 const weexComponents = {}
@@ -34,7 +35,7 @@ export function registerComponents (newComponents) {
       if (typeof component === 'string') {
         weexComponents[component] = true
       }
-      else if (typeof component === 'object' && typeof component.type === 'string') {
+      else if (isPlainObject(component) && typeof component.type === 'string') {
         weexComponents[component.type] = component
         registerElement(component.type, component.methods)
       }

runtime/api/module.js

@@ -17,25 +17,29 @@
  * under the License.
  */
 
+import { hasOwn, isPlainObject } from '../shared/utils'
+
 const weexModules = {}
 
 /**
  * Register native modules information.
  * @param {object} newModules
  */
-export function registerModules (newModules) {
+export function registerModules (newModules = {}) {
   for (const name in newModules) {
-    if (!weexModules[name]) {
+    if (!hasOwn(weexModules, name)) {
       weexModules[name] = {}
     }
-    newModules[name].forEach(method => {
-      if (typeof method === 'string') {
-        weexModules[name][method] = true
-      }
-      else {
-        weexModules[name][method.name] = method.args
-      }
-    })
+    if (Array.isArray(newModules[name])) {
+      newModules[name].forEach(method => {
+        if (typeof method === 'string') {
+          weexModules[name][method] = true
+        }
+        else if (isPlainObject(method) && typeof method.name === 'string') {
+          weexModules[name][method.name] = method.args || []
+        }
+      })
+    }
   }
 }
 

runtime/frameworks/legacy/app/register.js

@@ -50,17 +50,19 @@ export function initModules (modules, ifReplace) {
     }
 
     // push each non-existed new method
-    modules[moduleName].forEach(function (method) {
-      if (typeof method === 'string') {
-        method = {
-          name: method
+    if (Array.isArray(modules[moduleName])) {
+      modules[moduleName].forEach(function (method) {
+        if (typeof method === 'string') {
+          method = {
+            name: method
+          }
         }
-      }
 
-      if (!methods[method.name] || ifReplace) {
-        methods[method.name] = method
-      }
-    })
+        if (!methods[method.name] || ifReplace) {
+          methods[method.name] = method
+        }
+      })
+    }
   }
 }
 

runtime/shared/utils.js

@@ -30,6 +30,14 @@ export function typof (v) {
   return s.substring(8, s.length - 1)
 }
 
+export function isPlainObject (value) {
+  return Object.prototype.toString.call(value) === '[object Object]'
+}
+
+export function hasOwn (object, key) {
+  return isPlainObject(object) && Object.prototype.hasOwnProperty.call(object, key)
+}
+
 export function bufferToBase64 (buffer) {
   if (typeof btoa !== 'function') {
     return ''

runtime/vdom/WeexElement.js

@@ -37,7 +37,7 @@ const registeredElements = {}
  */
 export function registerElement (type, methods) {
   // Skip when no special component methods.
-  if (!methods || !methods.length) {
+  if (!Array.isArray(methods) || !methods.length) {
     return
   }