Ryan Cramer
Ryan Cramer
@adrianbj The user-view-* permissions there there to make the user viewable. It's not a permission I would recommend assigning to the guest user. But if it otherwise suits your purpose,...
@adrianbj I think your approach is fine if you are limiting it to your front-end users page. But it sounds like you aren't differentiating between those users and those in...
Thanks @cb2004 I've pushed an update, does this fix it?
@trk An instance of a class can have methods called, but a class name on its own can't, so isn't going to be useful here, unless maybe referring to a...
@matjazpotocnik Thanks I've pushed fixes for the items above. Though I'm not running PHP 8.5 yet so please let me know if I screwed up anything.
@adrianbj Any idea whether PDO or Pdo is the correct one? I see PHP referring to both. @matjazpotocnik I want to stick to using `version_compare` so that these PHP 8.5...
@BernhardBaumrock > So I think PW should be a little more specific about the check if PW is already installed and not only check for the "site" folder being present....
@matjazpotocnik Thanks. That check from that line is actually: 1. The file /site/install/install.sql does NOT exist; and 2. The directory /site/ does exist. But that check doesn't lead to anything...
@BernhardBaumrock The installer should be okay with that site/assets/files directory existing, as there's a check for it here: https://github.com/processwire/processwire/blob/dev/install.php#L1036 ... so I think I still don't understand what's needed? Also...
Actually, the more relevant line is probably this one: https://github.com/processwire/processwire/blob/dev/install.php#L1918 The built-in mkdir function counts it as a success if the directory already exists. So whether it exists already or...