You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Howard M. Lewis Ship (JIRA)" <de...@tapestry.apache.org> on 2007/10/06 17:14:50 UTC

[jira] Created: (TAPESTRY-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
-----------------------------------------------------------------------------------------------------

                 Key: TAPESTRY-1810
                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
             Project: Tapestry
          Issue Type: New Feature
          Components: tapestry-ioc
    Affects Versions: 5.0.5
            Reporter: Howard M. Lewis Ship


In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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] Updated: (TAPESTRY-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

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

Howard M. Lewis Ship updated TAPESTRY-1810:
-------------------------------------------

    Fix Version/s: 5.1

> ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1810
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
>             Project: Tapestry
>          Issue Type: New Feature
>          Components: tapestry-ioc
>    Affects Versions: 5.0.5
>            Reporter: Howard M. Lewis Ship
>             Fix For: 5.1
>
>         Attachments: jira1810.zip
>
>
> In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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] Assigned: (TAPESTRY-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

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

Igor Drobiazko reassigned TAPESTRY-1810:
----------------------------------------

    Assignee: Igor Drobiazko

> ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1810
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
>             Project: Tapestry
>          Issue Type: New Feature
>          Components: tapestry-ioc
>    Affects Versions: 5.0.5
>            Reporter: Howard M. Lewis Ship
>            Assignee: Igor Drobiazko
>             Fix For: 5.1
>
>         Attachments: jira1810.zip
>
>
> In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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] Updated: (TAPESTRY-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

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

Paul Cooley updated TAPESTRY-1810:
----------------------------------

    Attachment: jira1810.zip

Created test module and test to prove the ServiceBinderImpl checks to see if the class passed into the ServiceBinder.bind(Class<T>) method is an interface.  If so, then the binder checks to see if a class exists in the same package with "Impl" as its suffix.

> ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1810
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
>             Project: Tapestry
>          Issue Type: New Feature
>          Components: tapestry-ioc
>    Affects Versions: 5.0.5
>            Reporter: Howard M. Lewis Ship
>         Attachments: jira1810.zip
>
>
> In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

Posted by "Josh Canfield (JIRA)" <de...@tapestry.apache.org>.
    [ https://issues.apache.org/jira/browse/TAPESTRY-1810?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12610340#action_12610340 ] 

Josh Canfield commented on TAPESTRY-1810:
-----------------------------------------

Nice, but I put my Impl classes in an impl subpackage...

> ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1810
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
>             Project: Tapestry
>          Issue Type: New Feature
>          Components: tapestry-ioc
>    Affects Versions: 5.0.5
>            Reporter: Howard M. Lewis Ship
>            Assignee: Igor Drobiazko
>             Fix For: 5.0.14
>
>         Attachments: jira1810.zip
>
>
> In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

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

Igor Drobiazko closed TAPESTRY-1810.
------------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 5.1)
                   5.0.14

> ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1810
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
>             Project: Tapestry
>          Issue Type: New Feature
>          Components: tapestry-ioc
>    Affects Versions: 5.0.5
>            Reporter: Howard M. Lewis Ship
>            Assignee: Igor Drobiazko
>             Fix For: 5.0.14
>
>         Attachments: jira1810.zip
>
>
> In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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-1810) ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.

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

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

I appreciate the effort of this "patch", but just providing new code is actually less useful in most cases than providing a real patch that can be applied by an IDE.  Subversion can generate such a patch file from a modified workspace.

> ServiceBinder.bind() with an interface should check to see if there's an Impl class and bind to that.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1810
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1810
>             Project: Tapestry
>          Issue Type: New Feature
>          Components: tapestry-ioc
>    Affects Versions: 5.0.5
>            Reporter: Howard M. Lewis Ship
>         Attachments: jira1810.zip
>
>
> In a module bind() method, if you invoke ServiceBinder.bind() and pass an interface, the code should check to see if there's a class in the same package named "XYZImpl" (for interface XYZ).  If so, it should bind that Impl class as the implementation class.  Convention over configuration, even in the small details.

-- 
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