Title: [Plugin: WordPress MU Domain Mapping] undefined GETs
Last modified: August 20, 2016

---

# [Plugin: WordPress MU Domain Mapping] undefined GETs

 *  [cogmios](https://wordpress.org/support/users/cogmios/)
 * (@cogmios)
 * [13 years, 12 months ago](https://wordpress.org/support/topic/plugin-wordpress-mu-domain-mapping-undefined-gets/)
 * in redirect_login_to_orig it requests directly the $_GET[‘action’] this will 
   work but is not desired:
 * a) it will trigger a php notics “Notice: Undefined index: action in /opt/htdocs/
   html/wp-content/mu-plugins/domain_mapping.php on line 647” for users who have
   php turned on to show notices
    b) it is no longer preferred to query $_GET directly
   but use filter_input(INPUT_GET, ‘action’, FILTER_SANITIZE_STRING); instead (see
   [http://php.net/manual/en/function.filter-input.php](http://php.net/manual/en/function.filter-input.php))
 * e.g. replacing it with:
 *     ```
       $action = filter_input(INPUT_GET,  'action', FILTER_SANITIZE_STRING);
       	if ( !get_site_option( 'dm_remote_login' ) || $action == 'logout' || isset( $_GET[ 'loggedout' ] ) ) {
       		return false;
       	}
       ```
   
 * removes the php notice
 * You could also use
 * `$action = filter_input(INPUT_GET, 'action', FILTER_CALLBACK, $options);`
 * to callback to the list of available actions
 * now for the “loggedout” you could use FILTER_NULL_ON_FAILURE instead
 * [http://wordpress.org/extend/plugins/wordpress-mu-domain-mapping/](http://wordpress.org/extend/plugins/wordpress-mu-domain-mapping/)

Viewing 1 replies (of 1 total)

 *  Plugin Author [Ron Rennick](https://wordpress.org/support/users/wpmuguru/)
 * (@wpmuguru)
 * [13 years, 12 months ago](https://wordpress.org/support/topic/plugin-wordpress-mu-domain-mapping-undefined-gets/#post-2824267)
 * Thanks for reporting the warning 🙂 We’ll probably stick to an isset() check 
   to clear the warning since that’s what we;ve used throughout the plugin.

Viewing 1 replies (of 1 total)

The topic ‘[Plugin: WordPress MU Domain Mapping] undefined GETs’ is closed to new
replies.

 * ![](https://s.w.org/plugins/geopattern-icon/wordpress-mu-domain-mapping.svg)
 * [WordPress MU Domain Mapping](https://wordpress.org/plugins/wordpress-mu-domain-mapping/)
 * [Frequently Asked Questions](https://wordpress.org/plugins/wordpress-mu-domain-mapping/#faq)
 * [Support Threads](https://wordpress.org/support/plugin/wordpress-mu-domain-mapping/)
 * [Active Topics](https://wordpress.org/support/plugin/wordpress-mu-domain-mapping/active/)
 * [Unresolved Topics](https://wordpress.org/support/plugin/wordpress-mu-domain-mapping/unresolved/)
 * [Reviews](https://wordpress.org/support/plugin/wordpress-mu-domain-mapping/reviews/)

## Tags

 * [get](https://wordpress.org/support/topic-tag/get/)

 * 1 reply
 * 2 participants
 * Last reply from: [Ron Rennick](https://wordpress.org/support/users/wpmuguru/)
 * Last activity: [13 years, 12 months ago](https://wordpress.org/support/topic/plugin-wordpress-mu-domain-mapping-undefined-gets/#post-2824267)
 * Status: not resolved