Mykees / MkMediaBundle

Tests/Controller/MediasControllerTest.php

Spacing +12 added lines, -12 removed lines

@@ -32,28 +32,28 @@ 
     public function testListMedias()
     {
         $crawler = $this->client->request('GET', '/admin/medias/Post/MvcBlogBundle/1');
-        $this->assertEquals(200,$this->client->getResponse()->getStatusCode());
-        $this->assertEquals(2,$crawler->filter('.count-tr')->count());
+        $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
+        $this->assertEquals(2, $crawler->filter('.count-tr')->count());
 
 
         $crawler = $this->client->request('GET', '/admin/medias/Post/MvcBlogBundle/2');
-        $this->assertEquals(200,$this->client->getResponse()->getStatusCode());
-        $this->assertEquals(1,$crawler->filter('.count-tr')->count());
+        $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
+        $this->assertEquals(1, $crawler->filter('.count-tr')->count());
     }
 
     public function testFindAllMediasForAModel()
     {
-        $medias  = $this->manager->findMediasByModelAndId('Post',1);
-        $this->assertGreaterThan(1,count($medias));
+        $medias = $this->manager->findMediasByModelAndId('Post', 1);
+        $this->assertGreaterThan(1, count($medias));
         $this->assertEquals(2, count($medias));
     }
 
     public function testRemoveFile()
     {
         $this->client->request('GET', '/admin/medias/delete/Post/MvcBlogBundle/9');
-        $this->assertEquals(200,$this->client->getResponse()->getStatusCode());
+        $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
 
-        $medias  = $this->manager->findMediasByModelAndId('Post',2);
+        $medias = $this->manager->findMediasByModelAndId('Post', 2);
         $this->assertEquals(0, count($medias));
     }
 
@@ -75,8 +75,8 @@ 
             ['HTTP_X-Requested-With' => 'XMLHttpRequest']
         );
 
-        $this->assertEquals(200,$this->client->getResponse()->getStatusCode());
-        $medias  = $this->manager->findMediasByModelAndId('Post',13);
+        $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
+        $medias = $this->manager->findMediasByModelAndId('Post', 13);
 
         $this->assertEquals(1, count($medias));
 
@@ -99,10 +99,10 @@ 
             ['HTTP_X-Requested-With' => 'XMLHttpRequest']
         );
 
-        $this->assertEquals(500,$this->client->getResponse()->getStatusCode());
+        $this->assertEquals(500, $this->client->getResponse()->getStatusCode());
 
         $this->manager = $this->container->get('mk.media.manager');
-        $medias  = $this->manager->findMediasByModelAndId('Post',17);
+        $medias = $this->manager->findMediasByModelAndId('Post', 17);
         $this->assertEquals(0, count($medias));
     }
 

Interfaces/Mediable.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: Rafidion Michael
- * Date: 02/12/2014
- * Time: 01:56
- */
+     * Created by PhpStorm.
+     * User: Rafidion Michael
+     * Date: 02/12/2014
+     * Time: 01:56
+     */
 
 namespace Mykees\MediaBundle\Interfaces;
 

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 namespace Mykees\MediaBundle\Interfaces;
 
 
-Interface Mediable{
+Interface Mediable {
 
     public function getId();
 }

EventListener/UploadSubscriber.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: Rafidion Michael
- * Date: 30/11/2014
- * Time: 18:22
- */
+     * Created by PhpStorm.
+     * User: Rafidion Michael
+     * Date: 30/11/2014
+     * Time: 18:22
+     */
 
 namespace Mykees\MediaBundle\EventListener;
 

Spacing +28 added lines, -28 removed lines

@@ -36,57 +36,57 @@ 
     }
 
 
-    public function uploadProcess( UploadEvent $event )
+    public function uploadProcess(UploadEvent $event)
     {
-        $file  = $event->getFile()->all();
+        $file = $event->getFile()->all();
         $fileUploaded = $file['file'];
-        $model   = $event->getMediableModel();
+        $model = $event->getMediableModel();
         $model_id = $event->getMediableId();
         $resize_option = $event->getContainer()->getParameter('mykees.media.resize');
         $extension = pathinfo($fileUploaded->getClientOriginalName(), PATHINFO_EXTENSION);
         $DS = DIRECTORY_SEPARATOR;
 
 
-        if( $this->isValidExtension($extension,$event) )
+        if ($this->isValidExtension($extension, $event))
         {
             //create dir
             $webroot = $event->getRootDir().'/../web';
             $dir = $webroot.'/img';
 
-            if(!file_exists($dir)){mkdir($dir,0777);}
+            if (!file_exists($dir)) {mkdir($dir, 0777); }
 
             $dir .= $DS.date('Y');
-            if(!file_exists($dir)){mkdir($dir,0777);}
+            if (!file_exists($dir)) {mkdir($dir, 0777); }
 
             $dir .= $DS.date('m');
-            if(!file_exists($dir)){mkdir($dir,0777);}
+            if (!file_exists($dir)) {mkdir($dir, 0777); }
 
             //clean and define path filename
-            $filename = $this->cleanFilename( $fileUploaded->getClientOriginalName(), $extension );
+            $filename = $this->cleanFilename($fileUploaded->getClientOriginalName(), $extension);
             //test duplicate
-            $name = $this->mediaExist( $filename,$webroot );
+            $name = $this->mediaExist($filename, $webroot);
 
             $filePath = date('Y').'/'.date('m').'/'.$name;
             $save_media = $this->saveMedia($filePath, $name, $model, $model_id);
 
-            if($save_media)
+            if ($save_media)
             {
                 //upload
-                $fileUploaded->move($dir,$name);
+                $fileUploaded->move($dir, $name);
                 $event->setMedia($save_media);
 
-                if(!empty($resize_option[$model]))
+                if (!empty($resize_option[$model]))
                 {
                     $resize = new ResizeHelper($resize_option[$model], $webroot);
                     $resize->resize($save_media->getFile());
                 }
 
                 return true;
-            }else{
+            }else {
 
                 return new Response();
             }
-        }else{
+        }else {
 
             return new Response();
         }
@@ -98,12 +98,12 @@ 
      * @param $event
      * @return bool
      */
-    private function isValidExtension($extension,$event)
+    private function isValidExtension($extension, $event)
     {
         $extensions = $event->getContainer()->getParameter('mykees.media.extension');
-        $valid_extensions = !empty($extensions) ? $extensions : ['jpg','jpeg','JPG','JPEG'];
+        $valid_extensions = !empty($extensions) ? $extensions : ['jpg', 'jpeg', 'JPG', 'JPEG'];
 
-        return in_array($extension,$valid_extensions);
+        return in_array($extension, $valid_extensions);
     }
 
     /**
@@ -112,10 +112,10 @@ 
      * @param $extension
      * @return string
      */
-    private function cleanFilename( $filename, $extension )
+    private function cleanFilename($filename, $extension)
     {
-        $f = explode('.',$filename);
-        $cleanFilename = Urlizer::urlize(implode('.',array_slice($f,0,-1)));
+        $f = explode('.', $filename);
+        $cleanFilename = Urlizer::urlize(implode('.', array_slice($f, 0, -1)));
 
         return $cleanFilename.'.'.$extension;
     }
@@ -127,23 +127,23 @@ 
      * @param int $count
      * @return string
      */
-    private function mediaExist( $filename, $webroot, $count=0 )
+    private function mediaExist($filename, $webroot, $count = 0)
     {
         $file = $filename;
         $filePath = false;
-        if($count > 0){
-            $f = explode('.',$filename);
+        if ($count > 0) {
+            $f = explode('.', $filename);
             $file = $f[0].'_'.$count.'.'.end($f);
             $filePath = $webroot.'/img/'.date('Y').'/'.date('m').'/'.$file;
         }
         $filePath = !$filePath ? $webroot.'/img/'.date('Y').'/'.date('m').'/'.$file : $filePath;
-        if(file_exists($filePath))
+        if (file_exists($filePath))
         {
             $count++;
 
-            return $this->mediaExist( $filename, $webroot, $count );
-        }else{
-            $filename =  $file;
+            return $this->mediaExist($filename, $webroot, $count);
+        }else {
+            $filename = $file;
 
             return $filename;
         }
@@ -160,7 +160,7 @@ 
     private function saveMedia($filePath, $filename, $model, $model_id)
     {
         $media = new Media();
-        $fn = explode('.',$filename);
+        $fn = explode('.', $filename);
         $media->setName($fn[0]);
         $media->setMediableModel($model);
         $media->setMediableId($model_id);

Braces +3 added lines, -3 removed lines

@@ -82,11 +82,11 @@ 
                 }
 
                 return true;
-            }else{
+            } else{
 
                 return new Response();
             }
-        }else{
+        } else{
 
             return new Response();
         }
@@ -142,7 +142,7 @@ 
             $count++;
 
             return $this->mediaExist( $filename, $webroot, $count );
-        }else{
+        } else{
             $filename =  $file;
 
             return $filename;

EventListener/MediasListener.php

Spacing +2 added lines, -2 removed lines

@@ -31,9 +31,9 @@
     {
         $model = $args->getEntity();
 
-        if($model instanceof Mediable)
+        if ($model instanceof Mediable)
         {
-            $manager = new MediaManager($this->managerRegistry,$this->rootDir,$this->resize_parameters);
+            $manager = new MediaManager($this->managerRegistry, $this->rootDir, $this->resize_parameters);
             $manager->removeAllMediasForModel($model);
         }
     }

Traits/MediableTrait.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: tetsu0o
- * Date: 28/12/14
- * Time: 12:41
- */
+     * Created by PhpStorm.
+     * User: tetsu0o
+     * Date: 28/12/14
+     * Time: 12:41
+     */
 
 namespace Mykees\MediaBundle\Traits;
 use Doctrine\Common\Collections\ArrayCollection;

Spacing +4 added lines, -4 removed lines

@@ -14,16 +14,16 @@
 
     protected $medias;
 
-    public function getMediableModel(){
+    public function getMediableModel() {
         return Reflection::getClassShortName($this);
     }
 
-    public function getMediableId(){
+    public function getMediableId() {
         return $this->getId();
     }
 
-    public function getMedias(){
-        $this->medias = $this->medias ? : new ArrayCollection();
+    public function getMedias() {
+        $this->medias = $this->medias ?: new ArrayCollection();
         return $this->medias;
     }
 }

Util/Reflection.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: tetsu0o
- * Date: 28/12/14
- * Time: 12:45
- */
+     * Created by PhpStorm.
+     * User: tetsu0o
+     * Date: 28/12/14
+     * Time: 12:45
+     */
 
 namespace Mykees\MediaBundle\Util;
 

Spacing +8 added lines, -8 removed lines

@@ -19,25 +19,25 @@
     }
 
 
-    public static function getClassShortName ( $model ) {
-        $reflection = new \ReflectionClass( $model );
+    public static function getClassShortName($model) {
+        $reflection = new \ReflectionClass($model);
         return $reflection->getShortName();
     }
 
 
-    public static function getBundlePath ( $model  ){
-        $explode = explode('\\', self::getClassName($model ));
+    public static function getBundlePath($model) {
+        $explode = explode('\\', self::getClassName($model));
         return $explode[0].'\\'.$explode[1].'\\'.$explode[2].'\\'.$explode[3];
     }
 
 
-    public static function getBundleRepository ( $model  ){
-        $explode = explode('\\', self::getClassName($model ));
+    public static function getBundleRepository($model) {
+        $explode = explode('\\', self::getClassName($model));
         return $explode[0].$explode[1].':'.$explode[3];
     }
 
-    public static function getShortBundleRepository ( $model  ){
-        $explode = explode('\\', self::getClassName($model ));
+    public static function getShortBundleRepository($model) {
+        $explode = explode('\\', self::getClassName($model));
         return $explode[0].$explode[1];
     }
 

Manager/AbstractManager.php

Spacing +16 added lines, -16 removed lines

@@ -16,35 +16,35 @@ 
 
     public function getModelInfos(array $datas)
     {
-        $ids= [];
+        $ids = [];
         $models = [];
         $model_exist = false;
 
-        foreach( $datas as $k=>$data )
+        foreach ($datas as $k=>$data)
         {
-            if($data instanceof Mediable)
+            if ($data instanceof Mediable)
             {
-                array_push($ids,$data->getMediableId());
-                if( $model_exist != $data->getMediableModel() )
+                array_push($ids, $data->getMediableId());
+                if ($model_exist != $data->getMediableModel())
                 {
-                    array_push($models,$data->getMediableModel());
+                    array_push($models, $data->getMediableModel());
                     $model_exist = $data->getMediableModel();
                 }
             }
         }
 
-        return [ 'ids'=>$ids, 'models'=>$models ];
+        return ['ids'=>$ids, 'models'=>$models];
     }
 
     public function refreshMediasArray(array $medias, array $models)
     {
         $this->clean($models);
-        foreach($models as $model)
+        foreach ($models as $model)
         {
-            foreach($medias as $media){
-                if($model instanceof Mediable)
+            foreach ($medias as $media) {
+                if ($model instanceof Mediable)
                 {
-                    if($model->getId() == $media->getMediableId() && $model->getMediableModel() == $media->getMediableModel())
+                    if ($model->getId() == $media->getMediableId() && $model->getMediableModel() == $media->getMediableModel())
                     {
                         $model->getMedias()->add($media);
                     }
@@ -60,9 +60,9 @@ 
 
     public function addMedias(array $medias, Mediable $model)
     {
-        foreach($medias as $media){
-            if(!empty($media) && Reflection::getClassShortName($media) == 'Media'){
-                $this->addMedia($media,$model);
+        foreach ($medias as $media) {
+            if (!empty($media) && Reflection::getClassShortName($media) == 'Media') {
+                $this->addMedia($media, $model);
             }
         }
     }
@@ -70,12 +70,12 @@ 
     public function refreshMedias(array $medias, Mediable $model)
     {
         $model->getMedias()->clear();
-        $this->addMedias($medias,$model);
+        $this->addMedias($medias, $model);
     }
 
     public function clean(array $models)
     {
-        foreach($models as $model)
+        foreach ($models as $model)
         {
             $model->getMedias()->clear();
         }