Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8120610

EditingCellProperty is not set to null

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: 8
    • Fix Version/s: 8
    • Component/s: javafx

      Description

      In the description of the "editingCellProperty", we have :
      "Represents the current cell being edited, or null if there is no cell being edited."

      It's true at the very beginning, but once you've edited one cell. It will never be null again. Instead you do in "TableCell":
      // reset the editing cell on the TableView
                  table.edit(-1, null);

      So technically speaking, it's not set to null.. So I don't know if it's important or not, but maybe specify a bit the doc because the test :
      getEditingCell() == null
      is not a good test if someone wants to know if the TableView is editing something right now.

      And the test :
      getEditingCell().getRow() == -1

      is not good either because an NPE will be thrown the first time.

        Attachments

          Activity

            People

            Assignee:
            jgiles Jonathan Giles
            Reporter:
            shadzic Samir Hadzic
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Imported: