API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21File%21fi...

The documentation for the array returned from hook_filetransfer_info() still contain the description for file and file path, which aren't used anymore from Drupal 8, since it uses the autoload. The description for those keys should be removed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kiamlaluno created an issue. See original summary.

apaderno’s picture

There is also #2038271: Convert hook_filetransfer_info() to plugin system, but it seems far from being deployed. I think we should still remove the description of the not used keys.

Shashwat Purav’s picture

Assigned: Unassigned » Shashwat Purav
Shashwat Purav’s picture

Assigned: Shashwat Purav » Unassigned
Status: Active » Needs review
FileSize
1.04 KB

Created patch. The description for "file" and "file path" keys is removed.

Shashwat Purav’s picture

apaderno’s picture

Status: Needs review » Reviewed & tested by the community

It is quite smooth.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to all three 8.x branches, thanks!

  • catch committed db998a7 on 8.3.x
    Issue #2783849 by Shashwat Purav, kiamlaluno: hook_filetransfer_info()...

  • catch committed 337e9e9 on 8.2.x
    Issue #2783849 by Shashwat Purav, kiamlaluno: hook_filetransfer_info()...

  • catch committed 6b53dcf on 8.1.x
    Issue #2783849 by Shashwat Purav, kiamlaluno: hook_filetransfer_info()...

Status: Fixed » Closed (fixed)

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