Fix description overwritten

Author: oowekyala

src/main/java/net/sourceforge/pmd/util/fxdesigner/model/PropertyDescriptorSpec.java

@@ -147,8 +147,8 @@ public Var<String> valueProperty() {
      * @return An xml string
      */
     public String toXml() {
-        return String.format("<property name=\"%s\" type=\"%s\" value=\"%s\" />",
-                             getName(), getTypeId().getStringId(), getValue());
+        return String.format("<property name=\"%s\" type=\"%s\" value=\"%s\" description=\"%s\"/>",
+                             getName(), getTypeId().getStringId(), getValue(), getDescription());
     }
 
 

src/main/java/net/sourceforge/pmd/util/fxdesigner/popups/ExportXPathWizardController.java

@@ -8,7 +8,6 @@
 import static com.github.oowekyala.rxstring.ItemRenderer.indented;
 import static com.github.oowekyala.rxstring.ItemRenderer.surrounded;
 import static com.github.oowekyala.rxstring.ItemRenderer.wrapped;
-import static net.sourceforge.pmd.util.fxdesigner.util.AuxLanguageRegistry.getSupportedLanguageVersions;
 import static net.sourceforge.pmd.util.fxdesigner.util.DesignerUtil.stringConverter;
 import static net.sourceforge.pmd.util.fxdesigner.util.reactfx.ReactfxUtil.rewireInit;
 
@@ -173,10 +172,7 @@ public Subscription bindToRuleBuilder(ObservableXPathRuleBuilder ruleBuilder) {
 
 
     private void initialiseLanguageChoiceBox() {
-        languageChoiceBox.getItems().addAll(getSupportedLanguageVersions().stream()
-                                                                          .map(LanguageVersion::getLanguage)
-                                                                          .distinct()
-                                                                          .collect(Collectors.toList()));
+        languageChoiceBox.getItems().setAll(AuxLanguageRegistry.getSupportedLanguages().collect(Collectors.toList()));
 
         languageChoiceBox.setConverter(stringConverter(Language::getShortName, AuxLanguageRegistry::findLanguageByShortName));
     }
@@ -286,6 +282,7 @@ private static LiveTemplateBuilder<ObservableXPathRuleBuilder> liveTemplateBuild
                             .append("<property name=\"").bind(PropertyDescriptorSpec::nameProperty)
                             .append("\" type=\"").bind(PropertyDescriptorSpec::typeIdProperty, PropertyTypeId::getStringId)
                             .append("\" value=\"").bind(PropertyDescriptorSpec::valueProperty)
+                            .append("\" description=\"").bind(PropertyDescriptorSpec::descriptionProperty)
                             .appendLine("\"/>")
             )
             .appendIndent(2).append("<property name=\"version\" value=\"").bind(ObservableXPathRuleBuilder::xpathVersionProperty).appendLine("\"/>")

src/main/java/net/sourceforge/pmd/util/fxdesigner/util/controls/PopOverWrapper.java

@@ -19,6 +19,7 @@
 import net.sourceforge.pmd.util.fxdesigner.util.DesignerUtil;
 
 import javafx.application.Platform;
+import javafx.event.EventHandler;
 import javafx.stage.Stage;
 import javafx.stage.Window;
 import javafx.stage.WindowEvent;
@@ -34,6 +35,7 @@ public final class PopOverWrapper<T> {
     private final Var<PopOver> myPopover = Var.newSimpleVar(null);
     private BiFunction<T, PopOver, PopOver> rebinder;
     private T identity;
+    private EventHandler<WindowEvent> userOnHiding = we -> {};
 
     public PopOverWrapper(BiFunction<T, @Nullable PopOver, @Nullable PopOver> rebinder) {
         this.identity = null;
@@ -56,7 +58,12 @@ public void showOrFocus(Consumer<@NonNull PopOver> showMethod) {
     }
 
     public void rebind(T identity) {
-        preload(() -> rebinder.apply(identity, myPopover.getValue()));
+        if (identity != this.identity) {
+            this.identity = identity;
+            this.userOnHiding.handle(null);
+            this.userOnHiding = we -> {};
+            preload(() -> rebinder.apply(identity, myPopover.getValue()));
+        }
     }
 
     private void preload(Supplier<PopOver> supplier) {
@@ -68,6 +75,14 @@ private void preload(Supplier<PopOver> supplier) {
             myPopover.setValue(null);
             return;
         }
+
+        this.userOnHiding = popOver.getOnHiding();
+
+        popOver.setOnHiding(we -> {
+            this.identity = null;
+            this.userOnHiding.handle(we);
+        });
+
         popOver.getRoot().getStylesheets().addAll(DesignerUtil.getCss("popover").toString());
         popOver.getRoot().applyCss();
         myPopover.setValue(popOver);

src/main/java/net/sourceforge/pmd/util/fxdesigner/util/controls/PropertyCollectionView.java

@@ -7,7 +7,6 @@
 import java.io.IOException;
 import java.util.Comparator;
 import java.util.Objects;
-import java.util.Optional;
 
 import org.apache.commons.lang3.StringUtils;
 import org.checkerframework.checker.nullness.qual.NonNull;
@@ -175,7 +174,6 @@ private PopOver rebindPopover(PropertyDescriptorSpec newSpec, PopOver pop) {
             // create it
             return detailsPopOver(newSpec);
         }
-        Optional.ofNullable(pop.getOnHiding()).ifPresent(it -> it.handle(null));
 
         pop.titleProperty().bind(newSpec.nameProperty()
                                         .filter(StringUtils::isNotBlank)