Notice: Undefined index: options in /var/www/html/oncorps/docroot/sites/all/modules/contrib/views/theme/theme.inc

CommentFileSizeAuthor
#5 views-n3165178-5.interdiff.txt811 bytesdamienmckenna
#5 views-n3165178-5.patch1.81 KBdamienmckenna
#3 undefined-index-option-view-theme-inc-3165178.patch1.72 KBpradeepjha
#2 Screenshot 2020-08-14 at 12.02.12 PM.png155.83 KBpradeepjha

Comments

pradeepjha created an issue. See original summary.

pradeepjha’s picture

pradeepjha commented
Issue summary: View changes
StatusFileSize
new Screenshot 2020-08-14 at 12.02.12 PM.png155.83 KB
pradeepjha’s picture

pradeepjha commented
Status: Active » Needs review
StatusFileSize
new undefined-index-option-view-theme-inc-3165178.patch1.72 KB
surya.s’s picture

surya.s commented
Status: Needs review » Reviewed & tested by the community

The patch works fine in D7 site. Thank you!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Notice: Undefined index: options in /sites/all/modules/contrib/views/theme/theme.inc » Notice: Undefined index: options in theme/theme.inc
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new views-n3165178-5.patch1.81 KB
new views-n3165178-5.interdiff.txt811 bytes

It's not a good solution to check for an outer variable if there's an interior variable you're actually looking for. Please give this new patch a try.

Can you please describe the situation that lead to this notice displaying? What version of PHP are you using? Thank you.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-3.24 » 7.x-3.x-dev
renatog’s picture

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

Yes, @surya.s both will solve the problem but #5 really makes sense

+1 to it

  • RenatoG committed 22ea6a4 on 7.x-3.x authored by DamienMcKenna 
    Issue #3165178 by pradeepjha, DamienMcKenna, surya.s, RenatoG: Notice:...
renatog’s picture

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

Moved to the dev branch

Thank you so much everyone

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #3118500: Plan for Views 7.x-3.25

Status: Fixed » Closed (fixed)

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