Uploaded image for project: 'Commons BeanUtils'
  1. Commons BeanUtils
  2. BEANUTILS-56

populate method has mistaken the judgment which uses indexed poperty.

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • Nightly Builds
    • None
    • None
    • None
    • Operating System: All
      Platform: PC

    • 6125

    Description

      If populate method has indexed setter even when not attached propertyname
      [index] to name, it will use indexed property.
      If propertyname[index] is not attached to name, I think that I should not use
      indexed property.

      I think that I should make it as follows.

      --------------------------------------
      if (descriptor instanceof IndexedPropertyDescriptor)
      setter = ((IndexedPropertyDescriptor) descriptor).
      getIndexedWriteMethod();
      --------------------------------------
      to
      --------------------------------------
      if (descriptor instanceof IndexedPropertyDescriptor) &&
      name.indexOf(PropertyUtils.INDEXED_DELIM) > 0)
      setter = ((IndexedPropertyDescriptor) descriptor).
      getIndexedWriteMethod();
      ---------------------------------------

      Attachments

        Activity

          People

            Unassigned Unassigned
            mail@pocketclub.com Matoba Tatsunori
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: