Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
API page: http://api.drupal.org/api/drupal/modules%21taxonomy%21taxonomy.module/fu...
Enter a descriptive title (above) relating to taxonomy_term_load, then describe the problem you have found:
If $tid not found, FALSE is returned. NULL would be also assumable.
Comment | File | Size | Author |
---|---|---|---|
#10 | taxonomy_term_load-does-not-1689426-10.patch | 526 bytes | mjonesdinero |
#6 | taxonomy_term_load-does-not-1689426-6.patch | 582 bytes | mjonesdinero |
#4 | taxonomy_term_load-does-not-1689426-4.patch | 578 bytes | mjonesdinero |
#2 | taxonomy_term_load-does-not-1689426-2.patch | 514 bytes | mjonesdinero |
added_Return_message1.patch | 539 bytes | ro-no-lo | |
Comments
Comment #1
jhodgdonThanks for the patch! See http://drupal.org/node/1689394#comment-6244396 for a note on patch formatting.
So this patch needs to be rerolled using Git, and also:
- Line wrapping -- move the added text up to the previous line, and wrap at 80 characters.
- Add "the" to "FALSE if *the* term was not found.".
Comment #2
mjonesdinero CreditAttribution: mjonesdinero commentedre roll the patch..
Comment #4
mjonesdinero CreditAttribution: mjonesdinero commentedsorry about the last patch submitted it was for d7 i thought it is for d7 but also in d8..
attach is the patch now for d8.
Comment #5
jhodgdonThanks!
This line is missing a space:
And maybe it would be better to say "... term entity, or FALSE if the term ..." rather than putting that at the end?
Comment #6
mjonesdinero CreditAttribution: mjonesdinero commentedokie re-rolled the patch, git pull first for the update on the patch commited.
attach is the updated patch now..
Comment #7
mjonesdinero CreditAttribution: mjonesdinero commentedComment #8
jhodgdonLooks good, thanks! I'll get it committed shortly.
Comment #9
jhodgdonCommitted to 8.x. The patch doesn't apply to 7.x, so we need a reroll.
Comment #10
mjonesdinero CreditAttribution: mjonesdinero commentedre-rolled the patch for D7
Comment #11
jhodgdonThanks! I'll get this committed soon.
Comment #12
jhodgdonCommitted to 7.x now too. Thanks again!