_field_info_collate_fields() and field_info_collate_types() still use a direct static.

Patch moves them to drupal_static() - they need to use the 'advanced pattern', though.

CommentFileSizeAuthor
#7 field_info_static-705124-7.patch1.47 KByched
field_info_drupal_static.patch1.47 KByched

Comments

yched’s picture

yched commented

field_info_drupal_static.patch queued for re-testing.

yched’s picture

yched commented
Assigned: Unassigned » yched

easier tracking

Scott Reynolds’s picture

Scott Reynolds commented
Status: Needs review » Closed (duplicate)

#447862-24: Convert field to the new static caching API merged this patch into that.

yched’s picture

yched commented
Status: Closed (duplicate) » Needs review

#447862-24: Convert field to the new static caching API has API changes, highly undesirable at this point.
So reopening this one.

yched’s picture

yched commented

field_info_drupal_static.patch queued for re-testing.

Status: Needs review » Needs work

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

yched’s picture

yched commented
Status: Needs work » Needs review
StatusFileSize
new field_info_static-705124-7.patch1.47 KB

Rerolled

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Closed (won't fix)

Looks obsolete to me.