craftcms / cms

Build bespoke content experiences with Craft.
https://craftcms.com
Other
3.28k stars 635 forks source link

[4.x]: User photos volume not recognized as valid #12531

Closed TheFunkyMonk closed 1 year ago

TheFunkyMonk commented 1 year ago

What happened?

Description

With a valid volume (containing an uploaded image) set in user settings, I'm still getting "Please set a valid volume for storing the user photos in user settings page first." message under my user account.

Screen-Shot-2023-01-12-at-1 40 31-PM Screen Shot 2023-01-12 at 1 40 11 PM Screen-Shot-2023-01-12-at-1 40 45-PM

This is currently a Servd assets volume, but I have also tested with a local assets volume with the same results.

Steps to reproduce

  1. Create an assets filesystem/volume
  2. Assign the volume to user photo location in user settings

Expected behavior

User photo can be chosen/uploaded in user preferences

Actual behavior

User preferences panel displays "Please set a valid volume for storing the user photos in user settings page first."

Craft CMS version

Craft Pro 4.3.6.1

PHP version

8.1.13

Operating system and version

Linux 5.15.68-0-virt

Database type and version

MySQL 10.4.27

Image driver and version

Imagick 3.7.0 (ImageMagick 6.9.11-60)

Installed plugins and versions

"craftcms/cms": "4.3.6.1",
"craftcms/redactor": "^3.0",
"doublesecretagency/craft-cpcss": "^2.6",
"mmikkel/retcon": "^2.6",
"nystudio107/craft-imageoptimize": "^4.0",
"nystudio107/craft-minify": "4.0.0-beta.2",
"nystudio107/craft-retour": "^4.1",
"nystudio107/craft-seomatic": "4.0.19",
"servd/craft-asset-storage": "^3.2",
"spicyweb/craft-neo": "^3.5",
"verbb/cloner": "^2.0",
"verbb/field-manager": "^3.0",
"vlucas/phpdotenv": "^5.4.0"
brianjhanson commented 1 year ago

Thanks for reporting! That message will appear when the filesystem for the transforms has the "Files in this filesystem have public URLs" set to false. I'm going to close this out assuming that's what's happening here, but if that's not the case, feel free to reopen.

TheFunkyMonk commented 1 year ago

Ah that was it, thanks for the clarification @brianjhanson!

brandonkelly commented 1 year ago

User photos will no longer require a filesystem that has URLs starting in 4.4, via 303e5ddb3730190916f7fc5a939d7b3cea62b91a.

brandonkelly commented 1 year ago

4.4 is out!