Docs says that file_save_upload() accepts $form_state as a second parameter. But actual list of parameters is this:

function file_save_upload($form_field_name, $validators = array(), $destination = FALSE, $delta = NULL, $replace = FILE_EXISTS_RENAME) {
  /* ... */
}

From the look of function definition, the documentation part referencing $form_state can be removed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SiliconMind’s picture

Status: Active » Needs review
FileSize
625 bytes

Patch uploaded.

jhedstrom’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll, +Documentation

Needs a reroll.

adci_contributor’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
624 bytes

Rerolled

joachim’s picture

Component: file system » documentation
Status: Needs review » Reviewed & tested by the community
Issue tags: -Documentation

Looks good to me.

Setting to docs component so our docs maintainer spots it.

webchick’s picture

Version: 8.0.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)
Issue tags: +Needs backport to D7

Nice catch. Committed and pushed to 8.0.x. Looks like it applies to Drupal 7 as well.

  • webchick committed 7babe4c on 8.0.x
    Issue #2280485 by SiliconMind, the_contributor: Documentation for...
webchick’s picture

Version: 7.x-dev » 8.0.x-dev
Status: Patch (to be ported) » Fixed
Issue tags: -Needs backport to D7

Oops, nope I lie. :) 8.0.x only. Huh. Wonder how on earth that happened.

Status: Fixed » Closed (fixed)

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