From 5f9b5d54f3df3944f9a9adb88ad4f4d4abee6c2e Mon Sep 17 00:00:00 2001
From: jim1_lin <jim1_lin@asus.com>
Date: Wed, 2 Mar 2016 12:21:18 +0800
Subject: [PATCH] move check from classRouter to Schema.js

---
 spec/ParseRole.spec.js       | 2 +-
 src/Routers/ClassesRouter.js | 3 ---
 src/Schema.js                | 3 ++-
 3 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/spec/ParseRole.spec.js b/spec/ParseRole.spec.js
index ed483b22..a217e830 100644
--- a/spec/ParseRole.spec.js
+++ b/spec/ParseRole.spec.js
@@ -108,7 +108,7 @@ describe('Parse Role testing', () => {
     .then((r) => {
       fail("_Role object should not save without name.");
     }, (error) => {
-      expect(error.code).toEqual(135);
+      expect(error.code).toEqual(111);
       done();
     });
   });
diff --git a/src/Routers/ClassesRouter.js b/src/Routers/ClassesRouter.js
index 72e497e5..9742f5f9 100644
--- a/src/Routers/ClassesRouter.js
+++ b/src/Routers/ClassesRouter.js
@@ -81,9 +81,6 @@ export class ClassesRouter extends PromiseRouter {
   }
 
   handleCreate(req) {
-    if(req.params.className === '_Role' && req.body && !req.body.name){
-      throw new Parse.Error(135, 'Role names must be specified.');
-    }
     return rest.create(req.config, req.auth, req.params.className, req.body);
   }
 
diff --git a/src/Schema.js b/src/Schema.js
index 5c8a94d1..70ea57f7 100644
--- a/src/Schema.js
+++ b/src/Schema.js
@@ -73,7 +73,8 @@ var defaultColumns = {
 
 
 var requiredColumns = {
-  _Product: ["productIdentifier", "icon", "order", "title", "subtitle"]
+  _Product: ["productIdentifier", "icon", "order", "title", "subtitle"],
+  _Role: ["name"]
 }
 
 // Valid classes must: