Following Drupal best practices I think it would be better to use Drupal's ip_address() in stead of $_SERVER['REMOTE_ADDR']

CommentFileSizeAuthor
#1 1270058.patch772 bytesjelle_s

Comments

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
StatusFileSize
new 1270058.patch772 bytes

Fix is in the attached patch

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
Title: User drupal's ip_address() in stead of $_SERVER['REMOTE_ADDR'] » Use drupal's ip_address() in stead of $_SERVER['REMOTE_ADDR']
cyberwolf’s picture

cyberwolf commented

Subscribing.

attiks’s picture

attiks commented
Status: Active » Needs review
toddy’s picture

toddy commented
Assigned: Unassigned » toddy
Status: Needs review » Fixed

This is fixed now and will be part of the next release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Updated issue summary.