fix Illegal offset type in isset or empty and Undefined method Tracker::getSessionData()

[marlocorridor]

for issue #188 and #218

[marlocorridor]

I do not understand why this error occur

Argument 1 passed to PragmaRX\Tracker\Data\RepositoryManager::__construct() must be an instance of PragmaRX\Tracker\Data\GeoIP, instance of PragmaRX\Support\GeoIp\GeoIp given, called in *****\vendor\pragmarx\tracker\src\Vendor\Laravel\ServiceProvider.php on line 322

please help. :)

[fntneves]

@marlocorridor Please, go to file

src/Data/RepositoryManager.php

and add this to the top so it will get the correct type:

use PragmaRX\Support\GeoIp\GeoIp;

[marlocorridor]

Thanks a lot @fntneves . :)

[KeitelDOG]

Why hasn't this branch been merged yet? I got the "Call to undefined method PragmaRX\Tracker\Tracker::getSessionData()" when migrating from Laravel 4.2 to 5.0 .
Please @antonioribeiro or other Admin could you add this fix? Cause I really need this error fixed before I update my Staging Server.

[fntneves]

@marlocorridor You're welcome! :D

Would you please also commit that fix, if it is not already available?

[KeitelDOG]

Nice Job @marlocorridor . This change is very good and resolve the problem of : Illegal offset type in isset or empty and Undefined method Tracker::getSessionData() as suggested.
I just tested it directly by changing the codes in vendor library in files Tracker.php and ServiceProvider.php .

Accessor getSessionData() could be created to return the protected $sessionData attribute, but since it is an internal call, everything is perfect.

@antonioribeiro I definitely think that this change should be merged. I'm not a fan of package hacking, this leaves unstable results when updating packages.

[marlocorridor]

@fntneves included on my new commit. :)

[KeitelDOG]

@fntneves Could you please merge changes. I'm gonna upgrade from Laravel 4.2 to 5.0 on my Staging Server on Friday. And may be to Production the same day or on Saturday.

[fntneves]

@KeitelDOG I am not an admin of this repository. I would like to merge changes, actually.

\cc @antonioribeiro

[KeitelDOG]

@fntneves sorry I didn't know. I was just about to connect to my Staging server to upgrade to Laravel 5.0, and I check if it was merged. So I have no choice to do some code hacking on the files till then. Thanks.

[antonioribeiro]

Sorry for the delay, merged.