non-static fields should not be treated as static properties

Details

  • Type: Sub-task Sub-task
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.0
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Testcase included:
    yes

Description

> java.awt.Font.getName() // ok
> java.awt.Font.name
> throws
> java.lang.IllegalArgumentException

the 'name' field on Font is treated as a static property while the field not static.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul King added a comment -

Hi Gents, I placed a try catch around the offending piece of code. This hides the problem rather than solving the root cause. I didn't have time to investigate any further. At least we have a workaround and test in place now and we can tighten the noose around the real bug (and avoid the expensive of the exception) when we find time.

Show
Paul King added a comment - Hi Gents, I placed a try catch around the offending piece of code. This hides the problem rather than solving the root cause. I didn't have time to investigate any further. At least we have a workaround and test in place now and we can tighten the noose around the real bug (and avoid the expensive of the exception) when we find time.
Hide
Permalink
Alex Tkachman added a comment -

I prefer to postpone this issue till 1.2 or even till we rewrite of meta class. Following code shows that many things goes differently than expected.

package org.codehaus.groovy.classgen

class StaticPropertyTest extends GroovyTestCase{

    void testStaticProperty () {
        // U.class.getName () call
        assertEquals "U", U.getName()

        // should fail because getU () is private
        assertEquals "U", new U ().u

        // should fail because getU () is private
        assertEquals "U", new U ().getU()

        // should call U.class.getName () because 'name' is private in U
        assertEquals "org.codehaus.groovy.classgen.U", U.name

        assertEquals "U", U.testPrivate()
    }
}

class U {
    private static def name = "U"

    static def testPrivate () {
        return U.name
    }

    private static def getName () {"U"}

    private def getU () {"U"}
}
Show
Alex Tkachman added a comment - I prefer to postpone this issue till 1.2 or even till we rewrite of meta class. Following code shows that many things goes differently than expected. 
package org.codehaus.groovy.classgen

class StaticPropertyTest extends GroovyTestCase{

    void testStaticProperty () {
        // U.class.getName () call
        assertEquals "U", U.getName()

        // should fail because getU () is private
        assertEquals "U", new U ().u

        // should fail because getU () is private
        assertEquals "U", new U ().getU()

        // should call U.class.getName () because 'name' is private in U
        assertEquals "org.codehaus.groovy.classgen.U", U.name

        assertEquals "U", U.testPrivate()
    }
}

class U {
    private static def name = "U"

    static def testPrivate () {
        return U.name
    }

    private static def getName () {"U"}

    private def getU () {"U"}
}

People

  • Assignee:
    Unassigned
    Reporter:
    Dierk Koenig
Vote (0)
Watch (3)

Dates

  • Created:
    Updated: