Merge pull request #2117 from yama/1.4.x-refactor

merged from ce by cherry pick
evolution-cms · Jan 22, 2023 · 252d3f9 · 252d3f9
2 parents 9bb77ed + 84a7faf
commit 252d3f9
Show file tree

Hide file tree

Showing 328 changed files with 42,224 additions and 12,606 deletions.
diff --git a/.editorconfig b/.editorconfig
@@ -0,0 +1,26 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+indent_style = space
+tab_width = 4
+
+[**.js]
+tab_width = 4
+
+[**.coffee]
+tab_width = 4
+
+[**.cson]
+tab_width = 4
+
+[**.json]
+tab_width = 4
+
+[**rc]
+tab_width = 4
+
+[**php]
+tab_width = 4
diff --git a/.gitattributes b/.gitattributes
@@ -1,3 +1,4 @@
 .gitattributes export-ignore
 .gitignore     export-ignore
 .tx export-ignore
+.github export-ignore
diff --git a/.gitignore b/.gitignore
@@ -11,6 +11,7 @@
 !/assets/cache/.htaccess
 !/assets/cache/index.html
 !/assets/cache/siteManager.php
+!/assets/cache/sitePublishing.idx.php
 !/assets/cache/images/index.html
 !/assets/cache/rss/index.html
 !/assets/cache/updater/index.html

diff --git a/assets/.thumbs/.htaccess b/assets/.thumbs/.htaccess
@@ -1,5 +1,10 @@
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
 </Files>
diff --git a/assets/backup/.htaccess b/assets/backup/.htaccess
@@ -1,2 +1,8 @@
-order deny,allow
-deny from all
+<ifModule mod_authz_core.c>
+    Require all denied
+</ifModule>
+<ifModule !mod_authz_core.c>
+    order deny,allow
+    deny from all
+</ifModule>
+
diff --git a/assets/cache/.htaccess b/assets/cache/.htaccess
@@ -0,0 +1,7 @@
+<ifModule mod_authz_core.c>
+    Require all denied
+</ifModule>
+<ifModule !mod_authz_core.c>
+    order deny,allow
+    deny from all
+</ifModule>
diff --git a/assets/cache/images/.htaccess b/assets/cache/images/.htaccess
@@ -1,2 +1,7 @@
-order deny,allow
-allow from all
+<ifModule mod_authz_core.c>
+    Require all granted
+</ifModule>
+<ifModule !mod_authz_core.c>
+    order deny,allow
+    allow from all
+</ifModule>
diff --git a/assets/cache/images/ht.access b/assets/cache/images/ht.access
@@ -1,2 +1,7 @@
-order deny,allow
-allow from all
+<ifModule mod_authz_core.c>
+    Require all granted
+</ifModule>
+<ifModule !mod_authz_core.c>
+    order deny,allow
+    allow from all
+</ifModule>
diff --git a/assets/cache/sitePublishing.idx.php b/assets/cache/sitePublishing.idx.php
@@ -0,0 +1 @@
+<?php $cacheRefreshTime=0; ?>
diff --git a/assets/export/.htaccess b/assets/export/.htaccess
@@ -1,5 +1,10 @@
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
 </Files>
diff --git a/assets/files/.htaccess b/assets/files/.htaccess
@@ -1,5 +1,10 @@
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
 </Files>
diff --git a/assets/images/.htaccess b/assets/images/.htaccess
@@ -1,5 +1,10 @@
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
 </Files>
diff --git a/assets/import/.htaccess b/assets/import/.htaccess
@@ -1,5 +1,10 @@
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
 </Files>
diff --git a/assets/js/.htaccess b/assets/js/.htaccess
@@ -1,5 +1,11 @@
+
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
-</Files>
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
+</Files>
diff --git a/assets/js/euiuploader/css/euiuploader.css b/assets/js/euiuploader/css/euiuploader.css
@@ -29,6 +29,10 @@
 }
 #euiUploadState .sgrow3 {
     text-align: center;
+    overflow: visible;
+    display: block;
+    background: none;
+    height: auto;
 }
 #euiUploadState .progress .complete {
     color:green;

diff --git a/assets/lib/.htaccess b/assets/lib/.htaccess
@@ -1,5 +1,11 @@
+
 IndexIgnore */*
 <Files *.php>
-    Order Deny,Allow
-    Deny from all
-</Files>
+    <ifModule mod_authz_core.c>
+        Require all denied
+    </ifModule>
+    <ifModule !mod_authz_core.c>
+        order deny,allow
+        deny from all
+    </ifModule>
+</Files>
diff --git a/assets/lib/Helpers/Collection.php b/assets/lib/Helpers/Collection.php
@@ -33,6 +33,7 @@ public function create(array $data = array())
     /**
      * @return \ArrayIterator
      */
+    #[\ReturnTypeWillChange]
     public function getIterator()
     {
         return new \ArrayIterator($this->data);
@@ -149,6 +150,7 @@ public function add($data, $id = null)
     /**
      * @return int
      */
+    #[\ReturnTypeWillChange]
     public function count()
     {
         return count($this->data);
@@ -259,6 +261,7 @@ public function removeElement($element)
      * @param mixed $offset
      * @return bool
      */
+    #[\ReturnTypeWillChange]
     public function offsetExists($offset)
     {
         return $this->containsKey($offset);
@@ -268,6 +271,7 @@ public function offsetExists($offset)
      * @param mixed $offset
      * @return mixed|null
      */
+    #[\ReturnTypeWillChange]
     public function offsetGet($offset)
     {
         return $this->get($offset);
@@ -278,6 +282,7 @@ public function offsetGet($offset)
      * @param mixed $value
      * @return Collection
      */
+    #[\ReturnTypeWillChange]
     public function offsetSet($offset, $value)
     {
         if ($offset !== null) {
@@ -293,6 +298,7 @@ public function offsetSet($offset, $value)
      * @param mixed $offset
      * @return mixed|null
      */
+    #[\ReturnTypeWillChange]
     public function offsetUnset($offset)
     {
         return $this->remove($offset);

diff --git a/assets/lib/Helpers/Mailer.php b/assets/lib/Helpers/Mailer.php
@@ -252,4 +252,12 @@ public function getCFGDef($param, $default = null)
     {
         return \APIhelpers::getkey($this->config, $param, $default);
     }
+
+    /**
+     * @return MODxMailer
+     */
+    public function getMailProcessor()
+    {
+        return $this->mail;
+    }
 }
diff --git a/assets/lib/Helpers/PHPThumb.php b/assets/lib/Helpers/PHPThumb.php
@@ -26,6 +26,10 @@ class PHPThumb
     public function __construct()
     {
         $this->thumb = new \phpthumb();
+        $defaultCacheFolder = 'assets/cache';
+        $this->thumb->config_cache_directory = MODX_BASE_PATH . $defaultCacheFolder;
+        $this->thumb->config_temp_directory = $defaultCacheFolder;
+        $this->thumb->config_document_root = MODX_BASE_PATH;
         $this->fs = FS::getInstance();
     }
 

diff --git a/assets/lib/MODxAPI/modResource.php b/assets/lib/MODxAPI/modResource.php
@@ -525,13 +525,14 @@ public function save($fire_events = false, $clearCache = false)
         }
 
         $this->set('alias', $this->getAlias());
-
+        $data = $this->toArray(null, null, null, false);
         $this->invokeEvent('OnBeforeDocFormSave', array(
             'mode'   => $this->newDoc ? "new" : "upd",
             'id'     => isset($this->id) ? $this->id : '',
-            'doc'    => $this->toArray(),
+            'doc'    => &$data,
             'docObj' => $this
         ), $fire_events);
+        $this->fromArray($data);
 
         $fld = $this->encodeFields()->toArray(null, null, null, false);
         foreach ($this->default_field as $key => $value) {
@@ -632,7 +633,7 @@ public function save($fire_events = false, $clearCache = false)
         $this->invokeEvent('OnDocFormSave', array(
             'mode'   => $this->mode,
             'id'     => isset($this->id) ? $this->id : '',
-            'doc'    => $this->toArray(),
+            'doc'    => $this->toArray(null, null, null, false),
             'docObj' => $this
         ), $fire_events);
 

diff --git a/assets/lib/SimpleTab/plugin.class.php b/assets/lib/SimpleTab/plugin.class.php
@@ -87,7 +87,7 @@ public function __construct($modx, $lang_attribute = 'en')
      * @param array $ids
      * @param $folder
      */
-    public function clearFolders($ids = array(), $folder)
+    public function clearFolders(array $ids, $folder)
     {
         foreach ($ids as $id) {
             $this->fs->rmDir($folder . $id . '/');

diff --git a/assets/lib/class.modxRTEbridge.php b/assets/lib/class.modxRTEbridge.php
@@ -51,8 +51,8 @@ public function __construct($editorKey = NULL, $bridgeConfig = array(), $tvOptio
         $this->initLang($basePath);
 
         // Get modxRTEbridge-config from child-class
-        $this->bridgeParams = isset($bridgeConfig['bridgeParams']) ? $bridgeConfig['bridgeParams'] : array();
-        $this->gSettingsCustom = isset($bridgeConfig['gSettingsCustom']) ? $bridgeConfig['gSettingsCustom'] : array();
+        $this->bridgeParams           = isset($bridgeConfig['bridgeParams']) ? $bridgeConfig['bridgeParams'] : array();
+        $this->gSettingsCustom        = isset($bridgeConfig['gSettingsCustom']) ? $bridgeConfig['gSettingsCustom'] : array();
         $this->gSettingsDefaultValues = isset($bridgeConfig['gSettingsDefaultValues']) ? $bridgeConfig['gSettingsDefaultValues'] : array();
 
         // Determine settings from Modx
@@ -149,17 +149,17 @@ public function __construct($editorKey = NULL, $bridgeConfig = array(), $tvOptio
         $this->tvOptions = $tvOptions;
 
         // Set pluginParams
-        $this->editorKey = $editorKey;
-        $this->theme = $this->modxParams['theme'] ?? 'base';
-        $this->pluginParams = $modx->event->params ?? array();
-        $this->pluginParams['pluginName'] = $modx->event->activePlugin;
-        $this->pluginParams['editorLabel'] = $bridgeConfig['editorLabel'] ?? 'No editorLabel set for "' . $editorKey . '"';
-        $this->pluginParams['editorVersion'] = $bridgeConfig['editorVersion'] ?? 'No editorVersion set';
-        $this->pluginParams['editorLogo'] = $bridgeConfig['editorLogo'] ?? '';
+        $this->editorKey                      = $editorKey;
+        $this->theme                          = $this->modxParams['theme'] ?? 'base';
+        $this->pluginParams                   = $modx->event->params ?? array();
+        $this->pluginParams['pluginName']     = $modx->event->activePlugin;
+        $this->pluginParams['editorLabel']    = $bridgeConfig['editorLabel'] ?? 'No editorLabel set for "' . $editorKey . '"';
+        $this->pluginParams['editorVersion']  = $bridgeConfig['editorVersion'] ?? 'No editorVersion set';
+        $this->pluginParams['editorLogo']     = $bridgeConfig['editorLogo'] ?? '';
         $this->pluginParams['skinsDirectory'] = isset($bridgeConfig['skinsDirectory']) && !empty($bridgeConfig['skinsDirectory']) ? trim($bridgeConfig['skinsDirectory'], "/") . "/" : '';
         $this->pluginParams['skinthemeDirectory'] = isset($bridgeConfig['skinthemeDirectory']) && !empty($bridgeConfig['skinthemeDirectory']) ? trim($bridgeConfig['skinthemeDirectory'], "/") . "/" : '';
-        $this->pluginParams['base_path'] = $basePath;
-        $this->pluginParams['base_url'] = $baseUrl;
+        $this->pluginParams['base_path']      = $basePath;
+        $this->pluginParams['base_url']       = $baseUrl;
     }
 
     // Function to set editor-parameters
@@ -170,12 +170,10 @@ public function set($key, $value, $type = false, $emptyAllowed = false)
             $this->themeConfig[$key] = NULL;    // Delete Parameter completely from JS-initialization
         } else {
             if (!isset($this->themeConfig[$key])) $this->themeConfig[$key] = array();
-            $this->themeConfig[$key]['value'] = $value;
-            $this->themeConfig[$key]['default'] = !isset($this->themeConfig[$key]['default'])
-                ? $value
-                : $this->themeConfig[$key]['default'];
-            $this->themeConfig[$key]['type'] = $type == false ? 'string' : $type;
-            $this->themeConfig[$key]['empty'] = $emptyAllowed;
+            $this->themeConfig[$key]['value']   = $value;
+            $this->themeConfig[$key]['default'] = !isset($this->themeConfig[$key]['default']) ? $value : $this->themeConfig[$key]['default'];
+            $this->themeConfig[$key]['type']    = $type == false ? 'string' : $type;
+            $this->themeConfig[$key]['empty']   = $emptyAllowed;
         }
     }
 
@@ -798,7 +796,7 @@ public function getModxSettings()
     public function parsePlaceholders($content, $ph)
     {
         foreach ($ph as $key => $value) {
-            $content = str_replace('[+' . $key . '+]', $value, $content);
+            $content = str_replace('[+' . $key . '+]', $value ?? '', $content);
         }
         return $content;
     }