If the rrssb JavaScript library is not installed properly, you will get a notice as follows:

Notice: Undefined index: description in rrssb_requirements() (line 183 of modules/contrib/rrssb/rrssb.module).

It looks like $requirements['rrssb']['description'] needs to be initialized at the top of the function.

CommentFileSizeAuthor
#4 2668476-4.patch584 bytesheykarthikwithu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GuyPaddock created an issue. See original summary.

AdamPS’s picture

Thanks for the report. Please describe how to hit the bug. From a glance at the code, perhaps if the library is not installed then you go to the status report in Drupal UI.

The 'not installed' branch of code looks like the one missing a description anyway, so can set it there rather than at top of function.

slinky’s picture

I just installed the module. Went to configuration and the error message appeared at the top of the configuration menu:

Notice: Undefined index: description in rrssb_requirements() (line 372 of /home/mysite/public_html/d/sites/all/modules/rrssb/rrssb.module).

heykarthikwithu’s picture

Status: Active » Needs review
FileSize
584 bytes

A check is been added before rendering 'description'.

AdamPS’s picture

Status: Needs review » Fixed

Thanks. I committed a slightly different patch. I thought it was worth keeping the text explaining how to download the library.

  • AdamPS committed 68a358c on 7.x-1.x
    Issue #2668476 by heykarthikwithu, AdamPS: Notice: Undefined index:...

Status: Fixed » Closed (fixed)

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