Merge branch 'develop' into 4.8

CHANGELOG.md

@@ -1,5 +1,21 @@
 # Release Notes for Craft CMS 4
 
+## Unreleased
+
+- It’s now possible to select the temp asset volume within Assets fields, if the temp upload location includes a subpath. ([#14246](https://github.com/craftcms/cms/pull/14246))
+- Fixed a bug where it wasn’t possible to set the “Formatting Locale” user preference back to “Same as language” once another value had been selected.
+- Fixed a bug where layout components provided by disabled plugins weren’t getting omitted. ([#14236](https://github.com/craftcms/cms/pull/14236))
+- Fixed a bug where “Move to the left” and “Move to the right” actions within field layout designers weren’t always getting enabled when they should, if a new tab was added.
+- Fixed a bug where “Move to the left” and “Move to the right” actions within field layout designers were labelled incorrectly for right-to-left languages.
+
+## 4.7.1 - 2024-01-29
+
+- Unpublished drafts no longer show “Created at” or “Updated at” metadata values. ([#14204](https://github.com/craftcms/cms/issues/14204))
+- Fixed a bug where empty Dropdown fields were getting treated as dirty when unchanged.
+- Fixed a bug where Recent Entries widgets were getting mangled when new entries were created via Quick Post widgets.
+- Fixed an error that occurred when adding a Dropdown field condition rule, if the field contained any optgroups. ([#14224](https://github.com/craftcms/cms/issues/14224))
+- Fixed a bug where Dropdown field condition rules weren’t displaying `0` options. ([#14232](https://github.com/craftcms/cms/pull/14232))
+
 ## 4.7.0 - 2024-01-23
 
 > [!NOTE]  

packages/craftcms-vue/admintable/App.vue

@@ -104,21 +104,19 @@
             </template>
             <template slot="title" slot-scope="props">
               <span
-                v-if="props.rowData.status !== undefined"
+                v-if="props.rowData.status"
                 class="status"
                 :class="{enabled: props.rowData.status}"
               ></span>
               <a
-                :class="{'cell-bold': props.rowData.status === undefined}"
+                :class="{'cell-bold': !props.rowData.status}"
                 v-if="props.rowData.url"
                 :href="props.rowData.url"
                 >{{ props.rowData.title }}</a
               >
-              <span
-                :class="{'cell-bold': props.rowData.status === undefined}"
-                v-else
-                >{{ props.rowData.title }}</span
-              >
+              <span :class="{'cell-bold': !props.rowData.status}" v-else>{{
+                props.rowData.title
+              }}</span>
             </template>
 
             <template slot="handle" slot-scope="props">
@@ -134,7 +132,7 @@
                   }}<template
                     v-if="
                       props.rowData.menu.showCount ||
-                      props.rowData.menu.showCount === undefined
+                      typeof props.rowData.menu.showCount === 'undefined'
                     "
                   >
                     ({{ props.rowData.menu.items.length }})</template
@@ -172,8 +170,7 @@
                 @click="handleDetailRow(props.rowData.id)"
                 v-if="
                   props.rowData.detail.content &&
-                  (!props.rowData.detail.handle ||
-                    props.rowData.detail.handle == undefined) &&
+                  !props.rowData.detail.handle &&
                   (Object.keys(props.rowData.detail.content).length ||
                     props.rowData.detail.content.length)
                 "
@@ -202,7 +199,7 @@
                 v-on:finishloading="loading(false)"
                 v-on:reload="remove(props.rowIndex, props.rowData.id)"
                 v-if="
-                  props.rowData._showDelete == undefined ||
+                  typeof props.rowData._showDelete === 'undefined' ||
                   props.rowData._showDelete == true
                 "
               ></admin-table-delete-button>
@@ -830,8 +827,8 @@
 
       canReorder() {
         if (
-          this.$refs.vuetable == undefined ||
-          this.$refs.vuetable.tableData == undefined
+          typeof this.$refs.vuetable === 'undefined' ||
+          typeof this.$refs.vuetable.tableData === 'undefined'
         ) {
           return false;
         }

packages/craftcms-vue/admintable/components/AdminTableActionButton.vue

@@ -41,7 +41,7 @@
               :class="{
                 ...(act.class ? act.class : {}),
                 ...{
-                  error: act.error !== undefined && act.error,
+                  error: act.error,
                   disabled:
                     act.allowMultiple !== undefined &&
                     !act.allowMultiple &&
@@ -53,7 +53,7 @@
               :data-ajax="act.ajax"
               @click.prevent="
                 !(
-                  act.allowMultiple !== undefined &&
+                  typeof act.allowMultiple !== 'undefined' &&
                   !act.allowMultiple &&
                   hasMultipleSelected
                 )
@@ -136,7 +136,7 @@
         this.$emit('click', param, value, action, ajax);
 
         // Is the action button the one to deal with the click?
-        if (handleClick !== undefined && !handleClick) {
+        if (typeof handleClick !== 'undefined' && !handleClick) {
           return;
         }
 

packages/craftcms-vue/admintable/components/AdminTableDeleteButton.vue

@@ -35,26 +35,23 @@
 
     computed: {
       success() {
-        var successMessage =
-          this.successMessage !== undefined
-            ? Craft.t('site', this.successMessage, {name: this.name})
-            : Craft.t('app', '“{name}” deleted.', {name: this.name});
+        var successMessage = this.successMessage
+          ? Craft.t('site', this.successMessage, {name: this.name})
+          : Craft.t('app', '“{name}” deleted.', {name: this.name});
         return Craft.escapeHtml(successMessage);
       },
       confirm() {
-        var confirmationMessage =
-          this.confirmationMessage !== undefined
-            ? Craft.t('site', this.confirmationMessage, {name: this.name})
-            : Craft.t('app', 'Are you sure you want to delete “{name}”?', {
-                name: this.name,
-              });
+        var confirmationMessage = this.confirmationMessage
+          ? Craft.t('site', this.confirmationMessage, {name: this.name})
+          : Craft.t('app', 'Are you sure you want to delete “{name}”?', {
+              name: this.name,
+            });
         return Craft.escapeHtml(confirmationMessage);
       },
       failed() {
-        var failMessage =
-          this.failMessage !== undefined
-            ? Craft.t('site', this.failMessage, {name: this.name})
-            : Craft.t('app', 'Couldn’t delete “{name}”.', {name: this.name});
+        var failMessage = this.failMessage
+          ? Craft.t('site', this.failMessage, {name: this.name})
+          : Craft.t('app', 'Couldn’t delete “{name}”.', {name: this.name});
         return Craft.escapeHtml(failMessage);
       },
     },

packages/craftcms-vue/admintable/components/AdminTableDetailRow.vue

@@ -1,10 +1,7 @@
 <template>
   <div>
     <div
-      v-if="
-        rowData.detail.content &&
-        (!rowData.detail.showAsList || rowData.detail.showAsList === undefined)
-      "
+      v-if="rowData.detail.content && !rowData.detail.showAsList"
       v-html="rowData.detail.content"
     ></div>
     <div v-if="rowData.detail.content && rowData.detail.showAsList">

src/base/Element.php

@@ -5082,10 +5082,10 @@ public function getMetadata(): array
                 return $icon . Html::tag('span', $label);
             },
         ], $event->metadata, [
-            Craft::t('app', 'Created at') => $this->dateCreated
+            Craft::t('app', 'Created at') => $this->dateCreated && !$this->getIsUnpublishedDraft()
                 ? $formatter->asDatetime($this->dateCreated, Formatter::FORMAT_WIDTH_SHORT)
                 : false,
-            Craft::t('app', 'Updated at') => $this->dateUpdated
+            Craft::t('app', 'Updated at') => $this->dateUpdated && !$this->getIsUnpublishedDraft()
                 ? $formatter->asDatetime($this->dateUpdated, Formatter::FORMAT_WIDTH_SHORT)
                 : false,
             Craft::t('app', 'Notes') => function() {

src/config/app.php

@@ -3,7 +3,7 @@
 return [
     'id' => 'CraftCMS',
     'name' => 'Craft CMS',
-    'version' => '4.7.0',
+    'version' => '4.7.1',
     'schemaVersion' => '4.5.3.0',
     'minVersionRequired' => '3.7.11',
     'basePath' => dirname(__DIR__), // Defines the @app alias

src/console/controllers/SetupController.php

@@ -609,7 +609,7 @@ public function actionCloud(): int
         $io = new ConsoleIO($input, $output, new HelperSet([new QuestionHelper()]));
 
         Craft::$app->getComposer()->install([
-            'craftcms/cloud' => '^1.0.0',
+            'craftcms/cloud' => '*',
         ], $io);
 
         $message = sprintf('Extension %s', $moduleInstalled ? 'updated' : 'installed');

src/controllers/ElementsController.php

@@ -1686,9 +1686,7 @@ public function actionUpdateFieldLayout(): ?Response
             'initialDeltaValues' => Craft::$app->getView()->getInitialDeltaValues(),
         ];
 
-        return $this->_asSuccess(Craft::t('app', '{type} saved.', [
-            'type' => Craft::t('app', 'Draft'),
-        ]), $element, $data, true);
+        return $this->_asSuccess('Field layout updated.', $element, $data, true);
     }
 
     private function _fieldLayoutData(ElementInterface $element): array

src/controllers/UsersController.php

@@ -1143,7 +1143,7 @@ public function actionEditUser(mixed $userId = null, ?User $user = null, ?array
             ArrayHelper::multisort($allLocales, fn(Locale $locale) => $locale->getDisplayName());
 
             $localeOptions = [
-                ['label' => Craft::t('app', 'Same as language'), 'value' => ''],
+                ['label' => Craft::t('app', 'Same as language'), 'value' => '__blank__'],
             ];
             array_push($localeOptions, ...array_map(fn(Locale $locale) => [
                 'label' => $locale->getDisplayName(Craft::$app->language),
@@ -1484,9 +1484,13 @@ public function actionSaveUser(): ?Response
 
         if ($isCurrentUser) {
             // Save their preferences too
+            $preferredLocale = $this->request->getBodyParam('preferredLocale', $user->getPreference('locale')) ?: null;
+            if ($preferredLocale === '__blank__') {
+                $preferredLocale = null;
+            }
             $preferences = [
                 'language' => $this->request->getBodyParam('preferredLanguage', $user->getPreference('language')),
-                'locale' => $this->request->getBodyParam('preferredLocale', $user->getPreference('locale')) ?: null,
+                'locale' => $preferredLocale,
                 'weekStartDay' => $this->request->getBodyParam('weekStartDay', $user->getPreference('weekStartDay')),
                 'alwaysShowFocusRings' => (bool)$this->request->getBodyParam('alwaysShowFocusRings', $user->getPreference('alwaysShowFocusRings')),
                 'useShapes' => (bool)$this->request->getBodyParam('useShapes', $user->getPreference('useShapes')),

src/fields/Assets.php

@@ -239,7 +239,14 @@ protected function defineRules(): array
         ];
 
         $rules[] = [
-            ['sources', 'defaultUploadLocationSource', 'restrictedLocationSource'], 'validateNotTempVolume',
+            [
+                'sources',
+                'defaultUploadLocationSource',
+                'restrictedLocationSource',
+                'defaultUploadLocationSubpath',
+                'restrictedLocationSubpath',
+            ],
+            'validateNotTempVolume',
         ];
 
         $rules[] = [['previewMode'], 'in', 'range' => [self::PREVIEW_MODE_FULL, self::PREVIEW_MODE_THUMBS], 'skipOnEmpty' => false];
@@ -255,18 +262,32 @@ protected function defineRules(): array
      */
     public function validateNotTempVolume(string $attribute): void
     {
-        [$tempVolume] = Craft::$app->getAssets()->getTempVolumeAndSubpath();
+        [$tempVolume, $tempSubpath] = Craft::$app->getAssets()->getTempVolumeAndSubpath();
         if ($tempVolume !== null) {
             $tempVolumeKey = "volume:$tempVolume->uid";
             $inputSources = $this->getInputSources();
 
-            if (
-                (in_array($attribute, ['source', 'sources']) && in_array($tempVolumeKey, $inputSources)) ||
-                ($attribute == 'defaultUploadLocationSource' && $this->defaultUploadLocationSource === $tempVolumeKey) ||
-                ($attribute == 'restrictedLocationSource' && $this->restrictedLocationSource === $tempVolumeKey)
-            ) {
-                // intentionally not translating this since it's short-lived (>= 4.7, < 5.0) and dev-facing only.
-                $this->addError($attribute, "Volume “{$tempVolume->name}” is used to store temporary asset uploads, so it cannot be used in an Assets field.");
+            if (empty($tempSubpath)) {
+                if (
+                    (in_array($attribute, ['source', 'sources']) && in_array($tempVolumeKey, $inputSources)) ||
+                    ($attribute == 'defaultUploadLocationSource' && $this->defaultUploadLocationSource === $tempVolumeKey) ||
+                    ($attribute == 'restrictedLocationSource' && $this->restrictedLocationSource === $tempVolumeKey)
+                ) {
+                    // intentionally not translating this since it's short-lived (>= 4.7, < 5.0) and dev-facing only.
+                    $this->addError($attribute, "Temporary asset uploads are being stored in $tempVolume->name, so the same volume cannot be used by an Assets field.");
+                }
+            } else {
+                if (
+                    ($attribute == 'defaultUploadLocationSource' &&
+                    $this->defaultUploadLocationSource === $tempVolumeKey &&
+                    $this->defaultUploadLocationSubpath == $tempSubpath) ||
+                    ($attribute == 'restrictedLocationSource' &&
+                    $this->restrictedLocationSource === $tempVolumeKey &&
+                    $this->restrictedLocationSubpath == $tempSubpath)
+                ) {
+                    // intentionally not translating this since it's short-lived (>= 4.7, < 5.0) and dev-facing only.
+                    $this->addError($attribute, "Temporary asset uploads are being stored in $tempVolume->name/$tempSubpath, so the same location cannot be used by an Assets field.");
+                }
             }
         }
     }
@@ -278,15 +299,15 @@ public function getSourceOptions(): array
     {
         $sourceOptions = [];
         /** @var Volume|null $tempVolume */
-        [$tempVolume] = Craft::$app->getAssets()->getTempVolumeAndSubpath();
+        [$tempVolume, $tempSubpath] = Craft::$app->getAssets()->getTempVolumeAndSubpath();
         if ($tempVolume) {
             $tempVolumeKey = 'volume:' . $tempVolume->uid;
         } else {
             $tempVolumeKey = null;
         }
 
         foreach (Asset::sources('settings') as $volume) {
-            if ($tempVolumeKey !== null && $volume['key'] === $tempVolumeKey) {
+            if ($tempVolumeKey !== null && $volume['key'] === $tempVolumeKey && empty($tempSubpath)) {
                 // only allow it if already selected
                 if (
                     (!is_array($this->sources) || !in_array($tempVolumeKey, $this->sources)) &&
@@ -715,13 +736,13 @@ public function getInputSources(?ElementInterface $element = null): array|string
             $sources = array_merge($this->sources);
         } else {
             $sources = [];
-            [$tempVolume] = Craft::$app->getAssets()->getTempVolumeAndSubpath();
+            [$tempVolume, $tempSubpath] = Craft::$app->getAssets()->getTempVolumeAndSubpath();
             $tempVolumeKey = $tempVolume ? "volume:$tempVolume->uid" : null;
 
             foreach (Craft::$app->getElementSources()->getSources(Asset::class) as $source) {
                 if (
                     $source['type'] !== ElementSources::TYPE_HEADING &&
-                    $source['key'] !== $tempVolumeKey
+                    ($source['key'] !== $tempVolumeKey || !empty($tempSubpath))
                 ) {
                     $sources[] = $source['key'];
                 }

src/fields/BaseOptionsField.php

@@ -323,7 +323,7 @@ public function normalizeValue(mixed $value, ?ElementInterface $element = null):
             $value = Json::decodeIfJson($value);
         } elseif ($value === '' && $this->multi) {
             $value = [];
-        } elseif ($value === '__BLANK__') {
+        } elseif (strtolower($value) === '__blank__') {
             $value = '';
         } elseif ($value === null && $this->isFresh($element)) {
             $value = $this->defaultValue();

src/fields/Country.php

@@ -43,7 +43,7 @@ public static function valueType(): string
      */
     public function normalizeValue(mixed $value, ElementInterface $element = null): mixed
     {
-        return !in_array($value, ['', '__BLANK__']) ? $value : null;
+        return !in_array(strtolower($value), ['', '__blank__']) ? $value : null;
     }
 
     /**
@@ -52,7 +52,7 @@ public function normalizeValue(mixed $value, ElementInterface $element = null):
     protected function inputHtml(mixed $value, ?ElementInterface $element = null): string
     {
         $options = Craft::$app->getAddresses()->getCountryRepository()->getList(Craft::$app->language);
-        array_unshift($options, ['label' => '', 'value' => '__BLANK__']);
+        array_unshift($options, ['label' => ' ', 'value' => '__blank__']);
 
         return Cp::selectizeHtml([
             'id' => $this->getInputId(),