From 2729201e9dc2eec05ea3f68d2b0b7eeb9c871cec Mon Sep 17 00:00:00 2001
From: Diatrex <diahshalabi@hotmail.com>
Date: Mon, 24 Aug 2020 15:59:25 +0300
Subject: [PATCH 1/3] #1995 SEO fixes for w3 validator

---
 .../base-theme/resources/views/layouts/default.twig    | 10 +++++-----
 .../base-theme/resources/views/layouts/page.twig       | 10 +++++-----
 .../resources/views/notifications/email.twig           |  2 +-
 .../base-theme/resources/views/partials/assets.twig    |  9 +++------
 .../resources/views/partials/categories-mobile.twig    |  2 +-
 .../base-theme/resources/views/partials/messages.twig  |  4 ++--
 .../base-theme/resources/views/partials/metadata.twig  |  5 +----
 .../base-theme/resources/views/partials/metatags.twig  |  2 +-
 .../base-theme/resources/views/partials/settings.twig  |  1 -
 .../base-theme/resources/views/partials/styles.twig    |  2 +-
 10 files changed, 20 insertions(+), 27 deletions(-)
diff --git a/addons/default/visiosoft/base-theme/resources/views/layouts/default.twig b/addons/default/visiosoft/base-theme/resources/views/layouts/default.twig
index daf89d630..76d07b450 100644
--- a/addons/default/visiosoft/base-theme/resources/views/layouts/default.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/layouts/default.twig
@@ -1,15 +1,15 @@
 <!doctype html>
 
 <!--[if lt IE 7]>
-<html class="nojs ms lt_ie7" lang="en"><![endif]-->
+<html class="nojs ms lt_ie7" lang="{{ config('app.locale') }}"><![endif]-->
 <!--[if IE 7]>
-<html class="nojs ms ie7" lang="en"><![endif]-->
+<html class="nojs ms ie7" lang="{{ config('app.locale') }}"><![endif]-->
 <!--[if IE 8]>
-<html class="nojs ms ie8" lang="en"><![endif]-->
+<html class="nojs ms ie8" lang="{{ config('app.locale') }}"><![endif]-->
 <!--[if gt IE 8]>
-<html class="nojs ms" lang="en"><![endif]-->
+<html class="nojs ms" lang="{{ config('app.locale') }}"><![endif]-->
 
-<html>
+<html lang="{{ config('app.locale') }}">
 
 <head>
     {% include "visiosoft.theme.base::partials/metadata" %}
diff --git a/addons/default/visiosoft/base-theme/resources/views/layouts/page.twig b/addons/default/visiosoft/base-theme/resources/views/layouts/page.twig
index 7ac84f568..e4cb45c72 100644
--- a/addons/default/visiosoft/base-theme/resources/views/layouts/page.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/layouts/page.twig
@@ -1,15 +1,15 @@
 <!doctype html>
 
 <!--[if lt IE 7]>
-<html class="nojs ms lt_ie7" lang="en"><![endif]-->
+<html class="nojs ms lt_ie7" lang="{{ config('app.locale') }}"><![endif]-->
 <!--[if IE 7]>
-<html class="nojs ms ie7" lang="en"><![endif]-->
+<html class="nojs ms ie7" lang="{{ config('app.locale') }}"><![endif]-->
 <!--[if IE 8]>
-<html class="nojs ms ie8" lang="en"><![endif]-->
+<html class="nojs ms ie8" lang="{{ config('app.locale') }}"><![endif]-->
 <!--[if gt IE 8]>
-<html class="nojs ms" lang="en"><![endif]-->
+<html class="nojs ms" lang="{{ config('app.locale') }}"><![endif]-->
 
-<html>
+<html lang="{{ config('app.locale') }}">
 
 <head>
     {% include "visiosoft.theme.base::partials/metadata" %}
diff --git a/addons/default/visiosoft/base-theme/resources/views/notifications/email.twig b/addons/default/visiosoft/base-theme/resources/views/notifications/email.twig
index e296b1b31..e65663688 100644
--- a/addons/default/visiosoft/base-theme/resources/views/notifications/email.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/notifications/email.twig
@@ -9,7 +9,7 @@
                 {{ title ?: setting_value('streams::name','Visiosoft LTD.') }}
             </title>
 
-            <style type="text/css" rel="stylesheet" media="all">
+            <style rel="stylesheet" media="all">
                 /* Media Queries */
                 @media only screen and (max-width: 500px) {
                     .button {
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/assets.twig b/addons/default/visiosoft/base-theme/resources/views/partials/assets.twig
index 7b843d4d3..e623a9d63 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/assets.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/assets.twig
@@ -2,9 +2,6 @@
     Main CSS can be found in metadata.twig
 #}
 
-<!-- VueJS Requirements -->
-<script type="text/javascript" src="{{ asset('js/app.js') }}"></script>
-
 {# Core Theme Components #}
 {{ asset_add("theme.js", "visiosoft.theme.base::js/popper.min.js") }}
 {{ asset_add("theme.js", "visiosoft.theme.base::js/bootstrap.min.js") }}
@@ -24,20 +21,20 @@
 {{ asset_script("theme.js") }}
 
 {# Include Styles Loaded From Outside #}
-<style type="text/css">
+<style>
     {% for style in asset_inlines("styles.css", ["min"]) %}
     {{ style|raw }}
     {% endfor %}
 </style>
 
 {# Include Scripts Loaded From Outside #}
-<script type="text/javascript">
+<script>
     {% for script in asset_inlines("scripts.js", ["min"]) %}
     {{ script|raw }}
     {% endfor %}
 </script>
 
-<script type="text/javascript">
+<script>
     {% for script in asset_inlines("customjs.js") %}
     {{ script|raw }}
     {% endfor %}
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/categories-mobile.twig b/addons/default/visiosoft/base-theme/resources/views/partials/categories-mobile.twig
index 5a323512a..b2ce5c599 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/categories-mobile.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/categories-mobile.twig
@@ -11,7 +11,7 @@
                 <img src="{{ img('visiosoft.theme.base::images/default-categories-icon.png').url }}" alt="{{ main_category.name }}"
                      class="img-responsive">
             {% endif %}
-            <font>{{ main_category.name }}</font>
+            <span>{{ main_category.name }}</span>
             <p class="p-0 sub-categories">
                 {% set sub_categories = entries('cats','category').where('parent_category_id', main_category.id).where('deleted_at', null).orderBy('sort_order').get() %}
                 {% for sub_category in sub_categories|slice(0,sub_categories_limit) %}
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/messages.twig b/addons/default/visiosoft/base-theme/resources/views/partials/messages.twig
index 6ea3181dd..32a1d31b3 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/messages.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/messages.twig
@@ -1,4 +1,4 @@
-<section id="messages">
+<div id="messages">
 
     <!-- Success Messages -->
     {% if message_exists('success') %}
@@ -54,4 +54,4 @@
         </div>
     {% endif %}
 
-</section>
+</div>
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/metadata.twig b/addons/default/visiosoft/base-theme/resources/views/partials/metadata.twig
index 0c9932310..8f51c0847 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/metadata.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/metadata.twig
@@ -1,6 +1,3 @@
-<!-- Locale -->
-<meta http-equiv="Content-Language" content="{{ config('app.locale') }}">
-
 <!-- VueJS Requirements -->
 <meta name="csrf-token" content="{{ csrf_token() }}" />
 
@@ -27,7 +24,7 @@
 {% include "visiosoft.theme.base::partials/styles" %}
 
 {# Custom Theme Components #}
-<style type="text/css">
+<style>
     {% for style in asset_inlines("custom-theme.css", ["min"]) %}
     {{ style|raw }}
     {% endfor %}
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/metatags.twig b/addons/default/visiosoft/base-theme/resources/views/partials/metatags.twig
index bfcd78be2..47b13d70b 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/metatags.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/metatags.twig
@@ -1,12 +1,12 @@
 <!-- Meta Tags -->
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width"/>
-<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
 <meta name="generator" content="{{ config_get('streams::distribution.name') }}"/>
 <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0">
 <meta name="description"
       content="{{ (template.get('meta_description') ?: setting_value('streams::description'))|slice(0,160) }}"/>
 <meta name="keywords" content="{{ template.get('meta_keywords') }}" />
+{{ setting_value('visiosoft.module.advs::google_statistic_code')|raw }}
 <!-- Meta Tags -->
 
 <title>
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/settings.twig b/addons/default/visiosoft/base-theme/resources/views/partials/settings.twig
index 25e6a47aa..e69de29bb 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/settings.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/settings.twig
@@ -1 +0,0 @@
-{{ setting_value('visiosoft.module.advs::google_statistic_code')|raw }}
\ No newline at end of file
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/styles.twig b/addons/default/visiosoft/base-theme/resources/views/partials/styles.twig
index 595e1fad1..80b988429 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/styles.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/styles.twig
@@ -1,3 +1,3 @@
-<style type="text/css">
+<style>
     {{ setting_value("visiosoft.theme.base::style")|raw }}
 </style>
\ No newline at end of file

From 92d4a89809795991c6bbfaef047e5b8c75bd69de Mon Sep 17 00:00:00 2001
From: Diatrex <diahshalabi@hotmail.com>
Date: Wed, 26 Aug 2020 11:08:50 +0300
Subject: [PATCH 2/3] #2027 SEO small changes

---
 .../src/Http/Controller/advsController.php    | 16 +++++++---
 .../resources/views/partials/navigation.twig  | 32 ++++++++++++++-----
 2 files changed, 35 insertions(+), 13 deletions(-)

diff --git a/addons/default/visiosoft/advs-module/src/Http/Controller/advsController.php b/addons/default/visiosoft/advs-module/src/Http/Controller/advsController.php
index 65eca8331..d01054fbf 100644
--- a/addons/default/visiosoft/advs-module/src/Http/Controller/advsController.php
+++ b/addons/default/visiosoft/advs-module/src/Http/Controller/advsController.php
@@ -278,10 +278,6 @@ class AdvsController extends PublicController
             $radio = $returnvalues['radio'];
         }
 
-        if (!empty($param['user'])) {
-            $user = $this->userRepository->find($param['user']);
-        }
-
         Cookie::queue(Cookie::make('last_search', $this->requestHttp->getRequestUri(), 84000));
 
         $viewType = $this->requestHttp->cookie('viewType');
@@ -302,6 +298,12 @@ class AdvsController extends PublicController
             $this->template->set('meta_title', $catText);
         }
 
+        if (!empty($param['user'])) {
+            $user = $this->userRepository->find($param['user']);
+            $this->template->set('showTitle', false);
+            $this->template->set('meta_title', $user->name() . ' ' . trans('visiosoft.module.advs::field.ads'));
+        }
+
         $compact = compact('advs', 'countries', 'mainCats', 'subCats', 'checkboxes', 'request', 'param',
             'user', 'featured_advs', 'viewType', 'topfields', 'selectDropdown', 'selectRange', 'selectImage', 'ranges',
             'seenList', 'searchedCountry', 'radio', 'categoryId', 'cityId', 'allCats', 'catText');
@@ -400,7 +402,11 @@ class AdvsController extends PublicController
 
         $this->template->set('meta_keywords', implode(',', explode(' ', $adv->name)));
         $this->template->set('meta_description', strip_tags($adv->advs_desc, ''));
-        $this->template->set('meta_title', $adv->name . "|" . end($categories)['name']);
+        $this->template->set('showTitle', false);
+        $this->template->set(
+            'meta_title',
+            $adv->name . " " . end($categories)['name'] . ' ' . setting_value('streams::domain')
+        );
         if (substr($adv->cover_photo, 0, 4) === "http") {
             $coverPhoto = $adv->cover_photo;
         } else {
diff --git a/addons/default/visiosoft/base-theme/resources/views/partials/navigation.twig b/addons/default/visiosoft/base-theme/resources/views/partials/navigation.twig
index 3b0bc3433..69fc9b22e 100644
--- a/addons/default/visiosoft/base-theme/resources/views/partials/navigation.twig
+++ b/addons/default/visiosoft/base-theme/resources/views/partials/navigation.twig
@@ -28,16 +28,24 @@
                 <ul class="nav navbar-nav navbar-right d-block d-sm-none">
                     <li class="nav-item">
                         {% if auth_check() %}
-                            <a class="nav-link" href="{{ url_route('anomaly.module.users::logout') }}">{{ trans('visiosoft.theme.base::field.logout.name') }}</a>
+                            <a class="nav-link" href="{{ url_route('anomaly.module.users::logout') }}">
+                                {{ trans('visiosoft.theme.base::field.logout.name') }}
+                            </a>
                         {% else %}
-                            <a class="nav-link" href="{{ url_route('anomaly.module.users::login') }}">{{ trans('visiosoft.theme.base::field.login.name') }}</a>
+                            <a class="nav-link" href="{{ url_route('anomaly.module.users::login') }}" rel="nofollow">
+                                {{ trans('visiosoft.theme.base::field.login.name') }}
+                            </a>
                         {% endif %}
                     </li>
                     <li class="nav-item">
-                        <a class="nav-link" href="{{ url_route('profile::profile') }}">{{ trans('visiosoft.theme.base::field.profile') }}</a>
+                        <a class="nav-link" href="{{ url_route('profile::profile') }}" rel="nofollow">
+                            {{ trans('visiosoft.theme.base::field.profile') }}
+                        </a>
                     </li>
                     <li class="nav-item">
-                        <a href="/advs/create_adv" class="btn btn-primary">{{ trans("visiosoft.theme.base::button.post_ad.name") }}</a>
+                        <a href="/advs/create_adv" class="btn btn-primary" rel="nofollow">
+                            {{ trans("visiosoft.theme.base::button.post_ad.name") }}
+                        </a>
                     </li>
                 </ul>
             {% endif %}
@@ -47,16 +55,24 @@
                 <ul class="nav navbar-nav navbar-right">
                     <li class="nav-item">
                         {% if auth_check() %}
-                            <a class="nav-link" href="{{ url_route('anomaly.module.users::logout') }}">{{ trans('visiosoft.theme.base::field.logout.name') }}</a>
+                            <a class="nav-link" href="{{ url_route('anomaly.module.users::logout') }}">
+                                {{ trans('visiosoft.theme.base::field.logout.name') }}
+                            </a>
                         {% else %}
-                            <a class="nav-link" href="{{ url_route('anomaly.module.users::login') }}">{{ trans('visiosoft.theme.base::field.login.name') }}</a>
+                            <a class="nav-link" href="{{ url_route('anomaly.module.users::login') }}" rel="nofollow">
+                                {{ trans('visiosoft.theme.base::field.login.name') }}
+                            </a>
                         {% endif %}
                     </li>
                     <li class="nav-item">
-                        <a class="nav-link" href="{{ url_route('profile::profile') }}">{{ trans('visiosoft.theme.base::field.profile') }}</a>
+                        <a class="nav-link" href="{{ url_route('profile::profile') }}" rel="nofollow">
+                            {{ trans('visiosoft.theme.base::field.profile') }}
+                        </a>
                     </li>
                     <li class="nav-item">
-                        <a href="/advs/create_adv" class="btn btn-primary">{{ trans("visiosoft.theme.base::button.post_ad.name") }}</a>
+                        <a href="/advs/create_adv" class="btn btn-primary" rel="nofollow">
+                            {{ trans("visiosoft.theme.base::button.post_ad.name") }}
+                        </a>
                     </li>
                 </ul>
             {% endif %}

From b96ec7d8c02deb36652c3d784b506adf063e816e Mon Sep 17 00:00:00 2001
From: Diatrex <diahshalabi@hotmail.com>
Date: Wed, 26 Aug 2020 15:32:42 +0300
Subject: [PATCH 3/3] #2011 openclassify field type background change

---
 .../resources/config/settings/sections.php    |  1 +
 .../resources/config/settings/settings.php    |  7 +++++
 .../advs-module/resources/lang/en/setting.php |  3 +++
 .../advs-module/src/Adv/AdvRepository.php     | 15 ++++++++---
 .../src/Http/Controller/UploadController.php  | 26 ++++++++++---------
 5 files changed, 37 insertions(+), 15 deletions(-)

diff --git a/addons/default/visiosoft/advs-module/resources/config/settings/sections.php b/addons/default/visiosoft/advs-module/resources/config/settings/sections.php
index 50d32db0f..35e671821 100644
--- a/addons/default/visiosoft/advs-module/resources/config/settings/sections.php
+++ b/addons/default/visiosoft/advs-module/resources/config/settings/sections.php
@@ -25,6 +25,7 @@ return [
                     'estimated_pending_time',
                     'default_published_time',
                     'default_GET',
+                    'add_canvas',
                     'thumbnail_width',
                     'thumbnail_height',
                     'picture_width',
diff --git a/addons/default/visiosoft/advs-module/resources/config/settings/settings.php b/addons/default/visiosoft/advs-module/resources/config/settings/settings.php
index 8a69b1286..0125a4a09 100644
--- a/addons/default/visiosoft/advs-module/resources/config/settings/settings.php
+++ b/addons/default/visiosoft/advs-module/resources/config/settings/settings.php
@@ -108,6 +108,13 @@ return [
             'default_value' => 270,
         ],
     ],
+    'add_canvas' => [
+        'type' => 'anomaly.field_type.boolean',
+        'config' => [
+            'default_value' => true,
+            'mode' => 'checkbox',
+        ],
+    ],
     'thumbnail_height' => [
         'type' => 'anomaly.field_type.integer',
         'config' => [
diff --git a/addons/default/visiosoft/advs-module/resources/lang/en/setting.php b/addons/default/visiosoft/advs-module/resources/lang/en/setting.php
index 026e78216..643e19f0a 100644
--- a/addons/default/visiosoft/advs-module/resources/lang/en/setting.php
+++ b/addons/default/visiosoft/advs-module/resources/lang/en/setting.php
@@ -79,6 +79,9 @@ return [
     'default_GET' => [
         'name' => 'Default Ad GET',
     ],
+    'add_canvas' => [
+        'name' => 'Add Canvas',
+    ],
     'thumbnail_width' => [
         'name' => 'Thumbnail Width',
     ],
diff --git a/addons/default/visiosoft/advs-module/src/Adv/AdvRepository.php b/addons/default/visiosoft/advs-module/src/Adv/AdvRepository.php
index a370ce3e3..90843381b 100644
--- a/addons/default/visiosoft/advs-module/src/Adv/AdvRepository.php
+++ b/addons/default/visiosoft/advs-module/src/Adv/AdvRepository.php
@@ -316,9 +316,18 @@ class AdvRepository extends EntryRepository implements AdvRepositoryInterface
                 // Create thumbnail image
                 $image = Image::make(file_get_contents($adv->files[0]->url()));
                 $image->resize(
-                    setting_value('visiosoft.module.advs::thumbnail_width'),
-                    setting_value('visiosoft.module.advs::thumbnail_height')
-                );
+                    null,
+                    setting_value('visiosoft.module.advs::thumbnail_height'),
+                    function ($constraint) {
+                        $constraint->aspectRatio();
+                    });
+                if (setting_value('visiosoft.module.advs::add_canvas')) {
+                    $image->resizeCanvas(
+                        setting_value('visiosoft.module.advs::thumbnail_width'),
+                        setting_value('visiosoft.module.advs::thumbnail_height'),
+                        'center', false, 'fff'
+                    );
+                }
                 $fileName = 'tn-' . $adv->files[0]->name;
                 $image->save(app_storage_path() . '/files-module/local/images/' . $fileName);
 
diff --git a/addons/default/visiosoft/media-field_type/src/Http/Controller/UploadController.php b/addons/default/visiosoft/media-field_type/src/Http/Controller/UploadController.php
index fb95602b8..bae2b9d24 100644
--- a/addons/default/visiosoft/media-field_type/src/Http/Controller/UploadController.php
+++ b/addons/default/visiosoft/media-field_type/src/Http/Controller/UploadController.php
@@ -62,21 +62,23 @@ class UploadController extends AdminController
             $fullImg = WaterMark::make($this->request->file('upload')->getRealPath())
                     ->resize(null, setting_value('visiosoft.field_type.media::imageResizeH', 600),function ($constraint) {
                             $constraint->aspectRatio();
-                    })
-                    ->resizeCanvas(
-                        setting_value('visiosoft.field_type.media::imageCanvasW', 800),
-                        setting_value('visiosoft.field_type.media::imageCanvasH', 600),
-                        'center', false, 'fff'
-                    );
+                    });
             $mdImg = WaterMark::make($this->request->file('upload')->getRealPath())
                     ->resize(null, setting_value('visiosoft.module.advs::picture_height'),function ($constraint) {
                         $constraint->aspectRatio();
-                    })
-                    ->resizeCanvas(
-                        setting_value('visiosoft.module.advs::picture_width', 400),
-                        setting_value('visiosoft.module.advs::picture_height', 300),
-                        'center', false, 'fff'
-                    );
+                    });
+            if (setting_value('visiosoft.module.advs::add_canvas')) {
+                $fullImg->resizeCanvas(
+                    setting_value('visiosoft.field_type.media::imageCanvasW', 800),
+                    setting_value('visiosoft.field_type.media::imageCanvasH', 600),
+                    'center', false, 'fff'
+                );
+                $mdImg->resizeCanvas(
+                    setting_value('visiosoft.module.advs::picture_width', 400),
+                    setting_value('visiosoft.module.advs::picture_height', 300),
+                    'center', false, 'fff'
+                );
+            }
             foreach ([$fullImg, $mdImg] as $index => $image) {
                 if ($watermarktype == 'image') {
                     $watermarkimage_id = $settings->value('visiosoft.module.advs::watermark_image');
