These got added inadvertently when drupal_get_js() was added to core. The result is that http://api.drupal.org/api/function/drupal_get_js/6 lists the parameters as:

eter $scope (optional) The scope for which the JavaScript rules should be returned. Defaults to 'header'.

eter $javascript (optional) An array with all JavaScript code. Defaults to the default JavaScript array for the given scope.

Should be applied to 7.x/6.x/5.x.

CommentFileSizeAuthor
#5 drupal-fix-param-doxygen-258128-d5.patch806 bytesgpk
#1 drupal-fix-param-doxygen-258128-1.patch860 byteswebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Needs review
StatusFileSize
new drupal-fix-param-doxygen-258128-1.patch860 bytes

Patch.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

I think this patch wins the Most Trivial Patch of the Month Award :)

tjholowaychuk’s picture

tjholowaychuk commented

Haha yikes.. second that.

dries’s picture

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

Committed to CVS HEAD and DRUPAL-6. Thanks Angie.

gpk’s picture

gpk commented
Version: 7.x-dev » 5.x-dev
Status: Fixed » Needs review
StatusFileSize
new drupal-fix-param-doxygen-258128-d5.patch806 bytes

Original patch applied to 5.x with offset and fuzz 2, this one should apply cleanly.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

Looks good. :)

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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