Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot export calendars - Exception thrown #4863

Closed
CapSel opened this issue Dec 31, 2022 · 6 comments
Closed

Cannot export calendars - Exception thrown #4863

CapSel opened this issue Dec 31, 2022 · 6 comments
Labels
1. to develop Accepted and waiting to be taken care of bug regression Regression of a previous working feature

Comments

@CapSel
Copy link

CapSel commented Dec 31, 2022

Steps to reproduce

  1. Go to Calendar App
  2. Click "pencil" next to calendar you want to export - for example "Personal"
  3. Click export (nothing happens)
  4. Click export (empty page)

Expected behavior

I wish to have an ICS file that I could import into another instance of nextcloud.

Actual behaviour

Nothing is shown.

Calendar app version

4.2.0

CalDAV-clients used

Web browser

Browser

Newest firefox (108?)

Client operating system

Arch Linux, MacOS

Server operating system

ArchLinux, Ubuntu with dockerized nextcloud:fpm-alpine

Web server

Nginx

Database engine version

PostgreSQL

PHP engine version

None

Nextcloud version

25.0.2

Updated from an older installed version or fresh install

Fresh install

List of activated apps

No response

Nextcloud configuration

No response

Web server error log

No response

Log file

{"reqId":"XH5JPZQHzAEEskBLQ4SJ","level":3,"time":"2022-12-31T11:24:39+00:00","remoteAddr":"89.69.69.250","user":"capsel","app":"index","method":"POST","url":"/index.php/apps/calendar/v1/config/view","message":"OCA\\Calendar\\Controller\\SettingsController::setConfig(): Argument #2 ($value) must be of type string, null given, called in /var/www/html/lib/private/AppFramework/Http/Dispatcher.php on line 225 in file '/var/www/html/custom_apps/calendar/lib/Controller/SettingsController.php' line 71","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:108.0) Gecko/20100101 Firefox/108.0","version":"25.0.2.3","exception":{"Exception":"Exception","Message":"OCA\\Calendar\\Controller\\SettingsController::setConfig(): Argument #2 ($value) must be of type string, null given, called in /var/www/html/lib/private/AppFramework/Http/Dispatcher.php on line 225 in file '/var/www/html/custom_apps/calendar/lib/Controller/SettingsController.php' line 71","Code":0,"Trace":[{"file":"/var/www/html/lib/private/AppFramework/App.php","line":172,"function":"dispatch","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[{"__class__":"OCA\\Calendar\\Controller\\SettingsController"},"setConfig"]},{"file":"/var/www/html/lib/private/Route/Router.php","line":298,"function":"main","class":"OC\\AppFramework\\App","type":"::","args":["OCA\\Calendar\\Controller\\SettingsController","setConfig",{"__class__":"OC\\AppFramework\\DependencyInjection\\DIContainer"},["view","calendar.settings.setConfig"]]},{"file":"/var/www/html/lib/base.php","line":1047,"function":"match","class":"OC\\Route\\Router","type":"->","args":["/apps/calendar/v1/config/view"]},{"file":"/var/www/html/index.php","line":36,"function":"handleRequest","class":"OC","type":"::","args":[]}],"File":"/var/www/html/lib/private/AppFramework/Http/Dispatcher.php","Line":165,"Previous":{"Exception":"TypeError","Message":"OCA\\Calendar\\Controller\\SettingsController::setConfig(): Argument #2 ($value) must be of type string, null given, called in /var/www/html/lib/private/AppFramework/Http/Dispatcher.php on line 225","Code":0,"Trace":[{"file":"/var/www/html/lib/private/AppFramework/Http/Dispatcher.php","line":225,"function":"setConfig","class":"OCA\\Calendar\\Controller\\SettingsController","type":"->","args":["view",null]},{"file":"/var/www/html/lib/private/AppFramework/Http/Dispatcher.php","line":133,"function":"executeController","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[{"__class__":"OCA\\Calendar\\Controller\\SettingsController"},"setConfig"]},{"file":"/var/www/html/lib/private/AppFramework/App.php","line":172,"function":"dispatch","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[{"__class__":"OCA\\Calendar\\Controller\\SettingsController"},"setConfig"]},{"file":"/var/www/html/lib/private/Route/Router.php","line":298,"function":"main","class":"OC\\AppFramework\\App","type":"::","args":["OCA\\Calendar\\Controller\\SettingsController","setConfig",{"__class__":"OC\\AppFramework\\DependencyInjection\\DIContainer"},["view","calendar.settings.setConfig"]]},{"file":"/var/www/html/lib/base.php","line":1047,"function":"match","class":"OC\\Route\\Router","type":"->","args":["/apps/calendar/v1/config/view"]},{"file":"/var/www/html/index.php","line":36,"function":"handleRequest","class":"OC","type":"::","args":[]}],"File":"/var/www/html/custom_apps/calendar/lib/Controller/SettingsController.php","Line":71},"CustomMessage":"--"},"id":"63b01c028a013"}

Browser log

No response

Additional info

No response

@CapSel CapSel added 0. to triage Pending approval or rejection bug labels Dec 31, 2022
@CapSel
Copy link
Author

CapSel commented Dec 31, 2022

I have this issue on:

  • freshly installed Nextcloud in Docker from tag fpm-alpine + postgres:latest with nginx outside of container on ubuntu
  • old instance (as in fresh install was looong time ago) of Nextcloud unpacked (not from distro package or docker) on ArchLinux with Apache and php-fpm

@miaulalala
Copy link
Contributor

Please make sure to look for existing issues frst!

Duplicate of #4844

@miaulalala
Copy link
Contributor

Sorry, my mistake - this is valid.

@miaulalala miaulalala reopened this Jan 3, 2023
@tcitworld
Copy link
Member

From what I can found, setTimezone and setInitialView don't check that they have indeed a valid value before doing the call to set the value.

@miaulalala
Copy link
Contributor

Issue is also solved by #4873 : Vue-router (:to="") should only be used for internal/frontend routes and not for external routes. The initialization of route params goes crazy because of this and finally causes the backend issues.

@st3iny
Copy link
Member

st3iny commented Jan 4, 2023

BTW: This is indeed a regression from the sharing redesign (#4515). The export button used href before and I migrated it to param without thinking about the implications.

@miaulalala miaulalala added regression Regression of a previous working feature 1. to develop Accepted and waiting to be taken care of and removed 0. to triage Pending approval or rejection labels Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug regression Regression of a previous working feature
Projects
None yet
Development

No branches or pull requests

4 participants