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

New install or update without SMS function #859

Open
alexintour opened this issue Nov 22, 2022 · 0 comments
Open

New install or update without SMS function #859

alexintour opened this issue Nov 22, 2022 · 0 comments

Comments

@alexintour
Copy link
Contributor

Bug report

Please give us a short description of the bug
When updating from an old code, but also when installing completely from scratch (I guess), in the config.php file you can decide if a SMS sending system is active or not.

Expected behaviour

site working without SMS function

Actual behaviour

when entering user options, the file vendor/autoload.php is called, and since it is missing, the page is not loading

Steps to reproduce

enter in user options when the SMS supoprt is not present

in file lib/sms.php the file is called without checking if it exists or not.
require_once('vendor/autoload.php'); use Twilio\Rest\Client;

but it should be like
if(file_exists('vendor/autoload.php')) require_once 'vendor/autoload.php'; use Twilio\Rest\Client;

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

No branches or pull requests

1 participant