You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Benedikt Ritter (JIRA)" <ji...@apache.org> on 2012/06/18 21:09:42 UTC

[jira] [Created] (SANDBOX-422) [BeanUtils2] Wrap checked exceptions into unchecked exception

Benedikt Ritter created SANDBOX-422:
---------------------------------------

             Summary: [BeanUtils2] Wrap checked exceptions into unchecked exception
                 Key: SANDBOX-422
                 URL: https://issues.apache.org/jira/browse/SANDBOX-422
             Project: Commons Sandbox
          Issue Type: Bug
          Components: BeanUtils2
    Affects Versions: Nightly Builds
            Reporter: Benedikt Ritter
             Fix For: Nightly Builds


As [discussed on the ML|http://markmail.org/thread/yetl76y734dnt5zz] the checked exceptions that are declared by the java reflection API should be wrapped into a custom RuntimeException (BeanReflectionException).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (SANDBOX-422) [BeanUtils2] Wrap checked exceptions into unchecked exception

Posted by "Simone Tripodi (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SANDBOX-422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13423390#comment-13423390 ] 

Simone Tripodi commented on SANDBOX-422:
----------------------------------------

last check I do is using two priceless tools: the eyes! :)

there were just too much/less empty lines, nothing to be worried ;)
                
> [BeanUtils2] Wrap checked exceptions into unchecked exception
> -------------------------------------------------------------
>
>                 Key: SANDBOX-422
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-422
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>            Assignee: Simone Tripodi
>             Fix For: Nightly Builds
>
>         Attachments: SANDBOX-422.patch
>
>
> As [discussed on the ML|http://markmail.org/thread/yetl76y734dnt5zz] the checked exceptions that are declared by the java reflection API should be wrapped into a custom RuntimeException (BeanReflectionException).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (SANDBOX-422) [BeanUtils2] Wrap checked exceptions into unchecked exception

Posted by "Benedikt Ritter (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SANDBOX-422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benedikt Ritter updated SANDBOX-422:
------------------------------------

    Attachment: SANDBOX-422.patch

Hi,

as explained on SANDBOX-423, I had to create a patch that removes all checked exceptions from the API. Therefore I created a hierachy of unchecked exceptions, that serve as wrappers.

TIA for reviewing this huge patch!
Benedikt
                
> [BeanUtils2] Wrap checked exceptions into unchecked exception
> -------------------------------------------------------------
>
>                 Key: SANDBOX-422
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-422
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>             Fix For: Nightly Builds
>
>         Attachments: SANDBOX-422.patch
>
>
> As [discussed on the ML|http://markmail.org/thread/yetl76y734dnt5zz] the checked exceptions that are declared by the java reflection API should be wrapped into a custom RuntimeException (BeanReflectionException).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (SANDBOX-422) [BeanUtils2] Wrap checked exceptions into unchecked exception

Posted by "Simone Tripodi (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SANDBOX-422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Simone Tripodi resolved SANDBOX-422.
------------------------------------

    Resolution: Fixed
      Assignee: Simone Tripodi

patch applied, see r1366010 - thanks!!!

I just fixed some code format incongruences and dropped few useless statements ATM but didn't see a reason to keep it uncommitted
                
> [BeanUtils2] Wrap checked exceptions into unchecked exception
> -------------------------------------------------------------
>
>                 Key: SANDBOX-422
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-422
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>            Assignee: Simone Tripodi
>             Fix For: Nightly Builds
>
>         Attachments: SANDBOX-422.patch
>
>
> As [discussed on the ML|http://markmail.org/thread/yetl76y734dnt5zz] the checked exceptions that are declared by the java reflection API should be wrapped into a custom RuntimeException (BeanReflectionException).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (SANDBOX-422) [BeanUtils2] Wrap checked exceptions into unchecked exception

Posted by "Benedikt Ritter (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SANDBOX-422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13423335#comment-13423335 ] 

Benedikt Ritter commented on SANDBOX-422:
-----------------------------------------

Thanks a lot!

how do you make sure the formatting is okay? I just run mvn site and look at the reports. Do you use the eclipse checkstyle plugin? If so, can you add the configuration to the svn repo?

TIA!
                
> [BeanUtils2] Wrap checked exceptions into unchecked exception
> -------------------------------------------------------------
>
>                 Key: SANDBOX-422
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-422
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>            Assignee: Simone Tripodi
>             Fix For: Nightly Builds
>
>         Attachments: SANDBOX-422.patch
>
>
> As [discussed on the ML|http://markmail.org/thread/yetl76y734dnt5zz] the checked exceptions that are declared by the java reflection API should be wrapped into a custom RuntimeException (BeanReflectionException).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira