Remove UndoRedo calls trying to call removed EditorInspector::refresh()

[kleonc]

Fixes #61267.

EditorInspector::refresh() was removed in #45879. I think the removed code doesn't need to be replaced by anything as now inspector is meant to auto detect the changes and refresh itself automatically.
I've searched for "refresh", haven't found any more leftovers.

[akien-mga]

Thanks!

[Rindbee]

Regarding undo/redo in Inspector, if you want to refresh immediately, you still need to record Inspector::_edit_request_change. Otherwise, the refresh interval will depend on docks/property_editor/auto_refresh_interval (if you set it to 10s, the association will be easily discovered).

I had tried to fix this issue before. In #60792.

[kleonc]

Regarding undo/redo in Inspector, if you want to refresh immediately, you still need to record Inspector::_edit_request_change. Otherwise, the refresh interval will depend on docks/property_editor/auto_refresh_interval (if you set it to 10s, the association will be easily discovered).

Isn't it like that for any undoed/redoed property visible in the inspector though? I haven't checked the relevant code deeply so I'm not sure. But if that's the case then it would be strange to refresh immediately just in case of MultiNodeEdit (instead of for any edited Object).

Also Inspector::_edit_request_change is a private method and currently it's being used only within the EditorInspector. So calling it externally seems wrong. And doing so is possible because it's binded to ClassDB just so the EditorInspector itself could add such calls to its own UndoRedo actions (so this usage is kinda internal).

[Rindbee]

Just thought the intent of the previous code was to refresh immediately.

Here, wherever the refresh code is placed, it feels a little awkward. It feels like MultiNodeEdit is the part that separated from EditorInspector::_edit_set, but there's a strong connection.