Summary
We should read from the *_ip_hex column instead of the *_ip column when reading rows from any of cu_changes, cu_log_event, or cu_private_event
Background
- In T363487: Remove the cuc_ip, cule_ip, and cupe_ip columns from the cu_changes, cu_log_event, and cu_private_event tables respectively as duplicated to the IP hex columns we want to remove the cuc_ip, cule_ip, and cupe_ip columns as duplicated to the *_ip_hex equivalents
- One step in doing that task is to replace any uses of the *_ip columns with the *_ip_hex column
- A part of this is to do that conversion when reading from selected rows in CheckUser and other code which might read this data
- We can do this by passing the IP hex value through IPUtils::formatHex to produce format provided by *_ip
- A part of this is to do that conversion when reading from selected rows in CheckUser and other code which might read this data
Acceptance criteria
- No code reads data from the cuc_ip, cule_ip, or cupe_ip columns (using them in WHERE conditions will be handled separately) - This includes code outside the CheckUser extension