BitPrepared / d1b0

applications/workspace/src/BitPrepared/Bundle/D1b0Workspace/Controller/V1/BadgeController.php

Spacing +3 added lines, -3 removed lines

@@ -28,11 +28,11 @@
         $filter = $request->get('filterBy');
         echo $filter;
 
-        $badge = R::findAll("badge","type = ? AND enable = 1",[$filter]);
+        $badge = R::findAll("badge", "type = ? AND enable = 1", [$filter]);
 
         $res = [];
-        foreach($badge as $b){
-            array_push($res,[
+        foreach ($badge as $b) {
+            array_push($res, [
                 "id"=>$b->id,
                 "name"=>$b->name,
                 "descritpion"=>$b->description,

applications/workspace/src/BitPrepared/Bundle/D1b0Workspace/Controller/V1/WorkspaceController.php

Spacing +71 added lines, -71 removed lines

@@ -131,7 +131,7 @@ 
         return JsonResponse::create($res, 201, $headers)->setSharedMaxAge(300);
     }
 
-    public function putWorkspace($id,Request $request)
+    public function putWorkspace($id, Request $request)
     {
         $user_id = $this->getSessionId();
         $data = json_decode($request->getContent(), true);
@@ -145,7 +145,7 @@ 
         $unit = $data['team']['unit'];
         $group = $data['team']['group'];
 
-        $wp = R::load("workspace",$id);
+        $wp = R::load("workspace", $id);
             $ws->title = $title;
             $ws->description = $description;
             $ws->environment = $environment;
@@ -154,7 +154,7 @@ 
         $id = R::store($ws);
 
         //save the team
-        $team = R::findOne("team","workspace = ?",[$id]);
+        $team = R::findOne("team", "workspace = ?", [$id]);
             $team->patrol = $patrol;
             $team->unit = $unit;
             $team->group = $group;
@@ -201,12 +201,12 @@ 
         return JsonResponse::create($res, 201, $headers)->setSharedMaxAge(300);
     }
 
-    public function deleteWorkspace($id,Request $request)
+    public function deleteWorkspace($id, Request $request)
     {
         //Disassocia un utente da un workspace
         $user_id = $this->getSessionId();
 
-        $wp = R::findOne("userworkspace",["workspace = ?"],[$id]);
+        $wp = R::findOne("userworkspace", ["workspace = ?"], [$id]);
         R::trash($ws);
 
         $response = new Response();
@@ -251,16 +251,16 @@ 
 
         $key = $data['key'];
 
-        $share = R::findOne("share","key = ?",[$key]);
+        $share = R::findOne("share", "key = ?", [$key]);
         echo $share->inserttime;
-        if($share !== NULL){
+        if ($share !== NULL) {
             $date = new \DateTime();
             date_sub($date, date_interval_create_from_date_string('15 minutes'));
 
-            $wp_id=$share['workspace'];
+            $wp_id = $share['workspace'];
 
             $dateOld = new \DateTime($share->inserttime);
-            if($dateOld > $date){
+            if ($dateOld > $date) {
                 $usw = R::dispense("userworkspace");
                     $usw->user = $user_id;
                     $usw->workspace = $wp_id;
@@ -269,7 +269,7 @@ 
                 $headers = [];
                 $response = JsonResponse::create(["id"=>$wp_id], 200, $headers)->setSharedMaxAge(300);
 
-            }else{
+            }else {
                 $headers = [];
                 $response = JsonResponse::create(["message"=>"Key no more valid"], 498, $headers)->setSharedMaxAge(300);
             }
@@ -291,7 +291,7 @@ 
             $part->totalpoint = 0;
         $part_id = R::store($part);
 
-        foreach($data['part'] as $r){ //TODO va fixato nelle api
+        foreach ($data['part'] as $r) { //TODO va fixato nelle api
             $resource = R::dispense("resource");
                 $resource->part = $part_id;
                 $resource->inserttime = date('Y-m-d H:i:s');
@@ -303,7 +303,7 @@ 
             $resource_id = R::store($resource);
         }
 
-        foreach($data['badges'] as $badge_id){ //TODO va fixato nelle api
+        foreach ($data['badges'] as $badge_id) { //TODO va fixato nelle api
             $pb = R::dispense("partbadge");
                 $pb->badge = $badge_id;
                 $pb->part = $part_id;
@@ -316,20 +316,20 @@ 
         return JsonResponse::create($res, 201, $headers)->setSharedMaxAge(300);
     }
 
-    public function getPart($id,$part_id, Request $request) {
+    public function getPart($id, $part_id, Request $request) {
         $user_id = $this->getSessionId();
 
         $data = json_decode($request->getContent(), true);
 
-        $part = R::findOne("part","id = ?",[$part_id]);
+        $part = R::findOne("part", "id = ?", [$part_id]);
 
-        $resource = R::findAll("resource","part = ?",[$part_id]);
+        $resource = R::findAll("resource", "part = ?", [$part_id]);
 
-        $partecipants = R::findAll("cero","part = ?",[$part_id]);
+        $partecipants = R::findAll("cero", "part = ?", [$part_id]);
 
-        $badges = R::findAll("partbadge","part = ? AND deleted = 0",[$part_id]);
+        $badges = R::findAll("partbadge", "part = ? AND deleted = 0", [$part_id]);
 
-        $res= [
+        $res = [
             "id"=>$part->id,
             "creation"=>$part->inserttime,
             "points"=>$part->points,
@@ -339,28 +339,28 @@ 
             "partecipants"=>[]
         ];
 
-        foreach($badges as $b){
-            array_push($res['badges'],$b->id);
+        foreach ($badges as $b) {
+            array_push($res['badges'], $b->id);
         }
-        foreach($resource as $r){
-            array_push($res['part'],[
+        foreach ($resource as $r) {
+            array_push($res['part'], [
                 "type"=>$r->type,
                 "hash"=>$r->hash,
                 "ref"=>$r->ref
             ]);
         }
-        foreach($partecipants as $p){
-            array_push($res['partecipants'],$p->user);//TODO forse va usato l'id del c'ero e non l'id dell'utente
+        foreach ($partecipants as $p) {
+            array_push($res['partecipants'], $p->user); //TODO forse va usato l'id del c'ero e non l'id dell'utente
         }
 
         $headers = [];
         return JsonResponse::create($res, 201, $headers)->setSharedMaxAge(300);
     }
 
-    private function getPositionInArray($array,$id){
-        $count =0;
-        foreach($array as $a){
-            if($a->id === $id){
+    private function getPositionInArray($array, $id) {
+        $count = 0;
+        foreach ($array as $a) {
+            if ($a->id === $id) {
                 return $count;
             }
             $count = $count + 1;
@@ -368,22 +368,22 @@ 
         return -1;
     }
 
-    public function putPart($id,$part_id, Request $request) {
+    public function putPart($id, $part_id, Request $request) {
         $user_id = $this->getSessionId();
 
         $data = json_decode($request->getContent(), true);
 
-        $part = R::load("part",$part_id);
+        $part = R::load("part", $part_id);
             $part->workspace = $id;
             $part->user = $user_id;
             $part->lastupdatetime = date('Y-m-d H:i:s');
             $part->totalpoint = 0;
         $part_id = R::store($part);
 
-        $delete_res=R::findAll("resource","WHERE part = ?",[$part_id]);
+        $delete_res = R::findAll("resource", "WHERE part = ?", [$part_id]);
 
-        foreach($data['part'] as $r){ //TODO va fixato nelle api
-            $resource = R::findOne("resource","WHERE hash = ? AND deleted = 0",[$r->hash]);//TODO BISOGNA FARE IL DIFF TRA QUELLE PRESENTI E QUELLE NON PRESENTI
+        foreach ($data['part'] as $r) { //TODO va fixato nelle api
+            $resource = R::findOne("resource", "WHERE hash = ? AND deleted = 0", [$r->hash]); //TODO BISOGNA FARE IL DIFF TRA QUELLE PRESENTI E QUELLE NON PRESENTI
                 $resource->part = $part_id;
                 $resource->updatetime = date('Y-m-d H:i:s');
                 $resource->type = $r->type;
@@ -391,34 +391,34 @@ 
                 $resource->hash = $r->hash;
                 $resource->totalpoint = 0;
             $resource_id = R::store($resource);
-            $rem_id=getPositionInArray($delete_res,$resource_id);
-            if($rem_id != 0)
-                array_splice($delete_res,$rem_id,1); //RIMUOVO GLI ELEMENTI CHE HO MODIFICATO
+            $rem_id = getPositionInArray($delete_res, $resource_id);
+            if ($rem_id != 0)
+                array_splice($delete_res, $rem_id, 1); //RIMUOVO GLI ELEMENTI CHE HO MODIFICATO
         }
 
-        foreach($delete_res as $d){
+        foreach ($delete_res as $d) {
             //RIMUOVO REALMENTE DAL DB LE COSE CHE HO LASCIATO FUORI DALLA PUT (PRESENTI NEL DB MA NON NELLA NUOVA VERSIONE ODIO LE PUT)
-            $resource = R::load("resource",[$d->id]);
-            $resource->deleted=true;
+            $resource = R::load("resource", [$d->id]);
+            $resource->deleted = true;
             R::store($resource);
         }
 
-        $delete_badge=R::findAll("partbadge","WHERE part = ? AND deleted = 0",[$part_id]);
+        $delete_badge = R::findAll("partbadge", "WHERE part = ? AND deleted = 0", [$part_id]);
 
-        foreach($data['badges'] as $badge_id){
-            $pb = R::load("partbadge",$badge_id);
+        foreach ($data['badges'] as $badge_id) {
+            $pb = R::load("partbadge", $badge_id);
                 $pb->badge = $badge_id;
                 $pb->part = $part_id;
             $tmp = R::store($pb);
-            $rem_id=getPositionInArray($delete_badge,$tmp);
-            if($rem_id != 0)
-                array_splice($delete_badge,$rem_id,1); //RIMUOVO GLI ELEMENTI CHE HO MODIFICATO
+            $rem_id = getPositionInArray($delete_badge, $tmp);
+            if ($rem_id != 0)
+                array_splice($delete_badge, $rem_id, 1); //RIMUOVO GLI ELEMENTI CHE HO MODIFICATO
         }
 
-        foreach($delete_badge as $d){
+        foreach ($delete_badge as $d) {
             //RIMUOVO REALMENTE DAL DB LE COSE CHE HO LASCIATO FUORI DALLA PUT (PRESENTI NEL DB MA NON NELLA NUOVA VERSIONE ODIO LE PUT)
-            $badge = R::load("partbadge",[$d->id]);//FORSE RILOADARLI NON È NECESSARIO
-            $badge->deleted=true;
+            $badge = R::load("partbadge", [$d->id]); //FORSE RILOADARLI NON È NECESSARIO
+            $badge->deleted = true;
             R::store($badge);
         }
 
@@ -427,17 +427,17 @@ 
         return JsonResponse::create($res, 201, $headers)->setSharedMaxAge(300);
     }
 
-    public function deletePart($id,$part_id, Request $request) {
+    public function deletePart($id, $part_id, Request $request) {
         $user_id = $this->getSessionId();
-        $part = R::load("part",$part_id);
-            $part->deleted=true;
+        $part = R::load("part", $part_id);
+            $part->deleted = true;
         R::store($part);
 
 
-        $delete_badge=R::findAll("partbadge","WHERE part = ?",[$part_id]);
-        foreach($delete_badge as $d){
-            $badge = R::load("partbadge",[$d->id]);//FORSE RILOADARLI NON È NECESSARIO BASTA FARE $d->deleted=true; e store($d)
-                $badge->deleted=true;
+        $delete_badge = R::findAll("partbadge", "WHERE part = ?", [$part_id]);
+        foreach ($delete_badge as $d) {
+            $badge = R::load("partbadge", [$d->id]); //FORSE RILOADARLI NON È NECESSARIO BASTA FARE $d->deleted=true; e store($d)
+                $badge->deleted = true;
             R::store($badge);
         }
 
@@ -449,13 +449,13 @@ 
         $response->setSharedMaxAge(300);
     }
 
-    private function getPoint($badge_id,$badges){
-        foreach($badges as $b){
-            if($b->id === $badge_id){
-                if($b->completed === True){
+    private function getPoint($badge_id, $badges) {
+        foreach ($badges as $b) {
+            if ($b->id === $badge_id) {
+                if ($b->completed === True) {
                     echo "CASO 1;<BR />";
                     return $this->$POINT_FOR_USING_A_CONQUERED_BADGE;
-                }else{
+                }else {
                     echo "CASO 2;<BR />";
                     return $this->POINT_FOR_USING_A_BADGE;
                 }
@@ -464,16 +464,16 @@ 
         echo "CASO 3;<BR />";
         return $this->POINT_DEFAULT;
     }
-    public function checkin($id,$part_id, Request $request) {
+    public function checkin($id, $part_id, Request $request) {
         $user_id = $this->getSessionId();
 
-        $badges = R::findAll("partbadge","part = ? AND deleted = 0",[$part_id]);
-        $u_badges = R::findAll("userbadge","user = ? AND deleted = 0",[$user_id]);
+        $badges = R::findAll("partbadge", "part = ? AND deleted = 0", [$part_id]);
+        $u_badges = R::findAll("userbadge", "user = ? AND deleted = 0", [$user_id]);
 
         $point_earned = 0;
-        foreach($badges as $b){ //SE CI SONO DEI BADGE
-            $point = $this->getPoint($b->id,$u_badges);
-            if($point != $this->POINT_DEFAULT){ //SE SEI IN CAMMINO PER QUEI BADGE O SE LI POSSIEDI GIÀ
+        foreach ($badges as $b) { //SE CI SONO DEI BADGE
+            $point = $this->getPoint($b->id, $u_badges);
+            if ($point != $this->POINT_DEFAULT) { //SE SEI IN CAMMINO PER QUEI BADGE O SE LI POSSIEDI GIÀ
                 echo "PUNTI:".$point;
                 $point_earned = $point_earned + $point;
                 $pb = R::dispense("cero");
@@ -484,7 +484,7 @@ 
                     $pb->points = $point;
                 $tmp = R::store($pb);
 
-                if($point === $this->POINT_FOR_USING_A_BADGE){ //SE SEI IN CAMMINO MA NON LI HAI ANCORA RAGGIUNTI
+                if ($point === $this->POINT_FOR_USING_A_BADGE) { //SE SEI IN CAMMINO MA NON LI HAI ANCORA RAGGIUNTI
                     $ubc = R::dispense("userbadgeclove");
                         $ubc->user = $user_id;
                         $ubc->badge = $b->id;
@@ -495,7 +495,7 @@ 
             }
         }
 
-        if($point_earned <= 0){ //SE NON CI SONO BADGE O SE TU NON SEI IN CAMMINO PER NESSUNO DI LORO
+        if ($point_earned <= 0) { //SE NON CI SONO BADGE O SE TU NON SEI IN CAMMINO PER NESSUNO DI LORO
             $pb = R::dispense("cero");
                 $pb->user = $user_id;
                 $pb->part = $part_id;
@@ -509,13 +509,13 @@ 
 
     }
 
-    public function deleteCheckin($id,$part_id, Request $request) {
+    public function deleteCheckin($id, $part_id, Request $request) {
         $user_id = $this->getSessionId();
 
-        $u_badges = R::findAll("userbadge","user = ? AND part = ?",[$user_id,$part_id]);
+        $u_badges = R::findAll("userbadge", "user = ? AND part = ?", [$user_id, $part_id]);
         R::trashAll($u_badges);
 
-        $cero = R::findAll("cero","user = ? AND part = ?",[$user_id,$part_id]);
+        $cero = R::findAll("cero", "user = ? AND part = ?", [$user_id, $part_id]);
         R::trashAll($cero);
 
         $response = new Response();

applications/workspace/test/Tests/WorkspaceTest.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Tests;
 
 use Silex\WebTestCase;
-use JsonSchema\Validator;
 
 class WorkspaceTest extends WebTestCase
 {

Spacing +3 added lines, -3 removed lines

@@ -9,7 +9,7 @@ 
 {
     use AbstractAppTest;
 
-    public function testGetWorkSpace(){
+    public function testGetWorkSpace() {
         $schema = __DIR__.'/../../../../api/schemas/workspace.json';
 
         $client = $this->createClient();
@@ -18,12 +18,12 @@ 
         $crawler = $client->request('GET', '/api/v1/workspace/1');
         $response = $client->getResponse();
         $data = $client->getResponse()->getContent();
-        $validator = $this->askValidation($data,$schema);
+        $validator = $this->askValidation($data, $schema);
 
         if ($validator->isValid()) {
             echo "The supplied JSON validates against the schema.\n";
             $this->assertTrue(true);
-        } else {
+        }else {
             echo "JSON does not validate. Violations:\n";
             foreach ($validator->getErrors() as $error) {
                 echo "[{$error['property']}] {$error['message']}\n";

applications/workspace/test/Tests/WorkspacesTest.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Tests;
 
 use Silex\WebTestCase;
-use JsonSchema\Validator;
 
 class WorkspaceTest extends WebTestCase
 {

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 {
     use AbstractAppTest;
 
-    public function testGetWorkspaceList(){
+    public function testGetWorkspaceList() {
         $schema = __DIR__.'/../../../../api/schemas/workspaceList.json';
 
         //$client = $this->createClient();

applications/workspace/test/Tests/AbstractAppTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     public function askValidation($data, $schema)
     {
         $validator = new \JsonSchema\Validator;
-        $js_schema =  (object)['$ref' => 'file://' . $schema];
+        $js_schema = (object)['$ref' => 'file://'.$schema];
         $validator->check(json_decode($data), $js_schema);
         return $validator;
     }