Context Navigation

Changes between Initial Version and Version 1 of Ticket #4622, comment 13

initial	v1
3	3	For some added complexity, we use WooCommerce's registration forms to create the user. In the current production version, they improperly handle when the $user_id comes back as a WP_Error object. The resulting cast to int messes up the admin user: no fun. I've already submitted a patch for this.
4	4
5		Our current solution is to add a filter for `sanitize_user` (which I believe ~~validate_username()~~ calls). I couldn't use `bp_core_strip_username_spaces()` because it uses `username_exists()` which eventually calls `sanitize_user()` resulting in an endless loop until memory_limit was reached.
	5	Our current solution is to add a filter for `sanitize_user` (which I believe `validate_username()` calls). I couldn't use `bp_core_strip_username_spaces()` because it uses `username_exists()` which eventually calls `sanitize_user()` resulting in an endless loop until memory_limit was reached.
6	6
7	7	I understand that it's a tricky issue to fix in a compatible way. I think `sanitize_user` is the likely choice.