Needs review
Project:
Clickpath
Version:
6.x-1.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Aug 2008 at 16:19 UTC
Updated:
20 Nov 2010 at 00:00 UTC
Jump to comment: Most recent file
This is a FANTASTIC module! Thanks so much. I notice that it only stores the paths of page views, and if a page display of a View is visited, a blank line appears in the list of recent pages. It would be great if pages which were full Views page displays could be stored too.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 301611.patch | 722 bytes | simon georges |
Comments
Comment #1
coltrane#399008: Use menu_get_item() instead of drupal_get_title() may solve this
Comment #2
jchuck5612 commentedI don't know if this is applicable in all situations but I was able to get my view paths recognized using the following code. I am just beginning my Drupal development experience so I assume there is a lot wrong here but for now it's working.
Replaced:
$title = _clickpath_path_is_frontpage($path) ? t('Home') : $menu_item['title'];With:
Comment #3
coltraneI think this is a bug, so reclassifying.
Comment #4
coltraneGive the patch at #399008: Use menu_get_item() instead of drupal_get_title() a try and see if Views paths appear correctly. I'll try here soon if I can.
Comment #5
simon georges commentedActually, when Views paths have no associated menu entry (just paths), there is no
$menu_item['title'], but there is a$menu_item, sodrupal_get_title()is not used.I've managed to fix the bug using the following patch
Thanks to the folks on #399008: Use menu_get_item() instead of drupal_get_title() for their work !
=> This patch should fix #399008: Use menu_get_item() instead of drupal_get_title() & #275777: Duplicate Clicks and Clear Clicks.
Comment #6
asb commentedI'm confused... does 301611.patch include or supercede the mentioned patches from #399008 and #275777?
Maybe someone could roll a new release, of the whole module, with all recent patches, please?