bugfixing.

Author: JavaSaBr

build.gradle

@@ -28,7 +28,7 @@ ext.applicationMainClass = "com.ss.editor.JfxApplication"
 ext.applicationVendor = "[email protected]"
 ext.applicationTitle = "jMonkeyBuilder"
 ext.jmeVersion = "v3.3.dev-SNAPSHOT"
-ext.jmbExtVersion = "2.1.0"
+ext.jmbExtVersion = "develop-SNAPSHOT"
 ext.jme3_xbuf_version = '0.9.1'
 ext.junitPlatformVersion = "1.0.0"
 ext.junitJupiterVersion = "5.0.0"

embedded-plugins/jmb-tree-generator/jmb-tree-generator-1.1.0.jar renamed to embedded-plugins/jmb-tree-generator/jmb-tree-generator-1.1.1.jar

@@ -1,10 +1,7 @@
 package com.ss.editor.part3d.editor.impl.scene.handler;
 
-import com.jme3.bullet.control.PhysicsControl;
-import com.jme3.bullet.control.RigidBodyControl;
 import com.jme3.scene.Spatial;
-import com.ss.editor.util.ControlUtils;
-import com.ss.editor.util.NodeUtils;
+import com.ss.editor.extension.util.JmbExtUtils;
 import org.jetbrains.annotations.NotNull;
 
 import java.util.function.Consumer;
@@ -18,22 +15,6 @@ public class PhysicsControlTransformationHandler implements Consumer<Spatial> {
 
     @Override
     public void accept(@NotNull final Spatial spatial) {
-        NodeUtils.children(spatial)
-            .flatMap(ControlUtils::controls)
-            .filter(PhysicsControl.class::isInstance)
-            .filter(ControlUtils::isEnabled)
-            .forEach(control -> {
-                if (control instanceof RigidBodyControl) {
-                    final RigidBodyControl bodyControl = (RigidBodyControl) control;
-                    final boolean kinematic = bodyControl.isKinematic();
-                    final boolean kinematicSpatial = bodyControl.isKinematicSpatial();
-                    bodyControl.setKinematic(true);
-                    bodyControl.setKinematicSpatial(true);
-                    bodyControl.clearForces();
-                    bodyControl.update(0);
-                    bodyControl.setKinematic(kinematic);
-                    bodyControl.setKinematicSpatial(kinematicSpatial);
-                }
-            });
+        JmbExtUtils.resetPhysicsControlPositions(spatial);
     }
 }

src/main/java/com/ss/editor/part3d/editor/impl/scene/handler/PhysicsControlTransformationHandler.java

@@ -23,6 +23,7 @@ public void accept(@NotNull final Spatial spatial) {
                 .filter(RigidBodyControl.class::isInstance)
                 .map(RigidBodyControl.class::cast)
                 .filter(RigidBodyControl::isEnabled)
+                .filter(control -> Float.compare(control.getMass(), 0.0F) != 0)
                 .filter(control -> !control.isActive())
                 .forEach(PhysicsRigidBody::activate);
     }

src/main/java/com/ss/editor/part3d/editor/impl/scene/handler/ReactivatePhysicsControlsTransformationHandler.java

@@ -33,9 +33,9 @@
 /**
  * The base implementation of the property control.
  *
- * @param <C> the type of a change consumer
- * @param <D> the type of an editing object
- * @param <T> the type of an editing property
+ * @param <C> the type of a change consumer.
+ * @param <D> the type of an editing object.
+ * @param <T> the type of an editing property.
  * @author JavaSaBr
  */
 public class PropertyControl<C extends ChangeConsumer, D, T> extends VBox implements UpdatableControl {
@@ -171,10 +171,11 @@ public PropertyControl(@Nullable final T propertyValue, @NotNull final String pr
      * @return the six object consumer
      */
     @FromAnyThread
-    public @NotNull SixObjectConsumer<@NotNull C, @NotNull D, @NotNull String, @Nullable T, @Nullable T, @NotNull BiConsumer<D, T>> newChangeHandler() {
+    public @NotNull SixObjectConsumer<C, D, String, T,  T, BiConsumer<D, T>> newChangeHandler() {
         return (changeConsumer, object, propName, newValue, oldValue, handler) -> {
 
-            final PropertyOperation<ChangeConsumer, D, T> operation = new PropertyOperation<>(object, propName, newValue, oldValue);
+            final PropertyOperation<ChangeConsumer, D, T> operation =
+                new PropertyOperation<>(object, propName, newValue, oldValue);
             operation.setApplyHandler(handler);
 
             changeConsumer.execute(operation);
@@ -200,7 +201,11 @@ public void setEditObject(@NotNull final D editObject) {
     @FxThread
     public void setEditObject(@NotNull final D editObject, final boolean needReload) {
         setEditObject(editObject);
-        if (!needReload) return;
+
+        if (!needReload) {
+            return;
+        }
+
         setIgnoreListener(true);
         try {
             reload();

src/main/java/com/ss/editor/ui/control/property/PropertyControl.java

@@ -32,6 +32,7 @@
 /**
  * The implementation of the {@link PropertyControl} to edit the {@link AudioData}.
  *
+ * @param <C> the change consumer's type.
  * @author JavaSaBr
  */
 public class AudioKeyPropertyControl<C extends ChangeConsumer> extends PropertyControl<C, AudioNode, AudioKey> {
@@ -148,14 +149,18 @@ private void addAudioData(@NotNull final Path file) {
     protected void processOpen() {
 
         final AudioKey element = getPropertyValue();
-        if (element == null) return;
+        if (element == null) {
+            return;
+        }
 
         final String assetPath = element.getName();
         if (StringUtils.isEmpty(assetPath)) return;
 
         final Path assetFile = Paths.get(assetPath);
         final Path realFile = notNull(getRealFile(assetFile));
-        if (!Files.exists(realFile)) return;
+        if (!Files.exists(realFile)) {
+            return;
+        }
 
         final RequestedOpenFileEvent event = new RequestedOpenFileEvent();
         event.setFile(realFile);

src/main/java/com/ss/editor/ui/control/property/impl/AudioKeyPropertyControl.java

@@ -21,8 +21,8 @@
 /**
  * The implementation of the {@link PropertyControl} to change boolean values.
  *
- * @param <C> the type of a {@link ChangeConsumer}
- * @param <T> the type of an editing object
+ * @param <C> the type of a {@link ChangeConsumer}.
+ * @param <T> the type of an editing object.
  * @author JavaSaBr
  */
 public class BooleanPropertyControl<C extends ChangeConsumer, T> extends PropertyControl<C, T, Boolean> {

src/main/java/com/ss/editor/ui/control/property/impl/BooleanPropertyControl.java

@@ -17,7 +17,7 @@
 /**
  * The implementation of the {@link PropertyControl} to edit a color values.
  *
- * @param <C> the type parameter
+ * @param <C> the change
  * @param <T> the type parameter
  * @author JavaSaBr
  */
@@ -89,7 +89,10 @@ protected void reload() {
      */
     @FxThread
     private void updateValue() {
-        if (isIgnoreListener()) return;
+
+        if (isIgnoreListener()) {
+            return;
+        }
 
         final ColorPicker colorPicker = getColorPicker();
         final ColorRGBA newColor = UiUtils.from(colorPicker.getValue());

src/main/java/com/ss/editor/ui/control/property/impl/ColorPropertyControl.java

@@ -17,9 +17,9 @@
 /**
  * The default implementation of the property control.
  *
- * @param <C> the type parameter
- * @param <D> the type parameter
- * @param <T> the type parameter
+ * @param <C> the change consumer's type.
+ * @param <D> the edited object's type.
+ * @param <T> the edited property's type.
  * @author JavaSaBr
  */
 public class DefaultPropertyControl<C extends ChangeConsumer, D, T> extends PropertyControl<C, D, T> {

src/main/java/com/ss/editor/ui/control/property/impl/DefaultPropertyControl.java

@@ -10,9 +10,9 @@
 /**
  * The default implementation of the property control.
  *
- * @param <C> the type parameter
- * @param <D> the type parameter
- * @param <T> the type parameter
+ * @param <C> the change consumer's type.
+ * @param <D> the edited object's type.
+ * @param <T> the edited property's type.
  * @author JavaSaBr
  */
 public class DefaultSinglePropertyControl<C extends ChangeConsumer, D, T> extends DefaultPropertyControl<C, D, T> {