Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
31 Mar 2010 at 19:17 UTC
Updated:
3 Jan 2014 at 01:42 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jhodgdonGood catch! This is also true of the file_copy() function, as you can see by looking a few lines into file_copy():
So, this should be added to the doc for $source for both functions.
http://api.drupal.org/api/function/file_copy/6
http://api.drupal.org/api/function/file_move/6
It's not relevant to Drupal 7, since the signature of these functions has changed and $source is always an object in D7.
Comment #2
sender commentedComment #3
sender commentedChanged wording, applied doxygen formatting standards and removed trailing spaces.
Comment #4
sender commentedComment #5
jhodgdonLooks pretty good...
We try to wrap lines as close to 80 characters as possible, so this needs a reformat:
Also, list formatting:
See standards at http://drupal.org/node/1354#general under "Lists" - should have a colon : after items and not a hyphen -
I realize that you didn't originate either of these problems, but it would be great if you could fix them. :)
Thanks!
Comment #6
sender commentedHere's a follow up.
Comment #7
crotown commentedWrapped lines correctly -- no needless short lines. And colons instead of hyphens in the list. Both of these changes for both of the two functions that needed this documentation change.
Comment #8
gábor hojtsyCommitted, thanks.