Return-Path: X-Original-To: apmail-commons-issues-archive@minotaur.apache.org Delivered-To: apmail-commons-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 3CF3D92D8 for ; Tue, 12 Jun 2012 20:57:45 +0000 (UTC) Received: (qmail 75867 invoked by uid 500); 12 Jun 2012 20:57:44 -0000 Delivered-To: apmail-commons-issues-archive@commons.apache.org Received: (qmail 75808 invoked by uid 500); 12 Jun 2012 20:57:44 -0000 Mailing-List: contact issues-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: issues@commons.apache.org Delivered-To: mailing list issues@commons.apache.org Received: (qmail 75797 invoked by uid 99); 12 Jun 2012 20:57:44 -0000 Received: from issues-vm.apache.org (HELO issues-vm) (140.211.11.160) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 12 Jun 2012 20:57:44 +0000 Received: from isssues-vm.apache.org (localhost [127.0.0.1]) by issues-vm (Postfix) with ESMTP id 7C52B1402B5 for ; Tue, 12 Jun 2012 20:57:44 +0000 (UTC) Date: Tue, 12 Jun 2012 20:57:42 +0000 (UTC) From: "Benedikt Ritter (JIRA)" To: issues@commons.apache.org Message-ID: <171411319.9239.1339534664511.JavaMail.jiratomcat@issues-vm> In-Reply-To: <830684732.9189.1339533823003.JavaMail.jiratomcat@issues-vm> Subject: [jira] [Updated] (SANDBOX-419) [BeanUtils2] Implement setIndexed(String propertyName) on DefaultBeanAccessor MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/SANDBOX-419?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benedikt Ritter updated SANDBOX-419: ------------------------------------ Attachment: SANDBOX-419.patch I've created a patch for this issue. Summary: * Changed {{DefaultIndexedPropertySetter}} to implement {{IndexedPropertySetterAccessor}} (rather than {{IndexedPropertyGetterAccessor}}) * Created a new implementation of {{BeanPropertySetter}} for indexed porperties. However this patch only servers for the purpose of discussion, since I'm unsure about two issues: # name of the new {{BeanPropertySetter}} implementation: Maybe it should be changed to "IndexedBeanPropertySetter" to make it clear that it is variant of DefaultBeanPropertySetter # checking of parameter types of the indexed setter method in {{IndexedPropertySetter}}: #* can we be sure that the first parameter type always will be of type int and the second parameter type will be the type of the indexed property? #* if so do we have to check if the first parameter is really of type int or do we only have to check if the second parameter type is assignment compatible to value? After we have agreed on this points I can create a new patch (or you can simply modify the patch). > [BeanUtils2] Implement setIndexed(String propertyName) on DefaultBeanAccessor > ----------------------------------------------------------------------------- > > Key: SANDBOX-419 > URL: https://issues.apache.org/jira/browse/SANDBOX-419 > Project: Commons Sandbox > Issue Type: Improvement > Components: BeanUtils2 > Affects Versions: Nightly Builds > Reporter: Benedikt Ritter > Attachments: SANDBOX-419.patch > > > Implement {{DefaultBeanAccessor.setIndexed(String propertyName)}} -- 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