I am using ip_geoloc for the views integration with various map providers (which is most awesome and versatile). I have disabled the HTML5 location and reverse geocoding features as I am only interested in the maps integration. Still, in hook_init() this module is storing a value in the $_SESSION variable, which by default causes Drupal's page cache to be disabled. It would be helpful if this module would avoid session storage where possible so as not to disable Drupal's page cache.

Comments

rdeboer’s picture

Install the Session Cache API module for exactly that purpose. After that $_SESSION won't be used.
Great to hear IPGVM is of use to you.
Rik

rdeboer’s picture

Issue summary: View changes
Status: Active » Closed (works as designed)
kristofferwiklund’s picture

Status: Closed (works as designed) » Needs work

I don't see how installing the session_cache will help. The module is still writing session data for anonymous. And Session Cache API only choose where to save it. I would like the session to be stored in the database. Maybe not for this module but for everything else.

I really would like the ip_geoloc modulue to not load anything in hook_init. This is not needed for just showing maps in views.

rdeboer’s picture

@kristofferwiklund

Yes on reading @onelittleant's original post properly... I originally read between the lines that this was about being able to use IPGV&M in combination with Boost or Varnish. But it doesn't actually say that...

You are both correct.

I will investigate whether I can put in a checkbox to switch off the use of any session storage altogether, as this has turned out to be such a biggy.

Naturally those that use the checkbox will disable any functionality around visitor location, including putting the visitor on the map, showing in a table distances from the visitor to various items in the View, etc.

Rik

rdeboer’s picture

Assigned: Unassigned » rdeboer
Status: Needs work » Fixed

The latest check in of 7.x-1.x-dev now has a check box on the IPGV&M configuration page, admin/config/system/ip_geoloc, under "Data collection options".

  • RdeBoer committed 8ee397c on
    Initial fixes for [#2107461], [#2432639]
    

Status: Fixed » Closed (fixed)

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