Getclicky module checks if the current url is an administrative page using the arg() function. The proper way should be using the path_is_admin function.

$arg = arg();
  
// Don't track page views in the admin sections
//@TODO: use path_get_admin_paths() instead of arg(0) comparison to admin.
  if ( __getclicky_track_or_not($user, $arg) ) {
...

I'll upload a patch fixing it using the path_is_admin function.

CommentFileSizeAuthor
#7 admin_path_check_fix_correct.patch1.25 KBgeorgi777
#2 admin_path_check_fix.patch1.25 KBgeorgi777

Comments

gopsan created an issue. See original summary.

georgi777’s picture

georgi777 commented
StatusFileSize
new admin_path_check_fix.patch1.25 KB

Here's the patch.

georgi777’s picture

georgi777 commented
Title: Is current url an admin path isn't checked properly » Is url an admin area not checked propery.
georgi777’s picture

georgi777 commented
Status: Active » Needs review
georgi777’s picture

georgi777 commented
Title: Is url an admin area not checked propery. » Properly check if path is admin
georgi777’s picture

georgi777 commented
georgi777’s picture

georgi777 commented
StatusFileSize
new admin_path_check_fix_correct.patch1.25 KB

This is the correct patch.

  • rsvelko committed 2a3e849 on 7.x-1.x 
    Issue #2797665 by gopsan, rsvelko: Properly check if path is admin
rsvelko’s picture

rsvelko commented
Status: Needs review » Reviewed & tested by the community

Applied, tested, tweaked a little, committed.

rsvelko’s picture

rsvelko commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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