Language is not a required parameter

While I was at it, I also provided an empty string to satisfy the trim() function in case $pet->getMailbodyPlain() returns null.

Thanks BenStallings, good find.

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

Hi, this issue doesn't seem to be fixed.

The issue persists because the code in pet_substitutions expects $params['language'] to be set but $params['language'] is set directly after pet_substitutions is called.

Can we reopen this issue?

@cwales95, yes, you can. Please test with latest code in branch as well.
I'll appreciate if you can post a patch as well.
Unfortunately, I don't have enough time to look into issues.

@Sharique I've updated the branch, but I am unable to reopen the issue. It says "Only maintainers can reopen."

@sharique yes, and I had already done so at https://git.drupalcode.org/project/pet/-/merge_requests/5/diffs?diff_id=... , but for some reason https://git.drupalcode.org/project/pet/-/merge_requests/5/diffs shows an incorrect "latest version" until you select "latest version" again.