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

Uncaught PHP Exception it\thecsea\simple_caldav_client\CalDAVException: "Recieved unknown HTTP status" #38

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

monkey3d
Copy link

Hi,
This message is received when the change method is called.
The http status received is 201.
The change is made, however, but invokes this exception.
By modifying the line
if ( $this->client->GetHttpResultCode() != '204' && $this->client->GetHttpResultCode() != '200')
through
if ( $this->client->GetHttpResultCode() != '204' && $this->client->GetHttpResultCode() != '200' && $this->client->GetHttpResultCode() != '201')
the exception no longer occurs.
cordially thank you

carduz and others added 22 commits December 1, 2015 23:51
Fix PHP notice in case tags are not present
Sorry the last fix did solve the displaying of errors but broke some functionality (the approach did not work all the time).
This fixes a tiny typo that caused FindCalendarHome to fail at its job.
FIX: FindCalendarHome broken due to typo
remove $etag as required param to delete() method. Using yandex calendar I can delete items without $etag.
Update SimpleCalDAVClient.php
Wrong namespace used for DOMDocument
Akarsh04 and others added 7 commits May 15, 2024 03:23
FREEPBX17-548 CalDav server http response issue resolved
Fix `Class "it\thecsea\simple_caldav_client\Exception" not found` and inconsistent indentation
Add function getEventByGuid, to search for an event by UID
Add protected definitions to prevent depreciation notices >PHP81, and make preg DAV header search and replace case insensitive as not all caldav servers (Nextcloud is one) serves uppercase DAV.
Small bugfixes and new function implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.