[weld-issues] [JBoss JIRA] Closed: (WELDX-187) Rename (or deprecate) @Veto in favor of @Vetoed

Pete Muir (JIRA) jira-events at lists.jboss.org
Mon Nov 8 10:19:01 EST 2010


     [ https://jira.jboss.org/browse/WELDX-187?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Pete Muir closed WELDX-187.
---------------------------

         Assignee: Pete Muir
    Fix Version/s:     (was: 1.0.0.Beta2)
       Resolution: Rejected


I'm not convinced for a number of reasons:

* Veto is both a noun and a verb, Vetoed is definitely a the past tense of the verb veto
* Consistency is not along the "axis" of annotation target, but along correctness in speech
* Changing the API now is not a good idea (unless critical)
* Consistency with CDI programmatic API (where you call processAnnotatedType.veto() to veto installation)
* I also think with the annotation you are requesting that an action be taken (that CDI vetoes the bean deployment)

In many ways @Vetoes would be most consistent IMO, however it looks very strange.

> Rename (or deprecate) @Veto in favor of @Vetoed
> -----------------------------------------------
>
>                 Key: WELDX-187
>                 URL: https://jira.jboss.org/browse/WELDX-187
>             Project: Weld Extensions
>          Issue Type: Feature Request
>    Affects Versions: 1.0.0.Beta1
>            Reporter: Dan Allen
>            Assignee: Pete Muir
>
> All the major type annotations in CDI are adjectives or nouns. For example:
> @Named
> @Decorator
> @Typed
> @Model
> @*Scoped
> @Alternative
> However, @Veto is an action (verb). Thus, to be consistent with the CDI naming convention, the @Veto annotation should be renamed to @Vetoed.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


More information about the weld-issues mailing list