Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Rename ReflectedXxx to ControllerXxx #1176

Closed
lodejard opened this issue Sep 24, 2014 · 1 comment
Closed

Rename ReflectedXxx to ControllerXxx #1176

lodejard opened this issue Sep 24, 2014 · 1 comment
Assignees
@sornaks
Labels
3 - Done enhancement
Milestone
1.0.0-beta1

Comments

@lodejard
Copy link
Contributor

ReflectedActionInvoker -> ControllerActionInvoker, ReflectedActionDescriptor -> ControllerActionDescriptor, etc.

The thing that makes them distinct is that they're based on controller classes, not that reflection is involved in the implementation
@danroth27 danroth27 added this to the 6.0.0-beta1 milestone Sep 30, 2014
sornaks pushed a commit that referenced this issue Oct 6, 2014
Issue #1176: Renaming ActionDescriptor, ActionDescriptorProvider, Act…
e4f959d 
…ionInvoker, ActionInvokerProvider, ActionExecutor, ModelBuilder, Tests. Either dropping "Reflected" or changing it to "Controller" depending on the context.
sornaks pushed a commit that referenced this issue Oct 7, 2014
Issue #1176: Renaming ActionDescriptor, ActionDescriptorProvider, Act…
a88f59f 
…ionInvoker, ActionInvokerProvider, ActionExecutor, ModelBuilder, Tests. Either dropping "Reflected" or changing it to "Controller" depending on the context.
@sornaks
Copy link

sornaks commented Oct 7, 2014

Checked in. a88f59f

@sornaks sornaks closed this as completed Oct 7, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sornaks sornaks

Labels
3 - Done enhancement
Projects
None yet
Milestone
1.0.0-beta1
Development

No branches or pull requests
4 participants
@yishaigalatzer @lodejard @danroth27 @sornaks