search

chkal / mvc-spec-migration-test

0 stars 0 forks source link

Consider renaming Mvc to MvcContext #54

Open chkal opened 8 years ago

chkal commented 8 years ago

Original issue MVC_SPEC-53 created by Santiago Pericas-Geertsen:

Should we do this or not?

chkal commented 8 years ago

Comment by rahmanusta:

I think, we should do

chkal commented 8 years ago

Comment by tommy94:

I think so

chkal commented 8 years ago

Comment by Christian Kaltepoth:

I think it is important to have short name used for EL expressions. ${mvc.contextPath} is much nicer compared to ${mvcContext.contextPath}. Especially because developers will need to type this very often.

One way to have a short EL name and a more descriptive class name would be to use something like this:

@Named("mvc")
public class MvcContext {}

But that looks inconsistent.

chkal commented 8 years ago

Comment by rahmanusta:

Yes, sure the bean name may remain "mvc"

chkal commented 8 years ago

Comment by Santiago Pericas-Geertsen:

Although not consistent with other specs like JSF, I also dislike a longer name in EL. I'm in favor of @Named("mvc").