@mattab opened this Issue on October 23rd 2012 Member

Reported in forums.

We should

  • Remove silent fail @ operator in all GeoIP drivers
  • Update the System check, to add a warning that, when the extension mbstring is not available or enabled, geoIP will not work
  • It would be nice & important to ensure that, any error happening during GeoIP tracking, would alos be surfaced in the GeoIP tooltips on the admin page, so that users can see the errors in the UI and do not have to debug the tracking requests to see the errrors.
@diosmosis commented on October 24th 2012 Member

(In [7290]) Fixes #3473, add "GeoIP won't work" warning to installation system check for mbstring, make sure GeoIP (Php) implementation checks for mbstring plugin, and remove silent fail calls in GeoIP plugins.

This Issue was closed on October 24th 2012
Powered by GitHub Issue Mirror