CommentFileSizeAuthor
#3 user_visits_adv-fixed-undefined-index-2375133-1.patch1.46 KBArkener
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

saru1683’s picture

saru1683 CreditAttribution: saru1683 commented
Category: Bug report » Support request

One more notice.
Notice: Use of undefined constant total - assumed 'total' in user_visits_adv_cron() (line 248 of user_visits_adv.module)

tanay_lakhani’s picture

tanay_lakhani CreditAttribution: tanay_lakhani commented

+1

Arkener’s picture

Arkener CreditAttribution: Arkener commented
Issue summary: View changes
Related issues: +#1941566: Cron deletes all visits, +#2458313: Data Clears on Cron Run
FileSize
user_visits_adv-fixed-undefined-index-2375133-1.patch1.46 KB

Attached patch will fix the problems from the original post and comment #1

  • Undefined offset: Now correctly checks if the given key is empty
  • Undefined constant: Total is changed to calculate the correct total
Arkener’s picture

Arkener CreditAttribution: Arkener commented
Status: Active » Needs review