Code style fixes

Dolphiq · Oct 6, 2017 · 3b5748d · 3b5748d
1 parent 0c80898
commit 3b5748d
Show file tree

Hide file tree

Showing 6 changed files with 51 additions and 55 deletions.
diff --git a/src/Plugin.php b/src/Plugin.php
@@ -18,17 +18,13 @@
 
 class Plugin extends \craft\base\Plugin
 {
-
     public function init()
     {
         parent::init();
 
         // Register field type
-        Event::on(Fields::class, Fields::EVENT_REGISTER_FIELD_TYPES, function(RegisterComponentTypesEvent $event) {
-          $event->types[] = Iconpicker::class;
+        Event::on(Fields::class, Fields::EVENT_REGISTER_FIELD_TYPES, function (RegisterComponentTypesEvent $event) {
+            $event->types[] = Iconpicker::class;
         });
     }
-
 }
-
-?>
diff --git a/src/assets/appAsset.php b/src/assets/appAsset.php
@@ -37,4 +37,4 @@ public function init()
 
         parent::init();
     }
-}
+}
diff --git a/src/assets/sharedAsset.php b/src/assets/sharedAsset.php
@@ -26,4 +26,4 @@ public function init()
 
         parent::init();
     }
-}
+}
diff --git a/src/fields/Iconpicker.php b/src/fields/Iconpicker.php
@@ -20,7 +20,6 @@
 use yii\db\Schema;
 use yii\helpers\ArrayHelper;
 
-
 class Iconpicker extends Field implements PreviewableFieldInterface
 {
     // Static
@@ -29,22 +28,22 @@ class Iconpicker extends Field implements PreviewableFieldInterface
     /**
      * @var string The directory where the fonts are
      */
-    CONST FONT_DIR = '@vendor/dolphiq/iconpicker/src/resources-shared/fonts/';
+    const FONT_DIR = '@vendor/dolphiq/iconpicker/src/resources-shared/fonts/';
 
     /**
      * @var array All extentions that are allowed to be imported as a font
      */
-    CONST FONT_EXT = ['*.woff', '*.ttf'];
+    const FONT_EXT = ['*.woff', '*.ttf'];
 
     /**
      * @var string Icon class
      */
-    CONST ICON_CLASS = 'dq-icon-';
+    const ICON_CLASS = 'dq-icon-';
 
     /**
      * @var string Pattern to format a safe file name
      */
-    CONST SAFE_NAME_PATTERN = '/[^A-Za-z0-9_\-]/';
+    const SAFE_NAME_PATTERN = '/[^A-Za-z0-9_\-]/';
 
     /**
      * @inheritdoc
@@ -167,13 +166,14 @@ public function normalizeValue($value, ElementInterface $element = null)
      * @return array
      */
 
-    private function getFonts(){
-        if(empty($this->fonts)) {
+    private function getFonts()
+    {
+        if (empty($this->fonts)) {
             $files = FileHelper::findFiles(Craft::getAlias(self::FONT_DIR), ['only' => self::FONT_EXT]);
             $filenames = [];
             $fonts = [];
 
-            foreach ($files as $file){
+            foreach ($files as $file) {
                 $pathInfo = pathinfo($file);
                 $safename = in_array($pathInfo['filename'], $filenames) ? $pathInfo['basename'] : $pathInfo['filename'];
                 $safename = $this->safeName($safename);
@@ -198,9 +198,10 @@ private function getFonts(){
      * Returns a options list for the settings dropdown when defining a field
      * @return array
      */
-    private function getFontOptions(){
+    private function getFontOptions()
+    {
         $f = $this->getFonts();
-        if(!empty($f)){
+        if (!empty($f)) {
             return ArrayHelper::map($f, 'safename', 'basename');
         }
     }
@@ -210,11 +211,11 @@ private function getFontOptions(){
      *
      * @return array|null
      */
-    private function getIcons(){
-
-        if(!empty($this->iconFont)) {
+    private function getIcons()
+    {
+        if (!empty($this->iconFont)) {
             $fonts = $this->getFonts();
-            if(!empty($fonts) && isset($fonts[$this->iconFont])) {
+            if (!empty($fonts) && isset($fonts[$this->iconFont])) {
                 $font = Font::load($fonts[$this->iconFont]['path']);
                 $font->parse();
 
@@ -231,10 +232,11 @@ private function getIcons(){
      * @param $font \FontLib\TrueType\File|null
      * @return array|null
      */
-    private function getUnicodeList($font){
-        if(!empty($font)){
+    private function getUnicodeList($font)
+    {
+        if (!empty($font)) {
             $unicodes = $font->getUnicodeCharMap();
-            if(!empty($unicodes)){
+            if (!empty($unicodes)) {
                 return array_keys($unicodes);
             }
         }
@@ -246,21 +248,20 @@ private function getUnicodeList($font){
     /**
      * Generate a css file that creates font families for each font file in the font directory
      */
-    public function getFontCss(){
-
+    public function getFontCss()
+    {
         $sharedAsset = new sharedAsset();
         $scss = "";
 
-        foreach ($this->getFonts() as $safeName => $pathInfo){
+        foreach ($this->getFonts() as $safeName => $pathInfo) {
             $fontFile = $pathInfo['path'];
             $font = Font::load($fontFile);
             $font->parse();
 
-            if(!empty($font)) {
+            if (!empty($font)) {
                 $iconFontName = $safeName;
 
                 if (!empty($iconFontName)) {
-
                     $scss .= "
 @font-face {
     font-family: 'dq-iconpicker-".$iconFontName."';
@@ -288,9 +289,6 @@ public function getFontCss(){
   -moz-osx-font-smoothing: grayscale;
 
 }'."\n\n";
-
-
-
                 }
             }
         }
@@ -299,17 +297,16 @@ public function getFontCss(){
 
         // Register the assetbundle that loads the generated css
         Craft::$app->view->registerAssetBundle(sharedAsset::className());
-
     }
 
     // Get the fontname of the currently selected font.
-    public function getIconFontName(){
-
+    public function getIconFontName()
+    {
         return $this->iconFont;
-
     }
 
-    private function safeName($filename){
+    private function safeName($filename)
+    {
         $name = preg_replace(self::SAFE_NAME_PATTERN, '-', $filename);
         return $name;
     }

diff --git a/src/models/IconpickerModel.php b/src/models/IconpickerModel.php
@@ -20,10 +20,8 @@
 use Craft;
 use craft\base\Model;
 
-
-
-class IconpickerModel extends Model{
-
+class IconpickerModel extends Model
+{
     public $icon;
     public $iconFont;
 
@@ -34,24 +32,28 @@ public function rules()
         ];
     }
 
-    public function getIconChar(){
+    public function getIconChar()
+    {
         return '&#'.$this->icon.';';
     }
 
-    public function getIconCharHex(){
+    public function getIconCharHex()
+    {
         return '&#x'.$this->getIconHex().';';
     }
 
-    public function getIconHex(){
+    public function getIconHex()
+    {
         return dechex($this->icon);
     }
 
-    public function getIconSpan(){
+    public function getIconSpan()
+    {
         return '<span class="'.$this->getIconClass().'">'.$this->getIconCharHex().'</span>';
     }
 
-    public function getIconClass(){
+    public function getIconClass()
+    {
         return 'dq-icon-'.$this->iconFont;
     }
-
-}
+}
diff --git a/src/views/main/_field.php b/src/views/main/_field.php
@@ -13,8 +13,7 @@
 
 $field->getFontCss();
 
-if(count($icons) > 0) {
-
+if (count($icons) > 0) {
     ?>
 
 
@@ -45,8 +44,7 @@
                   }
               } else {
                   echo 'No icons found in the selected font';
-              }
-              ?>
+              } ?>
           </div>
         </div>
       </div>
@@ -61,6 +59,9 @@
     <div class="resizehandle"></div>
   </div>
 
-<?php } else { ?>
+<?php
+} else {
+                  ?>
     <p>There is no font uploaded to the font folder of this plugin, no font selected in the field settings, or the font contains no icons</p>
-<?php } ?>
+<?php
+              } ?>
-Original file line number
+Diff line change
@@ Expand Up / @@ -37,4 +37,4 @@ public function init() @@
             parent::init();
         }
-    }
+    }