From b69ae943a74a3fcce4b149d18316f98bc79034bf Mon Sep 17 00:00:00 2001
From: Sonny Le <sonlv@templaza.com>
Date: Mon, 10 Oct 2022 11:57:23 +0700
Subject: [PATCH] Update Font Code Style

---
 framework/library/astroid/Helper/Font.php | 59 +++++++++--------------
 1 file changed, 24 insertions(+), 35 deletions(-)

diff --git a/framework/library/astroid/Helper/Font.php b/framework/library/astroid/Helper/Font.php
index cd4048cf..5faee2c9 100644
--- a/framework/library/astroid/Helper/Font.php
+++ b/framework/library/astroid/Helper/Font.php
@@ -114,50 +114,39 @@ public static function getUploadedFonts($template)
             return [];
         }
         $fonts = [];
-        $font_extensions = ['otf', 'ttf', 'woff'];
         if (file_exists($template_media_fonts_path) || file_exists($template_fonts_path)) {
             if (file_exists($template_media_fonts_path)) $template_fonts_path = $template_media_fonts_path;
-            foreach (scandir($template_fonts_path) as $font_path) {
-                if (is_file($template_fonts_path . '/' . $font_path)) {
-                    $pathinfo = pathinfo($template_fonts_path . '/' . $font_path);
-                    if (in_array($pathinfo['extension'], $font_extensions)) {
-                        $font = \FontLib\Font::load($template_fonts_path . '/' . $font_path);
-                        $font->parse();
-                        $fontname = $font->getFontFullName();
-                        $fontid = 'library-font-' . Helper::slugify($fontname);
-                        if (!isset($fonts[$fontid])) {
-                            $fonts[$fontid] = [];
-                            $fonts[$fontid]['id'] = $fontid;
-                            $fonts[$fontid]['name'] = $fontname;
-                            $fonts[$fontid]['files'] = [];
-                        }
-                        $fonts[$fontid]['files'][] = $font_path;
-                    }
-                }
-            }
+            $fonts  =   self::getLocalFonts($template_fonts_path);
         }
 
         if (file_exists($template_custom_fonts_path)) {
-            foreach (scandir($template_custom_fonts_path) as $font_path) {
-                if (is_file($template_custom_fonts_path . '/' . $font_path)) {
-                    $pathinfo = pathinfo($template_custom_fonts_path . '/' . $font_path);
-                    if (in_array($pathinfo['extension'], $font_extensions)) {
-                        $font = \FontLib\Font::load($template_custom_fonts_path . '/' . $font_path);
-                        $font->parse();
-                        $fontname = $font->getFontFullName();
-                        $fontid = 'library-font-' . Helper::slugify($fontname);
-                        if (!isset($fonts[$fontid])) {
-                            $fonts[$fontid] = [];
-                            $fonts[$fontid]['id'] = $fontid;
-                            $fonts[$fontid]['name'] = $fontname;
-                            $fonts[$fontid]['files'] = [];
-                        }
-                        $fonts[$fontid]['files'][] = $font_path;
+            $fonts  =   array_merge($fonts, self::getLocalFonts($template_custom_fonts_path));
+        }
+        Framework::getDebugger()->stop('local-fonts');
+        return $fonts;
+    }
+
+    public static function getLocalFonts($template_fonts_path) {
+        $fonts = [];
+        $font_extensions = ['otf', 'ttf', 'woff'];
+        foreach (scandir($template_fonts_path) as $font_path) {
+            if (is_file($template_fonts_path . '/' . $font_path)) {
+                $pathinfo = pathinfo($template_fonts_path . '/' . $font_path);
+                if (in_array($pathinfo['extension'], $font_extensions)) {
+                    $font = \FontLib\Font::load($template_fonts_path . '/' . $font_path);
+                    $font->parse();
+                    $fontname = $font->getFontFullName();
+                    $fontid = 'library-font-' . Helper::slugify($fontname);
+                    if (!isset($fonts[$fontid])) {
+                        $fonts[$fontid] = [];
+                        $fonts[$fontid]['id'] = $fontid;
+                        $fonts[$fontid]['name'] = $fontname;
+                        $fonts[$fontid]['files'] = [];
                     }
+                    $fonts[$fontid]['files'][] = $font_path;
                 }
             }
         }
-        Framework::getDebugger()->stop('local-fonts');
         return $fonts;
     }