I have tested this item ✅ successfully on 37b2f4f

I have tested this item ✅ successfully on 37b2f4f

on code review

Why not make it work like anybody would expect: respect the third parameter $email and use it to send the email to. It is not much use to send it to the site admin!

EDIT: My bad, the second parameter is the recipient email which makes $email obsolete. The naming is very confusing, it can be anybody's email. The calling program determines that. It doesn't have to be the site administrator or an administrator at all. The calling program can invent the name and email as they want.

I've never used the method so I don't know what the expectation is honestly. For now this PR just fixes the behavior I broke by using a bad variable. Though by the method name just at a guess I'd say sendAdminMail() should be sending an admin an email, but why that needs a dedicated method that you still have to pass an email address to (with no validation it's actually an admin receiving the message) seems odd to say the least.

this was created (see doc bloc) specifically imho to send mail to admin when a user has submitted new item in frontend. that's why i think it is never used... and we have no lang strings

That explains why it is not used in the content plugin. There they implemented it for all that are to receive mails (sendEmail = 1).

Thank you @mbabker!