nextcloud-docker-dev icon indicating copy to clipboard operation
nextcloud-docker-dev copied to clipboard

Settings exception on php80+

Open erik78se opened this issue 2 years ago • 6 comments

I'm getting an exception with nextcloud 25 (latest at date) with php80

I've got this in my .env

PHP_VERSION=80

image

It works on PHP_VERSION=74

erik78se avatar Jul 24 '22 15:07 erik78se

Do you have the viewer app installed and enabled? I don't think there should be a difference between 7.4 and 8.0 but that would explain the error

juliusknorr avatar Jul 25 '22 05:07 juliusknorr

Probably https://github.com/nextcloud/server/blob/master/apps/settings/lib/Settings/Personal/ServerDevNotice.php#L82 is not covering that for dev environments as for the releases viewer is always available.

juliusknorr avatar Jul 25 '22 05:07 juliusknorr

I'm not sure but I'm just following the instructions to get the development environment up.

I'm trying to use it in conjunction with the tutorial here: https://docs.nextcloud.com/server/latest/developer_manual/app_development/tutorial.html

So, php80 would be very much needed to have working somehow.

erik78se avatar Jul 27 '22 12:07 erik78se

Can you check if you have the viewer app cloned in workspace/server/apps-extra/viewer and if it is on the latest master branch?

juliusknorr avatar Jul 27 '22 17:07 juliusknorr

In my .env I have

REPO_PATH_SERVER=/root/nextcloud-docker-dev/server
ADDITIONAL_APPS_PATH=/root/nextcloud-docker-dev/server/apps-extra
# NEXTCLOUD_AUTOINSTALL_APPS="viewer activity"
PHP_VERSION=80
  • There is a viewer in /root/nextcloud-docker-dev/workspace/server/apps-extra/

  • But I don't fully understand from the docs what those PATHS:s are all for and what to set them to etc.

  • There is no viewer in the /root/nextcloud-docker-dev/server path.

erik78se avatar Jul 27 '22 17:07 erik78se

  • There is a viewer in /root/nextcloud-docker-dev/workspace/server/apps-extra/

This would be the expected directory. Can you check if there are any files in it and if it is using the latest master branch from GitHub? In addition, please check that all files are also readable by your current user.

  • But I don't fully understand from the docs what those PATHS:s are all for and what to set them to etc.

They look fine, with using the default bootstrap.sh script there should be no need to adjust the env to get it running.

  • There is no viewer in the /root/nextcloud-docker-dev/server path.

As mentioned above, that is expected as it should be in /root/nextcloud-docker-dev/apps-extra/viewer

juliusknorr avatar Aug 01 '22 21:08 juliusknorr

bootstrap.sh should now automatically cloning the viewer repo so closing this. Feel free to comment if you are still seeing this issue.

juliusknorr avatar Nov 28 '22 07:11 juliusknorr