You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "David Peterson (JIRA)" <de...@tapestry.apache.org> on 2007/06/12 20:57:25 UTC

[jira] Created: (TAPESTRY-1565) Rename 'BeginRender' annotation as 'BeforeRender'

Rename 'BeginRender' annotation as 'BeforeRender'
-------------------------------------------------

                 Key: TAPESTRY-1565
                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1565
             Project: Tapestry
          Issue Type: Improvement
          Components: tapestry-core
    Affects Versions: 5.0.5
         Environment: N/A
            Reporter: David Peterson
            Priority: Trivial


For consistency with the other component rendering annotations, @BeginRender should be named @BeforeRender.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org


[jira] Closed: (TAPESTRY-1565) Rename 'BeginRender' annotation as 'BeforeRender'

Posted by "David Peterson (JIRA)" <de...@tapestry.apache.org>.
     [ https://issues.apache.org/jira/browse/TAPESTRY-1565?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Peterson closed TAPESTRY-1565.
------------------------------------

    Resolution: Won't Fix

I hear what you say.  I agree. In fact, it's probably not worth spending another second thinking about. I'll close the issue as "won't fix".

> Rename 'BeginRender' annotation as 'BeforeRender'
> -------------------------------------------------
>
>                 Key: TAPESTRY-1565
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1565
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.0.5
>         Environment: N/A
>            Reporter: David Peterson
>            Priority: Trivial
>
> For consistency with the other component rendering annotations, @BeginRender should be named @BeforeRender.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org


[jira] Commented: (TAPESTRY-1565) Rename 'BeginRender' annotation as 'BeforeRender'

Posted by "Howard M. Lewis Ship (JIRA)" <de...@tapestry.apache.org>.
    [ https://issues.apache.org/jira/browse/TAPESTRY-1565?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12503985 ] 

Howard M. Lewis Ship commented on TAPESTRY-1565:
------------------------------------------------

I realize there's a consistency issue here, but I'm not sure BeforeRender is really any better, because it kind of implies that there's a Render phase (and there isn't).  Naming it BeginRender really identifies the phase as the main phase where rendering begins (or at least starts).  Still open for discussion while the code is alpha, but still.

> Rename 'BeginRender' annotation as 'BeforeRender'
> -------------------------------------------------
>
>                 Key: TAPESTRY-1565
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1565
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.0.5
>         Environment: N/A
>            Reporter: David Peterson
>            Priority: Trivial
>
> For consistency with the other component rendering annotations, @BeginRender should be named @BeforeRender.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org