Small cleaning up + Fix for crash when looping through empty Vec

bevy_mod_scripting_core/src/hosts.rs

@@ -360,22 +360,25 @@ impl<T: Asset> Script<T> {
         event_writer: &mut EventWriter<ScriptLoaded>,
     ) {
         debug!("reloading script {}", script.id);
-        // retrieve owning entity
-        let entity = contexts.script_owner(script.id()).unwrap();
-
-        // remove old context
-        contexts.remove_context(script.id());
 
-        // insert new re-loaded context
-        Self::insert_new_script_context::<H>(
-            host,
-            script,
-            entity,
-            script_assets,
-            providers,
-            contexts,
-            event_writer,
-        );
+        // retrieve owning entity
+        if let Some(entity) = contexts.script_owner(script.id()) {
+            // remove old context
+            contexts.remove_context(script.id());
+            // insert new re-loaded context
+            Self::insert_new_script_context::<H>(
+                host,
+                script,
+                entity,
+                script_assets,
+                providers,
+                contexts,
+                event_writer,
+            );
+        } else {
+            // remove old context
+            contexts.remove_context(script.id());
+        }
     }
 
     /// checks if a script has loaded, and if so loads (`ScriptHost::load_script`),

bevy_script_api/src/common/std.rs

@@ -131,10 +131,11 @@ impl<T: FromReflect + TypePath> IntoIterator for ScriptVec<T> {
             // TODO?: end used to be an Option, and this check moved into the next method but
             // I am not sure if this will ever realistically fail, so if you do get this exception happening
             // hit me with an issue
+            // if len > 0, subtract 1, otherwise set to 0
             end: self
                 .len()
-                .map(|v| v - 1)
-                .expect("Could not access length when creating iterator"),
+                .map(|len| if len > 0 { len - 1 } else { 0 })
+                .expect("Failed to get length of ScriptVec"),
             base: self,
         }
     }

bevy_script_api/src/rhai/bevy/mod.rs

@@ -40,13 +40,6 @@ impl CustomType for ScriptWorld {
                     .map(Dynamic::from)
                     .unwrap_or_default()
             })
-            .with_fn("get_children", |self_: ScriptWorld, parent: Entity| {
-                self_
-                    .get_children(parent)
-                    .into_iter()
-                    .map(Dynamic::from)
-                    .collect::<Vec<_>>()
-            })
             .with_fn(
                 "add_default_component",
                 |self_: ScriptWorld, entity: Entity, type_registration: ScriptTypeRegistration| {
@@ -138,15 +131,19 @@ impl CustomType for ScriptWorld {
                     })
                 },
             )
-            .with_fn("get_children", |self_: &ScriptWorld, parent: Entity| {
+            .with_fn("get_parent", |self_: ScriptWorld, entity: Entity| {
+                if let Some(parent) = self_.get_parent(entity) {
+                    Dynamic::from(parent)
+                } else {
+                    Dynamic::from(-1)
+                }
+            })
+            .with_fn("get_children", |self_: ScriptWorld, parent: Entity| {
                 self_
                     .get_children(parent)
                     .into_iter()
                     .map(Dynamic::from)
-                    .collect::<Vec<Dynamic>>()
-            })
-            .with_fn("get_parent", |self_: &ScriptWorld, entity: Entity| {
-                self_.get_parent(entity)
+                    .collect::<Vec<_>>()
             })
             .with_fn(
                 "push_child",