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

php v7.3 caldav and vcard continue break warning. #13089

Closed
cdmlb opened this issue Dec 15, 2018 · 3 comments
Closed

php v7.3 caldav and vcard continue break warning. #13089

cdmlb opened this issue Dec 15, 2018 · 3 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug

Comments

@cdmlb
Copy link

cdmlb commented Dec 15, 2018

Since updating to php v7.3

I get these two errors constantly, they are just warnings accord to the php documentation, but one day the code might not be backwards compatible anymore, idk???

"continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? at
/3rdparty/sabre/dav/lib/CalDAV/ICSExportPlugin.php#321
/3rdparty/sabre/vobject/lib/Component/VCard.php#514

@cdmlb cdmlb added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Dec 15, 2018
@rEnr3n
Copy link

rEnr3n commented Jan 16, 2019

nextcloud/calendar#1064

@J0WI J0WI added this to the Nextcloud 15.0.3 milestone Jan 16, 2019
@J0WI
Copy link
Contributor

J0WI commented Jan 16, 2019

Backport for NC 15: nextcloud/3rdparty#213

@MorrisJobke
Copy link
Member

Will be fixed in 15.0.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug
Projects
None yet
Development

No branches or pull requests

4 participants