Return-Path: Delivered-To: apmail-harmony-commits-archive@www.apache.org Received: (qmail 10757 invoked from network); 24 Apr 2009 13:39:54 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 24 Apr 2009 13:39:54 -0000 Received: (qmail 23980 invoked by uid 500); 24 Apr 2009 13:39:54 -0000 Delivered-To: apmail-harmony-commits-archive@harmony.apache.org Received: (qmail 23942 invoked by uid 500); 24 Apr 2009 13:39:54 -0000 Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@harmony.apache.org Delivered-To: mailing list commits@harmony.apache.org Received: (qmail 23933 invoked by uid 99); 24 Apr 2009 13:39:54 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Apr 2009 13:39:54 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Apr 2009 13:39:52 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 3B735234C003 for ; Fri, 24 Apr 2009 06:39:31 -0700 (PDT) Message-ID: <921143809.1240580371229.JavaMail.jira@brutus> Date: Fri, 24 Apr 2009 06:39:31 -0700 (PDT) From: "Tim Ellison (JIRA)" To: commits@harmony.apache.org Subject: [jira] Resolved: (HARMONY-6177) [classlib][beans] java.beans.PropertyDescriptor(String propertyName, Class beanClass, String getterName, String setterName) sets the wrong write method In-Reply-To: <865202908.1240565430335.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HARMONY-6177?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tim Ellison resolved HARMONY-6177. ---------------------------------- Resolution: Fixed Thanks Kevin. Patch applied to BEANS module at repo revision r768309. Please check it was applied as you expected. > [classlib][beans] java.beans.PropertyDescriptor(String propertyName, Class beanClass, String getterName, String setterName) sets the wrong write method > ---------------------------------------------------------------------------------------------------------------------------------------------------------- > > Key: HARMONY-6177 > URL: https://issues.apache.org/jira/browse/HARMONY-6177 > Project: Harmony > Issue Type: Bug > Components: Classlib > Affects Versions: 5.0M9 > Reporter: Kevin Zhou > Assignee: Tim Ellison > Fix For: 5.0M10 > > Attachments: HARMONY-6177.diff > > Original Estimate: 48h > Remaining Estimate: 48h > > Given a test case [1], RI passes it while HY fails. > The MockJSplitPane class extends javax.swing.JSplitPane and overwrite its setDividerLocation(int) method. > The test case creates a java.beans.PropertyDescriptor object using "dividerLocation" propertyName, null read method, and "setDividerLocation" write method. > Then, RI returns the correct method: > "public void PropertyDescriptorTest$MockJSplitPane.setDividerLocation(int)" > but HY returns a homony method but from its super class: > "public void javax.swing.JSplitPane.setDividerLocation(double)" > [1] Test Case: > class MockJSplitPane extends javax.swing.JSplitPane { > private static final long serialVersionUID = 3109235561744543400L; > public void setDividerLocation(int dividerLocation) { super.setDividerLocation(dividerLocation); } > } > public void test_setWriteMethod_MockJSplitPane() throws Exception { > Class beanClass = MockJSplitPane.class; > String propertyName = "dividerLocation"; > String setterName = "setDividerLocation"; > PropertyDescriptor pd = new PropertyDescriptor(propertyName, beanClass, null, setterName); > assertEquals(propertyName, pd.getName()); > assertNull(pd.getReadMethod()); > Method expectedMethod = beanClass.getMethod(setterName, new Class[] { int.class }); > assertEquals(expectedMethod, pd.getWriteMethod()); > } -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.