Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.I've found out that this module defines its own function get_client_ip() to get the client ip address. However, there is already a function provided by drupal core for the same purpose: ip_address()
Is there any explicit reason not using this core function?
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | why_not_using_core-2642796-2.patch | 1.42 KB | malcomio |
Comments
Comment #2
malcomio commentedGood question - probably just didn't realise it was there.
Here's a patch that uses ip_address() instead.
Comment #3
malcomio commentedComment #5
malcomio commentedThanks - committed to 7.x-1.x