hook_file_insert doc has no function body [#1260162]

Comment	File	Size	Author
#6	adding_hook_file_insert_example-1260162-6.patch	906 bytes	Bès

#2	adding_hook_file_insert_example-1260162-2.patch	1.15 KB	michaellenahan

Comment #1

michaellenahan CreditAttribution: michaellenahan commented 26 August 2011 at 09:24

Assigned:

Unassigned

» michaellenahan

i'm at drupalcon london doc sprint. working on this with kay_v.

Log in or register to post comments

Comment #2

michaellenahan CreditAttribution: michaellenahan commented 26 August 2011 at 12:09

File	Size
adding_hook_file_insert_example-1260162-2.patch	1.15 KB

Hello from DrupalCon in sunny South London.
We're at the doc sprint and have added an example.
Thanks to: bes and kay_v

Log in or register to post comments

Comment #3

Bès CreditAttribution: Bès commented 26 August 2011 at 12:45

Status:

Active

» Needs review

Log in or register to post comments

Comment #4

26 August 2011 at 14:18

Status:

Needs review

» Needs work

The last submitted patch, adding_hook_file_insert_example-1260162-2.patch, failed testing.

Log in or register to post comments

Comment #5

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 2 September 2011 at 14:39

Seems to have too many parens in this line:

+  if (empty($validate))) {

Also, this bit at the end of the patch needs to be removed:

diff --git a/scripts/drupal.sh b/scripts/drupal.sh
old mode 100755
new mode 100644
diff --git a/scripts/password-hash.sh b/scripts/password-hash.sh
old mode 100755
new mode 100644
diff --git a/scripts/run-tests.sh b/scripts/run-tests.sh
old mode 100755
new mode 100644