Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Jan 2016 at 14:51 UTC
Updated:
1 Mar 2016 at 05:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
rakesh.gectcrComment #3
David Hernández commentedConfirmed. The function doesn't return anything.
Comment #4
jhodgdonTrue, it shouldn't be a @return. But ... we seem to be losing some information that should still be in the documentation somewhere?
Comment #5
gaydabura commentedComment #6
andypostLooks good, when batch passed by reference there's no other way to update its set
Comment #7
jhodgdonThanks for the patch! It needs a little fix though:
If this is meant to be two paragraphs, you need to leave a blank line between them.
If it is meant to be one paragraph, you need to rewrap so that each docs line is as close to 80 characters as possible without going over.
Also moving this to 8.0.x to indicate it should be committed to both 8.0 and 8.1 branches. And fixing the title/summary since this is about a wrong @return, not @param.
Comment #8
gaydabura commentedthnx @jhodgdon
new patch is attached
Comment #9
gaydabura commentedComment #10
devaraj johnson commentedComment #11
jhodgdonGood enough, thanks!
Comment #12
catchCommitted/pushed to 8.1.x and cherry-picked to 8.0.x. Thanks!