PHP terminating tags ?>

Your plugin contains two php files, each of which include terminatings tags

?>

and a line of whitespace after the terminating tag.

This can lead to a “headers already sent” error message.

It is best practice to omit the terminating tag to ensure that no html is output.

Your files will both output whitespace which can cause undesirable side effects. In addition to the header issue, it can mess up the html output with extraneous whitespace.