PopupPro admin-ajax call insecure with domain mapping

We have a multisite network in which we declare
define('FORCE_SSL_ADMIN', true);

We also allow for domain mapping. 99% of the users that map their domain do not have an SSL certificate. So, when popup pro makes the admin-ajax call, it uses HTTPS rather than the scheme based on the page load due to the use of FORCE_SSL_ADMIN. The admin_url function obeys that declaration unless otherwise noted. We fixed this by just adding a simple test to the ajax call like so in class-public-popup (line 235)

protected function load_method_ajax() {
		global $pagenow;

		if ( ! in_array( $pagenow, array( 'wp-login.php', 'wp-register.php' ) ) ) {
			// Data is loaded via a normal WordPress ajax request.
			$scheme = is_ssl() ? 'https' : 'http';
			$this->script_data['ajaxurl'] = admin_url( 'admin-ajax.php', $scheme );
			$this->script_data['ajax_data']['orig_request_uri'] = $_SERVER['REQUEST_URI'];

Curious what you thought of this and if you think it can be added to any future updates so it's not something we have to continuously fix?