Closed (fixed)
Project:
Drupal core
Version:
9.4.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
20 Mar 2013 at 23:20 UTC
Updated:
25 Apr 2022 at 20:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jhodgdonHm. Do you know what this region callback is supposed to do? I couldn't figure it out. I'm also not sure where we should document it.
Comment #2
joachim commentedGit blame tells us it was added in #616240: Make Field UI screens extensible from contrib - part II. It looks like it's responsible for telling a FormAPI pre-render callback which of the regions of the table to put a row into. On the manage fields page, the regions are 'main' and 'add_new'; on the display fields page, they are 'visible' and 'hidden'.
Comment #13
quietone commentedThis came up as the random bug to triage in #bugsmash a few days.
Setting to a task because this is adding new documentation because are for 'incorrect or misleading documentation'.
#region_callback is only used in \Drupal\field_ui\Element\FieldUiTable::tablePreRender()
I then made a patch for review.
Comment #14
joachim commentedThanks! Patch looks good.
(But surely missing documentation is a bug?)
Comment #17
catchIf we're missing documentation for something that many people are likely to actually use, then it probably counts as 'incorrect/misleading'.
This though is barely an API at all - although since you opened the issue perhaps you were trying to use it?