Merge branch '4.1-dev' into 4.2-finder-memorytable

joomla · Jan 24, 2022 · c878272 · c878272
2 parents 0bd0413 + 33d80d2
commit c878272
Show file tree

Hide file tree

Showing 105 changed files with 309 additions and 258 deletions.
diff --git a/.drone.yml b/.drone.yml
@@ -57,13 +57,13 @@ steps:
       - php -v
       - ./libraries/vendor/bin/phpunit --testsuite Unit
 
-  - name: php81-unit
-    depends_on: [ phpcs ]
-    image: joomlaprojects/docker-images:php8.1
-    failure: ignore
-    commands:
-      - php -v
-      - ./libraries/vendor/bin/phpunit --testsuite Unit
+#  - name: php81-unit
+#    depends_on: [ phpcs ]
+#    image: joomlaprojects/docker-images:php8.1
+#    failure: ignore
+#    commands:
+#      - php -v
+#      - ./libraries/vendor/bin/phpunit --testsuite Unit
 
   - name: php72-integration
     depends_on: [ npm ]
@@ -420,6 +420,6 @@ steps:
 
 ---
 kind: signature
-hmac: daa8e5588cc88058bc57aadf35e3d9dd55eefaa9419c457ce374a0dd19eb7cda
+hmac: a6f3f2ed21261b612d052724e5f43a5dd55168f98cbaa48eeaa358816bb255df
 
 ...
diff --git a/.github/workflows/create-translation-pull-request-v4.yml b/.github/workflows/create-translation-pull-request-v4.yml
@@ -32,7 +32,7 @@ jobs:
           git remote add upstream https://github.com/joomla/joomla-cms.git
           git fetch upstream
           git checkout --progress --force -B translation refs/remotes/origin/translation          
-          git merge upstream/4.0-dev
+          git merge upstream/4.1-dev
 
       - name: Fetch and extract translations
         run: |
@@ -67,4 +67,4 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.API_TOKEN_GITHUB }}      
         run: |
           gh pr list -R joomla/joomla-cms --state open --author joomla-translation-bot -S "Translation Update" | grep -v "No pull" || \
-            gh pr create --title "Translation Update" --body "Automatically created pull request based on core-translation repository changes" -R joomla/joomla-cms --base 4.0-dev
+            gh pr create --title "Translation Update" --body "Automatically created pull request based on core-translation repository changes" -R joomla/joomla-cms --base 4.1-dev
diff --git a/administrator/components/com_associations/src/Helper/AssociationsHelper.php b/administrator/components/com_associations/src/Helper/AssociationsHelper.php
@@ -106,7 +106,7 @@ public static function getItem($extensionName, $typeName, $itemId)
 	{
 		if (!self::hasSupport($extensionName))
 		{
-			return array();
+			return null;
 		}
 
 		// Get the extension specific helper method
@@ -176,7 +176,7 @@ private static function loadHelper($extensionName)
 	 *
 	 * @param   string  $extensionName  The extension name with com_
 	 *
-	 * @return  boolean
+	 * @return  string
 	 *
 	 * @since  3.7.0
 	 */

diff --git a/administrator/components/com_config/src/View/Component/HtmlView.php b/administrator/components/com_config/src/View/Component/HtmlView.php
@@ -66,7 +66,7 @@ public function display($tpl = null)
 
 			if (!$component->enabled)
 			{
-				return false;
+				return;
 			}
 
 			$form = $this->get('form');
@@ -76,7 +76,7 @@ public function display($tpl = null)
 		{
 			Factory::getApplication()->enqueueMessage($e->getMessage(), 'error');
 
-			return false;
+			return;
 		}
 
 		$this->fieldsets   = $form ? $form->getFieldsets() : null;

diff --git a/administrator/components/com_content/forms/article.xml b/administrator/components/com_content/forms/article.xml
@@ -920,11 +920,13 @@
 		<field
 			name="introtext"
 			label="COM_CONTENT_FIELD_INTROTEXT"
+			filter="JComponentHelper::filterText"
 		/>
 
 		<field
 			name="fulltext"
 			label="COM_CONTENT_FIELD_FULLTEXT"
+			filter="JComponentHelper::filterText"
 		/>
 	</fields>
 

diff --git a/administrator/components/com_fields/src/Plugin/FieldsPlugin.php b/administrator/components/com_fields/src/Plugin/FieldsPlugin.php
@@ -148,11 +148,6 @@ public function onCustomFieldsPrepareField($context, $item, $field)
 		// Get the path for the layout file
 		$path = PluginHelper::getLayoutPath('fields', $this->_name, $field->type);
 
-		if (!file_exists($path))
-		{
-			$path = PluginHelper::getLayoutPath('fields', $this->_name, $field->type);
-		}
-
 		// Render the layout
 		ob_start();
 		include $path;

diff --git a/administrator/components/com_finder/src/Field/ContentmapField.php b/administrator/components/com_finder/src/Field/ContentmapField.php
@@ -79,7 +79,7 @@ protected function getGroups()
 		}
 		catch (\RuntimeException $e)
 		{
-			return;
+			return [];
 		}
 
 		// Build the grouped list array.

diff --git a/administrator/components/com_finder/src/Indexer/Query.php b/administrator/components/com_finder/src/Indexer/Query.php
@@ -1054,7 +1054,6 @@ protected function processString($input, $lang, $mode)
 
 					// Adjust the loop.
 					$i += 2;
-					continue;
 				}
 				// Handle the OR operator.
 				elseif ($op === 'OR' && isset($terms[$i + 2]))
@@ -1124,7 +1123,6 @@ protected function processString($input, $lang, $mode)
 
 					// Adjust the loop.
 					$i += 2;
-					continue;
 				}
 			}
 			// Handle an orphaned OR operator.
@@ -1172,7 +1170,6 @@ protected function processString($input, $lang, $mode)
 
 				// Adjust the loop.
 				$i++;
-				continue;
 			}
 			// Handle the NOT operator.
 			elseif (isset($terms[$i + 1]) && array_search($terms[$i], $operators, true) === 'NOT')

diff --git a/administrator/components/com_finder/tmpl/maps/default.php b/administrator/components/com_finder/tmpl/maps/default.php
@@ -112,8 +112,13 @@
 							<?php if (!$branchFilter) : ?>
 							<td class="text-center btns itemnumber">
 							<?php if ($item->rgt - $item->lft > 1) : ?>
-								<a href="<?php echo Route::_('index.php?option=com_finder&view=maps&filter[branch]=' . $item->id); ?>">
-									<span class="btn btn-info"><?php echo floor(($item->rgt - $item->lft) / 2); ?></span></a>
+								<a href="<?php echo Route::_('index.php?option=com_finder&view=maps&filter[branch]=' . $item->id); ?>"
+									aria-describedby="tip-map<?php echo $i; ?>">
+									<span class="btn btn-info"><?php echo floor(($item->rgt - $item->lft) / 2); ?></span>
+								</a>
+								<div role="tooltip" id="tip-map<?php echo $i; ?>">
+									<?php echo Text::_('COM_FINDER_HEADING_CHILDREN'); ?>
+								</div>
 							<?php else : ?>
 								-
 							<?php endif; ?>

diff --git a/administrator/components/com_languages/src/Model/InstalledModel.php b/administrator/components/com_languages/src/Model/InstalledModel.php
@@ -239,7 +239,6 @@ public function getData()
 					&& stripos($installedLanguage->language, $search) === false)
 				{
 					unset($installedLanguages[$key]);
-					continue;
 				}
 			}
 		}

diff --git a/administrator/components/com_mails/forms/template.xml b/administrator/components/com_mails/forms/template.xml
@@ -28,7 +28,8 @@
 			name="htmlbody"
 			type="editor"
 			label="COM_MAILS_FIELD_HTMLBODY_LABEL"
-			buttons="false"
+			buttons="true"
+			hide="fields,pagebreak,readmore,module"
 			class="inputbox"
 			filter="JComponentHelper::filterText"
 		/>

diff --git a/administrator/components/com_media/resources/scripts/components/browser/items/image.vue b/administrator/components/com_media/resources/scripts/components/browser/items/image.vue
@@ -4,15 +4,17 @@
     @dblclick="openPreview()"
     @mouseleave="hideActions()"
   >
-    <div class="media-browser-item-preview">
+    <div class="media-browser-item-preview"
+    :title="item.name">
       <div class="image-background">
         <div
           class="image-cropped"
           :style="{ backgroundImage: getHashedURL }"
         />
       </div>
     </div>
-    <div class="media-browser-item-info">
+    <div class="media-browser-item-info"
+    :title="item.name">
       {{ item.name }} {{ item.filetype }}
     </div>
     <span

diff --git a/administrator/components/com_media/src/Adapter/AdapterInterface.php b/administrator/components/com_media/src/Adapter/AdapterInterface.php
@@ -104,7 +104,7 @@ public function createFolder(string $name, string $path): string;
 	 *
 	 * @param   string  $name  The name
 	 * @param   string  $path  The folder
-	 * @param   binary  $data  The data
+	 * @param   string  $data  The data
 	 *
 	 * @return  string
 	 *
@@ -118,7 +118,7 @@ public function createFile(string $name, string $path, $data): string;
 	 *
 	 * @param   string  $name  The name
 	 * @param   string  $path  The folder
-	 * @param   binary  $data  The data
+	 * @param   string  $data  The data
 	 *
 	 * @return  void
 	 *

diff --git a/administrator/components/com_media/src/Controller/DisplayController.php b/administrator/components/com_media/src/Controller/DisplayController.php
@@ -36,7 +36,7 @@ class DisplayController extends BaseController
 	 * @param   string  $prefix  The class prefix. Optional.
 	 * @param   array   $config  Configuration array for view. Optional.
 	 *
-	 * @return  \Joomla\CMS\MVC\View\AbstractView  Reference to the view or an error.
+	 * @return  \Joomla\CMS\MVC\View\ViewInterface  Reference to the view or an error.
 	 *
 	 * @since   3.0
 	 * @throws  \Exception

diff --git a/administrator/components/com_media/src/Model/ApiModel.php b/administrator/components/com_media/src/Model/ApiModel.php
@@ -220,7 +220,7 @@ public function createFolder($adapter, $name, $path, $override)
 	 * @param   string   $adapter   The adapter
 	 * @param   string   $name      The name
 	 * @param   string   $path      The folder
-	 * @param   binary   $data      The data
+	 * @param   string   $data      The data
 	 * @param   boolean  $override  Should the file being overridden when it exists
 	 *
 	 * @return  string
@@ -286,7 +286,7 @@ public function createFile($adapter, $name, $path, $data, $override)
 	 * @param   string  $adapter  The adapter
 	 * @param   string  $name     The name
 	 * @param   string  $path     The folder
-	 * @param   binary  $data     The data
+	 * @param   string  $data     The data
 	 *
 	 * @return  void
 	 *

diff --git a/administrator/components/com_menus/src/Field/MenuOrderingField.php b/administrator/components/com_menus/src/Field/MenuOrderingField.php
@@ -35,7 +35,7 @@ class MenuOrderingField extends ListField
 	 * Method to get the list of siblings in a menu.
 	 * The method requires that parent be set.
 	 *
-	 * @return  array  The field option objects or false if the parent field has not been set
+	 * @return  array|boolean  The field option objects or false if the parent field has not been set
 	 *
 	 * @since   1.7
 	 */

diff --git a/administrator/components/com_menus/src/Helper/MenusHelper.php b/administrator/components/com_menus/src/Helper/MenusHelper.php
@@ -136,7 +136,7 @@ public static function getMenuTypes($clientId = 0)
 	 * @param   array    $languages  Optional array of specify which languages we want to filter
 	 * @param   int      $clientId   Optional client id - viz 0 = site, 1 = administrator, can be NULL for all (used only if menutype not given)
 	 *
-	 * @return  array
+	 * @return  array|boolean
 	 *
 	 * @since   1.6
 	 */

diff --git a/administrator/components/com_modules/tmpl/module/edit.php b/administrator/components/com_modules/tmpl/module/edit.php
@@ -107,7 +107,7 @@
 							<p><?php echo $short_description; ?></p>
 							<?php if ($long_description) : ?>
 								<p class="readmore">
-									<a href="#" onclick="document.querySelector('#tab-description').click();">
+									<a href="#" onclick="document.getElementById('myTab').activateTab(document.getElementById('description'));">
 										<?php echo Text::_('JGLOBAL_SHOW_FULL_DESCRIPTION'); ?>
 									</a>
 								</p>

diff --git a/administrator/modules/mod_sampledata/tmpl/default.php b/administrator/modules/mod_sampledata/tmpl/default.php
@@ -32,7 +32,7 @@
 		<?php foreach($items as $i => $item) : ?>
 			<li class="list-group-item sampledata-<?php echo $item->name; ?>">
 				<div class="d-flex justify-content-between align-items-center">
-					<div class="sample-data__title me-2">
+					<div class="sample-data__title">
 						<span class="sample-data__icon icon-<?php echo $item->icon; ?> me-1" aria-hidden="true"></span>
 						<?php echo htmlspecialchars($item->title, ENT_QUOTES, 'UTF-8'); ?>
 					</div>

diff --git a/build/media_source/plg_installer_webinstaller/scss/client.scss b/build/media_source/plg_installer_webinstaller/scss/client.scss
@@ -20,6 +20,19 @@ $extension-types: (
   // Extension type badges
   .badge {
     margin: 1px;
+    // BS5 Dropped all .badge-* color classes
+    &.badge-success {
+      background-color: var(--success);
+    }
+    &.badge-secondary {
+      background-color: var(--secondary);
+    }
+    &.badge-danger {
+      background-color: var(--danger);
+    }
+    &.badge-warning {
+      background-color: var(--warning);
+    }
   }
 
   .fader {
@@ -57,7 +70,7 @@ $extension-types: (
       margin-bottom: 2rem;
 
       &:hover {
-        background-color: $white;
+        background-color: var(--gray-200);
       }
     }
 
@@ -111,58 +124,23 @@ $extension-types: (
   .form-row {
     display: flex;
     .col-md:first-child {
-      margin-right: .8rem;
+      margin-inline-end: .8rem;
     }
 
     .input-group {
-      > input#com-apps-searchbox {
-        border-top-left-radius: .25rem;
-        border-bottom-left-radius: .25rem;
-      }
-
       .input-group-append {
         button#search-extensions {
-          color: var(--template-text-light);
-          background-color: var(--template-link-color);
-          border-color: var(--template-link-color);
-          border-top-left-radius: 0;
-          border-bottom-left-radius: 0;
+          border-start-start-radius: 0;
+          border-end-start-radius: 0;
         }
 
         button#search-reset {
           margin-inline-start: .8rem;
-          background-color: var(--template-bg-dark);
-          border-radius: .25rem;
-        }
-      }
-    }
-  }
-}
-
-/* stylelint-disable selector-max-compound-selectors */
-html[dir=rtl] .com-apps-container {
-  .form-row {
-    .col-md:first-child {
-      margin-left: .8rem;
-    }
-    .input-group {
-      > input#com-apps-searchbox {
-        border-top-right-radius: .25rem;
-        border-bottom-right-radius: .25rem;
-      }
-
-      .input-group-append {
-        button#search-extensions {
-          border-top-left-radius: .25rem;
-          border-top-right-radius: 0;
-          border-bottom-right-radius: 0;
-          border-bottom-left-radius: .25rem;
         }
       }
     }
   }
 }
-/* stylelint-enable selector-max-compound-selectors */
 
 // Disallow scrolling when overlay is active
 .ifw-busy {

diff --git a/components/com_content/src/View/Category/HtmlView.php b/components/com_content/src/View/Category/HtmlView.php
@@ -140,10 +140,6 @@ public function display($tpl = null)
 				{
 					$this->link_items[] = $item;
 				}
-				else
-				{
-					continue;
-				}
 			}
 		}
 

diff --git a/components/com_users/src/Controller/DisplayController.php b/components/com_users/src/Controller/DisplayController.php
@@ -29,7 +29,7 @@ class DisplayController extends BaseController
 	 * @param   array|boolean  $urlparams  An array of safe URL parameters and their variable types,
 	 *                                     for valid values see {@link \Joomla\CMS\Filter\InputFilter::clean()}.
 	 *
-	 * @return  static  This object to support chaining.
+	 * @return  void
 	 *
 	 * @since   1.5
 	 * @throws  \Exception

diff --git a/components/com_users/src/Controller/ProfileController.php b/components/com_users/src/Controller/ProfileController.php
@@ -72,7 +72,7 @@ public function edit()
 	/**
 	 * Method to save a user's profile data.
 	 *
-	 * @return  void
+	 * @return  void|boolean
 	 *
 	 * @since   1.6
 	 * @throws  \Exception