From 60f67db88aa5e21a1b64d3614023aa393c9964e8 Mon Sep 17 00:00:00 2001 From: codex <103840984+codex128@users.noreply.github.com> Date: Wed, 20 Mar 2024 15:42:14 -0400 Subject: [PATCH 1/4] switched to severe logging --- .../main/java/com/jme3/asset/ImplHandler.java | 9 +++--- .../java/jme3test/asset/TestIssue2230.java | 30 +++++++++++++++++++ 2 files changed, 35 insertions(+), 4 deletions(-) create mode 100644 jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java diff --git a/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java b/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java index 1b6275d12d..00000771df 100644 --- a/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java +++ b/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java @@ -124,10 +124,11 @@ protected T initialValue() { } catch (InstantiationException | IllegalAccessException | IllegalArgumentException | InvocationTargetException | NoSuchMethodException | SecurityException ex) { - logger.log(Level.SEVERE, "Cannot create locator of type {0}, does" - + " the class have an empty and publicly accessible" - + " constructor?", type.getName()); - logger.throwing(type.getName(), "", ex); +// logger.log(Level.SEVERE, "Cannot create locator of type {0}, does" +// + " the class have an empty and publicly accessible" +// + " constructor?", type.getName()); +// logger.throwing(type.getName(), "", ex); + logger.log(Level.SEVERE, "An exception occured while instantiating asset locator: " + type.getName(), ex); } return null; } diff --git a/jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java b/jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java new file mode 100644 index 0000000000..f1ea9942d2 --- /dev/null +++ b/jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java @@ -0,0 +1,30 @@ +/* + * Click nbfs://nbhost/SystemFileSystem/Templates/Licenses/license-default.txt to change this license + * Click nbfs://nbhost/SystemFileSystem/Templates/Classes/Class.java to edit this template + */ +package jme3test.asset; + +import com.jme3.app.SimpleApplication; +import com.jme3.asset.plugins.FileLocator; + +/** + * + * @author codex + */ +public class TestIssue2230 extends SimpleApplication { + + public static void main(String[] args) { + new TestIssue2230().start(); + } + + @Override + public void simpleInitApp() { + + assetManager.registerLocator("NonExistantFolder", FileLocator.class); + assetManager.loadModel("NonExistantModel.j3o"); + + stop(); + + } + +} From 7c0d1795473e1af6e134474f3c73636416082f20 Mon Sep 17 00:00:00 2001 From: codex <103840984+codex128@users.noreply.github.com> Date: Wed, 20 Mar 2024 15:44:57 -0400 Subject: [PATCH 2/4] removed previous code --- jme3-core/src/main/java/com/jme3/asset/ImplHandler.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java b/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java index 00000771df..a389570edc 100644 --- a/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java +++ b/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java @@ -124,10 +124,6 @@ protected T initialValue() { } catch (InstantiationException | IllegalAccessException | IllegalArgumentException | InvocationTargetException | NoSuchMethodException | SecurityException ex) { -// logger.log(Level.SEVERE, "Cannot create locator of type {0}, does" -// + " the class have an empty and publicly accessible" -// + " constructor?", type.getName()); -// logger.throwing(type.getName(), "", ex); logger.log(Level.SEVERE, "An exception occured while instantiating asset locator: " + type.getName(), ex); } return null; From 934dedc0c34dc45c3b63fca706b5ab4b32d5ba21 Mon Sep 17 00:00:00 2001 From: codex <103840984+codex128@users.noreply.github.com> Date: Wed, 20 Mar 2024 15:47:45 -0400 Subject: [PATCH 3/4] removed test --- .../java/jme3test/asset/TestIssue2230.java | 30 ------------------- 1 file changed, 30 deletions(-) delete mode 100644 jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java diff --git a/jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java b/jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java deleted file mode 100644 index f1ea9942d2..0000000000 --- a/jme3-examples/src/main/java/jme3test/asset/TestIssue2230.java +++ /dev/null @@ -1,30 +0,0 @@ -/* - * Click nbfs://nbhost/SystemFileSystem/Templates/Licenses/license-default.txt to change this license - * Click nbfs://nbhost/SystemFileSystem/Templates/Classes/Class.java to edit this template - */ -package jme3test.asset; - -import com.jme3.app.SimpleApplication; -import com.jme3.asset.plugins.FileLocator; - -/** - * - * @author codex - */ -public class TestIssue2230 extends SimpleApplication { - - public static void main(String[] args) { - new TestIssue2230().start(); - } - - @Override - public void simpleInitApp() { - - assetManager.registerLocator("NonExistantFolder", FileLocator.class); - assetManager.loadModel("NonExistantModel.j3o"); - - stop(); - - } - -} From 87383999c30ab2180d65342a10a0f0551c7a7353 Mon Sep 17 00:00:00 2001 From: SceneMax3D Date: Mon, 15 Apr 2024 09:47:45 +0300 Subject: [PATCH 4/4] Update ImplHandler.java fix typo --- jme3-core/src/main/java/com/jme3/asset/ImplHandler.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java b/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java index a389570edc..90af79d803 100644 --- a/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java +++ b/jme3-core/src/main/java/com/jme3/asset/ImplHandler.java @@ -124,7 +124,7 @@ protected T initialValue() { } catch (InstantiationException | IllegalAccessException | IllegalArgumentException | InvocationTargetException | NoSuchMethodException | SecurityException ex) { - logger.log(Level.SEVERE, "An exception occured while instantiating asset locator: " + type.getName(), ex); + logger.log(Level.SEVERE, "An exception occurred while instantiating asset locator: " + type.getName(), ex); } return null; }