Summary

Store and organize action views by attachment point.

Review Request #14663 — Created Oct. 31, 2025 and updated Nov. 7, 2025, 10:31 a.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-7.1.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

ActionView now takes the attachment point that the view is placed in,
for easy lookup and for registration.

The views are no longer stored in one long list. Instead, they're
organized by the attachment point. This will ultimately allow for
multiple views for the same action in different attachment points, and
to enable looking up the correct action within the right attachment
point.

Testing Done

Unit tests passed.

Tested that all actions, including menus and Quick Access actions,
all worked and expected.

Tested that Quick Access actions could be customized.

Commits

Summary	ID
Store and organize action views by attachment point. `ActionView` now takes the attachment point that the view is placed in, for easy lookup and for registration. The views are no longer stored in one long list. Instead, they're organized by the attachment point. This will ultimately allow for multiple views for the same action in different attachment points, and to enable looking up the correct action within the right attachment point.	0acd57412d565c59c180330c9250261a7dc23cd2

Summary

Store and organize action views by attachment point.

`ActionView` now takes the attachment point that the view is placed in, for easy lookup and for registration. The views are no longer stored in one long list. Instead, they're organized by the attachment point. This will ultimately allow for multiple views for the same action in different attachment points, and to enable looking up the correct action within the right attachment point.

0acd57412d565c59c180330c9250261a7dc23cd2

Issues

Description	From	Last Updated
This needs a "Version Changed" block.	david	Nov. 7, 2025, 10:31 a.m.
I'm not sure I have all your changes in my head at the same time, so it's possible that this …	david	Nov. 7, 2025, 10:31 a.m.
This needs to be added to the Args section.	maubin	Nov. 4, 2025, 8:17 p.m.
It might be nice to fail more gracefully if attachmentPointID is not a valid key.	david	Nov. 7, 2025, 10:31 a.m.
It might be nice to fail more gracefully if attachmentPointID is not a valid key. Right now the call to …	david	Nov. 7, 2025, 10:31 a.m.
Should we \|escapejs this? I guess there could be a scenario where a malicious action author sets the attachment_point_id to …	maubin	Nov. 4, 2025, 8:17 p.m.

flake8 passed.

JSHint passed.

Fix it!

reviewboard/static/rb/js/common/views/pageView.ts (Diff revision 1)
An issue was opened. Show all issues
```
This needs to be added to the Args section.
```

reviewboard/templates/actions/action.js (Diff revision 1)

An issue was opened. Show all issues

Should we |escapejs this? I guess there could be a scenario where a malicious action author sets the attachment_point_id to something hacky.

Fix it!

reviewboard/static/rb/js/common/views/pageView.ts (Diff revision 1)
An issue was opened. Show all issues
```
This needs a "Version Changed" block.
```

reviewboard/static/rb/js/common/views/pageView.ts (Diff revision 1)

An issue was opened. Show all issues

I'm not sure I have all your changes in my head at the same time, so it's possible that this is addressed in a different change, but I see that MenuActionView calls page.getActionView() with just the child ID, and there's nothing about that in this change. Is that changed elsewhere?

reviewboard/static/rb/js/common/views/pageView.ts (Diff revision 1)
An issue was opened. Show all issues
```
It might be nice to fail more gracefully if attachmentPointID is not a valid key.
```

reviewboard/static/rb/js/common/views/pageView.ts (Diff revision 1)

An issue was opened. Show all issues

It might be nice to fail more gracefully if attachmentPointID is not a valid key. Right now the call to _.filter will probably error out.