Undefined index when send fails and no message is set.

Created on 8 January 2018, about 7 years ago
Updated 10 December 2024, 4 months ago

Gateways are not required to return a message, only 'status' => FALSE, when sending fails. sms_handle_result() checks for a message before emitting the one set if there is one, but it doesn't check that the 'message' key is set as well. If it isn't then there will be a PHP notice:

Notice: Undefined index: message in sms_handle_result() (line 248 of sites/all/modules/smsframework/sms.module).

The attached patch fixes the check.

πŸ› Bug report
Status

Closed: outdated

Version

1.0

Component

Core Framework

Created by

πŸ‡ΊπŸ‡ΈUnited States jacob.embree

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024