Here is a patch for hook_field_update_forbid(). Trying to help out with #675046: Make sure all hooks in D7 have documentation.

CommentFileSizeAuthor
#11 775936-revised.patch1.16 KBfp
#8 775936-revised.patch1.16 KBfp
#6 775936-revised.patch1.12 KBfp
#5 775936-revised.patch1.15 KBjhodgdon
#3 hook_field_update_forbid_doc-0.patch1.16 KBnvahalik
#1 hook_field_update_forbid_doc.patch838 bytesnvahalik

Comments

nvahalik’s picture

nvahalik commented
StatusFileSize
new hook_field_update_forbid_doc.patch838 bytes

Here is the patch, it never works when I create the issue...

Status: Needs review » Needs work

The last submitted patch, hook_field_update_forbid_doc.patch, failed testing.

nvahalik’s picture

nvahalik commented
StatusFileSize
new hook_field_update_forbid_doc-0.patch1.16 KB

Let's try this one...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs work » Needs review

You have to set the status to "needs review" to launch the test bot. :)

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
StatusFileSize
new 775936-revised.patch1.15 KB

Thanks, this is mostly good! I've just made a few small edits. I like the example - seems very clear.

fp’s picture

fp commented
StatusFileSize
new 775936-revised.patch1.12 KB

Unnecessary capitalization fixed.

Status: Needs review » Needs work

The last submitted patch, 775936-revised.patch, failed testing.

fp’s picture

fp commented
StatusFileSize
new 775936-revised.patch1.16 KB

And this one from root...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs work » Needs review

launching test bot

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Needs work

Ooops. "FieldUpdateForbiddenObjection" should be Exception. I think that one was my fault. :)

fp’s picture

fp commented
StatusFileSize
new 775936-revised.patch1.16 KB

And I missed it.

fp’s picture

fp commented
Status: Needs work » Needs review

test

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

:)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Awesome! Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

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