Once #2022509: Deprecate taxonomy_vocabulary_load_multiple() and taxonomy_term_load_multiple() in favour of entity_load_multiple() lands, we need to remove calls to taxonomy_vocabulary_load and taxonomy_vocabulary_load_multiple because they will be deprecated.

We must use entity_load('taxonomy_vocabulary', $term->bundle()); and entity_load_multiple('taxonomy_term', $tids); instead.

CommentFileSizeAuthor
#5 rename-taxonomy-load-multi-2049369-5.patch1.7 KBrvilar
#2 rename-taxonomy-load-multi-2049369-2.patch1.61 KBrvilar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Postponed » Active

The issue got in, not postponed anymore

rvilar’s picture

rvilar
Primary language Catalan
Location Montcada i Reixac, Catalonia
CreditAttribution: rvilar commented
Assigned: Unassigned » rvilar
Status: Active » Needs review
FileSize
rename-taxonomy-load-multi-2049369-2.patch1.61 KB

Removed calls for old functions and replaced for entity_load_multiple.

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented

2: rename-taxonomy-load-multi-2049369-2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: rename-taxonomy-load-multi-2049369-2.patch, failed testing.

rvilar’s picture

rvilar
Primary language Catalan
Location Montcada i Reixac, Catalonia
CreditAttribution: rvilar commented
Issue summary: View changes
FileSize
rename-taxonomy-load-multi-2049369-5.patch1.7 KB
pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Needs work » Needs review
pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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