The API functions should return some sort of value, at the very least a TRUE or FALSE if worked. Ideally some sort of array of errors or something like that.

This is important because if someone needs to do something afterwords that depends on if that API call was successful, they currently have no way of knowing.

Thanks for the great module!

Comments

ebeyrent’s picture

Assigned: Unassigned » ebeyrent

I agree, and will add this to the functions.

Thanks for the feedback!

ebeyrent’s picture

Status: Active » Fixed

Fixed in the 2.6 release. Almost all functions return either TRUE or FALSE. In cases where FALSE is returned, errors are logged via watchdog.

zzolo’s picture

Awesome! Thank you.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.