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

Issues since v1.11 release #4850

Merged
merged 11 commits into from Apr 18, 2019

Conversation

Projects
None yet
3 participants
@protich
Copy link
Member

commented Apr 16, 2019

Bug fixes and enhancements for v1.11

protich added some commits Apr 16, 2019

Show resolved Hide resolved include/class.forms.php Outdated

@protich protich force-pushed the protich:issue/v1.11 branch from 2c4d4fc to 58206ba Apr 17, 2019

@aydreeihn

This comment has been minimized.

Copy link
Contributor

commented Apr 17, 2019

I see one more continue 2 warning:

PHP Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /var/www/html/develop-next/osTicket/setup/test/tests/class.php_analyze.php on line 78

Should this one be changed as well or was it intentionally left out?

@protich

This comment has been minimized.

Copy link
Member Author

commented Apr 17, 2019

@aydreeihn - good catch, I'll add it to the PR shortly.

@protich protich merged commit eaafe74 into osTicket:develop-next Apr 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.