Currently the activity_update_status function has the following line:

$handler = activity_handler_load($record->actions_id);

...and proceeds to work with this handler on the assumption that it is a valid handler object. However, it is possible for the activity_handler_load() function to return FALSE, so we need a check on that. Patch forthcoming...

CommentFileSizeAuthor
#1 1134898-check-for-valid-handler.patch1.82 KBkatbailey

Comments

katbailey’s picture

katbailey commented
Title: Need error handling in activity_update_status function » Need to check for a valid handler after calling activity_handler_load
StatusFileSize
new 1134898-check-for-valid-handler.patch1.82 KB

as promised...
The check is possibly needed elsewhere in the code but this is a start.

_shy’s picture

_shy
Primary language Ukrainian
Location Ukraine, Lutsk 🇺🇦
commented
Issue summary: View changes
Status: Active » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.