How make the block visible by anonymous users ? Possible ?
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | clickpath.module.txt | 6.29 KB | giorgosk |
| #10 | clickpath.patch | 1.61 KB | dkruglyak |
How make the block visible by anonymous users ? Possible ?
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | clickpath.module.txt | 6.29 KB | giorgosk |
| #10 | clickpath.patch | 1.61 KB | dkruglyak |
Comments
Comment #1
vstmusic commentedNo response ?
Comment #2
coltraneAs the module stands there is a critical bug #216308: call to undefined drupal_get_path_alias but it otherwise works with anonymous users, you just have to set the permission for anonymous to see the block.
Note, the latest patch in that issue at this time would mean clickpath wouldn't work for anonymous users ...
Comment #3
coltraneVersion 1.1 effectively keeps paths from being stored by the module for anonymous users. To provide anonymous user support there will need to be a solution to the problems identified in #216308: call to undefined drupal_get_path_alias. I am open to a solution but will not being pursuing one myself any time soon.
Comment #4
dropchew commented+1 subscribe
Comment #5
coltraneChanging priority and status.
Comment #6
Bricks and Clicks MarketingSubscribing
Comment #7
dkruglyak commentedWhat more info is needed?
Seems to me this feature could work just fine if page caching is turned off. With caching turned off all the right modules would be loaded and the problem from http://drupal.org/node/216308 (call to undefined drupal_get_path_alias) would never show itself.
I propose to replace the check for
$user->uidwith a test forfunction_exists('drupal_get_path_alias')and be done with it.Thoughts?
Comment #8
coltraneI believe that would work fine, but needs a patch (hence status back to active) and testing.
Comment #9
coltraneI think there should be some help text in the README but probably also the admin page explaining that page caching must be off for clickpath to work for anonymous users.
Comment #10
dkruglyak commentedOK, I rolled the patch and everything works great for anon users, even with normal (not aggressive) caching on.
Note, I added another enhancement to this patch, concerning the home page treatment. The default behavior (hardcoded 'Home') is not very helpful. I added the ability to specify this string in module settings and if it is not specified, default to 'Home' and if it is set to nothing, use the site name.
Everything is tested and ready to commit.
Comment #11
dddave commentedRTBTC indeed!
Works like a charm. Thx!
Comment #12
haveabreak commentedCould you attach full clickpath.module to work for anonymous users ?
Comment #13
giorgoskpatch from #10 works as expected
Comment #14
giorgoskhere is the 1.4 .module modified
everything else is the same
Comment #15
coltraneI've committed this to DRUPAL-6--1 and will try and roll a new release soon. Thanks dkruglyak, dddave and GiorgosK