From ffded5b78ee6e7eb86f5049a375eca2b4288a40a Mon Sep 17 00:00:00 2001 From: capt-glorypants Date: Mon, 1 Jan 2024 17:12:24 +0100 Subject: [PATCH] Rename `ArchetypeEntity::entity` into `ArchetypeEntity::id` (#11118) # Objective Fixes #11050 Rename ArchetypeEntity::entity to ArchetypeEntity::id to be consistent with `EntityWorldMut`, `EntityMut` and `EntityRef`. ## Migration Guide The method `ArchetypeEntity::entity` has been renamed to `ArchetypeEntity::id` --- crates/bevy_ecs/src/archetype.rs | 2 +- crates/bevy_ecs/src/query/iter.rs | 10 +++++----- crates/bevy_ecs/src/world/mod.rs | 4 ++-- crates/bevy_scene/src/scene.rs | 4 ++-- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/crates/bevy_ecs/src/archetype.rs b/crates/bevy_ecs/src/archetype.rs index 3e23da6457..9201f154ac 100644 --- a/crates/bevy_ecs/src/archetype.rs +++ b/crates/bevy_ecs/src/archetype.rs @@ -272,7 +272,7 @@ pub struct ArchetypeEntity { impl ArchetypeEntity { /// The ID of the entity. #[inline] - pub const fn entity(&self) -> Entity { + pub const fn id(&self) -> Entity { self.entity } diff --git a/crates/bevy_ecs/src/query/iter.rs b/crates/bevy_ecs/src/query/iter.rs index 0b9950f4ec..e79371c0d7 100644 --- a/crates/bevy_ecs/src/query/iter.rs +++ b/crates/bevy_ecs/src/query/iter.rs @@ -178,7 +178,7 @@ impl<'w, 's, D: QueryData, F: QueryFilter> QueryIter<'w, 's, D, F> { // Caller assures `index` in range of the current archetype. if !F::filter_fetch( &mut self.cursor.filter, - archetype_entity.entity(), + archetype_entity.id(), archetype_entity.table_row(), ) { continue; @@ -188,7 +188,7 @@ impl<'w, 's, D: QueryData, F: QueryFilter> QueryIter<'w, 's, D, F> { // Caller assures `index` in range of the current archetype. let item = D::fetch( &mut self.cursor.fetch, - archetype_entity.entity(), + archetype_entity.id(), archetype_entity.table_row(), ); @@ -719,7 +719,7 @@ impl<'w, 's, D: QueryData, F: QueryFilter> QueryIterationCursor<'w, 's, D, F> { let archetype_entity = self.archetype_entities.get_unchecked(index); Some(D::fetch( &mut self.fetch, - archetype_entity.entity(), + archetype_entity.id(), archetype_entity.table_row(), )) } @@ -817,7 +817,7 @@ impl<'w, 's, D: QueryData, F: QueryFilter> QueryIterationCursor<'w, 's, D, F> { let archetype_entity = self.archetype_entities.get_unchecked(self.current_row); if !F::filter_fetch( &mut self.filter, - archetype_entity.entity(), + archetype_entity.id(), archetype_entity.table_row(), ) { self.current_row += 1; @@ -831,7 +831,7 @@ impl<'w, 's, D: QueryData, F: QueryFilter> QueryIterationCursor<'w, 's, D, F> { // - fetch is only called once for each `archetype_entity`. let item = D::fetch( &mut self.fetch, - archetype_entity.entity(), + archetype_entity.id(), archetype_entity.table_row(), ); self.current_row += 1; diff --git a/crates/bevy_ecs/src/world/mod.rs b/crates/bevy_ecs/src/world/mod.rs index 8fb7658bd8..6fb2d2cffb 100644 --- a/crates/bevy_ecs/src/world/mod.rs +++ b/crates/bevy_ecs/src/world/mod.rs @@ -507,7 +507,7 @@ impl World { .iter() .enumerate() .map(|(archetype_row, archetype_entity)| { - let entity = archetype_entity.entity(); + let entity = archetype_entity.id(); let location = EntityLocation { archetype_id: archetype.id(), archetype_row: ArchetypeRow::new(archetype_row), @@ -536,7 +536,7 @@ impl World { .iter() .enumerate() .map(move |(archetype_row, archetype_entity)| { - let entity = archetype_entity.entity(); + let entity = archetype_entity.id(); let location = EntityLocation { archetype_id: archetype.id(), archetype_row: ArchetypeRow::new(archetype_row), diff --git a/crates/bevy_scene/src/scene.rs b/crates/bevy_scene/src/scene.rs index d09df9d0c7..93e15f55f0 100644 --- a/crates/bevy_scene/src/scene.rs +++ b/crates/bevy_scene/src/scene.rs @@ -96,7 +96,7 @@ impl Scene { for scene_entity in archetype.entities() { let entity = *instance_info .entity_map - .entry(scene_entity.entity()) + .entry(scene_entity.id()) .or_insert_with(|| world.spawn_empty().id()); for component_id in archetype.components() { let component_info = self @@ -117,7 +117,7 @@ impl Scene { } }) })?; - reflect_component.copy(&self.world, world, scene_entity.entity(), entity); + reflect_component.copy(&self.world, world, scene_entity.id(), entity); } } }